Ruby: Posso utilizzare i metodi di istanza all'interno di un metodo di classe?

Question

Ho una classe che contiene questo metodo di una classe:

def self.get_event_record(row, participant) 
    event = Event.where(
     :participant_id => participant.id, 
     :event_type_code => row[:event_type], 
     :event_start_date => self.format_date(row[:event_start_date]) 
).first 

    event = Event.new(
     :participant_id => participant.id, 
     :event_type_code => row[:event_type], 
     :event_start_date => self.format_date(row[:event_start_date]) 
) if event.blank? 

    event 
end 

e ho anche, nella stessa classe, un metodo di istanza:

def format_date(date) 
    parsed_date = date.split('/') 

    # if month or day are single digit, make them double digit with a leading zero 
    if parsed_date[0].split("").size == 1 
    parsed_date[0].insert(0, '0') 
    end 
    if parsed_date[1].split("").size == 1 
    parsed_date[1].insert(0, '0') 
    end 

    parsed_date[2].insert(0, '20') 

    formatted_date = parsed_date.rotate(-1).join("-") 
    formatted_date 
end 

Ricevo un 'metodo non definito' errore per #format_date. (L'ho provato senza lo self davanti, all'inizio). Non puoi usare i metodi di istanza nei metodi di classe della stessa classe?

Answer 1

risposta è no, non è possibile utilizzare i metodi istanza di una classe all'interno di un metodo di classe unles s avete qualcosa di simile:

class A 
    def instance_method 
    # do stuff 
    end 

    def self.class_method 
    a = A.new 
    a.instance_method 
    end 
end 

Ma, per quanto posso vedere, format_date non deve essere un metodo di istanza. Quindi scrivere format_date come

def self.format_date(date) 
    # do stuff 
end 

Answer 2

Si potrebbe fare YourClassName.new.format_date(your_date), anche se penso che sia abbastanza chiaro si dovrebbe essere ristrutturazione del codice - questo metodo probabilmente non appartiene a un'istanza. Perché non estendi la classe Date o fai il format_date un metodo di classe sulla classe che stai utilizzando?

EDIT: Qui ci sono alcune altre cose a cui pensare con il tuo codice:

• Tutto il tuo metodo format_date va a un sacco di lunghezze per manipolare le date come stringhe. Perché non usare Ruby's Date Class? Utilizzando Date.parse o Date.strptime o addirittura "01/01/2001".to_date potrebbe essere utile a seconda del proprio locale

• considerare di estendere la classe String per il metodo, se si ha realmente bisogno per rendere il proprio metodo:

  class String 
      def to_friendly_formatted_date 
      Date.strptime(self, "%d/%m/%y") 
      end 
  end 
  "01/08/09".to_friendly_formated_date 
  

• Il vostro metodo di classe sta piangendo la nostra per i metodi helper find_or_initialize_by:

  self.get_event_record(row, participant) 
      find_or_initialize_by_participant_id_and_event_type_code_and_event_start_date(:participant_id => participant.id, :event_type_code => row[:event_type_code], :event_start_date => row[:event_start_date].to_friendly_formatted_date) 
  end 
  

da Dio è lungo, ma realizza quello che stai cercando di fare più elegante (anche se io sono aperto a tesi!)

Answer 3

Basta creare metodo di classe

def self.format_date (..) 
    ... 
end 

E se u bisogno metodo di istanza, delegato al metodo di classe

def format_date *args 
    self.class.format_date *args 
end 

E io non credo che sia una buona idea quella di chiamare i metodi di istanza dal campo di applicazione della classe