[Ruby] えっ？こんなんあるの？-ウンコード・マニア

Ussy_tail 2013-12-16 12:59:18

[Ruby] えっ？こんなんあるの？

投稿者からのアピールポイント

レビューでやられました。陽の光を浴びない子がいます。さらに、未だ隠れているやつが、、、

  def next_exp(level, exp)
    level_a = all(name: "LevelA")

    infomation = level_a.select{|x| x["level"] == level}.first
    exp_to = infomation["exp_to"]
    next_exp = exp_to + 1 - exp

    return next_exp
    next_exp
  end

使い方ヒント: 「これは臭う」という行を見付けたら、各行の smell をクリックしてマーキングしておきましょう(要Twitter OAuth認証)

コメント(7)

#1 akimatter 2013-12-17 01:24:41

selectしてfirstてwww detectでいいじゃんwww

#2 akimicyu 2013-12-17 01:33:51

データモデルからも臭いが。exp_fromないのだろうか？最終レベル(ex:Lv99)に達したときの結果は99999999999999231312とかになるのだろうけど、それはいいんだろうか？

#3 akimatter 2013-12-17 01:58:23

そうか、そもそもallしたものをselectするべきじゃなくて、firstを使ったら済む話なんだw

#4 sandayuu_en 2013-12-17 02:02:08

exp_to は恐らくそのレベルの経験値の上限だから、exp_to + 1 したら次の経験値、もしくは次のレベルを取得すればnext_expなので、どう考えても、ここで返却されているのは、次の経験値じゃないですね、、、

#5 hokuts 2013-12-17 02:03:35

infomationとかいう変数名をつける人はたぶん将来ハゲる

#6 gatpai 2013-12-17 02:52:48

メソッド名と同名の変数をつけて、しかもreturnしているので幻惑を見ている気分になりがち

#7 andoshin11 2017-06-01 05:11:06

変数名とメソッド名に同じ名前使っちゃうのロックだ

コメント投稿には、twitter認証が必要です。

Twitter認証

« Ruby一覧に戻る

このウンコードに臭った人は、こちらのウンコードにも臭ってます

[Java] 長すぎるor短すぎる関数名

Javadocでやれ

public static User findByFirstNameFamily...

鑑賞する »

[Java] 連番

もはや人間が読むものではない。

...

package com.renban.erq053.czp008;

/**...

鑑賞する »

[Java] is禁止令

ウンコードの趣旨とは違い、レビューで指摘...

// Mod yamada Start

// 一般的に考えて真偽値を返すメ...

鑑賞する »