Славена обнови решението на 19.12.2011 00:35 (преди около 13 години)
-
require
го слагай в началото на файла, няма смисъл да е в модулаGameOfLife
- Думата
neighboards
ме озадачава :) - На места ти липсват някои интервали — не спазваш стриктно конвенцията за whitespace в Ruby и това ме кара да се чувствам некомфортно, като чета кода ти :)
Иначе, на пръв поглед, решението изглежда добре.
Благодаря за коментара :)
- Искам да попитам каква е конвенцията за скобите при дефиниране и извикване на метод с параметри.
Вероятно най-популярната е:
Скоби се пишат във всички случаи при дефиниране или извикване на метод, освен ако:
- Методът няма параметри или се извиква без параметри
- Извиквате метода само с ключове/стойности на хеш и сте изпуснали
{}
-скобите
Пример за второто:
User.create email: 'me@example.org', name: 'Greg Bell'
Макар че втората точка, и изобщо тази конвенция за скобите при извикване, не е изкована в каменните плочи на Руби-конвенциите и зависи донякъде от проекта/авторите на даден код.
Ако нямаш други причини да пишеш по различен начин, ти предлагам да се придържаш към това.
Така като гледам промените, които си направила, си мисля, че може би не е много добре така със скобите. Примерно, аз бих изпуснал скобите в тези два случая: Set.new(cells)
и Board.new(*(alive_cells | new_cells))
. Другото изглежда окей. С две думи, гледай да е достатъчно четимо и да е консистентно в кода ти и ще си окей.
P.S. И ако трябва да съм честен, аз се старая да изпускам скобите доста често, когато викам функции. Просто ми изглежда по-четимо.