OutSystems愛知ユーザー会#3使用　回答例

Transcript

1. 回答例

2. 使用していない変数、Entityは削除する • GroupEntity • TaskEntity: Attrebute GroupId • ｈome ：ローカル変数Count

   • Tasks Preparation：GetGroupAggregate • ServerActions: GetTasksByStatus

3. 命名規則に沿った内容にする • 意味が伝わりやすい名前にする – WebScreen : Detail→TaskDetail – InputParameter :

   commentInsert Task → TaskId • パスカルケースを使用する – WebScreen : home → Home commentInsert → CommentInsert • 外部キーには「Id」というサフィックスを付ける – Entity : Comment User → UserId Task → TaskId Task Priority → PriorityId

4. ハードコーディングは避ける • home：Preparationでのデータ取得 GetTasksTodo

5. ハードコーディングは避ける • home：Preparationでのデータ取得 GetTasksdoing,GetTasksComplete

6. ハードコーディングは避ける • Tasks の優先度アイコンの表示設定

7. SQLよりはAggregateを使う • home：PreparationGetTasksdoing,GetTasksComplete GetTasksTodoと同じ方法で代替可能

8. クエリの数を最小限にする • home：ChartStatusCountはForEachを使わなくても取得可能

9. クエリの数を最小限にする • home：ChartStatusCountはForEachを使わなくても取得可能

10. 再利用化する • タスク一覧にある優先度アイコンはhomeにもTasksにもあるの でWebBlockにまとめる – InputParameterによってアイコンを変えるWebBlockを作る – home、TasksにWebBlockを配置

11. 再利用化する • タスク一覧にある優先度アイコンはhomeにもTasksにもあるの でWebBlockにまとめる – InputParameterによってアイコンを変えるWebBlockを作る – home、TasksにWebBlockを配置

12. 再利用化する(ロジック)① • homePreparationの GetＴask～はStatus毎の件数 を出してExpressionに表示して いる

13. 再利用化する(ロジック)① • homePreparationの GetＴask～はStatus毎の件数 を出してExpressionに表示して いる

14. 再利用化する(ロジック)② • StatusId:InputParameter 件数:OutputParameterの ServerActionを作成 (FunctionはYesにする)

15. 再利用化する(ロジック)③ • ExpressionValueにServerActionが 選択できるように

16. モジュールを分割する① モジュールをCloneする以外の手順 ①データを保持するモジュールを作成 ②分割前のモジュールのEntityを参照する ③参照元のEntityのコピーを作成する

17. モジュールを分割する② ③参照元のEntityのデータをコピーしたEntityに保存する アクションを作る ④③のアクションを実行する(タイマーなどで) ⑤コピーしたEntityを参照して動作確認できたら 分割前のモジュールのEntityを削除する

18. その他細かい箇所 • Is Mandatoryプロパティを忘れずに設定する – Entity:Task Titleや Comment:Text (Formも同様に設定する) •

    サイトプロパティはロジック内で書き換えない – Tasks：PreparationのFileter（Site.Tasks_SearchStatusId） • 表示、非表示の定義の仕方は統一したほうがよい – Commentformの表示・非表示だけ プロパティ：Displayに設定している