PHPerはSpikaのどこを見たのか？

レガシーズ 公式ブログ レガシーコードと戦い続ける謎の集団、レガシーズの公式ブログです。 ※レガシーズはレガシーコードを通じて正しいコードの構造や改善の手法を模索する覆面集団です。特定のプロダクトやサービスを誹謗中傷する事が目的ではありません。 みなさん、こんにちは。マスク・ド・レガシーです。 PHPカンファレンスも無事に終わり、ゆっくりと休暇を楽しんでいたところ、にわかにタイムラインが騒がしくなっていました。今回は話題の中心になったSpikaを題材にPHPのコーディングについて考えてみようと思います。 Spikaの概要SpikaはMITライセンス化のオープンソースとして公開されたメッセンジャーアプリケーションです。プロジェクトにはネイティブで実装されたiOS、Android向けのモバイルアプリケーションとPHPで実装されたサーバーサイドを組み合わせる事で独自のメッセンジャーサービスを構築

[Yamashiro0217]

SpikaがコードをOSS化したことは良いことだと思う。色んなアプリが、メッセージ機能あったほうが価値が高まる。ただ、セキュリティ的に脆弱性があるのは事実。だからみんなPR送ればいいと思う。

[dtan4]

“ベタな実装”

[suginoy]

“※レガシーズはレガシーコードを通じて正しいコードの構造や改善の手法を模索する覆面集団です。特定のプロダクトやサービスを誹謗中傷する事が目的ではありません。”

[se-mi]

誰なんだ、マスクドレガシー

[ghostbass]

「ソースコードの手動での書き換えはデプロイやテストの自動化の妨げになります。」げに。

[uunfo]

正直これはつらいな。公開したのはものすごく立派だけど。

[brtRiver]

global 怖い

[k-holy]

WordPressにしてもそうだけど、ユーザー層と開発者層があまり被ってないのかな。文字列でのSQL生成というより適切にエスケープしないことが問題なので、プリペアードステートメント不使用＝悪という風潮は賛成できない

[fukken]

これ、「真っ先に目に付く部分」であって、「重要な部分」では必ずしもない気がする

[luccafort]

なるほど、ソースコードみたら思わず心中で「ぎゃああああああああ！」ってなったわ。10年前にこれがあったらここまで叩かれてなかったんだろうな。

[h5y1m141]

“公開されたソースコードを見た際にPHPerがどのようなポイントに目を止めて、話題にしたのかを見てみましょう”

[oldfish]

直しどころの解説

[sugimori]

例のメッセの件『 PHPerはSpikaのどこを見たのか？』