Списъци от символи с %i( foo bar baz )

Програмиране с Ruby

Курс във Факултета по Математика и Информатика към СУ

Решение на Първа задача от Иван Кавалджиев

Обратно към всички решения

Към профила на Иван Кавалджиев

Резултати

  • 6 точки от тестове
  • 0 бонус точки
  • 6 точки общо
  • 12 успешни тест(а)
  • 0 неуспешни тест(а)

Код

def series(type, number)
case type
when 'fibonacci' then series_element_generator(1, 1, number)
when 'lucas' then series_element_generator(2, 1, number)
when 'summed' then series('fibonacci', number) + series('lucas', number)
end
end
def series_element_generator(first, second, number)
(number - 1).times { first, second = second, first + second }
first
end

Лог от изпълнението

............

Finished in 0.01318 seconds
12 examples, 0 failures

История (7 версии и 7 коментара)

Иван обнови решението на 10.10.2014 03:00 (преди над 9 години)

+def series(type,number)
+ return fibonacci(number) if(type == 'fibonacci')
+ return lucas(number) if(type == 'lucas')
+ return summed(number) if(type == 'summed')
+end
+
+def fibonacci(number)
+ first, second = 0, 1
+ number.times { first,second = second, first + second }
+ return first
+end
+
+def lucas(number)
+ first, second = -1, 2
+ (number).times { first ,second = second, first + second }
+ return first
+end
+
+def summed(number)
+ fibonacci(number) + lucas(number)
+end

Добро решение. Лесно се чете и е с линейна сложност. Въпреки това имам няколко забележки:

  • Функциите fibonacci и lucas са почти идентични, опитай се да изведеш общия код в отделна функция.

  • При функциите fibonacci и lucas ще бъде по-ясно какво се случва, ако началните ти стойности са началните стойности от математическите редици.

  • Разгледай case конструкцията.

  • Whitespace-a при запетаите на двойното присвояване не е добре.

  • Няма нужда да пишеш return в края на функциите. Функциите винаги връщат последния остойностен израз.

  • Скобите около number на 15-ти ред са излишни. Бъди консистентен :).

Съгласен съм с всичко :). Само не знам дали има итерактивен подход, при който да се започне с началните стойности от математическите редици? Грешка ли е, ако оставя началните стойности да са тези, с които съм работил сега?

Има :).

Не е грешка, ако са такива началните стойности, просто е хубаво кодът да е максимално изразителен и ясен. При теб е така, но и задачата е лека. Ще ти дам hint :

Помисли над number.times. Например 0.times { ... } няма да изпълни блока.

Иван обнови решението на 10.10.2014 22:14 (преди над 9 години)

def series(type,number)
- return fibonacci(number) if(type == 'fibonacci')
- return lucas(number) if(type == 'lucas')
- return summed(number) if(type == 'summed')
+ case type
+ when 'fibonacci' then fibonacci(number)
+ when 'lucas' then lucas(number)
+ else summed(number)
+ end
end
def fibonacci(number)
- first, second = 0, 1
- number.times { first,second = second, first + second }
- return first
+ auxiliary(1,1,number)
end
def lucas(number)
- first, second = -1, 2
- (number).times { first ,second = second, first + second }
- return first
+ auxiliary(2,1,number)
end
+
+def summed(number)
+ fibonacci(number) + lucas(number)
+end
+
+def auxiliary(first,second,number)
+ (number-1).times { first,second = second,first + second }
+ first
+end
+
def summed(number)
fibonacci(number) + lucas(number)
end

Иван обнови решението на 11.10.2014 00:55 (преди над 9 години)

def series(type,number)
case type
when 'fibonacci' then fibonacci(number)
when 'lucas' then lucas(number)
else summed(number)
end
end
def fibonacci(number)
auxiliary(1,1,number)
end
def lucas(number)
auxiliary(2,1,number)
end
def summed(number)
fibonacci(number) + lucas(number)
end
def auxiliary(first,second,number)
(number-1).times { first,second = second,first + second }
first
end
-
-def summed(number)
- fibonacci(number) + lucas(number)
-end

Иван обнови решението на 11.10.2014 01:20 (преди над 9 години)

def series(type,number)
case type
- when 'fibonacci' then fibonacci(number)
- when 'lucas' then lucas(number)
+ when 'fibonacci' then series_element_generator(1,1,number)
+ when 'lucas' then series_element_generator(2,1,number)
else summed(number)
end
end
def fibonacci(number)
- auxiliary(1,1,number)
+ series_element_generator(1,1,number)
end
def lucas(number)
- auxiliary(2,1,number)
+ series_element_generator(2,1,number)
end
def summed(number)
fibonacci(number) + lucas(number)
end
-def auxiliary(first,second,number)
+def series_element_generator(first,second,number)
(number-1).times { first,second = second,first + second }
first
end
+

Добре се получава. :)

Хубаво е, че си се сетил да преименуваш auxiliary, защото това име не носи никаква информация. Помощна функция, но какво точно прави, не става ясно от името. series_element_generator е по-добре. Аз лично бих предпочел глагола compute вместо generate, но и така е ок.

Функциите fibonacci и lucas не са ти нужни вече, защото в series case-a е достатъчно описателен. Там е написано when 'fibonacci' then ... това ти наименова логиката след then и тъй като въпросната логика е достатъчно кратка, няма нужда да я именоваш още веднъж (като отделна функция), освен ако не искаш да я преизползваш някъде. Това можеш да го направиш и за summed. Може да махнеш функцията summed, а самото название summed да го вкараш в when при case-a. Така ще получиш и допълнителна функционалност - ако някой реши да подаде тип редица, който не се поддържа от series ще получи nil (case връща nil, ако никое условие не съвпадне и нямаш else), вместо summed. Получавайки nil веднага става ясно, че функцията не поддържа тази редица, докато връщайки summed, може да подведе ползвателя, че си имплементирал и тази хипотетична редица.

Предполагам че и сам си се досетил за това, но си запазил нещата по този начин, защото ако махнеш summed и използваш series_element_generator 5-ти ред ще ти стане много дълъг :). Помисли коя друга функция можеш да извикаш на 5-ти ред, която ти смята елемент от редицата на Фибоначи и елемент от редицата на Лукас.

P.S.

Виж в style guide-a как се индентира case и къде се оставя whitespace при оператори (на 22 ред има такъв проблем). Винаги оставяй място след запетая.

Да, не махнах функцията summed, защото 5-ти ред щеше да стане много дълъг. И поради тази причина оставих и fibonacci и lucas функциите, защото се използват в summed. Ще помисля и за друг вариант, обаче.

Да, не махнах функцията summed, защото 5-ти ред щеше да стане много дълъг. И поради тази причина оставих и fibonacci и lucas функциите, защото се използват в summed. Ще помисля и за друг вариант, обаче.

Иван обнови решението на 12.10.2014 00:47 (преди над 9 години)

def series(type,number)
case type
- when 'fibonacci' then series_element_generator(1,1,number)
- when 'lucas' then series_element_generator(2,1,number)
- else summed(number)
+ when 'fibonacci' then series_element_generator(1,1,number)
+ when 'lucas' then series_element_generator(2,1,number)
+ when 'summed' then series('fibonacci',number) + series('lucas',number)
end
end
-def fibonacci(number)
- series_element_generator(1,1,number)
-end
-def lucas(number)
- series_element_generator(2,1,number)
-end
-
-def summed(number)
- fibonacci(number) + lucas(number)
-end
-
def series_element_generator(first,second,number)
- (number-1).times { first,second = second,first + second }
+ (number-1).times { first, second = second, first + second }
first
end
+

Иван обнови решението на 13.10.2014 22:19 (преди над 9 години)

-def series(type,number)
+def series(type, number)
case type
- when 'fibonacci' then series_element_generator(1,1,number)
- when 'lucas' then series_element_generator(2,1,number)
- when 'summed' then series('fibonacci',number) + series('lucas',number)
+ when 'fibonacci' then series_element_generator(1, 1, number)
+ when 'lucas' then series_element_generator(2, 1, number)
+ when 'summed' then series('fibonacci', number) + series('lucas', number)
end
end
-def series_element_generator(first,second,number)
- (number-1).times { first, second = second, first + second }
+def series_element_generator(first, second, number)
+ (number - 1).times { first, second = second, first + second }
first
end

Готино :) Последни много дребни забележки.

  • Оставяй по 1 ред между дефинициите на функциите.
  • Махни празните редове накрая.

  • Може да подравниш then-овете в case един под друг. Ето така:

    when 'happy'   then ..
when 'hungry'  then ..
when 'thirsty' then ..

    Това ти дава някакъв по-структуриран, табличен вид.

Иван обнови решението на 14.10.2014 16:18 (преди над 9 години)

def series(type, number)
case type
when 'fibonacci' then series_element_generator(1, 1, number)
- when 'lucas' then series_element_generator(2, 1, number)
- when 'summed' then series('fibonacci', number) + series('lucas', number)
+ when 'lucas' then series_element_generator(2, 1, number)
+ when 'summed' then series('fibonacci', number) + series('lucas', number)
end
end
-
def series_element_generator(first, second, number)
(number - 1).times { first, second = second, first + second }
first
-end
-
+end
-
-