На мен ми се струва, че работи. Какво точно се случва?

Ред 21. Гледам го. Трябва да работи. Очаквам да работи. Не работи. Ще се радвам на коментар или насока по въпроса.

Сега на втори поглед го гледам и това с unless не ми изглежда ок, не работи и с ? накрая, нито с класически if .. then .. end . Със сигурност ми е пред очите, но не го виждам.

Тестовете са ми в Github.

Това ми е теста

it 'contains only unique numbers' do
    set = NumberSet.new << 1 << 1 #<< Rational(2,2)
    expect(set.size).to eq 1
  end 

Така гърми.

От тестовете, които не си коментирала виждам проблем на 33 ред set.incl(2). Нямаш incl метод в NumberSet.

Също така, за да можеш да правиш така numbers << 1 << 2 << 3, NumberSet#<< метода ти трябва да връща self. Това ще позволи такова chain-ване на извиквания на <<.

expect(NumberSet.new<<1).to eq [1] не е ясно колко коректен тест е тъй като резултатът от NumberSet#<< може да не е Array. По скоро бих го тествал с include от колкото с eq.

Да, това е останало от дебъг. Пушнах актуална версия на теста. Методът не работи и така

  def <<(number)
    if !(@number_set.include?(number)) then @number_set << number end
  end

EDIT: куул, това с чейнването не го знаех. Ще пробвам тъй. Благодаря!

EDIT 2: Стана. Много благодаря!

Пробвай с

def <<(number)
  @number_set << number unless @number_set.include?(number)
  self
end

Все още е work in progress. Не знам доколко е ок да задавам въпрос в коментар към недовършено решение - но ще опитам. Имам чувството, че копая в съвсем грешна посока.

Опитвам се да направя следното:

Вариант 1:

Всички филтри при инициализирането си връщат съответна анонимна функция, която [] да ползва като критерий за филтриране. Операторите & и | се предефинират, но като цяло [] работи с обединение /сечение на два блока като аргумент.

В този вариант не ми е ясно как да взема блока, с който е извикан Filter.new {this_block} и да го върна като резултат на initialize. С yield мога да изпълня блока, block_given? дава дали го има, но самият той има ли как да се вземе като функция/lambda, block/ от вътре в метода initialize ?

Вариант 2:

class Filter
  def initialize
    filtered_set = NumberSet.new
    @number_set.each {|current| filtered_set << current if yield current}
    filtered_set
  end
end

За това не ми е ясно как ще взема @number_set, която не е в същия клас.

Някаква насока – разбира се, колкото прецените, че е честно : ) ?

Опитът ти е успешен.

Вариант едно отпада. Без значение какво връщаш от initialize, не забравяй, че той се вика от метода new, който винаги връща нов обект - инстанция на класа, за който е извикан. Кодът за new в интерпретатора е нещо от сорта на:

VALUE method_call_new(VALUE klass, int argc, VALUE *args) {
    VALUE object = create_new_object(klass);
    invoke_initialize(object, argc, args);
    return object
}

Като изключим жалкия ми опит да пиша C, искам да се фокусираш върху invoke_initialize извикването. Това е твоя initialize метод. Това, което искам да илюстрирам е, че резултатът, който той връща се игнорира и new връща новосъздадения обект. Това прави ли смисъл?

Скачаме на вариант две. Filter#initialize трябва да приема блок. Кодът, който ти си написала е по-скоро имплементацията на NumberSet#[]. Там трябва да стои създаването на нов NumberSet и пълненето му с елементи на базата на филтъра. Филтрите трябва да могат да кажат за едно число - минава ли филтъра, или не. Толкова. :)

Благодаря за бързия коментар.

Тъй като това е първото парче С, което виждам в живота си, няма нужда да го окачествяваш като жалък опит :D. Моля, не стреляй по пианиста.

На четвъртия път, в който ти прочетох коментара, разбрах горе-долу какво трябва да прави филтъра. Със сигурност не се пише така – основно понеже нищо от написаното не работи.

Ако мога да вметна въпрос

„Филтрите трябва да могат да кажат за едно число - минава ли филтъра, или не. Толкова. :) ”

Това число като аргумент ли трябва да им се зададе? Защото извикването им със синтаксиса от условието numbers[SignFilter.new(:non_negative) & Filter.new { |number| number.even? }] не мога да си представя как точно ще си го вземе.

Чувствам се длъжна да кача последната (неработеща) версия на задачата, преди да отида на работа и дедлайна да се хлопне.

В момента се опитваш да връщаш резултат от initialize. Направи си един метод във филтъра, който се казва accepts?(number) / satisfied_by?(number) / met_by?(number). Този метод ще ни казва дали number удовлетворява филтъра. Съответно в NumberSet#[] ще разчиташ на този метод вместо на call. :)