Здравейте,
Четвъртото предизвикателство е публикувано. Примерен тест, както обикновено, можете да откриете в репото в Github.
Питайте тук, ако има нещо неясно.
Приятно решаване 
!
П.П. Надявам се този път да спазвате конвенциите за добър стил на кода...
partition -5 #=> []
т'ва ок ли е за тестовете?
@Иван, очаква се масивът да съдържа всички двойки естествени числа, чийто сбор е равен на number. Можеш да се досетиш колко са на брой тези двойки за отрицателен number. :)
изтрит
@Ангел Венчев, сумата на числата във вътрешените масиви трябва да се развнява на подаденото число на partition.
Моя грешка, не бях доразбрал условието
мерси @Николай Хубанов
Въпрос за тестовете: защо това работи:
expect { partition(-10) }.to raise_error
а това не работи:
partition(-1).should raise_error
?
Ако е въпрос на синтактична разлика м/у rspec 1 и 2 защо останалите тестове с should работят?
@Алекс, https://www.relishapp.com/rspec/rspec-expectations/v/2-6/docs/built-in-matchers/raise-error-matcher, осмисли написаното там, и пиши ако (не) ти е станало ясно :)
Замисли се все пак дали това е подходящото поведение на partition, когато му е подаден отрицателен аргумент.
@Петко - мерси за линка, явно това е просто ситнаксиса на rspec 2. Въпросът ми се поради от факта, че на някои места видях варианта в should, но там си беше казано, че това е за rspec 1. Иначе за поведението - ползвал съм ArgumentError със стринг, но не исках да поствам прекалено конкретно, че нали не е изтекъл срока. Според мен е коректно да raise-ва error/exception (не знам все още каква точно е разликата м/у двете и кога кое се ползва), защото по дефиниця естествени числа са или всички положителни цели числа или всички не-отрицателни цели числа. Подавайки му отрицателно число подаваш грешен аргумент.
Във условието пише, че приема за аргумент цяло число и връща двойки естествени числа.
@Алекс, Методи ти го каза ¯\_(ツ)_/¯.
Иначе за rspec, не си вникнал в това, което авторът е казал. Нека ти насоча вниманието...
...Use the
raise_errormatcher to specify that a block of code raises an error...
Искам да допълня казаното от Петко - налага се да сложите в блок кода, който искате да проверите, че хвърля изключение. Това не е просто синтаксис.
Въпреки, че за изключения все още не сме говорили, обяснението е много просто и се корени в реда на изпълнение на кода. Това:
partition(-1).should raise_error
...няма да работи, просто защото извикваш should на резултата от изпълнението на partition(-1). Тоест, още преди да си казал should, partition е извикан и е предизвикал изкючение, заради което и тестът ти ще гръмне.
Затова пък тук expect се възползва от блоковете в Руби, за да отложи изпълнението на парче код. Кодът се парсва и се вика to на резултата от изпълнението на expect. Блокът се изпълнява със закъснение, като е обгърнат с клаузи, които прихващат определени изключения:
expect { partition(-10) }.to raise_error
partition 0 => [[0, 0]] нали ?
@Методи - мерси, не бях прочел достатъчно внимателно.
@Петко - бях чел внимателно, но не го бях разбрал. :)
@Митьо - евалата, след това обяснение го зацепих.
@Иван - виж примерните тестове.
В предизвикателствата има ли изискванията на skeptic ? проблем ли е ако имам 82 символа на 1 ред ?
@Ангел, това, дали ще прекарваме решенията през skeptic, не трябва да е фактор, на базата на който да решаваш дали да спазваш конвенциите за добър стил на кода.
Трябва да сте влезли в системата, за да може да отговаряте на теми.