Вот мой код:

Код

LPSTR CMyDoc::mMyTrim(LPSTR szStr){  //MessageBox(NULL, szStr, "", MB_OK);    size_t  nStrLen = 0;  StringCchLength(szStr, 4096, &nStrLen);  LPSTR szTemp = new CHAR[nStrLen+1];  StringCchCopy(szTemp, nStrLen, szStr);  szTemp = szStr + nStrLen -1;  //  First trim spaces on the right  while( (szTemp >= szStr) && (*szTemp == ' ') )    *szTemp = '\0';//Здесь и возникает ошибка

Переменная szStr приходит из WinMain (это lpCmdStr). На самом деле в том месте где происходит присваивание, *szTemp точно содержит пробел, то есть * не разывеновывает указатель на не проинициализированную область памяти, соответственно и присваивание тоже законно, но по какой-то причине именно в этом месте вылетает исключение

IMHO, видимо, проблема заключается вот в этой строке:

Цитата

szTemp = szStr + nStrLen -1;

Объясняю, почему:
в строке

Цитата

LPSTR szTemp = new CHAR[nStrLen+1];

была выделена память в области динамического обмена, указатель szTemp содержит теперь АДРЕС начала выделенной области памяти. Затем, в строке

Цитата

szTemp = szStr + nStrLen -1;

этому указателю присваивается адрес хр$н знает какой области памяти, которая, кстати, может быть УЖЕ использована для других целей!
...и затем, в эту хр$н знает какую область памяти, ты записываешь в строке

Цитата

*szTemp = '\0';

символ окончания строки '\0' !!!
По твоему, это корректно???

Цитата

этому указателю присваивается адрес хр$н знает какой области памяти, которая, кстати, может быть УЖЕ использована для других целей!

А по-моему, у него всё нормально тут: szTemp присваивается адрес последнего ненулевого символа строки szStr...
Только не понятно - зачем там было создавать и копировать строку ?... Задумка не совсем ясна - надо обрезать исходную строку или же обрезать скопированную строку ?
У меня такой код обрезает справа пробелы в szStr нормально, без ошибок:

Код

WINAPI WinMain(HINSTANCE, HINSTANCE, LPSTR szStr, int) { size_t nStrLen = strlen(szStr); LPSTR szTemp = szStr + nStrLen -1; //  First trim spaces on the right while( (szTemp >= szStr) && (*szTemp == ' ') )   *szTemp-- = '\0'; MessageBox (0, szStr, szStr, 0); }

(добавил только --, чтобы все пробелы обрезались, а не только один последний)
PS: есть функция StrTrim...

Признаю, насчет

Цитата

этому указателю присваивается адрес хр$н знает какой области памяти, которая, кстати, может быть УЖЕ использована для других целей!

ПОГОРЯЧИЛСЯ, но постойте, куда же девался post от mr.DUDA (так, кажется, его зовут?)

[quote]LPSTR CMyDoc::mMyTrim(LPSTR szStr){
//MessageBox(NULL, szStr, "", MB_OK);

size_t nStrLen = 0;
StringCchLength(szStr, 4096, &nStrLen);
LPSTR szTemp = new CHAR[nStrLen+1];

StringCchCopy(szTemp, nStrLen, szStr);
szTemp = szStr + nStrLen -1;


// First trim spaces on the right
while( (szTemp >= szStr) && (*szTemp == ' ') )
*szTemp = '\0';//Здесь и возникает ошибка

Какой-то хитровыдуманный способ. Но впринципе дожно работать. Только здесь надо

Код

szTemp = szTemp + nStrLen -1;

и можно попрощаться с началом строки - хрен потом найдёшь. Ну да ладно.
И проверять надо, чтоб nStrLen!=0. И чтоб строка из одних пробелов не состояла - то начало-то потеряялось.
И цикл чуть-чуть переделать

Код

// while( /*(szTemp >= szStr)  один чёрт не срабатывает*/ && (*szTemp == ' ') ) //   *szTemp = '\0';//Здесь и возникает ошибка while (*szTemp==' ') *(szTemp--)=0;

Атак всё круто. Иногда даже будет работать

думаю действительно lpCmdStr лежит на страничке с атрибутами PAGE_READONLY и при попытке ее модифицировать и происходит exception.

надо не так:
szTemp = szStr + nStrLen -1;
а так:
szTemp = szTemp + nStrLen -1;

только начало строки действительно теряем

2Nimbus
Твоя мысль ясна и поэтому я несколько переделал алгоритм, дабы избавиться от подобных недорозумений, и хоть он получился несколько плосковатым, зато по-мойму стал более безопасным

2p0s0l
Ради одной функции StrTrim подрубать библу слишком жирно

2Олег М
Насчет nStrLen!=0 согласен, но менять условия цикла я не стал, так как есть вероятность того что байт перед 0-м символом строки тоже окажется пробелом и тогда будем считать левые пробелы лежащие где - то там...

2AndyY

Цитата

думаю действительно lpCmdStr лежит на страничке с атрибутами PAGE_READONLY и при попытке ее модифицировать и происходит exception.

Не думаю, я в одном случае получаю строчку из "Configuration Properties > Debugging > Command Arguments" проекта, а второй раз из самой командной строки, а результат один и тот же.

Это сообщение отредактировал(а) Royan - 6.7.2004, 14:45

И что, это работает?!

Код

szTemp = szStr + nStrLen -1;

А где же тип переменной szTemp? ...хотя, если она была уже объявлена где-то...
И вот это ещё:

Код

int nLastPos = nStrLen;

В данном случае, IMHO, переменная nLastPos ВСЕГДА будет равна 0 (что-то я не просёк, где nStrLen присваивается хоть что-то после инициализации нулём). Но, как я подозреваю, задумка была совсем иной!
Остальное проверю завтра, а то поздно уже...

...да, похоже на сервере со временем что-то (на моих часах 01:11)???

...да, похоже на сервере со временем что-то (на моих часах 01:11)???

Цитата

Вобщем вот в итоге что я изобрел:

Да уж, точно - изобрёл. Велосипед с треугольными колёсами. Ты её хоть скомпилировал? О запуске речи пока и не идёт

Прошу прощения вот правильный вариант (то что выше писалось на работе из головы поэтому и наделал ошибок)

Цитата

LPSTR CMyStrFormat::mMyTrim(LPSTR szStr){     if(szStr ==  0){     return '\0';   }   size_t nStrLen = 0;//Total string length     StringCchLength(szStr, 4096, &nStrLen);    LPSTR szTemp = szStr + nStrLen -1;   size_t nLastPos = nStrLen;   // Count spaces on the right   while( (szTemp >= szStr) && (*szTemp == ' ') ){     szTemp--;     nLastPos--;   }   //If string contained only paces return '\0'   if(nLastPos == 0){     return '\0';   }   szTemp = szStr;   int nCurPos = 0;   // Now count spaces on the left   while (*szTemp == ' '){     nCurPos++;     szTemp++;   }   //The lenght of the trimmed string   size_t nTrimmedStrLen = nStrLen - nCurPos - (nStrLen - nLastPos);   szFormattedStr = new CHAR[nTrimmedStrLen];     VALID_GLE_EX(szFormattedStr ,"Not enough memory. Please close other applications and try again.",false);     //Copy the resulting trimmed string   while (nTrimmedStrLen > 0){     nTrimmedStrLen --;     *szFormattedStr++ = *szTemp++;   }   *szFormattedStr = '\0';//Terminating NULL character   szFormattedStr-=(nStrLen - nCurPos - (nStrLen - nLastPos));//Rollback to the begining of the string   return szFormattedStr; }

Это сообщение отредактировал(а) Royan - 7.7.2004, 14:02

Нормально только надо {if(nLastPos == 0) { return ""; }

как вариант (тоже из головы) - почти тоже самое

Код

static TCHAR _szEmpty[]=_T(""); LPTSTR CMyStrFormat::mMyTrim(LPTSTR pszStr) { if (!szStr) return _szEmpty; LPTSTR pszStart=pszStr,pszEnd; for (;*pszStart==_T(' ');++pszStart); int nLen; StringCchLength(pszStart, 4096, &nLen) for (psz1=pszStart+nLen;pszEnd!=pszStart && *pszEnd=_T(' '); --pszEnd); if (pszStart==pszEnd) return _szEmpty; nLen=pszEnd-pszStart; TCHAR *pszRes=new TCHAR[nLen+1]; memcpy(pszRes,pszStart,nLen*sizeof(TCHAR)); pszRes[ nLen+1]=0; return pszRes; }