vestigial vestigial 2015-06-06 09:20:52

[PHP] 改行の仕方がわからなかった このエントリーをはてなブックマークに追加

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

ソースレビュー中にインデントがぐちゃぐちゃになっていてとても読みにくい部分を発見。 わざわざヒアドキュメントを使わなければいけない理由がわからなかったので問いただしてみると 「改行の仕方がわかりませんでした」とのこと。 print "…\n"を使っているところもあったのでこれが改行コードだと教えたら 「そこは先輩のソースのコピーです。なるほどこれで改行できるのか」って言っていました。 プログラマ暦10年ぐらいのはずだよね。

                    if ($hoge == false)
                    {
print<<<EOF
            <script type="text/javascript" src="./js/main.js"></script>
EOF;
print<<<EOF
            <script type="text/javascript">huga("$url");</script>
EOF;
                    }
                    else
                    {
                        $url = "xxxx";
print<<<EOF
            <script type="text/javascript" src="./js/main.js"></script>
EOF;
print<<<EOF
            <script type="text/javascript">foo("$url");</script>
EOF;
                    }

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

コメント(5)

#1 kuma056 kuma056 2015-06-06 11:22:30  

アピールポイントの「なるほどこれで改行できるのか」ってところにマーク付けたい

#2 bravotan bravotan 2015-06-06 11:29:04  

ヒアドキュメントを1行1行丁寧にやってるのもポイント(白目)

#3 rpSuxbJp rpSuxbJp 2015-06-11 15:31:07  

どこに改行が必要?

#4 elseorand elseorand 2015-07-02 16:16:55  

むしろPHPで==を使用してfalse判定もクソ

#5 AKYM21 AKYM21 2018-08-27 01:33:44  

if ($hoge == false)で凄い違和感が・・・。 falseの場合Trueというのはなかなか考えさせられる・・・。
返り値がFalse以外のものだった場合条件式的にFalseになるってことだから、True扱いのものがFalseになるわけで、$hogeが「ふが」とか「True」だと本来のBoolはTrueだけど条件式的にFalseになる。。。
とても気持ち悪い。
ヒアドキュメントは多用すると可読性が怪しくなるから取扱い注意だよね・・・。

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

Twitter認証

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

[PHP] つ、釣り!?

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

ツボって思い出し笑いが止まらなかったので...

class Hoge {
    /*
      * インストラクター
...

鑑賞する »

[C] 文字列の比較

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

JavaScriptとかと混同してしまっ...

char str[128];
int num1,num2;
scanf("%...

鑑賞する »

[PHP] 見た目はコメント、中身は未定義

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

シンタックスハイライトに騙されてしまいま...

<!-- 未定義なのでコメントアウト
<?php undefined_func...

鑑賞する »