2012-01-13 13 views
-1

こんにちはstaskoverflowコミュニティ。PHPコード改良リファクタ

PHP 5コードの次の構造をどのように改善できますか? モデル$月のリスト(12以上) $ month->日(31以上)

$日 - >クライアントクライアントのリスト $日 - >は$クライアント

のためのアクションのリストをsomeActions

すべての$クライアントに対してすべての$アクションを呼び出す必要があります。だから私の工事は:

私はこのコードを改善することができますか?

+3

このコードブロックのパフォーマンスや可読性を向上させることはできません。おそらく、これらの$アクションが何であるか、またはこのアプリケーションが何をしているのか説明できれば、助けてくれるかもしれません。例えば、あなたの 'days'と' clients'プロパティは、データベースから遅れて埋められていますか?その場合、このコードは多くのクライアントと数ヶ月で非常に遅くなる可能性があります。要するに、私は言っている:より多くの文脈が必要です。 –

+1

また、http://codereview.stackexchange.com/ – Mchl

+0

に興味があります。ありがとう、私はそのまま使用します。 – RusAlex

答えて

2

なぜネストされたループから移動したいのですか?私は最初の反復回数を減らすためにしようとします

  1. :あなたが持っている コードを理解することは非常に明確であり、あなたの目的は、この方法の性能を最適化することであるならば、私は行くだろうなアプローチは次のようになります。何ヶ月のデータを実際に外出先に表示したいのかなど、自分に質問することでこれを行うことができます。
  2. 類似した質問には日数とクライアントについて問い合わせることができます
  3. 休憩条件はありますか?これが意味するのは、

    $ result = $ action-> run($ client); if( "break_condition" == $ result){ break; }

これにより、反復回数が大幅に削減されます。

関連する問題