× The internal search function is temporarily non-functional. The current search engine is no longer viable and we are researching alternatives.
As a stop gap measure, we are using Google's custom search engine service.
If you know of an easy to use, open source, search engine ... please contact support@midrange.com.



David,

The 2nd WTF is when the next guy sees two constants defined with the
same value....

There's nothing wrong with that, in general, but it should be obvious
that either
it's two names for the basically the same value
d TRUE c '1'
d YES c '1'

or it's two completely different things that happen to have the same value
d DAYS_IN_A_WEEK....
d c 7

d MAX_RECURSION_DEPTH...
d c 7

In your case, the next guy is going to have to look at the code to
determine that there's two names for 'X0'.

Given the rest of the constants, and the inability to change all of
them, I would have just left cX0 by itself.

Charles


On Mon, Sep 6, 2010 at 3:17 AM, David FOXWELL <David.FOXWELL@xxxxxxxxx> wrote:
-----Message d'origine-----
De : rpg400-l-bounces@xxxxxxxxxxxx
[mailto:rpg400-l-bounces@xxxxxxxxxxxx] De la part de Charles Wilt

Bad idea IMHO, you should have either taken the time to
change all references or used the existing but poorly named one.

Charles, normally I would have, but then I didn't tell you about the other constants in the source, like

D cD1            C        'D1'
D cP2            C        'P2'

and all the others. What I feel really needed to be done was to enlighten the guy who wrote it. But I don't have the freedom to do so.


Having two names for the same thing makes things more
confusing for the next guy, not less.  You've went from one
WTF to two WTFs.

I don't agree. Now I have

IF MyProgrammer = cXO;

and then,

IF MyProgrammer = CantankerousOldGit;


Surely that's still only 1 WTF!!

WDSCi makes it easy to see every line with a X0, IIRC so does
SEU.  It shouldn't have been hard to change them.

If you're doing proper regression testing, it shouldn't
matter if you changed one line or fifty.

Even if you're not doing good regression testing, if you
can't change the name of a constant and feel good about your
changes you're probably in the wrong line of work.

That's a bit harsh. As I already said, the rules don't always let me proceed as I would like to do.
--
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.



As an Amazon Associate we earn from qualifying purchases.

This thread ...

Replies:

Follow On AppleNews
Return to Archive home page | Return to MIDRANGE.COM home page

This mailing list archive is Copyright 1997-2024 by midrange.com and David Gibbs as a compilation work. Use of the archive is restricted to research of a business or technical nature. Any other uses are prohibited. Full details are available on our policy page. If you have questions about this, please contact [javascript protected email address].

Operating expenses for this site are earned using the Amazon Associate program and Google Adsense.