toriimiyukki 2012-08-01 00:30:57

[PHP] PHPの型の概念を知らないと... このエントリーをはてなブックマークに追加

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

さらにローマ字表記にするだけで糞コードっぽく見える不思議! さらに三項演算子を使わないと糞コード!

function mojiga_aruka($bunsyou, $kensaku_moji) {
    if(strpos($bunsyou, $kensaku_moji)) {
        return true;
    }else{
        return false;
    }
}

コメント(4)

#1 Miraranran 2012-08-14 09:21:57  

関数呼び出しにオーバーヘッドがあるので、わざわざmojiga_arukaを定義して呼ぶだけ無駄。その場その場でstrposの返値をboolにキャストするだけでいい。

#2 fetus_hina 2012-08-14 10:13:31  

strpos や stripos の戻り値をキャストしてはいけません。strpos は (false と解釈される) 0 を返す可能性があります。 return strpos(...) !== false; 等とするのが良いでしょう。


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

[PHP] $select2「$selectがやられ...

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

コメントはすべて原文にはないものです

<?php

// 注: 実際のファイル名はちゃんとしたものです
incl...

鑑賞する »

[C] すげーちゃんと動く!

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

結合試験2日前に脱退したうんkプログラマ...

return 0;

鑑賞する »

[JavaScript] とっても長いswitch/case文

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

配列変数を全く理解できていなかった。

var rnd = Math.floor(Math.random() * 128...

鑑賞する »