Замечания по коду Бирюковой в блоке формальных языков

Originally reported on Google Code with ID 197

Читаем http://docs.moodle.org/dev/Coding_style -  и выправляем все в коде.
Самые явные ошибки

1. Следим за стилями именования переменных и функций
2. Следим за стилями фигурных скобок. Он должен соответствовать Moodle и быть всегда.
3. После операторов перед круглыми скобками пробел - например if ( ...) после функций
- нет.  И вообще посмотрите, как в другом коде пробелы стоят.
4. Вместо array_push часто проще и понятнее писать $arr[] =  Перед этим бывает полезно
написать типа $arr = array(); до начала цикла, чтобы начать с пустого массива.
5. В некоторых файлах тестов - например group_matches - неверные отступы, нет отступов
внутри тела функции например. И вообще тестов на основные функции - построение пар,
рекурсию - мало! Тесты - важнейший показатель, насколько я могу доверять вашему коду.

Пишите здесь по пунктам, когда исправите и вытолкните код (например - 1 и 2 - исправлено),
чтобы я мог посмотреть.

Reported by oasychev on 2013-05-22 16:14:16

Comments (1)