Pega general code review checklist

1.  Declaratives over procedural.
2. Reusability and Components.
3. Extension points
4. Avoid loops
5. Server side validation vs Client side validation
6. 
   

1) Reusability - While creating/refactoring rules, always evaluate the reusability aspect. Create/move the rules to WorkPool/Work- class for better reusability.

2) Prefer Declarative over procedural processing.

3) Avoid creating section rules with 1-2 fields.

4) Avoid creating the top level scalar properties on the Casetype class (or WorkPool class). Designing better data model always fetch the future maintenance of the application. 

5) For properties that hold 'Amount' (or other decimal values) create property with type Decimal (with the desired precision as qualifier)

6) Leverage Background requestor for better user experience - Delegate the processing to background requestor such as queue processor/Agent.

7) Do not use the OOTB top-level properties for handling the processing logic. (For example, pyErrorMessage property was being used in svcPerformFlowAction OOTB activity).

8) Keep in mind the DB column size of the below properties (Especially pyLabel, PyDescription..etc)