libraplanet libraplanet 2019-02-16 10:05:18

[Java] 無意味な論理値の反転 このエントリーをはてなブックマークに追加

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

小数の入力値チェックのロジック。

  • ポイント
    メソッド名と同じ変数名に論理値の結果を格納しているのに、反転している。
    戻り値で更にもう一度反転している。

(サイトの趣旨に合わない愚痴を記載してしまったので、削除しました。。。)

Boolean validateFloat(String str, float min, float max) {

  //文字列が数字だけかを判定している。
  Boolean isNumber = !NumberUtils.isNumber(str);

  if(isNumber) {
    //エラー時のメッセージ処理
    return false;
  }

  /*
   * 入力値が与えられた範囲内かチェック
   * 範囲外であればメッセージ処理をしてfalseを返す。
   */

  // 結局反転して戻している
  return !isNumber;
}

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

コメント(2)

#1 DsYochibe DsYochibe 2018-11-19 20:29:15  

ここで個人宛の愚痴を垂れられましても。

#2 InternalServerE InternalServerE 2018-12-29 06:02:33  

投稿者さんはこのサイトの主旨を理解するところからですねぇ。

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

Twitter認証

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

[Java] is禁止令

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

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

// Mod yamada Start

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

鑑賞する »

[Java] 内部的にunicodeだから日本語つかえ...

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

これを書いた先輩に「英語で書きましょうよ...

class 会員 {
    private int 会員番号;
    p...

鑑賞する »

[Java] もうコメント書くなお前

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

改修案件でぶち当たったソース マジでこ...

/**
  *
  *   〇〇の値を取得する
  *
  */
pu...

鑑賞する »