0

Есть следующий код:

case 10501:
GetWindowTextA(hMyEditControl, &static_text[0], 20);
try {
    drive_letter_string = static_text.substr(0, 1);
    SetWindowTextA(hMyStaticControl, drive_letter_string.data());
}
catch(const std::out_of_range & err)
{
    MessageBoxA(hWnd, "Sorry error while copying data to string", "Error", MB_ICONERROR | MB_OK);
    return DefWindowProc(hWnd, message, wParam, lParam);
}    
break;

Собственно говоря, в эдите у меня есть текст следующего вида: "C:\\bla-bla" (без кавычек, естественно). Я пытаюсь скопировать в переменную drive_letter_string букву C и ловлю исключение. В чем может быть дело?

3 ответа 3

4

Что-то мне не нравится, что Вы копируете текст по адресу &static_text[0]. Как правило, объект std::string устроен внутри довольно сложно и напоминает больше list, чем vector, т.е. данные в нем не обязательно расположены последовательно.

Лучше копируйте в обычный строковый буфер char[], а потом, если нужно, делайте из него std::string.

10
  • так это ж быдлокод, не?
    – PaulD
    Commented 16 авг. 2012 в 15:44
  • 1
    Почему? Использование типов Си в С++ никто в принципе не воспрещает, там, где это действительно необходимо.
    – skegg
    Commented 16 авг. 2012 в 15:45
  • 1
    Особенно это касается всяких WinAPI и прочих низкоуровневый вещей.
    – skegg
    Commented 16 авг. 2012 в 15:47
  • 2
    Память, выделенная под std::string [гарантированно представляет собой один блок][1] для случая, если length > 0. [1]: stackoverflow.com/questions/1986966/… Commented 16 авг. 2012 в 16:26
  • 1
    IMHO тут все достаточно понятно (раньше внимательно не смотрел). Данные копируются в "буфер" string (допустим там достаточно места). Поле "длины" данных в буфере естественно не меняется (допустим было 0). Соответственно .substr() бросает исключение.
    – avp
    Commented 17 авг. 2012 в 8:43
3

Очень нехороший код вы написали.

  • Происходит использование exception'ов для flow control'a на ровном месте. Эксепшны - для исключительных ситуаций, а не для ситуации, когда вам лень проверить длину строки.

  • SetWindowTextA не швыряет C++ исключения, поэтому ее можно смело вынести из блока.

  • Не проверяются коды ошибки от функций WinAPI.

  • В обработчике исключений написана ересь - std::out_of_range не имеет никакого отношения к копированию данных.

  • SetWindowTextA принимает LPCSTR, то есть нуль-терминированную строку, а вы передаете методу результат std::string::data(), который не содержит символа \0 в конце.

  • Ваш код элементарно переделывается, если спросить с помощью метода ::GetWindowTextA у контрола первую букву, а дальше (в зависимости от того, совпадает она с вашими ожиданиями или нет, и если метод WinAPI не сфейлился) установить ее путем вызова ::SetWindowTextA.

  • Не забудьте про то, что строка должна быть нуль-терминированной.
3
  • out_of_range имеет прямое отношение к взятию подстроки.
    – skegg
    Commented 16 авг. 2012 в 20:00
  • @mikillskegg Подразумевалось, что "Sorry error while copying data to string" не соотносится с реальным эксепшном от substing. Ну, и, опять же, см. первый пункт (про ненужность try-catch вообще). Commented 17 авг. 2012 в 7:20
  • Мне кажется, что по сравнению с главной проблемой - попыткой прямого копирования в string - все это не так принципиально. И насчет линейного расположения данный в стринге. Да, стандарт С++11 требует этого, а С++03 - нет, хотя реально все так и обстоит. Был неправ. Но все равно вот так копировать данные в стринг - полное безобразие.
    – skegg
    Commented 17 авг. 2012 в 8:00
1

Вот я, к сожалению, не совсем согласен с @mikillskegg, так как сам вызов копирования текста в static_text[0]:

  GetWindowTextA(hMyEditControl, &static_text[0], 20);

не находится внутри защищенного блока try, поэтому, эксепшн не может возникать там. Исключение возникает в одной из двух строк. Мне почему-то кажется, что во второй строке...


А вообще, тут надо дебажить самому. Код-то не сложный, ошибку будет найти нетрудно. Проверьте где возникает исключение, также убедитесь, что для static_text выделено достаточное количество памяти.

Ваш ответ

Нажимая «Отправить ответ», вы соглашаетесь с условиями пользования и подтверждаете, что прочитали политику конфиденциальности.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.