Project

General

Profile

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