Ussy_tail Ussy_tail 2013-12-16 21: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認証)

コメント(6)

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

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

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

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

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

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

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

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

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

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

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

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

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

Twitter認証

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

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

このエントリーをはてなブックマークに追加

Javadocでやれ

public static User findByFirstNameFamily...

鑑賞する »

[Java] 連番

このエントリーをはてなブックマークに追加

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

...

package com.renban.erq053.czp008;

/**...

鑑賞する »

[Java] is禁止令

このエントリーをはてなブックマークに追加

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

// Mod yamada Start

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

鑑賞する »