Re: Ext Print Files - In the Beginning *SPAM: 4*

[James Perkins <jrperkinsjr@xxxxxxxxx>]

The worst is when you can tell they ran out of indicators and you see
something like:
savind21 = *in21;
*in21 = (this <> that);
if (*in21);
  // do something;
endif;
*in21 = savind21;

And yes, I have seen this before. In this instance, stay on the soapbox ;-)
--
James R. Perkins
http://twitter.com/the_jamezp


On Mon, Jul 13, 2009 at 09:18, <BMay@xxxxxxxxx> wrote:

> I would agree that printerInd.overflow is pretty self explanatory.  Thus
> eliminating the need to clutter up the code with comments letting future
> developers know exactly what *in21 is.  Or even worse, having no
> explanation of *in21 at all.
>
> The argument I normally here is, "it is self explanitory.  It is specified
> on the OflInd keyword."  While this is true, I still have to look around
> elsewhere in the code to figure it out.  This wastes time.  If I see
> something like:
>
> If printerInd.overflow ;
>
> I know exactly what is happening without wasting my time.
>
> I'm off my soapbox now.
>
> Brian May
> Project Lead
> Management Information Systems
> Garan, Incorporated
> Starkville, Mississippi
>
> Young i Professionals
> http://www.youngiprofessionals.com