【Ruby】array を使ってソートした時に違う順番でソートされてしまう

Question

4台でレースをして順番を車種名で表示させたいです。

前回の質問で解決したと思ったのですが
コードをもっとオブジェクト指向っぽく書き換えたらなぜか、順位順（走行時間が小さい順）にソートできなくなってしまいました。

スクショ1枚目はarr.sort!で並び替えて何の数字か分からないものが出力された図
スクショ2枚目はarr.sort_by{|x| x.time}.map(&:type)でソートしても早い順にならなかった図。
レース時間である.time の計算式はcar.rbで定義しています。
自分としては arr メソッドが違うのかなぁとは思うのですが、間違いお気付きの方ご指摘お願いいたします。

require_relative 'car'
require_relative 'ferrari'
require_relative 'honda'
require_relative 'nissan'
require_relative 'toyota'

def main       #(type, speed, price, capacity, height, passenger)
  h = Honda.new("Honda", 100, 100, 8, 100, 1)
  n = Nissan.new("Nissan", 120, 50, 5, 100, 1)
  f = Ferrari.new("Ferrari", 200, 2000, 2, 100, 1)
  t = Toyota.new("Toyota", 100, 300, 2, 100, 1)
  
  output_information(h,n,f,t)

  puts '-----順位-------'
  arr = [h, n, f, t]
  puts arr.sort_by{|x| x.time}.map(&:type)
end

def output_information(h,n,f,t)
  puts "\t\t#{h.type}\t\t#{n.type}\t\t#{f.type}\t\t#{t.type}"
  puts "加速\t\t#{h.speed}\t\t#{n.speed}\t\t#{f.speed}\t\t#{t.speed}"
  puts "価格\t\t#{h.price}\t\t#{n.price}\t\t#{f.price}\t\t#{t.price}"
  puts "定員\t\t#{h.capacity}\t\t#{n.capacity}\t\t#{f.capacity}\t\t#{t.capacity}"
  puts "車高\t\t#{h.height}\t\t#{n.height}\t\t#{f.height}\t\t#{t.height}"
  puts "乗員\t\t#{h.passenger}\t\t#{n.passenger}\t\t#{f.passenger}\t\t#{t.passenger}"
  puts "走行時間\t\t#{h.time}\t\t#{n.time}\t\t#{f.time}\t\t#{t.time}"
end

if __FILE__ == $0
  main
end

# car.rb
# parent class
class Car
  attr_accessor :type, :speed, :price, :capacity, :height, :passenger, :distance, :brake_speed
  
  def initialize(type, speed,  price, capacity, height, passenger)
    @type = type
    @speed = speed
    @price = price
    @capacity = capacity
    @height = height
    @passenger = passenger
  end
  
  def equipment
    %w[アクセル ブレーキ]
  end
  
  def time
  #レースタイムを算出。
  @distance = 10000 #走行距離
  @brake_speed = @speed/2 #減速時の速度
  random = Random.new
  @brake_time = random.rand(1..5) #今回はブレーキを踏む回数を1〜5回でランダム設定
  @distance / (@brake_speed*@brake_time) + (@distance-@brake_speed*@brake_time) / @speed
  #タイム算出の計算式・・・ 減速時に進んだ時間 + 通常時に進んだ時間の和
  end
end

# ferrari.rb
class Ferrari < Car
end

# honda.rb
class Honda < Car
end

# nissan.rb
class Nissan < Car
end

# toyota.rb
class Toyota < Car
  # トヨタは価格と速度が比例する
  def initialize(type, speed,  price, capacity, height, passenger)
    super(type, price*0.8,  price, capacity, height, passenger) 
  end
end

【追記①】

car.rb で @race_time を使って定義しました。（走行時間を保存して出力するように定義）
実行ファイル ( main.rb ) のarray で x.race_time で出力するとうまくいく（とりあえず5回連続成功）

require_relative 'car'
require_relative 'ferrari'
require_relative 'honda'
require_relative 'nissan'
require_relative 'toyota'

def main       #(type, speed, price, capacity, height, passenger)
  h = Honda.new("Honda", 100, 100, 8, 100, 1)
  n = Nissan.new("Nissan", 120, 50, 5, 100, 1)
  f = Ferrari.new("Ferrari", 200, 2000, 2, 100, 1)
  t = Toyota.new("Toyota", 100, 300, 2, 100, 1)

  output_information(h,n,f,t)

  puts '-----順位-------'
  arr = [h, n, f, t]
  puts arr.sort_by{|x| x.race_time}.map(&:type)
end

def output_information(h,n,f,t)
  puts "\t\t#{h.type}\t\t#{n.type}\t\t#{f.type}\t\t#{t.type}"
  puts "加速\t\t#{h.speed}\t\t#{n.speed}\t\t#{f.speed}\t\t#{t.speed}"
  puts "価格\t\t#{h.price}\t\t#{n.price}\t\t#{f.price}\t\t#{t.price}"
  puts "定員\t\t#{h.capacity}\t\t#{n.capacity}\t\t#{f.capacity}\t\t#{t.capacity}"
  puts "車高\t\t#{h.height}\t\t#{n.height}\t\t#{f.height}\t\t#{t.height}"
  puts "乗員\t\t#{h.passenger}\t\t#{n.passenger}\t\t#{f.passenger}\t\t#{t.passenger}"
  puts "走行時間\t\t#{h.time}\t\t#{n.time}\t\t#{f.time}\t\t#{t.time}"
end

if __FILE__ == $0
  main
end

# car.rb
# parent class
class Car
  attr_accessor :type, :speed, :price, :capacity, :height, :passenger, :distance, :brake_speed, :race_time

  def initialize(type, speed,  price, capacity, height, passenger)
    @type = type
    @speed = speed
    @price = price
    @capacity = capacity
    @height = height
    @passenger = passenger
  end

  def equipment
    %w[アクセル ブレーキ]
  end

  def time
  #レースタイムを算出。
  @distance = 10000 #走行距離
  @brake_speed = @speed/2 #減速時の速度
  random = Random.new
  @brake_time = random.rand(1..5) #今回はブレーキを踏む回数を1〜5回でランダム設定
  @race_time = @distance / (@brake_speed*@brake_time) + (@distance-@brake_speed*@brake_time) / @speed #タイム算出の計算式・・・ 減速時に進んだ時間 + 通常時に進んだ時間の和
  # @race_time を使って下記の 「def race_time」で出力できるように
  end
  
  def race_time
    return @race_time
  end
end

【追記②】
①の方法だと def time で走行時間を計算しないと def race_time で計算できないという制約が生じてしまう。
def time の中に 走行時間を保存しようとしたが上手く行かず。。。

require_relative 'car'
require_relative 'ferrari'
require_relative 'honda'
require_relative 'nissan'
require_relative 'toyota'

def main       #(type, speed, price, capacity, height, passenger)
  h = Honda.new("Honda", 100, 100, 8, 100, 1)
  n = Nissan.new("Nissan", 120, 50, 5, 100, 1)
  f = Ferrari.new("Ferrari", 200, 2000, 2, 100, 1)
  t = Toyota.new("Toyota", 100, 300, 2, 100, 1)

  output_information(h,n,f,t)

  puts '-----順位-------'
  arr = [h, n, f, t]
  puts arr.sort_by{|x| x.time}.map(&:type) # 保存した走行時間をソートして表示。上手く行かない。
end

def output_information(h,n,f,t)
  puts "\t\t#{h.type}\t\t#{n.type}\t\t#{f.type}\t\t#{t.type}"
  puts "加速\t\t#{h.speed}\t\t#{n.speed}\t\t#{f.speed}\t\t#{t.speed}"
  puts "価格\t\t#{h.price}\t\t#{n.price}\t\t#{f.price}\t\t#{t.price}"
  puts "定員\t\t#{h.capacity}\t\t#{n.capacity}\t\t#{f.capacity}\t\t#{t.capacity}"
  puts "車高\t\t#{h.height}\t\t#{n.height}\t\t#{f.height}\t\t#{t.height}"
  puts "乗員\t\t#{h.passenger}\t\t#{n.passenger}\t\t#{f.passenger}\t\t#{t.passenger}"
  puts "走行時間\t\t#{h.time}\t\t#{n.time}\t\t#{f.time}\t\t#{t.time}"
end

if __FILE__ == $0
  main
end

# car.rb
# parent class
class Car
  attr_accessor :type, :speed, :price, :capacity, :height, :passenger, :distance, :brake_speed, :race_time

  def initialize(type, speed,  price, capacity, height, passenger)
    @type = type
    @speed = speed
    @price = price
    @capacity = capacity
    @height = height
    @passenger = passenger
  end

  def equipment
    %w[アクセル ブレーキ]
  end

  def time
  #レースタイムを算出。
  @distance = 10000 #走行距離
  @brake_speed = @speed/2 #減速時の速度
  random = Random.new
  @brake_time = random.rand(1..5) #今回はブレーキを踏む回数を1〜5回でランダム設定
  @race_time = @distance / (@brake_speed*@brake_time) + (@distance-@brake_speed*@brake_time) / @speed #タイム算出の計算式・・・ 減速時に進んだ時間 + 通常時に進んだ時間の和
  return @race_time #走行時間を保存
  end
end

【追記③】
car.rb で if 〜 return 〜 を使ったが走行時間とソートが合わず。。。

追記②と同じ

# car.rb
# parent class
class Car
  attr_accessor :type, :speed, :price, :capacity, :height, :passenger, :distance, :brake_speed, :race_time

  def initialize(type, speed,  price, capacity, height, passenger)
    @type = type
    @speed = speed
    @price = price
    @capacity = capacity
    @height = height
    @passenger = passenger
  end

  def equipment
    %w[アクセル ブレーキ]
  end

  def time #レースタイムを算出
  @distance = 10000 #走行距離
  @brake_speed = @speed/2 #減速時の速度
  random = Random.new
  @brake_time = random.rand(1..5) #今回はブレーキを踏む回数を1〜5回でランダム設定
  @race_time = @distance / (@brake_speed*@brake_time) + (@distance-@brake_speed*@brake_time) / @speed #タイム算出の計算式・・・ 減速時に進んだ時間 + 通常時に進んだ時間の和
    if @race_time > 0 #これだとソートが合わない。
    return @race_time
    end
  end
end

追記④ ようやく上手くいった

# main.rb
require_relative 'car'
require_relative 'ferrari'
require_relative 'honda'
require_relative 'nissan'
require_relative 'toyota'

def main #(type, speed, price, capacity, height, passenger)
  h = Honda.new("Honda", 100, 100, 8, 100, 1)
  n = Nissan.new("Nissan", 120, 50, 5, 100, 1)
  f = Ferrari.new("Ferrari", 200, 2000, 2, 100, 1)
  t = Toyota.new("Toyota", 100, 300, 2, 100, 1)

  output_information(h,n,f,t) # ここで走行時間が計算されて保存される。car.rb の 「def time」 のelseで走行時間が計算される

  puts '-----順位-------'
  arr = [h, n, f, t]
  puts arr.sort_by{|x| x.time}.map(&:type) # 保存した走行時間をソートして表示。 上記の「output_information(h,n,f,t)」で走行時間が既に計算されてるからここではその計算結果が表示される
end

def output_information(h,n,f,t)
  puts "\t\t#{h.type}\t\t#{n.type}\t\t#{f.type}\t\t#{t.type}"
  puts "加速\t\t#{h.speed}\t\t#{n.speed}\t\t#{f.speed}\t\t#{t.speed}"
  puts "価格\t\t#{h.price}\t\t#{n.price}\t\t#{f.price}\t\t#{t.price}"
  puts "定員\t\t#{h.capacity}\t\t#{n.capacity}\t\t#{f.capacity}\t\t#{t.capacity}"
  puts "車高\t\t#{h.height}\t\t#{n.height}\t\t#{f.height}\t\t#{t.height}"
  puts "乗員\t\t#{h.passenger}\t\t#{n.passenger}\t\t#{f.passenger}\t\t#{t.passenger}"
  puts "走行時間\t\t#{h.time}\t\t#{n.time}\t\t#{f.time}\t\t#{t.time}"
end

if __FILE__ == $0
  main
end

# car.rb
# parent class
class Car
  attr_accessor :type, :speed, :price, :capacity, :height, :passenger

  def initialize(type, speed,  price, capacity, height, passenger)
    @type = type
    @speed = speed
    @price = price
    @capacity = capacity
    @height = height
    @passenger = passenger
  end

  def equipment
    %w[アクセル ブレーキ]
  end

  def time #レースタイムを算出
  @distance = 10000 #走行距離
  @brake_speed = @speed/2 #減速時の速度
  random = Random.new
  @brake_time = random.rand(1..5) #今回はブレーキを踏む回数を1〜5回でランダム設定
  
    if @time # @time が存在するならその時間を表示（1回目は存在しない）
    @time 
    else # @time が存在しないなら計算する → 1回目だけ計算して2回目以降は計算結果だけでソートできるようにしてる
    @time = @distance / (@brake_speed*@brake_time) + (@distance-@brake_speed*@brake_time) / @speed #タイム算出の計算式・・・ 減速時に進んだ時間 + 通常時に進んだ時間の和
    end
  end
end

Answer 1

勘違いですね。
timeは呼び出す度に計算しているので、表を表示したときのtimeとソートしたときのtimeは別の値です。

初回のみ計算してその値を保存し、2回目以降は保存した値を返すようにすれば、やりたいことと合うでしょう。

Comments

[kazuki0714]

アドバイスありがとうございます。
保存して値を返すとは `return` を使えばよろしいでしょうか？
returnで色々試したのですが上手くソートできずでして。

他のメソッドを使うべき場合、ヒントを教えていただければ幸いです。

[otn]

> 初回のみ計算してその値を保存し、

インスタンス変数に保存します。

[kazuki0714]

追記①にインスタンス変数使ったコードを追加しました。
とりあえず5回くらいテストして上手くいったのですが合ってますでしょうか？

[otn]

間違ってはいないですが、これだとかならずtimeを呼び出してからしかrace_timeを呼べません。

普通は、def timeの中で初回だけ計算して、2回目以降の呼び出しでは計算せずに返すという処理にして、timeを呼び出します。

[kazuki0714]

追記②と③で試したのですが走行時間と順位が合わず上手くいきませんでした。
他に方法ありますでしょうか？

[otn]

def time
if @time
@time
else #1回目
計算して@timeに代入する
end
end

[kazuki0714]

できました！
追記④の `main.rb` の`def main` と `car.rb` の `def time` にコメントで自分なりの解釈書いたのですが認識は合ってますでしょうか？

[otn]

合ってますが、計算は全部elseの中に入れた方が良いですね。

[kazuki0714]

確かにそっちのほうが分かりやすいですね。
ありがとうございます、ようやく自分のやりたいコードになりました！