Inspection » History » Revision 4

Инспекция¶

Инспекция проведена в соответствию стилю QML Coding Conventions

Выявленные недочеты и ошибки перечисляются через буллеты. В конце недочёта или ошибки ставится его критичность для программы в целом.

0 - ошибка или недочёт не критичны, но следует подправить для большей структурированности и "красоты" написанного кода.
1 - ошибка или недочёт влекут за собой предупреждения при компиляции, портят общую структуру кода или влекут его неправильное понимание.
2 - ошибка или недочёт могут повлечь за собой падение программы при выполнении.

main.qml
- строка 31: большой комментированный участок кода - 1
MainPage.qml
- строки 55, 116: комментированные участки кода - 1
- строка 21: отладочная модель (List Model), следует убрать в финальной версии - 1
SearchPlacePage.qml
- нарушение порядка следования членов объекта - 0
- большие участки javascript кода следует выделить в отдельные функции - 0
Header.qml
- нарушение порядка следования членов объекта - 0
Общие замечания:
- в бинарных арифметических операциях, операнды и знаки следует разделять пробелами - 0
- все символьные строки явно прописанные в коде, нужно писать как аргумент функции qsTr()
  для облегчения дальнейших переводов.
- внутренние объекты следует разделять пробелами.
- для констант (размер текста, ширина полей...) следует завести отдельный javascript
  файл (лучше взять уже готовый, рекомендованный разработчиками фреймворка)- 1

MainPage.qml
- строки 121-126: последовательное присваивание одной переменной разных значений - 1
- строка 214, 219 : Отдельные страницы приложения типа Page следует перенести в main.qml - 1
CalendarView.qml
- наличие больших участков закомментированного кода - 0
Общие замечания:
- нарушена очередность следования объектов в файлах qml - 1
- присваивание членам объектов значении по умолчанию при объявлении - 0
- однотипные свойства объектов следует группировать - 0
- объект с одним-двумя свойствами следует располагать на одной строке - 0
- наличие магических чисел - 1

Files (0)