sevaorlov / active_type Goto Github PK

А ты давно спеки прогонял? Они сломаны, попробуй починить.

Давай пока "причешем" существующий код вместо добавления/улучшения функционала. В файле lib/active_type/active_type.rb:

Строка 18:
Тут и далее ты модифицируешь строку, пришедшую в аргументы функции. Так делать не стоит, потому что ты изменяешь объект, находящийся вне функции.

Строка 19:
Это можно написать, не используя str.length. Почитай про индексирование массивов в руби.

Строки с 27 по 41:
Этот код нужно переписать без использования переменной-счетчика i. Почитай про метод Array#map.

Строка 39:
Ужасный способ изменять аттрибуты у внешнего объекта. Такой способ нарушает один из главных принципов ООП - инкапсуляцию, т.к. ты не используешь какой-либо публичный интерфейс объекта, а напрямую изменяешь его данные. Подумай, как это можно сделать иначе.

Строки 51, 52:
То же самое, что в предыдущем пункте. instance_variable_get/set правильно использовать только для self (как у тебя делается в строке 9), а для стороннего объекта нужно использовать его публичный интерфейс, т.е. методы.

Строки 107-108:
Можно записать 1 строкой и без if-a.

Так же более правильно будет перенести логику из строчек 52-63 внутрь класса Property. Подумай, почему это будет правильней и как это сделать.

Продолжаем дальше.

Строка 69: если не вдаваться в детали, то для стороннего разработчика довольно трудно понять, зачем тут строка chop-ается. Вообще, расстановку запятых между значениями лучше всего делать с помощью метода Array#join. Кроме этого, тут тебе еще пригодится Array#map, чтобы получить массив, который ты потом будешь join-ить.

Строки 17, 46: Лучше не делать явного вызова этого метода, пусть он вызывается "лениво" при первом обращении к ActiveType.properties

Сева, вот ссылка на общепринятый кодинг стайл для руби:

https://github.com/bbatsov/ruby-style-guide

Прочитай и пофикси свой код в соответствии с этим документом, а так же настрой свой редактор, чтобы он не вставлял табы вместо пробелов (у тебя иногда встречаются табы, иногда пробелы).