Rubyのオブジェクト指向に関して

Question

class Team

  # インスタンスが持つ変数（値）
  attr_accessor :name, :win, :lose, :draw, 

  # インスタンスを初期化するための、特別なメソッド
  def initialize(name, win, lose, draw)
 end
 
 # インスタンスが持つメソッド（処理）
  def calc_win_rate
   return win.to_f/(win+lose)
  end
  
   def show_team_result()
    puts "#{self.name}の2020年の成績は#{self.win}勝#{self.lose}敗#{self.draw}分、勝率は#{self.calc_win_rate}"
  end
end

# インスタンスの生成と、変数への代入
giants = Team.new("Giants", 67, 45, 8)
tigers = Team.new("Tigers", 60, 53, 7)
dragons = Team.new("Dragons", 60, 55, 5)
baystars = Team.new("Baystars", 56, 58, 6)
carp = Team.new("Carp", 52, 56, 12)
swallows = Team.new("Swallows", 41, 69, 10)


# インスタンスの使用
giants.show_team_resul
tigers.show_team_result
dragons.show_team_result
baystars.show_team_result
carp.show_team_result
swallows.show_team_result
コード

Giants の2020年の成績は 67勝 45敗 8分、勝率は 0.5982142857142857です。
Tigers の2020年の成績は 60勝 53敗 7分、勝率は 0.5309734513274337です。
Dragons の2020年の成績は 60勝 55敗 5分、勝率は 0.5217391304347826です。
BayStars の2020年の成績は 56勝 58敗 6分、勝率は 0.49122807017543857です。
Carp の2020年の成績は 52勝 56敗 12分、勝率は 0.48148148148148145です。
Swallows の2020年の成績は 41勝 69敗 10分、勝率は 0.37272727272727274です。
と出力したいのですが
team.rb:22:in new': wrong number of arguments (given 4, expected 0) (ArgumentError) from team.rb:22:in <main>'
とエラー文が出ます。なぜでしょうか？よろしくお願いいたします。

Comments

[udon-ken]

コードは「コードの挿入」で記載してください。
そうしないとマークダウンと解釈されて訳分からない事になります（というかなってる）。

[attakei]

この質問と内容がほぼほぼ重複しています。このようなケースでは、既存の質問を編集してください
https://teratail.com/questions/352851

※参考
https://teratail.com/help/avoid-asking

[m.ts10806]

タイトルと内容の関連性がない

[tetrisuehara]

他の同様の投稿でもそうですが投稿者さんはあまりにも失礼じゃないですか？
ここにいる、特にちゃんと回答していただいている人は善意でやっているのにそれに対する返答もお礼もないですよね？
このサイトはあなたが通う学校じゃないんですよ？
中学生ぐらいでその程度の分別はわかりませんかね？

Answer 1

attr_accessor :name, :win, :lose, :draw,

末尾の,が余計です。

他に問題になりうるのはinitializeメソッドの中身がないことですね。

Struct使った方が楽です。

Team = Struct.new(:name, :win, :lose, :draw) do
  # インスタンスが持つメソッド（処理）
  def calc_win_rate
   return win.to_f/(win+lose)
  end

  def show_team_result
    puts "#{self.name}の2020年の成績は#{self.win}勝#{self.lose}敗#{self.draw}分、勝率は#{self.calc_win_rate}"
  end
end

Comments

[winterboum]

なんでエラーメッセージ載せてくれないのでしょうね

Answer 2

一つのclass に複数の同じ名前のmethodを書くと、最後のが有効になります。
ですから、みな同じinitializeをつかってます。
def initialize(name, win, lose, draw) とし
giants = Team.new("Giants", 67, 45, 8) の様にnewしましょう

Comments

[oppaiboinboin17]

# クラス定義
class Team

# インスタンスが持つ変数（値）
attr_accessor :name, :win, :lose, :draw,

# インスタンスを初期化するための、特別なメソッド
def initialize(name, win, lose, draw)
end

# インスタンスが持つメソッド（処理）
def calc_win_rate
return win.to_f/(win+lose)
end

def show_team_result()
puts "#{self.name}の2020年の成績は#{self.win}勝#{self.lose}敗#{self.draw}分、勝率は#{self.calc_win_rate}"
end
end

# インスタンスの生成と、変数への代入
giants = Team.new("Giants", 67, 45, 8)
tigers = Team.new("Tigers", 60, 53, 7)
dragons = Team.new("Dragons", 60, 55, 5)
baystars = Team.new("Baystars", 56, 58, 6)
carp = Team.new("Carp", 52, 56, 12)
swallows = Team.new("Swallows", 41, 69, 10)

# インスタンスの使用
giants.show_team_resul
tigers.show_team_result
dragons.show_team_result
baystars.show_team_result
carp.show_team_result
swallows.show_team_result

エラーが出るのですがなぜでしょうか？

[winterboum]

どんなエラーか書いていただかないとなんとも

[oppaiboinboin17]

team.rb:22:in `new': wrong number of arguments (given 4, expected 0) (ArgumentError)
from team.rb:22:in `<main>'
こちらがエラー文です。よろしくお願いいたします。

[winterboum]

２２行目ってどれ？

[oppaiboinboin17]

giants = Team.new("Giants", 67, 45, 8)が22行目です。よろしくお願いいたします。

[winterboum]

それと、code読みにくいので、質問を編集して<code>を使って書いてください

[oppaiboinboin17]

編集いたしました。よろしくお願いいたします。

[oppaiboinboin17]

引数のつけ方に問題があるようなのですがつけ方がわかりません。教えていただけないでしょうか？よろしくお願いいたします。

[winterboum]

で、22行ってどこ？

[oppaiboinboin17]

giants = Team.new("Giants", 67, 45, 8)です

[winterboum]

いま 質問欄にあるままのcodeでしたら、wrong number of arguments (given 4, expected 0) にはならないと思うのですが。
別のエラーがでるとは思うが

[asm]

> wrong number of arguments (given 4, expected 0) にはならないと思うのですが。
なりますよ
def initialize~が:initializeとなるので
attr_accessor ~,
:initialize
となって、
def initialize;@initialize;endが上書きされています

[winterboum]

ああ、なる

[winterboum]

ちょっと疑って、typoしてないかは見たのですが、上の行までは気が付かなかった

Answer 3

以下のようにしてください。

class Team

  # インスタンスが持つ変数（値）
  attr_accessor :name, :win, :lose, :draw

  # インスタンスを初期化するための、特別なメソッド
  def initialize(name, win, lose, draw)
    @name = name
    @win = win
    @lose = lose
    @draw = draw
  end

 # インスタンスが持つメソッド（処理）
  def calc_win_rate
   return win.to_f/(win+lose)
  end

  def show_team_result()
    puts "#{self.name}の2020年の成績は#{self.win}勝#{self.lose}敗#{self.draw}分、勝率は#{self.calc_win_rate}"
  end
end

# インスタンスの生成と、変数への代入
giants = Team.new("Giants", 67, 45, 8)
tigers = Team.new("Tigers", 60, 53, 7)
dragons = Team.new("Dragons", 60, 55, 5)
baystars = Team.new("Baystars", 56, 58, 6)
carp = Team.new("Carp", 52, 56, 12)
swallows = Team.new("Swallows", 41, 69, 10)

# インスタンスの使用
giants.show_team_result
tigers.show_team_result
dragons.show_team_result
baystars.show_team_result
carp.show_team_result
swallows.show_team_result