Skip to main content
Code Review

Return to Answer

Commonmark migration
Source Link

###Code Smells

Code Smells

###Naming Conventions

Naming Conventions

###Repetitive Code Patterns

Repetitive Code Patterns

###Exception Handling

Exception Handling

###Resource Leaks

Resource Leaks

###Code Smells

###Naming Conventions

###Repetitive Code Patterns

###Exception Handling

###Resource Leaks

Code Smells

Naming Conventions

Repetitive Code Patterns

Exception Handling

Resource Leaks

replaced http://stackoverflow.com/ with https://stackoverflow.com/
Source Link
expanded recommendations
Source Link

Objects and other Variables are nouns. Methods and other functions are verbs. Name them accordingly. Do not use acronyms or abbreviations, but do not use sentences for your names either. There is no need for joining words such as 'The', or 'With'. Most importantly, one should also always strive to maintain consistent naming convention, whatever style you choose. This is more in reference to @Shree's use of executeWithPreparedStatement.

Re-using names that you are deriving from is OK when you are recreating a function that does the same thing in a different way. It is not OK when you arehave extended the functionality of the code you are deriving from. Pick a new name that better describes why your method is different than the original.

Objects and other Variables are nouns. Methods and other functions are verbs. Name them accordingly. Do not use acronyms or abbreviations, but do not use sentences for your names either. There is no need for joining words such as 'The', 'With'. Most importantly, one should also always strive to maintain consistent naming convention, whatever style you choose. This is more in reference to @Shree's use of executeWithPreparedStatement.

Re-using names that you are deriving from is OK when you are recreating a function that does the same thing in a different way. It is not OK when you are extended the functionality of the code you are deriving from. Pick a new name that better describes why your method is different than the original.

Objects and other Variables are nouns. Methods and other functions are verbs. Name them accordingly. Do not use acronyms or abbreviations, but do not use sentences for your names either. There is no need for joining words such as 'The' or 'With'. Most importantly, one should also always strive to maintain consistent naming convention, whatever style you choose. This is more in reference to @Shree's use of executeWithPreparedStatement.

Re-using names that you are deriving from is OK when you are recreating a function that does the same thing in a different way. It is not OK when you have extended the functionality of the code you are deriving from. Pick a new name that better describes why your method is different than the original.

expanded recommendations
Source Link
Loading
Source Link
Loading
default

AltStyle によって変換されたページ (->オリジナル) /