Решение на Девета задача от Огнян Ангелов

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

Към профила на Огнян Ангелов

Резултати

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

Код

REPOSITORY = 'http://github.com/MightyPixel/ruby-retrospective-2015-1'
# Ретроспекция:
# 1-4 Задача
# Задача 1.
# 1. Променям функциите си да връщат нещо четимо и ясно
# 2. Преименувам аргументите на compare_prices да са по-стандартни
# Задача 2.
# 3. Ползвам between? функцията вместо две проверки: > и <=
# 4. За намиране на свободно мято вместо анонимна функция използвам "-"
# 5. Използвам "(0...x)" вместо "(0..x)"
# 6. Отделям функция "inside?"
# Задача 3.
# 7. Смених if ! на unless
# 8. Използвам range.each вместо while цикъл в FibonacciSequence
# 9. Преименувах previous, current на current, following
# 10. Рефакторирах iterate_rational
# 11. Добавих паралелно присвояване в each метода на RationalSequence
# 12. Подобрих стила като махнах ненужните редове и преместих блок
# аргумента на същия с този на блока
# Задача 4.
# 13. Замених max_by вместо max
# 14. Разделих функционалността на compare_cards в два метода
# 15. Премахвам префикса get_ от get_hand и get_hand_size
# 16. Премахнах ненужните редове
# 17. Добавих нужните спейсове съгласно ръководството за стил
# 18. Същото като 12
# 19. Замених двойни с единични кавички
# 20. Смених стила на обграждащия блок в suite_contains_ranks с do-end
# 21. Премахнах ненужните скоби и return-и

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

From https://github.com/fmi/ruby-retrospective-2015-1
 * branch            master     -> FETCH_HEAD
HEAD is now at 767dd8d Update the task name in the readme for clarity
Cloning into 'submission'...
HEAD is now at 30275f1 Fix rubocop offenses.
From /tmp/ruby-retrospective-2015-1/checker
 * branch            master     -> FETCH_HEAD
 * [new branch]      master     -> upstream/master

Changes URL:
http://github.com/MightyPixel/ruby-retrospective-2015-1/compare/767dd8dfe46...30275f13ed8

‘solutions/04.rb’ -> ‘/tmp/ruby-retrospective-2015-1/checker/solutions/04.rb’
‘solutions/02.rb’ -> ‘/tmp/ruby-retrospective-2015-1/checker/solutions/02.rb’
‘solutions/03.rb’ -> ‘/tmp/ruby-retrospective-2015-1/checker/solutions/03.rb’
‘solutions/01.rb’ -> ‘/tmp/ruby-retrospective-2015-1/checker/solutions/01.rb’
OK
........

Finished in 0.00481 seconds
8 examples, 0 failures
OK
....................

Finished in 0.01105 seconds
20 examples, 0 failures
OK
....................

Finished in 0.31432 seconds
20 examples, 0 failures
OK
.........................................................

Finished in 0.02741 seconds
57 examples, 0 failures
From https://github.com/fmi/ruby-homework
 * branch            master     -> FETCH_HEAD
HEAD is now at 9dd040c Modify a test in task 8 to not include empty cells
.

Finished in 0.0017 seconds
1 example, 0 failures

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

Огнян обнови решението на 27.01.2016 20:14 (преди около 9 години)

+REPOSITORY = 'http://github.com/MightyPixel/ruby-retrospective-2015-1'
+
+# Ретроспекция:
+# 1-4 Задача
+
+# Задача 1.
+# 1. Променям функциите си да връщат нещо четимо и ясно
+# 2. Преименувам аргументите на compare_prices да са по-стандартни
+
+# Задача 2.
+# 3. Ползвам between? функцията вместо две проверки: > и <=
+# 4. За намиране на свободно мято вместо анонимна функция използвам "-"
+# 5. Използвам "(0...x)" вместо "(0..x)"
+# 6. Отделям функция "inside?"
+
+# Задача 3.
+# 7. Смених if ! на unless
+# 8. Използвам range.each вместо while цикъл в FibonacciSequence
+# 9. Преименувах previous, current на current, following
+# 10. Рефакторирах iterate_rational
+# 11. Добавих паралелно присвояване в each метода на RationalSequence
+# 12. Подобрих стила като махнах ненужните редове и преместих блок
+# аргумента на същия с този на блока
+
+# Задача 4.
+# 13. Замених max_by вместо max
+# 14. Разделих функционалността на compare_cards в два метода
+# 15. Премахвам префикса get_ от get_hand и get_hand_size
+# 16. Премахнах ненужните редове
+# 17. Добавих нужните спейсове съгласно ръководството за стил
+# 18. Същото като 12
+# 19. Замених двойни с единични кавички
+# 20. Смених стила на обграждащия блок в suite_contains_ranks с do-end
+# 21. Премахнах ненужните скоби и return-и