arl: products: code review: sample_code_review_001
missing package definition
[translate]
  • [translate]
  • [translate]
  • [translate]


* imports
[translate]
  • [translate]
  • [translate]
  • [translate]


final static definitions
[translate]
  • [translate]
  • [translate]


// commenting
[translate]

missing Javadoc commenting
[translate]

servlet parametering
[translate] [translate]

use defined constants withing switch/case sentences
[translate]

servlet class usage
[translate] [translate] [translate]

java.util.Date class should not be used
[translate] [translate]

right JDBC usage
[translate] [translate] [translate]

Hashtable usage as object/class
[translate]

java.net.URL class should be used
[translate]

full class name usage in sample code
[translate]
  • [translate]
  • [translate]



© arl
updated: 20070711