Apparently there is no limit to naming conventions for method
-
public void SettingDeviceList_ReAddingAPreviouslyDeletedDevice_MustAddDevicesToRuleDevicesWithIsDeletedFalseAndIsIncludedTrue()
{
return;
}I feel like crying. X|
What is actually so bad about this? The length only? If yes, then WTF? You are kvetching about length? Why? If not, then fill it in here: __________________________
-
What is actually so bad about this? The length only? If yes, then WTF? You are kvetching about length? Why? If not, then fill it in here: __________________________
-
Eh? Unreadable in what sense? I can read quite clearly, as well as understand the intent. Long names are inherently self-documenting. I sincerely do NOT understand your response. To me, tersely abbreviated names are not only unreadable, but quite unnecessary in this day-and-age of auto-completion.
-
public void SettingDeviceList_ReAddingAPreviouslyDeletedDevice_MustAddDevicesToRuleDevicesWithIsDeletedFalseAndIsIncludedTrue()
{
return;
}I feel like crying. X|
I've used support software from a certain hardware manufacturer. I think their software was written by an intern. All of the names are of the form:
#define ACR1_ACR2_ACR3_..._ACRN 0x........
where each
ACR_i_is a three or four letter acronym for a hardware signal or state. Some of the names are well over 100 characters long, and completely unreadable.Software Zen:
delete this; -
Eh? Unreadable in what sense? I can read quite clearly, as well as understand the intent. Long names are inherently self-documenting. I sincerely do NOT understand your response. To me, tersely abbreviated names are not only unreadable, but quite unnecessary in this day-and-age of auto-completion.
-
Eh? Unreadable in what sense? I can read quite clearly, as well as understand the intent. Long names are inherently self-documenting. I sincerely do NOT understand your response. To me, tersely abbreviated names are not only unreadable, but quite unnecessary in this day-and-age of auto-completion.
The same reason that writing a 4 page essay in a comment in the lounge is unreadable. You have to parse a lot of text before you can begin to understand what the purpose is. Just get to the point, which I'm sure can be expressed in fewer than 15 words or however many are in that title.
-
Yeah, its like climbing Rapunzel's tower to find out that she is actually a he.
-
Eh? Unreadable in what sense? I can read quite clearly, as well as understand the intent. Long names are inherently self-documenting. I sincerely do NOT understand your response. To me, tersely abbreviated names are not only unreadable, but quite unnecessary in this day-and-age of auto-completion.
-
You guys have obviously never written any objective C. The following method signature (without defined parameter names) is a legit initializer method from the Apple code base for the NSBitmapImageRep[^][] object. God only knows what some users have made.
initWithBitmapDataPlanes:pixelsWide:pixelsHigh:bitsPerSample:samplesPerPixel:hasAlpha:isPlanar:colorSpaceName:bitmapFormat:bytesPerRow:bitsPerPixel:
-
Eh? Unreadable in what sense? I can read quite clearly, as well as understand the intent. Long names are inherently self-documenting. I sincerely do NOT understand your response. To me, tersely abbreviated names are not only unreadable, but quite unnecessary in this day-and-age of auto-completion.
In college, did you title your essays using the entire contents of the essays, or did you use something shorter? Longer variables take longer for humans to read, they can lead to extra horizontal scrolling (the Devil!), they take longer to type, and they are not necessary.
INSWSAEJETGTMA...
Im_Not_Saying_We_Should_Abbreviate_Excessively_Just_Enough_To_Get_The_Meaning_Across...
// If you want to explain something more, put it in a comment above the variable declaration so it doesn't follow the variable around.
abbreviateAppropriately... -
In college, did you title your essays using the entire contents of the essays, or did you use something shorter? Longer variables take longer for humans to read, they can lead to extra horizontal scrolling (the Devil!), they take longer to type, and they are not necessary.
INSWSAEJETGTMA...
Im_Not_Saying_We_Should_Abbreviate_Excessively_Just_Enough_To_Get_The_Meaning_Across...
// If you want to explain something more, put it in a comment above the variable declaration so it doesn't follow the variable around.
abbreviateAppropriately.../** <summary>That should go here.</summary> */
[Description("Or perhaps here.")]
abbreviateAppropriately...so it does follow the method around and is more localizable.
-
public void SettingDeviceList_ReAddingAPreviouslyDeletedDevice_MustAddDevicesToRuleDevicesWithIsDeletedFalseAndIsIncludedTrue()
{
return;
}I feel like crying. X|
-
public void SettingDeviceList_ReAddingAPreviouslyDeletedDevice_MustAddDevicesToRuleDevicesWithIsDeletedFalseAndIsIncludedTrue()
{
return;
}I feel like crying. X|
I thought I am the naming jerk :laugh:
-
Look on the bright side - it's broken up into easily manageable chunks by _'s Imagine it was not...
-
Nice ans well, but I expect some rcommendation any time now that will tell us to wear feathers on our heads and chant while dancing around the computer.
At least artificial intelligence already is superior to natural stupidity
-
What is actually so bad about this? The length only? If yes, then WTF? You are kvetching about length? Why? If not, then fill it in here: __________________________
SettingDeviceList_ReAddingAPreviouslyDeletedDevice_MustAddDevicesToRuleDevicesWithIsDeletedFalseAndIsIncludedTrueYou genuiely think this is readable? I got bored half way through, though it is 1 am where I am.Michael K Gray wrote:
If not, then fill it in here:
Well in probability this is several methods and attributes:
public void ReAddRequiredDeletedDevice()
{
//OK this if is a punt, but if gives the idea
if(!Device.IsDeleted && Device.IsIncluded)
{
//Add or whatever
}
}public void SetDeviceList()
{
ReAddRequiredDeletedDevice();
return; //Hey the one useful line of code from the OP!
}Sort of a cross between Lawrence of Arabia and Dilbert.[^]
-Or-
A Dead ringer for Kate Winslett[^] -
public void SettingDeviceList_ReAddingAPreviouslyDeletedDevice_MustAddDevicesToRuleDevicesWithIsDeletedFalseAndIsIncludedTrue()
{
return;
}I feel like crying. X|
That is not real! I can't believe
-
:(( :wtf: :(( :wtf: :(( :wtf: :(( Is that even called somewhere?!??! On thinking, it might be a stub that never got filled in for some reason, but the name is absolutely unreasonably ridiculously insanely crazy!
public class SysAdmin : Employee
{public override void DoWork(IWorkItem workItem) { if (workItem.User.Type == UserType.NoLearn){ throw new NoIWillNotFixYourComputerException(new Luser(workItem.User)); }else{ base.DoWork(workItem); } }}
Zac Greve wrote:
absolutely unreasonably ridiculously insanely crazy
private void absolutely_unreasonably_ridiculously_insanely_crazy()
{
return;
}FTFY
Alberto Bar-Noy --------------- “The city’s central computer told you? R2D2, you know better than to trust a strange computer!” (C3PO)
-
public void SettingDeviceList_ReAddingAPreviouslyDeletedDevice_MustAddDevicesToRuleDevicesWithIsDeletedFalseAndIsIncludedTrue()
{
return;
}I feel like crying. X|
I remember some years ago seeing this from one developer
void CommunicationsBlockScheduler::PerformScheduleAndExecuteOfCommunicationBlocksOnAProportialAndPriorityBasis(..)
And another developer on the same team took the oposite approach With gems like this:
void rssr()
This was replicate spread sheet rows
-
public void SettingDeviceList_ReAddingAPreviouslyDeletedDevice_MustAddDevicesToRuleDevicesWithIsDeletedFalseAndIsIncludedTrue()
{
return;
}I feel like crying. X|
I feel like laughing my arse off :D That's what you do when you: a) don't know how to use an IDE b) don't know what OOP stands for In particular, things like this, among others, were at the origin of OOP creation. Back in the seventies, structured programming hit a wall. Among other things, when your C program grew to 70 KLOCs or so, it seems, you ran out of reasonable identifiers. OOP, providing encapsulation and nice mechanisms to avoid global/public data, got past this issue.