Actions
Inspection » History » Revision 2
« Previous |
Revision 2/8
(diff)
| Next »
Никита Давыдовский, 03/27/2012 02:09 PM
Инспекция¶
Инспекция проведена в соответствию стилю QML Coding Conventions
Соглашение по написанию выявленных недочетов и ошибок¶
Выявленные недочеты и ошибки перечисляются через буллеты. В конце недочёта или ошибки ставится его критичность для программы в целом.- 0 - ошибка или недочёт не критичны, но следует подправить для большей структурированности и "красоты" написанного кода.
- 1 - ошибка или недочёт влекут за собой предупреждения при компиляции, портят общую структуру кода или влекут его неправильное понимание.
- 2 - ошибка или недочёт могут повлечь за собой падение программы при выполнении.
Отчет по инспектированию¶
Приложение "Places"
(инспектор Давыдовский Н., дата 26.03.2012)¶
- main.qml
- строка 31: большой комментированный участок кода - 1
- MainPage.qml
- строки 55, 116: комментированные участки кода - 1
- строка 21: отладочная модель (List Model), следует убрать в финальной версии - 1
- SearchPlacePage.qml
- нарушение порядка следования членов объекта - 0
- большие участки javascript кода следует выделить в отдельные функции - 0
- Header.qml
- нарушение порядка следования членов объекта - 0
- Общие замечания:
- в бинарных арифметических операциях, операнды и знаки следует разделять пробелами - 0
- все символьные строки явно прописанные в коде, нужно писать как аргумент функции qsTr()
для дальнейшего облегчения переводов. - внутренние объекты следует разделять пробелами.
Updated by Никита Давыдовский over 12 years ago · 2 revisions