Re: Does the recommendation against variable names starting with SQL apply to constants?

Dennis,

Just because something isn't broken, doesn't mean it can't be improved upon.

My first idiot test is, how many times per minute do I say WTF while
looking at the code...

If I can reduce the WTF's per minute with relative ease, I do so.
That will make things easier on the next poor slob to have to change
something. That saves $$$ in the long run.

Maybe it's left over from the boy scouts, but I try to leave places
better than I found them.

Charles

On Fri, Sep 3, 2010 at 5:51 AM, Dennis Lovelady <iseries@xxxxxxxxxxxx> wrote:

For me,  it would have to pass the first idiot test: "Is it broken," and a
source members search would likely not be involved.

Dennis Lovelady
http://www.linkedin.com/in/dennislovelady
--
I bought some powdered water, but I don't know what to add to it.

readability of the code is a big one for me.

Well here's a question for you.

What would you do ? :

Working on a RPGIII that has just been converted to IV.
Need to test a field to see if its value is 'X0'.
Search the source for 'X0' to see if I need to create a constant.
In the D specs I find :D cX0             C                   CONST('X0')
'c' being the RPGIII standard prefix for constants.
I realize that the length available for constant names did not make for
readability, but...
Reading further, I find that most of the constants are coded in the
same way.


Pride ( or plain stubborn ) will not let me use the constant cX0, so
I'm adding my own long constant name with the same value.



--
This is the RPG programming on the IBM i / System i (RPG400-L) mailing
list
To post a message email: RPG400-L@xxxxxxxxxxxx
To subscribe, unsubscribe, or change list options,
visit: http://lists.midrange.com/mailman/listinfo/rpg400-l
or email: RPG400-L-request@xxxxxxxxxxxx
Before posting, please take a moment to review the archives
at http://archive.midrange.com/rpg400-l.

--
This is the RPG programming on the IBM i / System i (RPG400-L) mailing list
To post a message email: RPG400-L@xxxxxxxxxxxx
To subscribe, unsubscribe, or change list options,
visit: http://lists.midrange.com/mailman/listinfo/rpg400-l
or email: RPG400-L-request@xxxxxxxxxxxx
Before posting, please take a moment to review the archives
at http://archive.midrange.com/rpg400-l.