Difference between revisions of "Coding standards"

From TempusServa wiki
Jump to navigation Jump to search
 
(10 intermediate revisions by the same user not shown)
Line 1: Line 1:
=== Naming standards (tobe) ===
=== Naming standards ===
Package / naming placement
==== Package / naming placement ====
* Package name all lowercase
* Package name all lowercase
* General platform entities
* General platform entities
** Servlets
** Servlets: '''com.tsnocode.server.servlet'''
** Fields
**Filters: '''com.tsnocode.server.filter'''
** Views
**Core: '''com.tsnocode.ui'''
** Fields: '''com.tsnocode.ui.field.<group>'''
** Views: '''com.tsnocode.ui.views.<type>  (item,list,statistics,)'''
**Codeunit: '''com.tsnocode.codeunit.common'''
* Solution specific code
* Solution specific code
** TS customer app:
*** '''com.tsnocode.solution.app'''
*** '''com.tsnocode.solution.customer'''
*** '''com.tsnocode.solution.customer.app'''
** Vendor app:
*** '''com.vendor.tsnocode.app'''
*** '''com.vendor.solution.app'''


Class naming
==== Class naming ====
Naming convention follows Java standard
* Starting with uppercase
* Starting with uppercase
* No special characters
* No special characters
* Seperate words with uppercase
* Seperate words with uppercase
In addition the following are recommended
* CodeunitPage post fix name "Page"
* CodeunitService post fix name "Service"
* Shared ressources include "Helper"
==== Table naming ====
System tables should follow this syntax
* lowercase
* no special characters
Reserved prefixes include
* data_
* _data_ (deleted tables)


=== Safety standards ===
=== Safety standards ===
Line 17: Line 43:
** Escape all string parameters using DbConnection.EscapeSQL
** Escape all string parameters using DbConnection.EscapeSQL
** Prepared statements  
** Prepared statements  
* Page codeunits should contain user group restrictions
* All Page codeunits should contain some user group restrictions, and set to be default off


=== Review standards ===
=== Review standards ===
The following changes requires a formal code review
The following changes requires a formal code review
* Changes to the API
* Changes to common objects
* Changes to common objects
** Command
** Command
Line 27: Line 54:
** DbConnection
** DbConnection
* Creation of OR changes to
* Creation of OR changes to
* Servlets
** Servlets
* CodeunitPagePublic
** CodeunitPagePublic
** Code containing SQL

Latest revision as of 11:57, 24 April 2024

Naming standards

Package / naming placement

  • Package name all lowercase
  • General platform entities
    • Servlets: com.tsnocode.server.servlet
    • Filters: com.tsnocode.server.filter
    • Core: com.tsnocode.ui
    • Fields: com.tsnocode.ui.field.<group>
    • Views: com.tsnocode.ui.views.<type> (item,list,statistics,)
    • Codeunit: com.tsnocode.codeunit.common
  • Solution specific code
    • TS customer app:
      • com.tsnocode.solution.app
      • com.tsnocode.solution.customer
      • com.tsnocode.solution.customer.app
    • Vendor app:
      • com.vendor.tsnocode.app
      • com.vendor.solution.app

Class naming

Naming convention follows Java standard

  • Starting with uppercase
  • No special characters
  • Seperate words with uppercase

In addition the following are recommended

  • CodeunitPage post fix name "Page"
  • CodeunitService post fix name "Service"
  • Shared ressources include "Helper"

Table naming

System tables should follow this syntax

  • lowercase
  • no special characters

Reserved prefixes include

  • data_
  • _data_ (deleted tables)

Safety standards

  • Input sanitization used in SQL, using either
    • Escape all string parameters using DbConnection.EscapeSQL
    • Prepared statements
  • All Page codeunits should contain some user group restrictions, and set to be default off

Review standards

The following changes requires a formal code review

  • Changes to the API
  • Changes to common objects
    • Command
    • Security
    • Controller
    • DbConnection
  • Creation of OR changes to
    • Servlets
    • CodeunitPagePublic
    • Code containing SQL