Week7

[Zvrbsky]

No description provided.

[piotrimbierowicz]

Trochę pachnie mi to programowaniem strukturalnym opakowanym w modele. Tzn. klasa Event musi wiedzieć wszystko o tym jak działają zamówienia, żeby móc zrealizować tę metodę. Zostawię tu propozycję tego jak ja bym to zrobił, ale nie trzeba się do tego stosować. Głównie chodzi o to, że metody mają tylko jedną odpowiedzialność i nie wiedzą za dużo o całym procesie.

class Order

def items_available?
  order_item.all? { |item| item.available? } # przydatna metoda https://ruby-doc.org/core-2.5.1/Enumerable.html#method-i-all-3F
end

def decrease_tickets_amount!
  order_item.each do |item|
    item.decrease_amount!
end

class OrderItem

def available?
  (event.amount - amount) >= 0
end

def decrease_amount!
  event.decrease_amount_by!(amount)
end

class Event

def decrease_amount_by!(number)
  update!(amount: amount - number)
end

def do_attachment
  if @order.items_available?
    @order.decrease_tickets_amount!
    @order.update owner: current_user
  else
     # ta część tak smao jak u Pana

[piotrimbierowicz]

if cat.blank? obsłuży zarówno cat == '' jak i cat.nil?

[piotrimbierowicz]

👍

[piotrimbierowicz]

Fajne, obiektowe programowanie. Każdy obiekt ma określoną odpowiedzialność.