[フロント]フロントパーツ

[kotani]

▼リファクタリング対象ファイル
/frontparts/bloc/LC_Page_FrontParts_Bloc.php
/frontparts/bloc/LC_Page_FrontParts_Bloc_Best5.php
/frontparts/bloc/LC_Page_FrontParts_Bloc_Calendar.php
/frontparts/bloc/LC_Page_FrontParts_Bloc_Cart.php
/frontparts/bloc/LC_Page_FrontParts_Bloc_Category.php
/frontparts/bloc/LC_Page_FrontParts_Bloc_Login.php
/frontparts/bloc/LC_Page_FrontParts_Bloc_News.php
/frontparts/bloc/LC_Page_FrontParts_Bloc_SearchProducts.php
/frontparts/LC_Page_FrontParts_LoginCheck.php

---

リファクタリングガイドライン
http://svn.ec-cube.net/open_trac/wiki/リファクタリングガイドライン

---

[kajiwara]

yomoro様　ありがとうございます！

取り急ぎ全ブロックの動作を確認してみました。
以下にて動作が不安定な部分がございますので、ご確認いただけると幸いです。

■ログインブロック
ログイン・ログアウト処理で下記エラー発生。
Fatal error: Call to a member function setParam() on a non-object in /data/class/pages/frontparts/LC_Page_FrontParts_LoginCheck.php on line 81

■カテゴリブロック
商品一覧ページを開いたとき、表示しているカテゴリーが開かない。

ということで、一旦チケットは差し戻しいたします。

また、別途ソースの確認を行い、フィードバックさせていただきます。

リファクタリングを開始したばかりで、色々分からないことがあると思いますが、随時フィードバックさせていただきますので、何卒よろしくお願い致します！！

[yomoro]

了解しました！チェック＆ご指摘有り難う御座います！ なるべく多くご協力できるようがんばります！

引き続き何卒宜しくお願い申し上げます！

[kajiwara]

yomoro様

株式会社ロックオンの梶原でございます。

今回、リファクタリングガイドラインが結構多岐にわたっており、そんな中で作業いただいており、本当にありがとうございます。
というところで、少し時間がかかってしまいましたが、ソースの確認をさせていただきました。

以下の点がリファクタリングガイドラインから外れておりますので、修正いただけるとありがたいです。

（1）http://svn.ec-cube.net/open_trac/wiki/リファクタリングガイドライン#ビジネスロジック

• 宣言を除き, 引数や返り値が無く, すべて内部のメンバ変数で処理するような関数は極力作成しない

こちらは分かりにくいと思いますので、申し訳ございませんが、以下チケットで同様の指摘をさせていただいておりますので、ご参照いただければ幸いです。

#979

▼その他注意事項
（1）関数の引数にオブジェクトのインスタンスを渡している場合, 明示的 に参照渡しをしていないので, PHP4 環境で予期せぬ不具合が発生する可能性があります.

（2）LC_Page_FrontParts_Bloc_Category.php に, 下記のようなコードが見られますが, SC_Display::detectDevice() は boolean を返しません.

if (SC_Display::detectDevice() === true) {

以上、姑みたいなチェックで申し訳ございません。

今回、「とりあえず動作する」⇒「振る舞いが綺麗なソースにする」　という観点で進めておりますので、ご了承いただきたく思います。
お忙しい中、ご協力いただいており恐縮ですが、ご確認のほどよろしくお願い致します。

また、分からないことがございましたら、他コミッター様への情報共有も兼ね、本チケットに返信いただけますと幸いです。

それでは、どうぞよろしくお願い致します。