Решение на Пета задача от Александър Антов
Към профила на Александър Антов
Резултати
- 0 точки от тестове
- 0 бонус точки
- 0 точки общо
- 0 успешни тест(а)
- 0 неуспешни тест(а)
Код
История (4 версии и 1 коментар)
Александър обнови решението на 21.01.2014 23:51 (преди почти 11 години)
Александър обнови решението на 21.01.2014 23:52 (преди почти 11 години)
Александър обнови решението на 21.01.2014 23:55 (преди почти 11 години)
Имаш много проблеми с нарушаване на конвенциите в Ruby, неспазване на skeptic ограниченията и прочее. Сериозни проблеми с идентацията, излишни return
клаузи и прочее. За задача-ретроспекция това е лош знак, защото изглежда така, сякаш си отсъствал тялом и духом от курса. Това е ръководството по стил, което трябва да "изядеш с кориците" за защитата на проекта си: https://github.com/fmi/ruby-course-guides/blob/master/course_projects.md#readme
Отделно, не минаваш и тестовете на всички задачи, затова няма как да ти дам точки за тях.
Относно нещата, които си научил:
- По т. 1 трябва да кажа, че колкото и забавно да е това, не трябва да се ползва в production :)
- По т. 2, по-добре да се ползва
split('')
, а най-добре --chars
:) - По т. 5 - не го прави. Прегледността в контекста на Ruby изисква да го изпускаш.
- По т. 6 - всъщност, символът е
\
и ако ти се налага да разделяш един ред така на много редове, по-добре помисли за рефакторинг. - По 7 - ако изпуснеш
return
, няма да има нужда от скоби; като общо правило, ако ти се наложи да слагаш скоби, ползвай&&
и||
. - По 20 - не е съвсем еквивалентно :) Най-често връща броя елементи на колекция.
Харесва ми, че си научил неща с тази ретроспекция, надявам се и ти да си доволен от себе си. Като следващи стъпки, бих ти препоръчал да разгледаш още примерен код на колегите си – например, избери си три от решенията на пета задача, на които съм дал най-много точки и разгледай научените от тях неща и написаните от тях решения на задачите. Гарантирано ще ти е полезно.