Szyny - Dodaj atrybuty nie w modelu i atrybut modelu aktualizacji

Question

Mam 3 pola w mojej formie, których nie ma w mojej bazie danych: opening_type, opening_hours, opening_minutes. Chcę zaktualizować główny atrybut "otwarcie" (w bazie danych) tymi 3 polami.

Próbowałem wielu rzeczy, które nie działają.

Właściwie mam:

attr_accessor :opening_type, :opening_hours, :opening_minutes 

    def opening_type=(opening_type) 
    end 
    def opening_type 
    opening_type = opening.split("-")[0] if !opening.blank? 
    end 

    def opening_hours=(opening_hours) 
    end 
    def opening_hours 
    opening_hours = opening.split("-")[1] if !opening.blank? 
    end 

    def opening_minutes=(opening_minutes) 
    end 
    def opening_minutes 
    opening_minutes = opening.split("-")[2] if !opening.blank?  
    end 

Próbowałem dodając coś takiego:

def opening=(opening) 
    logger.info "WRITE" 

    if !opening_type.blank? and !opening_hours.blank? and opening_minutes.blank? 
     opening = "" 
     opening << opening_type if !opening_type.blank? 
     opening << "-" 
     opening << opening_hours if !opening_hours.blank? 
     opening << "-" 
     opening << opening_minutes if !opening_minutes.blank? 
    end 
    write_attribute(:opening, opening) 
    end 

    def opening 
    read_attribute(:opening) 
    end 

Ale metody dostępowe nie są nazywane i myślę opening_type, opening_hours, opening_minutes były puste zbyt jeśli dostępowych zostały nazwane ...

Myślę, że nie potrzebuję oddzwaniania before_save i powinienem to przepisać akcesorów.

Uwagi: - Rails 3.0.5, - opening_type,:,: opening_minutes opening_hours mogłyby być pusty

EDIT: I uaktualniony mój kod

Answer 1

Zauważ, że attr_reader, attr_writer i attr_accessor tylko makr do definiowania własnych metod.

# attr_reader(:foo) is the same as: 
def foo 
    @foo 
end 

# attr_writer(:foo) is the same as: 
def foo=(new_value) 
    @foo = new_value 
end 

# attr_accessor(:foo) is the same as: 
attr_reader(:foo) 
attr_writer(:foo) 

W tej chwili wasze metody setter nie robi nic szczególnego, więc jeśli po prostu przejść do attr_accessor Twój kod będzie czystsze.

Innym problemem jest to, że twoja metoda opening= nigdy nie jest wywoływana, a to ma sens, ponieważ w twoim kodzie nie ma nigdzie go wywoływać. To, czego naprawdę chcesz, to to, że twój otwór zostanie ustawiony po ustawieniu wszystkich poszczególnych części. Teraz nie ma trywialny sposób, aby to zrobić, ale Rails ma before_validation zwrotnego, gdzie można umieścić kod, który biegnie po wartości zostały ustawione ale przed walidacja działa:

class Shop < ActiveRecord::Base 

    attr_accessor :opening_type, :opening_hours, :opening_minutes 

    before_validation :set_opening 

    private 
    def set_opening 
    return unless opening_type && opening_hours && opening_minutes 
    self.opening = opening_type + "-" + opening_hours + "-" + opening_minutes 
    end 
end 

Answer 2

zamiast

attr_reader :opening_type, :opening_hours, :opening_minutes 

trzeba

attr_accessor :opening_type, :opening_hours, :opening_minutes 
attr_reader :opening_type, :opening_hours, :opening_minutes 

hf ...

// Są: opening_type,: opening_hours,: opening_minutes real fields? Jeśli tak, potrzebujesz tego?

attr_accessor: otwarcie attr_reader: otwarcie