kamitsule kamitsule 2017-04-01 15:52:25

[PHP] 反対の賛成・・? このエントリーをはてなブックマークに追加

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

隣の先輩が保守で引き取った別会社からのソースコードなのですが、falseが返ってきたらしく、先輩が「どっち?どっち?」って涙目で訴えてきました。

function notExists()
{
  //処理がうんぬん
  if ($a == "aaa") {
    $res = true;

  } else {
    $res = false;
  }

  return $res;
}

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

コメント(3)

#1 InternalServerE InternalServerE 2017-04-09 04:50:07  

存在しないことが業務処理継続上必須となるような場合、あえて否定のチェック関数を設計することは ごくごく自然なわけですが。。 これがウンコードと称された所以はどの辺にあるんでしょうかね。

#3 umenojunji umenojunji 2017-04-26 21:03:49  

exist なのに、最後の判定がnull or empty でないところか? そんなにうんこーではないかも。 1'うんこつけました。

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

Twitter認証

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

[C] switch-caseは現場の臭いがする

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

"とりあえず"で書いた 2010 年 4...

// switch-case にしたいが、関数で値を取得しないといけないため、
...

鑑賞する »

[その他] [BATファイル]Echoの意味ない・・...

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

 

@echo off
echo "( ՞ةڼ◔)ンヌェアオオッツプェアンヌェル...

鑑賞する »

[PHP] 工数の無駄

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

メソッドといい引数の「デフォルト空文字」...

class hoge {

 /*
  前略
  */
// 変数の空...

鑑賞する »