5kg太ったのでイケメンがダイエットを応援してくれるLine bot作ろうとしてた話 #Rails

クソアプリ Advent Calendar 2019がバズってるのを見て、約２年前に作ろうとしたクソ設計☆クソアプリを思い出した。。。

どんなアプリ

イケメン(仮)に「目標体重」と「今日の体重」を教えて、その差分から罵倒されたり褒められたりするLineBot。

(実際の体重ではございません)

２年前の私

ワイ「おもちもおせちも美味しい。みかんも食べ放題。幸せだ...」
(久々に体重計乗る)
ワイ「うわ！5kg太った！！！！！！(泣)」
ワイ「痩せなきゃ。どうすればいいんだ。イケメンに叱られたら痩せるのに。」
ワイ「....」
ワイ「そうか、罵倒してくれるイケメンおらんかったら作ればいいのか(脳死)」

使ってた技術

Messaging API (LINE)
ruby on rails 5.2.3
Heroku

おそらく参考にしてた記事

今更ながらRails5+line-bot-sdk-ruby+HerokuでLineBot作成してみたら、色々詰まったのでまとめました。

今の私

とりあえず、Line Bot自体はみつからなかったから、コードだけよんでみよう...
ん？なんだこれは.....
クソアプリ以前にクソ設計じゃん!!!!(大声)

【クソポイント 1】私の目標体重はいずこへ...

とりあえず、データの構造を見てみよう...
ん...？ app/modelsの中に何も書いてない。もちろんdbの中にも何もない...

私の目標体重はいずこへ...

クラス変数に格納されてた。

このアプリでは、linebot_controller.rbにしかほとんんどコードが書いておらず、そこに全ての処理が書いてあった。

linebot_controller.rb

class LinebotController < ApplicationController
  require "line/bot"  # gem "line-bot-api"

  # callbackアクションのCSRFトークン認証を無効
  protect_from_forgery :except => [:callback]

  def client
    @client ||= Line::Bot::Client.new { |config|
      config.channel_secret = ENV["LINE_CHANNEL_SECRET"]
      config.channel_token = ENV["LINE_CHANNEL_ACCESS_TOKEN"]
    }
  end

  def callback
    body = request.body.read
    signature = request.env["HTTP_X_LINE_SIGNATURE"]
    unless client.validate_signature(body, signature)
      error 400 do "Bad Request" end
    end

    events = client.parse_events_from(body)

    ・・・

    events.each { |event|
      case event
      when Line::Bot::Event::Message
        case event.type
        when Line::Bot::Event::MessageType::Text

          if event.message["text"].match(/設定/)
            if t = event.message["text"].match(/([1-9][0-9]{0,2}|0)(\.[0-9])?/)
              @@targetWeight =  t[0].to_f
              ・・・
            end
          elsif ・・・
           ・・・
          end
          client.reply_message(event["replyToken"], message)
        end
      end
    }
  end
end

うん、つっこみどころ満載。

Messaging APIのリクエストボディでは、destination(ボットのユーザーID)とevents(ユーザーが発火)が格納されており、eventsの中は下記のようなjsonオブジェクトになってます。

{
  "replyToken": "",
  "type": "message",
  "mode": "active",
  "timestamp": 1462629479859,
  "source": {
    "type": "user",
    "userId": "..."
  },
  "message": {
    "id": "325708",
    "type": "text",
    "text": "目標体重を50kgに設定して"
  }
}

ユーザーを識別することは非常に簡単そう。

それなのにユーザーごとのデータを管理せずにクラス変数で目標体重を管理している。
つまり、この目標体重は全ユーザー共通で書き換え自由な状態になっていたのだ。
自分の目標体重、丸わかり。。。。

あれかな。みんなのイケメン(仮)になるのが嫌だったのかな。

【クソポイント 2】性癖.yml

DBがないけどイケメンはセリフを返す。
しかも、目標体重に近づけば優しくなる。
これはどうゆう仕組みなんだろ...と探っていったら...

※ これ以上はお見せできません

イケメン(仮)に言われたいセリフがみっちり書いてある性癖.ymlが発掘されました。
全部自分で羅列してったのか...痛いぞ、自分。

また、yamlも絶妙に読みづらい。
入れ子になった配列がずらーーーっとならんでおり、voice["message"][差分][乱数]で取ってくるって感じ。
配列にする必要なかった部分まで配列...

linebot_controller.rb

# ymlファイル読み込み
voice = YAML.load_file("#{Rails.root}/config/project.yml")

# diffは目標体重と現在の体重の差分
message_length = voice["message"][diff].length - 1
diet_message = voice["message"][diff][rand(message_length)]

message = {
  type: "text",
  text: "今日の体重は#{weight}kgで、目標体重の#{@@target}まであと#{sprintf("%.1f", diff)}kg減か！#{diet_message}"
}

【クソポイント 3】
ワイ『今日は23歳の誕生日！50kgだったよ！』
イケメン(仮)『今日の体重は23kg』

messageに数字があったら、必ず最初に出てくる数字を体重だと扱う処理がかいてあるため、イケメン(仮)と円滑な会話が全くできない仕様。
書き方もつっこみどころ満載...

if myWeight = event.message["text"].match(/([1-9][0-9]{0,2}|0)(\.[0-9])?/)
  myWeight = myWeight[0].to_f #やばい
 # 果てしなく痛い処理の連続
end

実際のところ

Q. このbotをつかいましたか？
A. 使ってないです。性癖.ymlを書いてる時に目が覚めました。

Q. 痩せました？
A. 全くもって痩せなかったです。

さいごに

しょうもない記事を最後まで読んでくださり、ありがとうございます。
いつもは結構真面目に記事を書いてるので、よかったら読んでください。

ちなみに、ダイエットに一番効力あったのは、好きな人に言われた「肉団子」のひとこと。
痩せたい。

5kg太ったのでイケメンがダイエットを応援してくれるLine bot作ろうとしてた話