Ruby on RailsでのJSON形式エラーでの解決方法

Question

お世話になります。

現在Ruby on Rails6で
掲示板アプりを作ろうとしているのですが
投稿メッセージを入力して送信しようとすると
JSONによる変換がうまく行かないのか
エラーが出ます。
@msg_data = JSON.parse(data)
という箇所でエラーが出ます。

エラー分を訳するかぎり
ハッシュから文字列への変換がうまく行ってない
ということぐらいはわかるのですが
どうすればいいか分からず、
すみませんがご教示ください。

発生している問題・エラーメッセージ

TypeError in MsgboardController#index
no implicit conversion of Hash into String

該当のソースコード

class MsgboardController < ApplicationController
  layout 'msgboard'
  
  def initialize
    super
    begin
      @msg_data = JSON.parse(File.read("data.text"))
    rescue
      @msg_data =Hash.new
  end
  @msg_data.each do |key,obj|
    if Time.now.to_i - key.to_i >24*60*60 then
      @msg_data.delete(key)
    end
  end
  File.write("data.txt",@msg_data.to_json)
  end
  def index
    if request.post? then
      obj= Mydata.new(msg:params['msg'], name:params['name'],mail:params['mail'])
      @msg_data[Time.now.to_i] = obj
      data = @msg_data_to_json
      File.write("data.txt", data)
      @msg_data = JSON.parse(data)
    end
  end
end

class Mydata
  attr_accessor :name
  attr_accessor :mail
  attr_accessor :msg

  def initialize msg:msg, name:name, mail:mail
    self.name = name
    self.mail = mail
    self.msg = msg
  end
end

補足情報（FW/ツールのバージョンなど）

ruby 2.6.6
Rails 6.0.3.2

Comments

[winterboum]

エラーを全文載せてください。
どのファイルのどの行で起きたか、という情報もあるかと思います。それも重要です。
codeから行数数えるのは間違えるかもなので、「ここだ」　とマークもしてください

@msg_data_to_json　が突然出てきます。どこでどのように定義してますか？

Answer 1

すみません　ありがとうございました。

data = @msg_data_to_json

の箇所が間違っておりました。

to_jsonメソッドの前は
_ではなく.にしなければいけないことに気がつき
修正したら動くようになりました。

Answer 2

すみません　ありがとうございました。

data = @msg_data_to_json

の箇所が間違っておりました。

to_jsonメソッドの前は
_ではなく.にしなければいけないことに気がつき
修正したら動くようになりました。

Answer 3

rails c でデバッグしたらいかがですかね。

ちょっと詳しい方ならここからわかるかもしれませんが、data.textというファイルの中身にも依存するかなという気がしましたので、私なら、というレベルで恐縮ですがおすすめいたします。

data.text

data.txt

と２種類いるのでそこも怪しそうですね。

Comments

[miyanon]

ありがとうございます、とりあえずdata.txtで統一したのですが変わらずでした。

[educ_gt]

```
@msg_data = JSON.parse(File.read("data.text"))
```
エラーここですよね。。

JSON.parse('[1,2,{"name":"tanaka","age":19}]')

こんな感じで値を渡したらどうでしょうか。

これでエラー箇所が変われば、File.readがおかしそうです。