RE: convert this code -- RPG400-L

× 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.

Subject: RE: convert this code
From: "Joe Pluta" <joepluta@xxxxxxxxxxxxxxxxx>
Date: Thu, 18 Apr 2002 18:58:55 -0500
Importance: Normal
List-archive: <http://archive.midrange.com/rpg400-l/>
List-help: <mailto:rpg400-l-request@midrange.com?subject=help>
List-id: RPG programming on the AS400 / iSeries <rpg400-l.midrange.com>
List-post: <mailto:rpg400-l@midrange.com>
List-subscribe: <http://lists.midrange.com/cgi-bin/listinfo/rpg400-l>,<mailto:rpg400-l-request@midrange.com?subject=subscribe>
List-unsubscribe: <http://lists.midrange.com/cgi-bin/listinfo/rpg400-l>,<mailto:rpg400-l-request@midrange.com?subject=unsubscribe>

> From: James Rich
>
>   for (i = 0; ((line[i] == stop) && (line[i]));)
>  (...)
>   for (x = 0; ((line[x]) && (line[x] != stop)); x++)

This is bad code.  It's called "for abuse", and is pretty unanimously
condemned.  Converting this to RPG is just asking for problems.

I heartily recommend Carel's advice to just use SCAN.

Joe

As an Amazon Associate we earn from qualifying purchases.

Follow-Ups:

RE: convert this code , James Rich

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.