元スレ【PHP】下らねぇ質問はID出して書き込みやがれ 111

php覧 / PC版 /

みんなの評価 :

前へ 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 次へ

/ 要望・削除依頼は掲示板へ / 管理情報はtwitterで

764 = :

すいません。出典の短縮URLが間違ってました。
正しくは　http://1cc.jp/hgogho　です。

765 = :

それらはみな、コードが雑だから。
phpspotは一見よさそうに見えるけど、すげえだめだから。
理由はくさるほどあるが、省く。
繰り返す。すげえだめだから、見てはいけない。

php関係なく、SEO対策とかを勉強したいなら見てもいい。

766 = :

なんか心配になったから書いとくよ。

安全性の問題
・キーの存在チェックをしていない
・name または message にカンマが含まれると意図しない出力になる
・サニタイジングを行っていない
・排他制御がない

一般的なコーディングと異なることによる速度低下
・ループの外に出せるcountをそのままにしている
・$i++じゃなくて++$i(この辺はまさにそのphpspotでも記載されていたと思うが)
・そもそもforeach
・ファイルの先頭に入れるのではなく、末尾にしたほうがI/O減る。
　この場合、forで末尾から出力するか、array_reverse→foreach

少なくとも上記の問題があるコード。

767 = :

>>764
イチイチURLを短縮しなくてもいいよ
みないからそういうの

768 = :

いちいち見ない宣言は要らない

769 = :

>>768=質問者

770 = :

同意。
質問するときに短縮URLなんて使うのは頭が狂ってる。

771 = :

質問者はエラーメッセージの文章をどの程度理解してるのか気になる

774 = :

>>773
エンコードされてない日本語で貼ればばいいんじゃね？

775 = :

>>774
お前は何を言っているんだ

776 = :

>>775
え、おかしい？これでしょ
http://phpspot.net/php/pg投稿フォームの作成.html

777 = :

デコードしちゃったURLから元のURLを一意に求めることは出来ない

780 = :

なるほど

782 = :

php関係ねえし
sqlの初歩から勉強しろ

784 = :

SQL文自体にがシングルクオートが必要なんだが

785 = :

>>782 >>784
ありがとう！

786 = :

正解じゃない。

787 = :

SQLインジェクション余裕

791 = 788 :

そのへんは気にしないで下さい。

言いたいことは使用者に速度を犠牲にしても書式を統一するか、
どういうデータが入ってるかまで把握してもらって使い分けてもらうか、
どちらにするのがいいのかな？という感じです。

792 = :

多態性で済ます

793 = 788 :

ポリモーフィズムですか。
よく耳にはしますが、難しそうな概念ですね。
データ型ということで言えば全てStringなのですが、
例えばある処理が必要な値なのかどうかということですね。（788の例だとnl2br。）

繰り返しになりますが、
ある処理が必要でない値でも、ある処理を入れても影響が出ないので、
全ての値において処理を統一することが可能です。（値によっては不必要な処理も入りますが。）
そこでちょっと悩んでいる感じです。

794 = :

そこまで難しいわけでもないよ。クラスでもいいしインターフェイスでもいいし。

> ある処理が必要でない値でも、ある処理を入れても影響が出ないので、
> 全ての値において処理を統一することが可能です。（値によっては不必要な処理も入りますが。）
というのが、すべての値において記述を統一することができ、必要な処理のみが行われるってなる。

けど、そういうことしたいなら既存のフレームワークとか当たったほうがいいかもしれないし、勉強がてら作ってみるのもいいかもしれない。

795 = 788 :

どうもありがとうございます。
おかげさまで少し賢くなった気がします。
とりあえず今回の件は統一することにしたいと思います。

アドバイス通りフレームワークも研究してみたいと思います。
夜遅くにお付き合い下さりありがとうございました。

796 = :

ただ、めんどくさいよ。
いろいろやってみるといいです。おやすみ

797 = :

PHPにはString.sliceが無いのだろうか？
これくらい言語が用意しろと思うんだが。。

798 = :

コード量が増えても速度優先するけどな。速度が速いということは負荷も軽くなるわけで。

799 = :

>>797
http://php.net/manual/ja/function.substr.php
ではダメなのか？