libraplanet libraplanet 2018-11-19 20:02:19

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

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

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

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

  • ポイント②
    このメソッド全体は昔に誰かが作った共通ライブラリ内にあり、
    入力値の範囲判定が正常に行われていなかったため報告したところ、
    今回の開発担当者はライブラリの不具合を言い訳に、現象を知覚しているのに直さなかった。

  • ポイント③
    結局直してもらったが、無意味な反転コードがそのままになっていた。
    指摘したところ『元のコードがそうなっていたから』

  • 私的意見
    普段から他人のコードが汚いとか、コピペはダメだとか言っている本人が、
    コピペで処理を書いて、ウンコードをそのまま残すのはどうなんだよ。
    お前のそういう行動がコードが汚くなる要因の1つではないのか?
    下らない言い訳をして、小数値の入力チェック ロジック如き自分で組めないなら、プログラマ辞めてしまえ。

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

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

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

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

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

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

コメント(1)

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

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

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

Twitter認証

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

[C#] 意味のない戻り値

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

とある判定を行う関数 結構あるある?

bool HogeHoge()
{
    bool result = fa...

鑑賞する »

[C] 冗長すぎるdefine文

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

実装する人意外にいそう

#ifndef TEST_MACRO_H
	#ifdef TEST_MACRO...

鑑賞する »

[C] 見た目重視

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

Cの講習がつまんなすぎて生まれたクソコー...

#include <stdio.h>

void sort(double a...

鑑賞する »