Re: IF NOT %FOUND

And if some maintenance programmer drops a READ into the loop?

It would only be a problem if he dropped his READ in between the last two
lines, in between the READE and the ENDDO.
Surely some basic level of competence can be expected?
No?

Ok in that case I have to assume the maintenance programmer is also bad
enough to copy your version of the read loop, change the file on the READEs
but NOT on the %EOF()

Then my version would work correctly and yours wouldn't.
That would be ironic - the very thing you put in to defend
against potential mistakes, caused an issue, where otherwise the code would
have been ok

I do take your point. It's just not my personal preference, and I've yet to
see it cause an issue.

On Tue, 12 Nov 2019 at 16:49, Jon Paris <jon.paris@xxxxxxxxxxxxxx> wrote:

And if some maintenance programmer drops a READ into the loop?

I prefer to always defend against the potential mistakes of others - I
consider it to be part of my job in writing the program.

On Nov 12, 2019, at 11:04 AM, Craig Richards <craig@xxxxxxxxxxxxxxxx>

wrote:

+1 for the DoW Not %Eof()

At the risk of being controversial, for a standard read loop I don't like
to pass the file name parameter.

There is no code between the Read and the test for End of File, it just
seems redundant to me.

Off the top of my head I don't recall the last time I actually needed to
specify this to get the code to work but I believe it was in a program

with

multiple subfiles and I don't see a lot of those myself these days.
--
This is the RPG programming on IBM i (RPG400-L) mailing list
To post a message email: RPG400-L@xxxxxxxxxxxxxxxxxx
To subscribe, unsubscribe, or change list options,
visit: https://lists.midrange.com/mailman/listinfo/rpg400-l
or email: RPG400-L-request@xxxxxxxxxxxxxxxxxx
Before posting, please take a moment to review the archives
at https://archive.midrange.com/rpg400-l.

Please contact support@xxxxxxxxxxxx for any subscription related

questions.

Help support midrange.com by shopping at amazon.com with our affiliate

link: https://amazon.midrange.com

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

Please contact support@xxxxxxxxxxxx for any subscription related
questions.

Help support midrange.com by shopping at amazon.com with our affiliate
link: https://amazon.midrange.com