(Apologizes in advance if this is a re-recording, but I did not find similar messages)
What bad method name templates you saw in the code and what he told you about the code.
For example, I continue to see:
public void preform___X___IfNecessary(...);
I think this is bad because operation X has an inverse of conditions. Note that this is a public method, as class methods may legally need private helpers such as
Sometimes developers seem to have a problem using a short wording. I had the one who called the procedure
InsertImportQueueRecord
I hated is the name. I changed it to
ImportItem
, , . , , , - QueueItemImport ScheduleImport, , . , - , , .
QueueItemImport
ScheduleImport
(), doThing() reallyDoThing()
, , . , , - , , . ?
. . , , . , , .
, , , .
ValidateFormData_PersistToDB_SendEmail().
ValidateFormData_PersistToDB_SendEmail()
#, .
, , :
private void SOMETHINGBecauseOf__a__(..); private void SOMETHINGBecauseOf__b__(..); private void SOMETHINGBecauseOf__c__(..);
, OfOf, . switch/if .
Source: https://habr.com/ru/post/1746910/More articles:C ++ interception process and display status - c ++Migrate an ASP.NET application to the cloud - asp.netHow can I quickly try out a large number of regular expressions and find out which one matches? - regexAssociationTypeMismatch with expected type in nested model forms - ruby ββ| fooobar.comLinq with a long clause where - c #PHP String Encoding Error - stringHow to create this array? (PHP) - arraysASP.NET MVC 2: Mechanics behind the order / order line in the editing form - asp.net-mvcHow to save service settings by installing InstallShield update - installerΠ‘ΠΈΠΌΠ²ΠΎΠ» "Ctypes" Π½Π΅ Π½Π°ΠΉΠ΄Π΅Π½ "Π΄Π»Ρ Π΄ΠΈΠ½Π°ΠΌΠΈΡΠ΅ΡΠΊΠΎΠΉ Π±ΠΈΠ±Π»ΠΈΠΎΡΠ΅ΠΊΠΈ Π² OSX - c++All Articles