Исходное сообщение
"Выпуск p2p-мессенджера Communist 1.4" Отправлено ProfessorNavigator, 25-Июн-22 17:50
>[оверквотинг удален] >   while (!f.eof ()) >     { >       std::string line; >       getline (f, line); >       if (line != "") > > https://github.com/ProfessorNavigator/communist/blob/8373173... > После проверки и перед открытием файл может быть удалён. В данном случае > ничего страшного не случится из-за проверки f.eof (), но в общем-то > filesystem::exists() лишняя. Принимается, но давайте тех.подробности или на GitHub/GitFlic, или на почту (смотрите readme), или на форум Альта. > Поскольку писать в поток не собираетесь, можно использовать ifstream. Можно, но зачем (опять же - давайте на GitHub/GitFlic и иже с ними)? > Сравнение std::string с "" приведёт у неявному вызову конструктора. Можно обойти, проверив > наличие содержимого строки вызовом line.empty(). Да, я в курсе. Но что-то у меня с ним при первом знакомстве пошло не так (уже не помню, что именно) - поэтому был выбран именно вариант с пустой строкой. По памяти есть потери, но на фоне возможностей современного железа и функционала программы - это не особо критично. Это мессенджер, а не программа обсчёта космических наблюдений. > Из этих мелочей читатель может сделать вывод, что автор не очень знаком > с Си++, тогда как от Профессора ожидается код на уровне Кнута, > Морриса и Пратта. Здесь Вы предупреждаете, что не программист, на фоне > некоторых профессиональных баш-разработчиков выглядит шикарно, но не github подобного > не увидел. На GitHub вы не видите подобного предупреждения, потому что пока никто не интересовался особо)) Да и зачем - вы смотрите на код, а не на автора. Кто и что автор - дело десятое. Кто понимает в разработке, тот и уровень автора оценит, а остальным оно без надобности.

Ваше сообщение
Имя*:
EMail:	Для отправки ответов на email укажите знак ! перед адресом, например, !user@host.ru (!! - не показывать email). Более тонкая настройка отправки ответов производится в профиле зарегистрированного участника форума.
Заголовок*:
Сообщение*:	>>[оверквотинг удален] >> while (!f.eof ()) >> { >> std::string line; >> getline (f, line); >> if (line != "") >> [/code] >> https://github.com/ProfessorNavigator/communist/blob/8373173118b76c80735be495cb02097cdd59a000/MainWindow.cpp#L57-L66 >> После проверки и перед открытием файл может быть удалён. В данном случае >> ничего страшного не случится из-за проверки f.eof (), но в общем-то >> filesystem::exists() лишняя. > Принимается, но давайте тех.подробности или на GitHub/GitFlic, или на почту (смотрите readme), > или на форум Альта. >> Поскольку писать в поток не собираетесь, можно использовать ifstream. > Можно, но зачем (опять же - давайте на GitHub/GitFlic и иже с > ними)? >> Сравнение std::string с "" приведёт у неявному вызову конструктора. Можно обойти, проверив >> наличие содержимого строки вызовом line.empty(). > Да, я в курсе. Но что-то у меня с ним при первом > знакомстве пошло не так (уже не помню, что именно) - поэтому > был выбран именно вариант с пустой строкой. По памяти есть потери, > но на фоне возможностей современного железа и функционала программы - это > не особо критично. Это мессенджер, а не программа обсчёта космических наблюдений. >> Из этих мелочей читатель может сделать вывод, что автор не очень знаком >> с Си++, тогда как от Профессора ожидается код на уровне Кнута, >> Морриса и Пратта. Здесь Вы предупреждаете, что не программист, на фоне >> некоторых профессиональных баш-разработчиков выглядит шикарно, но не github подобного >> не увидел. > На GitHub вы не видите подобного предупреждения, потому что пока никто не > интересовался особо)) Да и зачем - вы смотрите на код, а > не на автора. Кто и что автор - дело десятое. Кто > понимает в разработке, тот и уровень автора оценит, а остальным оно > без надобности.
	Введите код, изображенный на картинке: