Inspection » History » Revision 2
Revision 1 (Никита Артемов, 03/27/2012 10:26 AM) → Revision 2/8 (Никита Давыдовский, 03/27/2012 02:09 PM)
h1. Инспекция
Инспекция проведена в соответствию стилю "QML Coding Conventions":http://qt-project.org/doc/qt-4.8/qml-coding-conventions.html
h2. Соглашение по написанию выявленных недочетов и ошибок
Выявленные недочеты и ошибки перечисляются через буллеты. В конце недочёта или ошибки ставится его критичность для программы в целом.
* 0 - ошибка или недочёт не критичны, но следует подправить для большей структурированности и "красоты" написанного кода.
* 1 - ошибка или недочёт влекут за собой предупреждения при компиляции, портят общую структуру кода или влекут его неправильное понимание.
* 2 - ошибка или недочёт могут повлечь за собой падение программы при выполнении.
h2. Отчет по инспектированию
h3. Приложение "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()
для дальнейшего облегчения переводов.
** внутренние объекты следует разделять пробелами.