svetlik/core-ruby-1

The repitory for the Core Ruby course starting October 2014

core-ruby-1's Issues

Week 2: Tracks

Имаш още работа по задачата за траковете. Бутни ме тук (може да го направиш като напишеш коментар в който споменяваш @gsamokovarov) като я довършиш.

Занаятчийски идеи: не къмитвай код в който имаш дебъг информация (тези хвърчащи p-та).

Week 1: FMI Tribute

Тази задача ми харесва. Една малка забелешка, която имам: ползвал си Array#map за страничен ефект в Array#index_by. В този случай, e по-добре да ползваш Array#each.

Week 1: Hashes

Опитай се а имплементираш не-bang методите, чрез bang методите. Виж какво ще излезе :)

Week 2: Vectors

Сложи няколко аксесора на Vector3D/Vector2D класа, който онаследява от генеричният Vector. Иначе всичко е наред ;)

Week 0: Standard Types

Като цяло ми харесват решенията ти. Проблемите, които имаш са дребни.

Малко проблеми при форматирането, които може да решиш като пуснеш rubocop. Като цяло избягвай експлицитен then в if.

Това също, бих го счел като форматираща грешка, но някой оператори ти връщат истина или лъжа, няма смисъл да го проверяваш отново. Вкарва шум в кода. Примерът е:

def anagram?(word, other)
  histogram(word) == histogram(other) ? true : false
end

Имаш грешка в решението на remove_prefix, защото не проверяваш дали префикса наистина съществува. Може да си напишеш тест, в които се опитваш да премахнеш префикс, който не се среща в текста.

Week 1: Enumerable

И тази задача ми харесва. Малки забележки по често срещаните двойни отрицания: ... unless yield(element) != false, ... unless item != element. Мисля, че повечето може да го обърнеш до позитивни if проверки.

На няколко места сравняваш с true (... == true). Избягвай такива провекри. Погледни този слайд защо.