0

私は、Salesforce を介して当社での応募者のステータスを詳述する申請書を書いています。当社の従業員の 1 人が問い合わせ ID を入力すると、ステータス (クリア済み、未クリア) が表示され、クリアされていない場合は、プログラムを続行する前に申請者が修正する必要があるものが表示されます。

アプリケーションのさまざまな領域について正しく考えていることを確認したいと思います。ここに私が持っているものがあります:

  • モデル: 申請者クラスには などの動的関数がApplication.find_by_Enquiry_Token__c_and_Account_dot_LastName_from_Opportunityあり、要求されると Salesforce から情報を返します。
  • Controller : Salesforce から返されたデータを解析し、 などの情報でハッシュを作成します@applicant[:general_information] = {:first_name = data[:Account].first[:FirstName], :last_name = data[:Account].first[:LastName]}。
  • View : コントローラによって生成された情報を表示します。ただし、独自のロジックとチェックがあり、クリア ( )、クリアでない場合( )、条件付き情報がある場合( )divに応じてa の色を変更するなどです。class="success"class="danger"class="warning"

私はこれが正しいと思いますが、主に色付けするだけでなく特定のエラーを表示するために、返されたデータに基づいてチェックを実行するための Ruby コードが少しあるため、ビューが少し心配です。これは大丈夫ですか/これは基準に従っていますか? または、アプリケーションをリファクタリングして、これをコントローラーにプッシュする必要がありますか?

4

3 に答える 3

0

ビューで長いクエリを実行したり、インスタンス変数を設定したりしない限り、ビューで行うようなルビー コードを使用しても問題ないと思います。また、ビューからコントローラーへのコードの移動を開始するもう 1 つの兆候は、ビューが雑然として理解しにくくなっていると感じた場合です。おっしゃる通りですが、問題ないようです。

変更をお勧めすることの 1 つは、モデルのメソッド名を短くすることです。メソッド名が短いほど理解しやすくなり、メソッド名が非常に長くなり扱いにくくなります。それ以外は、すべてうまくいっていると思います！

于 2013-09-21T05:58:54.373 に答える