[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