VPF::Чеклист для проверки качества кода

sergejzr

Дата 18.6.2008, 12:48 (ссылка)

(нет голосов)

Загрузка ...

Un salsero
Group Icon

Профиль
Группа: Админ
Сообщений: 13285
Регистрация: 10.2.2004
Где: Германия г .Ганновер

Репутация: нет
Всего: 360

Очень осторожно запощу сюда свой вопрос smile

Итак требуется определить качество кода партнёра. Так как партнёров много и вце проходят кроссвалидацию, необходимо стандартизировать процесс. Для этого разрабатываю чеклист. Помогите расширить, ответить на возникшие вопросы:

o Correct use of Object Oriented programming (e.g. clear separation of concerns)
- a) Efficiency - Are the constructs efficiently designed?
  0 Yes
  0 No (add your remarks)
- b) Complexity – Do the constructs increase the architectural complexity?
  0 Yes (add your remarks)
  0 No
- c) Understandability - Does the design increase the psychological complexity?
  0 Yes (add your remarks)
  0 No
- d) Reusability - Does the design quality support possible reuse?
  0 Yes
  0 No (add your remarks)
- e) Testability/Maintainability - Does the structure support ease of testing and changes?
  0 Yes
  0 No (add your remarks)
o Proper use of try/catch and managing of Exceptions
По каким критериям определять?
o Java doc (or java doc like)
0 Not required/possible (explain why)
0 Exists in good quality
0 Could be improved (add your remarks)
0 Required but does not exists (add your remarks)
o User manual
0 Not required/possible (explain why)
0 Exists in good quality
0 Could be improved (add your remarks)
0 Required but does not exists (add your remarks)
o Inline comments
0 Not required/possible (explain why)
0 Exists in good quality
0 Could be improved (add your remarks)
0 Required but does not exists (add your remarks)
o Java source code conventions (e.g. Eclipse, D3.1)
- a) Comments formatted correctly? http://java.sun.com/docs/codeconv/html/Cod...s.doc4.html#385
  0 Yes
  0 Could be improved (add your remarks)
- b) Code conventions http://java.sun.com/docs/codeconv/html/Cod....doc5.html#2991
  0 Perfect
  0 Could be improved (add your remarks)
- c) Statements quality http://java.sun.com/docs/codeconv/html/Cod...s.doc6.html#430
  0 Perfect
  0 Could be improved (add your remarks)
- d) Naming Conventions http://java.sun.com/docs/codeconv/html/Cod...s.doc8.html#367
  0 Perfect
  0 Could be improved (add your remarks)
- e) Over aspects of coding quality
  0 Perfect
  0 Problematic (add your remarks)
- f) Are variable names human readable?
  0 Yes
  0 Not in all necessary cases (add your remarks)
Overall code quality
- a) Does the dead code exist?
  0 No
  0 Yes (add your remarks)
- b) Is the code efficient ?
  0 To my best knowledge - yes
  0 Not always (add your remarks)
- c) Are some items, that might be changed in the future hardcoded ?
  0 No
  0 Some of them (add your remarks)
- d) English quality within code and comments?
  0 Very good
  0 Good
  0 Acceptable
  0 Poor

--------------------

Me.
FB.
LinkedIn.
ORCID.
YT.

arilou

Дата 6.10.2008, 12:05 (ссылка)

(нет голосов)

Загрузка ...

Великий МунаБудвин

Профиль
Группа: Экс. модератор
Сообщений: 2646
Регистрация: 15.7.2004
Где: город-герой Минск

Репутация: 2
Всего: 61

sergejzr, Серег, я не совсем понял про "ответить на возникшие вопросы" - это ты о чем?

Добавлено через 1 минуту
ЗЫ. http://www.google.com/search?q=code+review+checklist - много материала.

Сорри, что так долго реагировал. Дел невпроворот.

--------------------

Wowa

Дата 27.2.2009, 17:52 (ссылка)

(нет голосов)

Загрузка ...

Эксперт

Профиль
Группа: Админ
Сообщений: 15017
Регистрация: 14.9.2000
Где: Винград

Репутация: нет
Всего: 290

sergejzr, а может стоит не просто чеклист, а баллы сделать?
типа:

0 Not required/possible (explain why)
1 Exists in good quality
2 Could be improved (add your remarks)
5 Required but does not exists (add your remarks)

Maksym

Дата 16.4.2009, 16:13 (ссылка)

(нет голосов)

Загрузка ...

Профиль
Группа: Участник Клуба
Сообщений: 1456
Регистрация: 19.8.2005
Где: Odessa, Black Sea

Репутация: нет
Всего: 62

sergejzr
К чему в результате пришел, если не секрет?

JackYF

Дата 19.4.2009, 10:39 (ссылка)

(нет голосов)

Загрузка ...

полуавантюрист

Профиль
Группа: Участник
Сообщений: 5814
Регистрация: 28.8.2004
Где: страна тысячи озё р

Репутация: нет
Всего: 162

Цитата

Хороший ли код?

О хороший
О плохой

Зачем мучаться? smile

Если серьёзно, какая-то уж очень грубая и бинарная оценка получается.

--------------------

Пожаловаться на меня как модератора можно здесь.

0 Пользователей читают эту тему (0 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема \| УП: Общие вопросы \| Следующая тема »