[生き残れるか]EC-CUBE[4店舗目]

**nobodyさん** · 2008/12/13(土) 14:11:28

【前スレ】
・[生き残れるか]EC-CUBE[3店舗目]　http://pc11.2ch.net/test/read.cgi/php/1209204054/

【過去スレ】
・EC-CUBE　http://pc11.2ch.net/test/read.cgi/php/1169631890/
・[生き残れるか]EC-CUBE[2店舗目]　http://pc11.2ch.net/test/read.cgi/php/1201784308/

【公式】
・EC-CUBE公式サイト　ttp://www.ec-cube.net/
・開発コミュニティ　ttp://xoops.ec-cube.net/
・マニュアルサイト　ttp://wiki.ec-cube.net/
・EC-CUBE Trac　ttp://svn.ec-cube.net/open_trac/
・メーリングリストの過去ログ　ttp://webml.ec-cube.net/

【その他】
・2ch EC-CUBE Wiki　ttp://www39.atwiki.jp/eccube2ch/
・[mixi] EC-CUBE　ttp://mixi.jp/view_community.pl?id=1272334
・EC-CUBEの学校　ttp://www.eccube-school.jp/

【EC-CUBEとは】
EC-CUBEは、株式会社ロックオンが創業から6年にわたって開発してきたECサイト構築システムをオープンソースとして公開したものです。ASPではできなかった独自性の高いECサイトが、無料で手に入ります。

**nobodyさん** · 2009/06/08(月) 06:00:49

ONE/DEPOのversion: 1.0.00 b1をｲﾝｽｺしてみましたが、いきなりまっ白ページでアウト
config.inc.phpの２１行目あたりから間違ってるって言われて
define('_ADMIN＿MAILADDR', 'someone@yourdomain.com'); // ONE/DEPOから発信されるメールのアドレス
の_ADMIN＿MAILADDの＿MAILADD部分のアンダースコアがマニアックに全角
そこ直したら、下にも全角アンダースコアがぞろぞろ
これって、仕様なのかな？

**nobodyさん** · 2009/06/08(月) 10:35:16

ONE/DEPOってPostgreSQLのみじゃなかったっけ

**nobodyさん** · 2009/06/08(月) 10:37:23

そうだよ

**nobodyさん** · 2009/06/08(月) 11:31:10

たびたびONE/DEPOのネタで恐縮です
先ほど、確認したら1.000 beta2にうｐされてたので、こっちで試してみました。
ｲﾝｽｺ環境は、CENTOS５．３とpostgresql-server-8.1.11-1、php-5.1.6です。
やはり、config.inc.phpの２１行目あたりかの全角アンダースコアはそのまんまなので
これは、パソコン側で半角に一括返還してFTPで差し替えました。
また、.htaccess内の/var/local/を/var/local/onedepo/に
php_value include_path ".:/var/local/onedepo/onedepocore/components
:/var/local/onedepo/onedepocore/include:/var/local/onedepo/onedepocore/smarty"

最後に、/var/lib/pgsql/data/pg_hba.confを
local all all md5
host all all 127.0.0.1/32 md5

以上で、すんなり動きました。
これから、ユーザ登録とか商品登録とかして、いじってみます。
中身空だからでしょうが、意外と軽快な感じどす、ひょっとして良いカモ！

**nobodyさん** · 2009/06/08(月) 11:59:50

ひつこくONE/DEPOですが、アドミン画面がEC-CUBEに良く似てますね
（つうか、日本人が日本人向けに作るとみんな同じような雰囲気になるんでしょうね）
これは、ZEN-CARTなんかとは段ちで、解り易くて、GOODです!
商品の画像一括登録は、一回に５００件単位。
商品データのインポート機能とか、後発だけに基本機能はそこそこ満足。
携帯でも確認しましたが、こちらもバッチリす。
入金管理ってとこに、返金依頼とか出品者支払一覧とかの機能があるのが特徴的すね。

ここまでの感触は、βのわりになかなか良いですね。
初期のEC-CUBE頃の、手作業修正の時代が、懐かしい。ｗ

**nobodyさん** · 2009/06/08(月) 13:30:07

個人的には有益情報だから㌧♪なんだけど
どうせなら専用スレ立てて書いた方がいいんでない？

**nobodyさん** · 2009/06/08(月) 19:15:10

大塚商会のアルファメールというサーバに入れたいと思っていますが、
問い合わせをしたら動作補償はしていないとのことでした。
どなたか既に利用されている方はいらっしゃいませんか？
プレミアではなく通常のアルファメールです。

大塚商会（アルファメール）
http://www.alpha-mail.jp/

**nobodyさん** · 2009/06/09(火) 01:45:24

今のところ、EC-CUBE 以外、送料周りはどれも融通が利かない。
ちょっと弄ったぐらいでは解決できないっぽい＜ONE/DEPO も

大規模店舗考えてんなら、送料一律でONE/DEPOでいけんじゃね？
小規模や産直考えてんなら、EC-CUBE で我慢し続けるか、
要望出しておいた Live 待つか・・・

**nobodyさん** · 2009/06/17(水) 12:01:29

ECの案件がきたらEC-CUBEを使おうと思っていますが、
どのレンタルサーバ(プラン)、どのクレジット決済が
相性いいのでしょうか？

**nobodyさん** · 2009/06/17(水) 12:05:46

商品数が多そうな案件ならMySQLはやめとけ。
激重になってクレームが来て1週間はつぶれるｗ

**nobodyさん** · 2009/06/17(水) 16:41:19

その商品数ってどれぐらいのことを言ってるの？

**nobodyさん** · 2009/06/18(木) 08:37:12

1000品目以上つかってるEC-CUBEのショップなんてあるのか？
そもそも費用対効果がほとんどないと思うが
どうせ楽天・Yahooの管理料が高くて逃げてきた連中が
倒産品打ってるだけだろ

**nobodyさん** · 2009/06/18(木) 09:14:35

>1000品目以上つかってるEC-CUBEのショップ
http://www.1-s.jp/products/list.php?name=&search.x=33&search.y=19
1万品目ぐらいあるぞｗ

**nobodyさん** · 2009/06/18(木) 12:47:30

20個くらいなら余裕そうだな。
問題はデザインの入れ替え費用。

**nobodyさん** · 2009/06/20(土) 23:28:52

問題なのはバージョンアップだろ。

**nobodyさん** · 2009/06/28(日) 13:29:31

>264
１万点以上てか、余計なもん省いちまって且つ、SQL所々見直せば問題ないよ。
やった後なら普通に使える。はずｗ
それをやらんで、デフォルトのままで・・・とか考えるとハマるよｗ

でだ、266氏言うよう、頭抱えることになるのがバージョンアップ(　＾∀＾)ｹﾞﾗｹﾞﾗ

**nobodyさん** · 2009/06/29(月) 06:51:34

MySQLのＥＲ図どこかにないですか？

**nobodyさん** · 2009/07/05(日) 10:25:09

作ってよければ配布して下さい

**nobodyさん** · 2009/07/06(月) 01:13:55

**nobodyさん** · 2009/07/06(月) 09:36:40

シャチにもらえ

**nobodyさん** · 2009/07/06(月) 19:07:11

はい

**nobodyさん** · 2009/07/08(水) 13:10:31

　　　　　　　　　7/8(水) EC-CUBE 2.4.1リリース…

　　　　　￣￣￣￣￣￣￣￣○￣￣￣￣￣￣￣￣￣￣
　　　　　　　　　　　　　　　　　 o
　　　　　　　　　　　　　　　　　ﾟ
　　　　　　　　　　　ヽ从/ ｡
　　　　　　　ｶﾞﾊﾞｯ　　 ∧ ∧　　　　／￣￣￣￣￣￣￣￣
　　　　　　　　　　　　　(ﾟДﾟ；):.　_　＜　はっ、夢か!!
　　　　　　　　　　　　ｒ'⌒と､j ミ　ヽ　＼＿＿＿＿＿＿＿＿
　　　　　　　　　　　ノ　,.ｨ'　｀ヽ.　/
　　　　　　　　　／　　　　　　 i!./
　　　　　　　（_,. 　　　　 //
　　　　　　　く.,_｀^''ｰ-､_,,..ノ/
　　　　　　　　　｀~`''ー--‐'

**nobodyさん** · 2009/07/11(土) 17:10:06

EC-CUBE2.4.1 正式版をリリースいたしました。（2009/07/08）
http://www.ec-cube.net/release/detail.php?release_id=205

今頃気づいた

**nobodyさん** · 2009/07/11(土) 20:44:23

いつもよりバグフィックスおおくね

**nobodyさん** · 2009/07/16(木) 02:31:21

つーかもう１から自分で作るわ

**nobodyさん** · 2009/07/16(木) 09:32:22

頼んだ

**nobodyさん** · 2009/07/18(土) 23:53:15

ごちゃごちゃしたEC解析していろいろ改造するより
１から作ったほうが絶対早いな

**nobodyさん** · 2009/07/19(日) 16:18:01

確かに、そういうにわかプログラマ今多いよね

**nobodyさん** · 2009/07/19(日) 19:40:30

まぁこういう意見もあるからな
http://d.hatena.ne.jp/zan-gyo/20090602/1243923480

**nobodyさん** · 2009/07/21(火) 00:27:39

IE7が糞なのはわかるが、上のリンク先行くとおそろしく表示が....

**nobodyさん** · 2009/07/21(火) 00:37:46

>>280
そこに書いてあることより、たいしたシステムでもないのに
ごちゃごちゃしたMVC採用してることが問題だと思う。

ちょっとした改造で手間がかかりすぎ。

**nobodyさん** · 2009/07/21(火) 10:23:52

非常にわかりにくく、バグ搭載版＝フリーで転がす
しこたま時間をかけていろいろ潰した版＝岩上商用版

て、噂もちらほら

**nobodyさん** · 2009/07/21(火) 17:36:16

>>282
ちょっとした改造で手間がかかりすぎなのは、まったく同意見ですが
あれのどこが MVC なの?

**nobodyさん** · 2009/07/21(火) 23:14:15

>>284
じゃああの構造はなんていうのが適切なんだい？
俺は知らん。

**nobodyさん** · 2009/07/22(水) 06:28:54

>>285
いや、俺も知らない。
だけど、MVCなら MとCが分離されている。

**nobodyさん** · 2009/07/22(水) 09:37:33

>>283
商用版はスペルミスも直ってるのかなｗ
そんな差違つけんなよと思うが

**nobodyさん** · 2009/07/22(水) 14:49:37

EC-CUBEってなんでインストール時に
全ファイルのパーミッション・実行権を与えないといけないの？

画像ファイルやCSSまで666にしないと動かないって・・・

**nobodyさん** · 2009/07/23(木) 20:06:44

さくらでインストールしてみた
PCの方はどうにか動いたんだが、携帯で見たときに500エラーが出てたので、
.htaccess削除してみたら、何とか500エラーは出なくなったんだけど
（PCの方は.htaccess削除してphp.iniに内容を書き換え済み）
携帯版、文字化けしています…

これって、html/mobileの.htaccessもphp.iniにしないと文字化けは直らなさそうだ、
というところまでは理解したんだけど、.htaccessみたいに
複数置いて大丈夫なのか、というのが分からないので誰かエロイ人に
教えていただきたいんですが

**nobodyさん** · 2009/07/23(木) 20:28:42

coreserverでインスコしなおすのがいいよ

**nobodyさん** · 2009/07/24(金) 09:15:15

>>289
php.ini複数置いても良いけど
メインのphp.iniだけでイケたよ？

mbstring.language = Japanese
magic_quotes_gpc = off
mbstring.internal_encoding = UTF-8
default_charset = UTF-8
mbstring.encoding_translation = 0
variables_order = EGPSC
session.auto_start = 0

あたりにしておけば携帯とPC両方おｋ
共有SSL使おうと思ったらEGPSCなんて並びに・・・

**289** · 2009/07/24(金) 21:31:31

>>291
PHPのことをほとんど何も分からないまま手探りでやってるから
実例まで出してもらえてホントに助かった
エロイ人本当にありがとう！！

さて今晩も頑張るか…

**nobodyさん** · 2009/07/25(土) 16:35:27

まあ、あなたったらいやらしい^^*

**nobodyさん** · 2009/07/27(月) 11:06:25

2.4.1いれてみたけど、未だにMySQLで文字化けするの直してないのかよ・・・

**nobodyさん** · 2009/07/29(水) 22:02:35

てか、使い勝手がやたら悪いんだけど、
もしかしてわざとそう作っておいて
「サポート料金払ってくれればちゃんとします」
というスタイルなのかな？

**nobodyさん** · 2009/07/30(木) 09:53:36

EC-CUBEoneの方はFOTTERからfooterに直ってるもんなぁｗ
この作り分けの労力に感心するがｗ

**nobodyさん** · 2009/07/30(木) 22:20:15

EC-CUBEoneなんてのがあるんだ

**nobodyさん** · 2009/08/02(日) 00:35:21

1ヶ月くらいecいじってたけど、だめだこりゃ
1から自分で作ろう・・・
何日かかるかもだけど・・・

**nobodyさん** · 2009/08/02(日) 19:03:50

>>280 のリンク先に

開発者がSmartyテンプレートエンジンを使いこなせていない。
{section} でループ回すんじゃなくて {foreach} 使おうよ。

ってあるんだけど、なぜ foreach を使う「べき」なのか教えて。

**nobodyさん** · 2009/08/03(月) 11:00:21

foreachの方が配列の出力がしやすい（わかりやすい）から

**nobodyさん** · 2009/08/03(月) 11:37:09

>>298
自分と同じ考えの人がいて安心したｗ

それなりに名の通ったオープンソースでこんなことになったの初めてだorz

**nobodyさん** · 2009/08/03(月) 12:56:38

http://namidame.2ch.net/test/read.cgi/manage/1248100374/97

**nobodyさん** · 2009/08/03(月) 16:09:00

>>301
phpのオープンソースではよくあるような気がするけど・・・
有名な wiki とか cms でもそんな気持ちにはなった。

ecは、変な構造のせいで開発スピードが極端に遅くなるし、
モチベーションも落ちる。

とりあえず週末でフレームワークとフロントエンドまでできた。
あとは面倒なバックエンドと画像作成が残ってるけど、
なんかもうどうでもよくなってきたなぁ。

**nobodyさん** · 2009/08/03(月) 20:14:04

ま、企業がオープンソースにする目的は明らかに「名を売りたい」だからな。
名前を売ってサポートやカスタマイズで金を取る。

**299** · 2009/08/03(月) 20:24:12

>>300
thx。パフォーマンスがどうのってわけじゃないんだね。安心した。
foreach の方が section の10倍コストパフォーマンスが良い、とかだったら、
手を施さなければいけない強迫観念箇所がまた１つ増えるところだった。

**nobodyさん** · 2009/08/03(月) 23:35:13

俺零細企業のweb担当なんだけど、
ロクにphpも理解してない（自分が）くせに自社ECに採用して死にそうになったｗ

バグ発覚して、フォーラムに解法がないと顔が青ざめる('A`)ﾓｳﾑﾘ

**nobodyさん** · 2009/08/04(火) 01:35:06

オープンソースのECシステムには
ろくなもんがないよな。

ソースいじくれない人はASPを利用することをお勧めする。

**306** · 2009/08/04(火) 01:47:17

過去の自分へ
「だからカラーミーにしとけとあれほど･･･」

**nobodyさん** · 2009/08/04(火) 11:25:32

セッションをUseRequestに設定したときの動作、少しでもチェックしてるのかな？
ポップアップのセッション引継は全滅だし

ケータイからアクセスしたときのリダイレクトが多すぎ
おかげでiModeブラウザは途中でリダイレクト止めちゃうし・・・

ドメインルートにアクセス
index.php?PHPSESSIDにリダイレクト
/mobile/にリダイレクト
/mobile/index.php?PHPSESSIDにリダイレクト

require.php弄って、ドメインルートからmobileに直接飛ばせばおｋ

Zenとかは目立ったバグもなく動かしやすいけど、日本人の感覚では使いにくいな
ケータイ対応も面倒だし

**nobodyさん** · 2009/08/05(水) 00:12:36

まぁいろいろ問題は・・・ありすぎるｗ
もれは、ロクオンはある意味、被害者だと思ってる。
1.x時代から諸問題ありまくり～で、開発陣が引きつっているところへ、
全国のﾊﾞｶﾄﾞﾓが無償ﾃｽﾀｰ引き受けてくれるだろうと
確信犯的に七節が独自のMVCに乗っけた 2.x を投入した。
国産ｵﾌﾟｿってあちこち宣伝されてたこともあって、
七節の読み通り、ﾜﾚｺｿﾜ~ってこぞってﾃｽﾀｰ気取り。
結果、七節は手元に置いたｵﾘﾁﾞﾅﾙの完成度を上げただけのような・・・
そのｵﾘﾁﾞﾅﾙはCUBEには使われていないと思われ。

そんなこととは知らず、のっちまったロクオンは、
把握するのにしこたま時間を費やすハメになり、
有償で請け負ったサイトもあるしで人手が足らず、
ぶち上げたビジネスモデルもうまくいかずに今に至る。

相手がビジネス優先の企業である以上、
道ばたに置いた空き缶に投入された小銭が、
有償サイト構築に使われないという保証はない。
ｺｼﾞｯｷｰに恵んであげてるとか思ってるｵﾏﾇｹな連中以外
小銭の投入、する気ある？

話がズレたが、そういうことだ。
これは投入後行方不明になった七節が責任とって大修正しないと治らん。

**nobodyさん** · 2009/08/05(水) 08:12:00

よくわからんが気持ち悪い文体だな。

**nobodyさん** · 2009/08/05(水) 13:42:26

要はシャチがあと全部無償で直せばいいだけのこと

**nobodyさん** · 2009/08/05(水) 16:27:30

よくわからんが、結局は開発元のロックオンの責任だな

**nobodyさん** · 2009/08/05(水) 17:52:58

> 七節が

だれでしたっけ？
詳しく

**nobodyさん** · 2009/08/06(木) 11:01:44

携帯向けの静的ページってどうやって登録するのですか？
デザイン管理＞ページ詳細で登録したらPC向けのページになります。

ちなみにVer2.X系です

**nobodyさん** · 2009/08/07(金) 00:11:47

結局はそれでおけしたロクオンの責任に違いない。

もれは１系に手を加えて使ってるんだが、
七節が２系投入後に、
本業が忙しくなったとかぬかして
掲示板から消えたのを機会に
開発者本人が見放した２系に手を焼くより、
１系を安定さして使ったほうが無難と判断したわけだ

**nobodyさん** · 2009/08/07(金) 01:57:58

IPAの言う「ver.2.4.1未満を使ってる人は2.4.1へのヴァージョンアップしろ」
という注意喚起は、どうしてる？
ttp://www.ipa.go.jp/security/vuln/documents/2009/200907_ec-cube.html

**nobodyさん** · 2009/08/07(金) 02:12:39

オープンソースのウェブアプリってろくなもんがないんだよ。
何でかというと、ウェブアプリってプログラミングしててもまったく面白くないから。
ある程度スキルが上がってくると、ウェブアプリなんて作りたくなくなる。
仕事で金になればいいけど、金にもならないのに、まともに開発継続するわけない。
おまけにユーザは素人ばっかだから、平気でFFFTPの使い方とか聞いてくるし。
ますますやる気はゼロというかマイナスに突入する。

**nobodyさん** · 2009/08/07(金) 12:02:57

>これは投入後行方不明になった七節が責任とって大修正しないと治らん。
行方不明にはなっていないが、大筋、そのとおりだと思う
とりあえずMVCとして先に完成させるべきかと

**nobodyさん** · 2009/08/08(土) 02:39:37

>>318
ま、そうだな。いくら知名度を上げる為とはいえ、
2ｃｈ見てたらとてもじゃないが、公開なんて出来ないw

**nobodyさん** · 2009/08/08(土) 21:06:56

七節がどういう経緯で、なんの仕事で忙しくなったか知ったら
腰抜かす奴多そうだなwww

**nobodyさん** · 2009/08/08(土) 21:51:57

んなわけねえだろ

**nobodyさん** · 2009/08/09(日) 14:42:16

たいした技術もないくせに、素人相手は疲れる～とか
うぬぼれた技術者ほど内に内にと籠もる典型だわな
うぬぼれ君の影で必死になって素人相手している営業君がいるわけで、
ある意味、スキル的に言うと、営業君のほうが上だったりしてな～い？
うぬぼれ君ってば、あっちの会社のほうがギャランティいいからとか、
移ってみたら、「ちみ、それしかできないの？」ってな(　＾∀＾)ｹﾞﾗｹﾞﾗ

問題ありあり～のEC-CUBE褒めるわけじゃあないが、
あれこれと酷評されるのわかってて放り出した気持ちにあっぱれ！を送るよ
ほっぽって逃げてった七節には喝か？

**nobodyさん** · 2009/08/09(日) 22:53:19

>>323
おまえは日本語の勉強したほうがいい

**nobodyさん** · 2009/08/10(月) 08:04:54

>>321
ｋｗｓｋ

**nobodyさん** · 2009/08/10(月) 15:10:17

七節ってのはどなたか知らんが
このソースをずっと相手にするとなると気が重くなる
よほどのみかえりでもあるなら別だけど
まあ、徐々に作り変えて最後には全く別物にするだろうけどね

**nobodyさん** · 2009/08/10(月) 15:43:46

しかし気になりますよね

違法なソフトを作って逮捕されて裁判中
宗教に走って神様のためにプログラム中
携帯端末に手を出してデスマーチ進行中
別のオープンソースにかかりっきり
趣味で本格的にマンガを描きはじめた
行方不明
病気
死亡

**nobodyさん** · 2009/08/10(月) 17:16:37

>>321
七節ってロックオンに関係ある人だったっけ？
社員ならなんか事情知ってるんだろうな。

**nobodyさん** · 2009/08/10(月) 17:28:16

ある会社のＥＣＡＳＰを利用しているんだけど、モバイルのアフィリエイトのタグも設定できない、メジャー機種対応もできないで困っています。
どうしたらいいですか？教えてエコいひと。

**nobodyさん** · 2009/08/10(月) 18:42:37

>>329
http://procart.web.fc2.com/

**nobodyさん** · 2009/08/11(火) 01:09:25

>>330
どれも使う気にならないのはなんでだろう・・・

**nobodyさん** · 2009/08/11(火) 06:21:09

がんばってもうあきらめるんだ！

**nobodyさん** · 2009/08/12(水) 06:16:43

>>329

このスレにいるって事はまさか…。
手遅れかも知れんがEC-CUBE関連だけは止めとけな。

**nobodyさん** · 2009/08/13(木) 22:11:39

ちょっとうんこしてくる

**nobodyさん** · 2009/08/14(金) 05:00:36

うんこしてきた

**nobodyさん** · 2009/08/14(金) 07:05:08

うんこ山盛り乙

**nobodyさん** · 2009/08/15(土) 01:36:24

EC-CUBEでやってちゃんと集客出来るの?

**nobodyさん** · 2009/08/15(土) 01:59:36

何を使うかじゃない。何を売るかが重要だ

**nobodyさん** · 2009/08/15(土) 02:34:40

トラブったら入れ直す方が速いな

**nobodyさん** · 2009/08/19(水) 02:32:13

>>326
七節ってどなたかわ知らんでもいいが、2.xを実質作った人
ロックオンがこんなかんぢって1.xを世に送り出し、
けいぢばんでＤＢがＤＢがってあがってるのは1.x系譲り
（ＤＢ関連の問題は今に引き継がれている）
まだまだバグやら不具合やら多かったのにもかからわず、
なかば強引に、現状に至る。

1.x系でショップってた連中はかな～り反発したんだが、ポイッと捨てられた。
ふまえて、2.x系であっちいぢりこっちいぢり頑張ってても、
3.xとか出たら再度ポイッと捨て（られ）る可能性が高い。

もしかしたらｗ
2.x ほっぽって、3.xの開発でもやってんじゃないか？＜七節

**nobodyさん** · 2009/08/19(水) 17:46:17

日本語でおｋ

**nobodyさん** · 2009/08/19(水) 17:55:46

何でPageってファイルにごりごりSQLが書かれてるの?
つーか、全体的にSQLが読みにくい。

決め打ちでメソッドを作ればいいのに、フィールドとか条件とかを1個1個足していくから何をやってるか全然分からん。

**nobodyさん** · 2009/08/19(水) 17:57:18

あと、フォーラムの業者がけちくさい。
簡単なカスタマイズくらいコード書いて教えてやれ。

**nobodyさん** · 2009/08/19(水) 19:31:47

簡単だというのなら、自分でやれば良い。
簡単なことすらできない人間が、
サーバーにアプリ設置して運営なんてやるな。
社会にとって迷惑だ。

**nobodyさん** · 2009/08/19(水) 21:30:26

友達からカスタマイズ頼まれソースを読んでるけど、これってカスタマイズしやすいの?

LC_Page_Admin_Products_Product.php
LC_Page_Products_Detail.php

この2つは管理側、ショップ側の違いはあるけど概ね似たような情報が必要なのは予測出来る。
実際に「商品を読み込む」というECで最も重要な処理が両方にあるけど、両方ともにSQLがべた書きされてる。

//$mode = "admin";
$mode = "shop";

$product->read( $product_id , $mode );

こういうメソッドを1本用意すれば良いのに、コントローラーにSQLをべた書きしている。

カスタマイズを頼まれたから頑張る、詳細はまだ不明だけど。

**nobodyさん** · 2009/08/19(水) 21:55:06

レビュー、関連商品、お勧め商品なんかを取得する処理もコントローラーにべた書きされてますね。
管理側でレビューを編集したい場合どうするんだろうと思ってみたら、管理側にもべた書きされてた。

//$mode = "admin";
$mode = "shop";

$review->readByProductId($product_id,$mode);
$review->readByReviewId($review_id,$mode);

こんな感じで必要な情報のみを取得したらいいですかね。

**nobodyさん** · 2009/08/19(水) 21:58:02

Object - Product - ProductAdmin or ProductShop

こんな感じでオブジェクトごと切り分けたら、$modeをいちいち渡す必要もなくなりますね。

**nobodyさん** · 2009/08/20(木) 00:57:54

pages以下のクラスはモデルクラスって扱いなんだろ。少なくともリクエストを直接受けるコントローラーはhtml以下にあるんだから。
まあ、SQLを実行するような処理は、独立したクラスでまとめた方がいいとは思うが、DBから引き出した商品情報でも、ユーザ画面で必要とされるカラムと管理画面で必要とされるカラムは違いがあるわけで、
むしろハッキリと別の処理にしておいた方がお互いに影響が出ず、やりやすいこともあるだろう。
あとは、もっと単純に、開発時にユーザ画面と管理画面を作ってるプログラマーが別で、それぞれ独自に開発していったのかもしれない。
まあ、この手のオープンソースのウェブアプリでは比較的綺麗に書かれてる方だと思う。

**nobodyさん** · 2009/08/20(木) 02:25:21

LC_Page_Shopping_Complete.phpのトランザクションのところは大丈夫ですか?

$objQuery->begin();
$order_id = $this->lfDoComplete($objQuery, $uniqid);
$objQuery->commit();

lfDoComplete()の中にあるINSERTやUPDATEでこけてもrollbackしてないようです。

$objQuery->insert()
$objQuery->update()

これらのメソッドに失敗したときに処理がないようです。

$this->conn->query()も見ましたが、ここでも失敗したときはメールを送ってるだけっぽいです。

何か見落としていたらすいません。

**nobodyさん** · 2009/08/20(木) 09:30:02

おそらく、製品版はちゃんとしてるんじゃないかな
オープンソースだからわざとやってるんだろ
（或いは、未経験の新入社員に書かせているとか

**nobodyさん** · 2009/08/20(木) 12:18:38

とにかく場当たり的にあらゆるファイルにSQLを書いていますね。
PAGE,HELPER,UTIL,SESSION・・・などなど。

今日も帰ったらソースを読みますが、手を出すと撃沈しそうな雰囲気がありますね。
OneDepoも見てみます。

**nobodyさん** · 2009/08/20(木) 12:24:42

CSV_HELPERがデータベースに商品レコードを読みに行ってますね。

$product_list = $product->getProductList($condition);

$output = $csv_helper->outPut($product_list);
$output = $pdf_helper->outPut($product_list);

print($output);

商品一覧を取得するメソッドを1個作れば、
管理ページでもSHOPページでもCSVでもPDFでもどこにでも使えますよね。

そうしてないから、総てのページに商品を取得するクエリを書いてる。

今日も帰ったらソースを読みます。