Димитър обнови решението на 08.10.2014 22:54 (преди около 10 години)
+# foo
Решение на Първа задача от Димитър Димитров
Към профила на Димитър Димитров
Резултати
- 6 точки от тестове
- 0 бонус точки
- 6 точки общо
- 12 успешни тест(а)
- 0 неуспешни тест(а)
Код
def generic_series(n, first_value, second_value)
return first_value if n == 1
return second_value if n == 2
generic_series(n - 1, first_value, second_value) +
generic_series(n - 2, first_value, second_value)
end
def series(type, n)
case type
when 'fibonacci' then generic_series(n, 1, 1)
when 'lucas' then generic_series(n, 2, 1)
when 'summed' then series('fibonacci', n) + series('lucas', n)
end
end
Лог от изпълнението
............ Finished in 0.0232 seconds 12 examples, 0 failures
История (2 версии и 6 коментара)
Димитър обнови решението на 08.10.2014 22:54 (преди около 10 години)
-# foo
+def generic_series(n, first_value, second_value)
+ return first_value if n == 1
+ return second_value if n == 2
+
+ generic_series(n - 1, first_value, second_value) +
+ generic_series(n - 2, first_value, second_value)
+end
+
+def series(type, n)
+ case type
+ when 'fibonacci' then generic_series(n, 1, 1)
+ when 'lucas' then generic_series(n, 2, 1)
+ when 'summed' then series('fibonacci', n) + series('lucas', n)
+ end
+end
Само на нашето решение никой не е коментирал. Тъжна работа.
Не трябва ли when да е едно ниво по-надълбоко (според ръководството по стил: 'Indent when and else one level deeper than case.') ?
София, така е, права си.
Само че мисля да ревизираме нашето ръководство по стил, за да уеднаквим някои неща с това за community-то, където, например, case и when се идентират на едно ниво.
Ясно! :))
Името на този параметър 'n' във функциите 'generic_series','series' не е ок.
В контекста е ясно какво е 'n', но ще е по-добре ако се замени със number или някакво друго по-подходящо име. :D
@Димитър, благодаря за коментара! Ще гледам да си поправя грешките в решението! :))