ウンコード・マニア
Pei_Useless 2012-10-09 16:07:17

[PHP] その関数名、おしい このエントリーをはてなブックマークに追加

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

PHPのバージョンアップ対応の作業中に見つけたコードです。 結果はそのままHTMLとして表示されておりました。

なお、ソースの改行コードはCRLFでした。

function str_nl_to_br($str) {
    $ret  = "";

    if( $str != null ) {
        $ret = str_replace("
","<BR>",$str); // 改行を削除すると動かなくなるので気を付けてください。
    }

    return $ret; 
}

コメント(9)

#1 Chiether 2012-10-11 10:55:52  

nl2br「」

#2 kuma056 2012-10-11 16:42:18  

$str != null も少々匂います。わかってて使ってるならいいんですが

#3 Miraranran 2012-10-31 01:20:12  

なぜそこを is_string($str) にしなかったのか... nullになんの恨みがあったんだ。

#4 token7 2013-01-13 04:49:36  

分かってて正規表現使わないとしたらひどい悪戯だ。。

#5 InternalServerE 2013-01-13 20:33:38  

改行を<br>に置き換えるのに正規表現なんか要るん？

#6 uten_dev 2013-08-08 04:35:20  

タブコードでも一緒だねー。コード全体の改行コードが変わったら動かなくなる上に非表示文字だから混乱をまねく。

#9 S_F__ 2013-12-02 04:39:21  

何故エスケープしない！
あ、サイズ削減の為か。
しかしそれならシングルクォート使えばいいのに…
なぜnl2brを使わなかったのかも疑問である

« PHP一覧に戻る

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

[Java] フルHD推奨。

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

画面の左側がわりと無駄な気がします。

public int hoge(){
    int result = 0;
...

鑑賞する »

[Java] スベテカンスウ

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

何でもやってくれる関数って、便利ですよね...

class Processor
{
	public bool process...

鑑賞する »

[その他] そのプレフィックスいる？　＆　予備・・・...

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

ネーミングが日本語のローマ字っていうのも...

create table syain (
syain_bango int no...

鑑賞する »