[Devel] Разработка на C++
Vitaly Lipatov
lav на etersoft.ru
Ср Сен 5 17:58:18 MSK 2012
Пример синтаксически не согласованной конструкции:
«Заглянув в код, у меня возникло несколько предложений».
В общем, посмотрев на проект «Варгус», у меня возникло несколько
замечаний:
1. Не надо писать циклы
for (it=; it!=end; it++) с помощью while
1.1. Возможно, в Qt будет удобно использовать foreach для перебора
2. Не надо брать указатели в скобочки:
delete (*itv);
(*its)->stopPlay();
3. Не надо проверять указатель на NULL перед delete и free
4. Не надо ставить пробелы перед и после ->:
s -> hide();
никто же не пишет s . hide()
5. Участки кода разного смысла можно отделать пустыми строками, как это
делается в прозе для абзацев.
Также проект уверенно держится впереди по степени отсутствия
комментариев.
Просьба разработчикам узнать себя и исправиться.
--
С уважением,
Виталий Липатов,
Etersoft
Подробная информация о списке рассылки Devel