LocalJumpError (Ruby on Rails)

Question

Ich habe gesucht/gegoogled, aber ich habe mit dem folgenden Problem zu kämpfen.

Ich baue eine Rails 2.3.2 Anwendung und eine der Anforderungen ist es, den Median einer Reihe von Ergebnissen zu berechnen. Ich benutze Code für die Berechnung des Medians aus dem Ruby Cookbook, laufe aber weiter auf ein Problem mit dem Fehler 'LocalJumpError - no block given', wenn ich versuche, den Median eines Arrays mit einer ungeraden Anzahl von Mitgliedern zu finden.

Das Codebeispiel meiner Meinung nach ist wie folgt:

<%= survey_response.median([6,4,5,4,4,2]) %> 

dann in survey_response.rb Modellen sind die Methoden, wie folgt:

def mean(array) 
    array.inject(array.inject(0) { |sum, x| sum += x }/array.size.to_f) 
end 

def median(array,already_sorted=false) 
    return nil if array.empty? 
    array = array.sort unless already_sorted 
    m_pos = array.size/2 
    return array.size % 2 == 1 ? array[m_pos] : mean(array[m_pos-1..m_pos])  
end 

Der Fehler wird verursacht, wenn die Median-Methode zurück verweist zu der mittleren Methode, um das Medium mit einer ungeraden Anzahl von Elementen im Array zu erhalten. Ich kann einfach nicht herausfinden, warum ich diesen Fehler bekomme oder wie ich ihn beheben kann - also würde ich jede Hilfe/Anleitung/Lachen, die jemand mir anbieten könnte, sehr schätzen!

Dank

Simon

Answer 1

Lis sieht aus wie es ist wegen Ihnen einen Fractional-Index in das Array verwendet wird. Versuchen Sie ersetzen:

m_pos = array.size/2 

mit:

m_pos = (array.size/2) .ceil

Versuchen Sie auch, Ihre mittlere Funktion dieser Wechsel:

def mean(array) 
    array.inject(0) { |sum, x| sum += x }/array.size.to_f 
end 

Answer 2

Bessere Code :

def mean(array) 
    array.inject { |sum, n| sum + n }/array.length.to_f 
end 

def median(array) 
    return nil if array.empty? 
    array.sort! 
    middle = array.length/2 
    (array.length % 2 == 1) ? array[middle] : mean([array[middle-1], array[middle]]) 
end 

puts median([5,11,12,4,8,21]) # => 9.5 

Answer 3

Das mean Methode sieht schrecklich verpfuscht. Versuchen Sie dies:

 
def mean(array) 
    a.inject(0) { |sum,x| sum += x }/a.size.to_f 
end