Не присваивается значение элементу массива LPWSTR
Есть кусок кода некоторой программы на C++
Пишу в Visual Studio 2010.
Вопрос такой: Почему строка [COLOR="#800080"]dir[j]='\0';[/COLOR] игнорируется и не выполняется как будто в коде ее вообще нет?
Вот поставил я брекпоинты на всех строках этого кода, запустил, и брекпоинт со строки "[COLOR="#800080"]dir[j]='\0';[/COLOR]" исчезает! И когда программа выполняется, то эту строку вообще пропускает, как будто там ничего нет.
В коде, что я привел выше, вы видите третью строку закоментированной. Если ее раскоментировать, то эта строчка выполнится без проблем. Но как только ее поместить ниже цикла, так тут же она игнорируется! :confused:
Я вообще не въежаю, как объяснить такое странное поведение?? Объясните пожалуйста, а то у меня от этого странного поведения уже мозг кипит.
P.S. Сама задача этого кода в следующем: получить полный путь к текущей программе и отбросить в пути имя программы, оставив таким образом только путь к папке программы.
Код:
LPWSTR dir;
GetModuleFileName(NULL, dir, 300);
//dir[3]='!';
int i, j=0;
for(i=0;dir!='\0';i++)
if(dir=='\\')
j=i;
dir[j]='\0';
j++;
GetModuleFileName(NULL, dir, 300);
//dir[3]='!';
int i, j=0;
for(i=0;dir!='\0';i++)
if(dir=='\\')
j=i;
dir[j]='\0';
j++;
Пишу в Visual Studio 2010.
Вопрос такой: Почему строка [COLOR="#800080"]dir[j]='\0';[/COLOR] игнорируется и не выполняется как будто в коде ее вообще нет?
Вот поставил я брекпоинты на всех строках этого кода, запустил, и брекпоинт со строки "[COLOR="#800080"]dir[j]='\0';[/COLOR]" исчезает! И когда программа выполняется, то эту строку вообще пропускает, как будто там ничего нет.
В коде, что я привел выше, вы видите третью строку закоментированной. Если ее раскоментировать, то эта строчка выполнится без проблем. Но как только ее поместить ниже цикла, так тут же она игнорируется! :confused:
Я вообще не въежаю, как объяснить такое странное поведение?? Объясните пожалуйста, а то у меня от этого странного поведения уже мозг кипит.
P.S. Сама задача этого кода в следующем: получить полный путь к текущей программе и отбросить в пути имя программы, оставив таким образом только путь к папке программы.
Код:
LPWSTR dir
Во-первых, у тебя алгоритмически неправильно задача решена.
[QUOTE=Tolias28]
Сама задача этого кода в следующем: получить полный путь к текущей программе и отбросить в пути имя программы, оставив таким образом только путь к папке программы.
[/QUOTE]
Ты отрежешь таким кодом букву диска, а не полный путь. Читать строку с конца нужно для того, чтобы сделать то, что ты хочешь. А правильный код выглядит следующим образом:
Обрати внимание, что я убрал переменную j, она там просто лишняя.
[QUOTE=Tolias28]
Сама задача этого кода в следующем: получить полный путь к текущей программе и отбросить в пути имя программы, оставив таким образом только путь к папке программы.
[/QUOTE]
Ты отрежешь таким кодом букву диска, а не полный путь. Читать строку с конца нужно для того, чтобы сделать то, что ты хочешь. А правильный код выглядит следующим образом:
Код:
#include <stdio.h>
#include <windows.h>
int main(void) {
wchar_t dir[MAX_PATH] = {0};
GetModuleFileName(GetModuleHandle(NULL), dir, MAX_PATH);
int i = 0;
for (i = lstrlen(dir); i >= 0; i--) {
if (dir == L'\\') {
dir = L'\0';
break;
}
}
i++;
wprintf(L"%s\n", dir);
return 0;
}
#include <windows.h>
int main(void) {
wchar_t dir[MAX_PATH] = {0};
GetModuleFileName(GetModuleHandle(NULL), dir, MAX_PATH);
int i = 0;
for (i = lstrlen(dir); i >= 0; i--) {
if (dir == L'\\') {
dir = L'\0';
break;
}
}
i++;
wprintf(L"%s\n", dir);
return 0;
}
Обрати внимание, что я убрал переменную j, она там просто лишняя.
Алгоритмически было решено вполне правильно.
Там, цикл запоминает '\\' каждый раз, но цикл не заканчивается после этого. А закончится, когда встретится с '\0'. Таким образом, когда цикл закончится, в переменной j будет торчать номер последний '\\'.
-------------------------------------------------------------------------------------------
UPD
На самом деле тот вариант, может, быть и быстрее. Там нет lstrlen, которая сделает один проход по массиву, чтобы узнать его длину, но с другой стороны, при длинных именах и директориях будет быстрее ваш вариант, так как в lstrlen нет лишней проверки. Так что бросаться про алгоритмически неверно не стоит сразу.
Там, цикл запоминает '\\' каждый раз, но цикл не заканчивается после этого. А закончится, когда встретится с '\0'. Таким образом, когда цикл закончится, в переменной j будет торчать номер последний '\\'.
-------------------------------------------------------------------------------------------
UPD
На самом деле тот вариант, может, быть и быстрее. Там нет lstrlen, которая сделает один проход по массиву, чтобы узнать его длину, но с другой стороны, при длинных именах и директориях будет быстрее ваш вариант, так как в lstrlen нет лишней проверки. Так что бросаться про алгоритмически неверно не стоит сразу.
У меня, кстати, заработал код, который автор привел на компиляторе от Борланда. Не думал, что они сильно отличаются от микрософтовавского, но нет, правда оказалось жестокой.
Автор начни с того, что ты не выделяешь память под dir и в итоге все время пишешь по какому то случайному указателю. После такого можешь ждать от программы чего угодно не удивляясь. =)
Цитата: aks
Автор начни с того, что ты не выделяешь память под dir и в итоге все время пишешь по какому то случайному указателю. После такого можешь ждать от программы чего угодно не удивляясь. =)
Кстати, да. Я что-то не заметил этот косяк, а проверял константой. Пичаль мне.:facepalm:
Цитата: hivewarrior
Алгоритмически было решено вполне правильно.
Там, цикл запоминает '\\' каждый раз, но цикл не заканчивается после этого. А закончится, когда встретится с '\0'. Таким образом, когда цикл закончится, в переменной j будет торчать номер последний '\\'.
-------------------------------------------------------------------------------------------
UPD
На самом деле тот вариант, может, быть и быстрее. Там нет lstrlen, которая сделает один проход по массиву, чтобы узнать его длину, но с другой стороны, при длинных именах и директориях будет быстрее ваш вариант, так как в lstrlen нет лишней проверки. Так что бросаться про алгоритмически неверно не стоит сразу.
Там, цикл запоминает '\\' каждый раз, но цикл не заканчивается после этого. А закончится, когда встретится с '\0'. Таким образом, когда цикл закончится, в переменной j будет торчать номер последний '\\'.
-------------------------------------------------------------------------------------------
UPD
На самом деле тот вариант, может, быть и быстрее. Там нет lstrlen, которая сделает один проход по массиву, чтобы узнать его длину, но с другой стороны, при длинных именах и директориях будет быстрее ваш вариант, так как в lstrlen нет лишней проверки. Так что бросаться про алгоритмически неверно не стоит сразу.
Виноват, не заметил. Принято.
Цитата: hivewarrior
Алгоритмически было решено вполне правильно.
Там, цикл запоминает '\\' каждый раз, но цикл не заканчивается после этого. А закончится, когда встретится с '\0'. Таким образом, когда цикл закончится, в переменной j будет торчать номер последний '\\'.
-------------------------------------------------------------------------------------------
UPD
На самом деле тот вариант, может, быть и быстрее. Там нет lstrlen, которая сделает один проход по массиву, чтобы узнать его длину, но с другой стороны, при длинных именах и директориях будет быстрее ваш вариант, так как в lstrlen нет лишней проверки. Так что бросаться про алгоритмически неверно не стоит сразу.
Там, цикл запоминает '\\' каждый раз, но цикл не заканчивается после этого. А закончится, когда встретится с '\0'. Таким образом, когда цикл закончится, в переменной j будет торчать номер последний '\\'.
-------------------------------------------------------------------------------------------
UPD
На самом деле тот вариант, может, быть и быстрее. Там нет lstrlen, которая сделает один проход по массиву, чтобы узнать его длину, но с другой стороны, при длинных именах и директориях будет быстрее ваш вариант, так как в lstrlen нет лишней проверки. Так что бросаться про алгоритмически неверно не стоит сразу.
Спасибо за разъяснение) Я уже хотел это самое писать, как увидел, что вы ниже это уже сделали за меня)
Да да, именно потому я вырешил не использовать strlen, а узнавать последний слеш за один проход в цикле.
Спасибо за пример, в котором цикл идет от последнего символа. Работает. Но я все же хотел склониться к своему решению, в котором не используется strlen...
Цитата:
Автор начни с того, что ты не выделяешь память под dir и в итоге все время пишешь по какому то случайному указателю. После такого можешь ждать от программы чего угодно не удивляясь. =)
Каюсь... Признаюсь, я в самом начале память под dir выделял, но тогда компилятор ругался на невозможность преобразования:
Цитата:
IntelliSense: argument of type "LPWSTR *" is incompatible with parameter of type "LPWSTR"
потому я искусился объявить просто указатель, довольствовашись тем, что компилятор молча компилирует и не ругается.
Upd:
Выделил память под dir, но строка dir[j]='\0'; всеравно не исполняется(( Почему?
wchar_t* dir=new wchar_t [MAX_PATH] нэ?
не:) А какая в даном случае разница, объявлять массив статически или динамически:) Проблема оказалась не в этом. Как мне подсказали на другом форуме, трабл заключался в том, что я все эти отладки и правки кода делал в релиз, а не в дебаг! Элементарно то как... Никогда не думал, что при отладке в релизе могут быть такие приколы. Теперь буду знать. Проблема решена и мой вопрос полностью исчерпан.
aks
Вот и я это заметил и думаю,что-то тут не так:)
ТС:
Для означенной задачи есть более удобная функция,которая по пути получает и имя папки отдельно,и имя программы[COLOR="#c0c0c0"],и блэкджек со шлюхами…[/COLOR]
Вот и я это заметил и думаю,что-то тут не так:)
ТС:
Для означенной задачи есть более удобная функция,которая по пути получает и имя папки отдельно,и имя программы[COLOR="#c0c0c0"],и блэкджек со шлюхами…[/COLOR]
Цитата: @pixo $oft
aks
Вот и я это заметил и думаю,что-то тут не так:)
ТС:
Для означенной задачи есть более удобная функция,которая по пути получает и имя папки отдельно,и имя программы[COLOR="#c0c0c0"],и блэкджек со шлюхами…[/COLOR]
Вот и я это заметил и думаю,что-то тут не так:)
ТС:
Для означенной задачи есть более удобная функция,которая по пути получает и имя папки отдельно,и имя программы[COLOR="#c0c0c0"],и блэкджек со шлюхами…[/COLOR]
Не разрушай этот мир велосипедов.