Survey Results

an underscore and plain english (camel case)

fabulous7-Apr-06 16:10

7-Apr-06 16:10

because of the features in IDEs nowadays, I have left hungarian for good (I think)

I make my locals easy to understand like this underscore then camel case

_lastName

And if any one checking my code cannot understand what that is, I assume that they will point the mouse at it and Visual Studio (I use .NET) will tell them how it was declared, or they should not be programming.

Let's make things simpler than possible.

-- modified at 22:12 Friday 7th April, 2006

vobject7-Apr-06 1:53

vobject

7-Apr-06 1:53

I used HN for quite a long time, but i always had a feeling that it was not "good".

This article finally told me why:

http://www.cuj.com/documents/s=7989/cujcexp1911hyslop/hyslop.htm

And after reading "C++ Coding Standards" I finally stopped using hungarian notation. in the beginning it was hard to think about variable names, but now it is no problem anymore Smile | :)

C++ is a typesafe language and if you name your variables reasonable there is no more need for HN.

For C style coding it might seem to be useful but as Herb Sutter already said: "Avoid Hungarian notation. It will make a liar out of you. Warts are not information, but disinformation."

lpatchRegards.

Just us GUIDs

Troy Marchand6-Apr-06 18:23

Troy Marchand

6-Apr-06 18:23

If you use GUIDs you never have to worry about naming conflicts, and variable scoping is highly over-rated anyways. Plus your code is automatically obfuscated. And its also great for job security (unless everyone realizes that you can't figure out your code either) OMG | :OMG:

Troy

I don't have local variables

Marek.T6-Apr-06 11:52

Marek.T

6-Apr-06 11:52

I don't use local variables, Did you forget about the lispers? Smile | :)

K.I.S.S.

David Veeneman6-Apr-06 9:27

David Veeneman

6-Apr-06 9:27

Keep it short and simple

camelCasingForLocalVariables
m_PrefixForMemberVariables
p_PrefixForPropertyVariables

David Veeneman
www.veeneman.com

Re: K.I.S.S.

Alvaro Mendez7-Apr-06 5:08

Alvaro Mendez

7-Apr-06 5:08

David Veeneman wrote:
m_PrefixForMemberVariables
p_PrefixForPropertyVariables

Why do you find it necessary to distinguish these two? It's very easy for a member to be exposed as a property. Do you then rename your members?

I keep it extra simple:

camelCasingForLocalVariables
m_camelCasingForFields (class members)

Alvaro

The bible was written when people were even more stupid than they are today. Can you imagine that? - David Cross

Using C# .Net then use Microsoft's

Daniel Vaughan6-Apr-06 3:22

Daniel Vaughan

6-Apr-06 3:22

http://msdn.microsoft.com/library/default.asp?url=/library/en-us/cpgenref/html/cpconNamingGuidelines.asp

Honestly, Hungarian notation makes my eyes bleed.

Daniel Vaughan
Zen Diaphragm

Using C++ and using C# style

FocusedWolf7-Apr-06 5:22

FocusedWolf

7-Apr-06 5:22

Well i got into c# first before getting to know c++, so now i write my c++ classes with the attitude of trying to make it more C# like...If i got a public variable, i capitialize its first letter to make it like a c# property...private variables are lower case on first letter with each word after starting with upper case.

I hate hungarian to...i think it was invented for back when code was written in a notepad setting...like what kinda noob would do that now (thinks of some linux code tools that are nothing but elaborate textpads)... basically cause you didn't have something like intellisense that could tell you what type of variable something was without looking for where it was declared... now hungarian is just noise in the code! (IMHO)

C# style is as good as code can appear to me lol

thx for link: http://msdn.microsoft.com/library/default.asp?url=/library/en-us/cpgenref/html/cpconNamingGuidelines.asp

now when i work on a c++ game project, and invite other members, i can say...THIS is the style!!! Laugh | :laugh:

Re: Using C++ and using C# style

vobject9-Apr-06 21:42

vobject

9-Apr-06 21:42

Of course public member variables are no style at all Big Grin | :-D

Trance Junkie4-Apr-06 23:51

Trance Junkie

4-Apr-06 23:51

Local = GoodPerson ;
Global = good_person ;
Global controls = m_ctrlname ;
Classes = CPerson ;
:->

-- modified at 5:52 Wednesday 5th April, 2006

Re: Me

Bob Stanneveld5-Apr-06 23:34

Bob Stanneveld

5-Apr-06 23:34

Hello,

Not really a consistent approach.. Your locals are in CamelCase. Your globals don't follow a specific guideline. Your global controls are prefixed with the scope m_(ember), prefixed with hungarian notation and the actual name doesn't have a capital letter? Your classnames follow the Microsoft convention.

Why?

Behind every great black man...
... is the police. - Conspiracy brother

Blog[^]

Re: Me

Malik Nasir31-May-06 0:07

Malik Nasir

31-May-06 0:07

not a good approach there should be some difference

Regards

My use of "Scope and Hungarian prefix"

Kochise4-Apr-06 21:40

Kochise

4-Apr-06 21:40

To keep some consistency of my source code and help people understanding it afterward, I use I slightly modified "Scope and Hungarian prefix" notation, I explained in my CSkinProgress control :

// g_Global      : g_ = Global
// i_Input       : i_ = Input
// no z...
// m_Member      : m_ = Member
// o_Output      : o_ = Output
// l_Local       : l_ = Local
// ...           : use your imagination

// aArray        : a  = Array    (array)
// bBool         : b  = Boolean  (boolean, TRUE/FALSE)
// cConstant     : c  = Constant (constant, whatever...)
// dDefine       : d  = Define   (simple define or defined value)
// eEnum         : e  = Enum     (enum list element)
// iIterator     : i  = Iterator (STL iterator)
// nNumber       : n  = Number   (char, long, int, whatever...)
// oObject       : o  = Object   (C++ class)
// pPointer      : p  = Pointer  (typicaly a 32 bits ulong address)
// sStruct       : s  = Struct   (structure)
// tTemplate     : t  = Template (template)
// uUnion        : u  = Union    (join two or more values of the same size under a common name)
// vVector       : v  = Vector   (STL vector, mostly like an array)

// poRemoteClass : po = Pointer on Object
// cdConstantDef : cd = Constant Define, typicaly a constant defined value
// usUnionStruct : us = Union of Structures
// ...           : use your imagination

// o_psBitmap    : CBitmap::GetBitmap((BITMAP*) o_psBitmap); // s = BITMAP, p = *, o_ means it's an output

For private/protected members of a class, I start with 'mp_' (means MemberPrivate_). All these informations, though not used by the compiler, could help to understand the focus and life time of a variable (g_ stands for the whole application life time, m_ for the class life time, l_ for the method life time, ...).

The hungarian prefix helps to remind the reader what kind of variable it is. You don't always have the constructor/declaration on screen (even if all methods are supposed to fits in one screen), so it's kind to know what you are dealing with. Not every IDE have IntelliSense, I sometimes code on notepad...

This is also quite useful to chage a variable from a scope and type to another, without changing other variable. For instance, changing the mp_nSerial to m_nSerial when it has to become public, or m_nSerial to m_oStrSerial when there is no more enough space on the int value and/or you may support characters in your serial.

Don't forget you cannot use a int like a string. You may not do m_nSerial.Format("%d", GetSerial()), but m_nSerial = GetSerial() instead... That's where hungarian prefix helps, it prevent you from even thinking doing some kind of .Format(stuff) Smile | :)

Things to read though :

http://www.joelonsoftware.com/printerFriendly/articles/Wrong.html

http://discuss.fogcreek.com/joelonsoftware/default.asp?cmd=show&ixPost=35396

Kochise

In Code we trust !

-- modified at 3:43 Wednesday 5th April, 2006

Datagrid new line

Macky1004-Apr-06 3:26

Macky100

4-Apr-06 3:26

Alright folks

having a bit of a problem with a datagrid. i set up the datagrid as read only to start with then using tablestyles set some columns as readonly = false.

When i first insert a new row into the datagrid it actually shows 2 lines but only 1 line is active, which i want but i don't need the want the other line.

So all i want when i insert a line into the datagrid is the line i create. It seems to do it when all datagrid read only. Is their any functions or validate checks that stop this from happening

any help is appreciated

regards

mack100

Re: Datagrid new line

Arch4ngel4-Apr-06 10:52

Arch4ngel

4-Apr-06 10:52

Wrong thread dude.

This is a thread about the result of the survey.

You get more chance getting an answer on the proper thread.

If someone says "Die mortal!", don't stay to see if he isn't.

CListCtrl

Phil J Pearson4-Apr-06 1:11

Phil J Pearson

4-Apr-06 1:11

Well ... someone had to vote for it!

thrakazog3-Apr-06 9:17

thrakazog

3-Apr-06 9:17

For job security shouldn't we just name all variables X? Or, just one letter names from a-z. In C# you could use both "X" and "x" in the same block. You would be hard to replace if nobody but you understood your code.

Re: X

Rohit Wason3-Apr-06 9:58

Rohit Wason

3-Apr-06 9:58

Good idea! Bur, the trouble would be when you'll spend better part of the day understanding your own code
Big Grin | :-D

Cheers,
Rohit Wason

Re: X

Gary R. Wheeler3-Apr-06 15:40

Gary R. Wheeler

3-Apr-06 15:40

thrakazog wrote:
just one letter names from a-z

Believe it or not, one of the first versions of BASIC for personal computers (called 'Tiny BASIC') supported exactly 26 variables, named 'A' through 'Z'.

Software Zen: delete this;

Fold With Us![^]

hairy_hats7-Apr-06 5:19

hairy_hats

7-Apr-06 5:19

One of the machine tool programming languages we use used to only allow variables called V - V1, V2, V3, V4... Made programming in Brainf--k look appealing!

Re: X

toxcct4-Apr-06 0:58

toxcct

4-Apr-06 0:58

you know, code obfuscating tool exist !! Unsure | :~

what if you (or one of your colleagues) have to maintain the code several months later ?
very bad convention sir. i strongly deprecate it

Re: X

KarstenK4-Apr-06 2:58

KarstenK

4-Apr-06 2:58

It also can you the wrong way, that your boss is guessing you arent a serious programmer Unsure | :~

PS: Writing maintainble code belongs to the duties of a programmer.

Greeting from Germany

Re: X

KevinHall4-Apr-06 4:59

KevinHall

4-Apr-06 4:59

I just don't get these other replies. Future maintenance? What your boss thinks? Bahh!!!! Personally, I name all my variables and functions with 'l' and '1':

l, ll, l1, ll1, l1l, l11, llll, lll1, (and so on...)

I mean, take a look at the beauty of the following code:

<br />
     for(ll1ll1l=l11llll; ll1ll1l=l111lll; ++ll1ll1l)<br />
     {<br />
         ll1ll1l1[ll1ll1l].ll1l1l();<br />
     }<br />

Re: X

WillemM5-Apr-06 21:09

WillemM

5-Apr-06 21:09

Very handy dont you think? Sigh | :sigh:

If I was your boss and you would make code like that, you would be fired the next day. This will cost a lot of money to maintain and if you wait to long to fix code like this, you will spend even more money to fix it...

WM.

What about weapons of mass-construction?

Re: X

KevinHall6-Apr-06 7:57

KevinHall

6-Apr-06 7:57

I can't believe anyone actually believes I or thrakazog is serious. Laugh | :laugh:

(At least I hope thrakazog isn't serious or he will have a hard time keeping a job.)

Obviously such code will get you fired in a heartbeat (should someone review it that is).

Here's a more subtle approach which will probably eventually get you fired, just not right away: How to Write Unmaintainable Code[^]

Use Ctrl+Left/Right to switch messages, Ctrl+Up/Down to switch threads, Ctrl+Shift+Left/Right to switch pages.

Option	Votes	%
Pascal Cased	171	10.56
camel Cased	702	43.36
Fixed letter prefix (eg `lLocal`)	81	5.00
Hungarian prefix (eg `strLocal`)	482	29.77
Scope prefix (eg `l_Local`)	36	2.22
Scope and Hungarian prefix (eg `l_strLocal`)	125	7.72
Total	1,619	100%

Coding style: How do you name your local variables? [Edit]