元スレ【PHP】フレームワーク CakePHP 16ホール目【v2.4】

php覧 / PC版 /

みんなの評価 :

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

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

853 = :

>>850
おいおい、俺がこの業界にまだ入りかけで、アルバイトでPHP書いてたときですら、
アクセス修飾子書いてたぞ。
もう5年以上は前の話だ。

856 = :

>>855
すげー！
そんな違いがあるのか。
勉強になりました。
テストは書いてないです。
ありがとう。

857 = :

>>854
アルバイトが長かっただけで俺もそれなりにおっさんだけど。
varで書かれたコードは見た目が気に食わない。

というのも、varは互換性のためpublicと同じになるから、
private, protectedが使えないわけだけど、
JavaScriptみたくアクセス修飾子がないけどうまくやってる言語もあったりするから。

とはいえ、途中参加のPHPプロジェクトに入ったりしたら、既存のコードを読むときに、
publicなのかそうじゃないかは、コードを読むときに手がかりにするし、
というかpublicメソッドだけ追ったほうが概要がつかみやすいと思う。

まぁ、1人で作ってたり少人数チームだったりすると、アクセス修飾子なんてなくても、
なんとなくわかったりするし、privateには _ つけるといった運用カバー的方法もあったりするし、
どうしても徹底しないととは、正直言いづらい。
だから、見た目が気に食わない。

860 = :

>>859
じゃぁ

コントローラをFacadeにするのは正解。
ビジネスロジックを別クラスにするのも正解。
一般的に、ビジネスロジックはModelに書くのが良いとされている。
一つのModelばかりになんでも詰め込んで肥大化させるのはあまり良くない。
複数のモデルで共通する振る舞いはBehaviorを作るのが正解。
Modelとテーブルの関係を外して新たなModelを書くのはOK。
Libに新しいクラス作ってもいいよ。

ってところ。

862 = :

作るものにもよるかもしれんけど、コントローラーやモデルに、
そんな沢山書かないとな場合て、そんなにないよな。

もしそうなら、コントローラーも、新たに作ったコントローラーもモデルも、
数行しか書いてないみたいには、しないほうがいいと思うよ。

find2回して、片方をforeachでまわしてちょっと加工して、
あとは表示なんてのは、そのページでしか使わないんだったら、
もうコントローラーに書いておけばいいと思ってる。

863 = :

>>862
それだと、テストが難しくなる。

864 = :

>>863
すべてをテストする必要はないんじゃね。
find2 foreach1だけなら、ほぼ間違えないし、
テスト可能にする事も含めて、テストのコストのほうが高いと思う。
foreachの中身が1行2行という想定だけど、
もしこれが複雑なものなら、foreachのとこだけ切り出してテストできるようにはしてもいいと思うけど。

865 = :

>>860
Cakeって案外不親切だよな
自由度が高いといえば聞こえはいいが

866 = :

CakePHPは自由度高くないでしょ

867 = :

2014年にもなって「varって何が悪いの？」レベルのことを言う経験10年って・・
不勉強とかそういうレベルじゃねーだろ

868 = :

化石だな

869 = :

（俺も正確に理解せずに使ってた、なんて言えない・・・）

870 = :

var使ってますってのは
私はいまだにOOも理解してない手続き型しか書けない老害です
って宣言と同義

871 = :

アクセス権はOOに必須ではないがな

872 = :

カプセル化はOOのキモだと思う

873 = :

そりゃそうだ

874 = :

全部publicにstaticに…ってただのファイル分割だわ
そういやそんなおじさんがいたな…

881 = :

Amazonみたいにさ、
この商品を見た人は、以下の商品も見ています。

ってのを作りたい場合、どのように作ればいいか教えてもらえませんか？

883 = :

FacebookとかAmazonレベルだと複雑なスコア算出の式をしてるだろうが、
とりあえず何らかの指標で弾き出したスコアを元にリストアップってのはどこも同じはず。

884 = :

オライリーから、集合知プログラミングって本が出てたはず
わりとそのものズバリな内容が書いてあった
中のコードはphpじゃないけどpythonだから読みやすいし

885 = :

奥が深い分野だから、Amazonと同じ精度出すだけでも、
難易度相当高いよ。

886 = :

同じ精度出す必要無いよ。過剰実装だ。
単純に文字通り受け取って、リファラから商品IDを集計すれば良いだけだろ。

889 = :

>>888
おっけーです

890 = :

CakePHPがあまり関係ない+そんな基礎的なことをと言われそうだけど、
よく見てるスレってここくらいだし、一応CakePHP使って作るし、質問させてくれ。

入力、確認、完了というよくあるフローの投稿機能なんだけど、
入力の段階でテキストやプルダウンなどのほかに、画像アップロードがあるんだ。
確認ページでは画像アップロードは完了させて、確認できるように表示する。
完了すれば、画像は所定のディレクトリに保持されて、ファイルパスが他のテキストとあわせてDBに入る。
確認の段階で戻ったり、完了に到達しなかったら、テキストも画像も保持しない。

良くある話だとは思うけど、確認ページ+画像アップロードが、なんか久しぶりで、
さらにcronが使えないサーバーで動かす予定なんだ。
前やったときは、daily cronで完了に至らなかったケースの画像を削除してたんだけど、
その方法が使えないから、どうしたものかと悩んじゃって。

この画像の後始末って、普通どうやってやるもんなのかな？

892 = :

>>890
というか、いちいち俺は削除しないな。
下手に削除プログラム走らせるとバグの原因になる。
ゴミデータあっても誰も見ないし。

894 = :

乱数でハズレ引いたユーザーに負担してもらう

895 = :

>>890
俺は正式アップロード用のフォルダとそうでないのと分けてるよ。
後から手動で消しても大差ないし、SSHからなら一瞬だしな。

896 = :

今まさに正式アップロードになろうとしてるやつを手動で消さないように注意ね

897 = :

それは日付指定して削除すればいいだけだよね

898 = :

解決
しゅーーりょーー！