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



Glad to see this happening ..... I know one of your people(consultant) who is in the 25+ category whose solution to check and report on duplicates in a file was building uniquely key logical..get error on create.. build work file with duplicates and write a report program over it.

A lot of code .... when he could have just use something like ....

EXCEPT HDR1;
EXCEPT HDR2;

EXEC SQL Declare CheckDups Cursor
For SELECT CATITEM, Count(*) FROM CATITEMS
GROUP BY CATITEM
HAVING count(*) > 1;


EXEC SQL Open CheckDups;

EXEC SQL Fetch CheckDups into :DUPITEM,:DUPCOUNT;

DOW SQLCOD = *Zeros and SQLSTT = '00000';


IF *INOF;
EXCEPT HDR1;
EXCEPT HDR2;

ENDIF;


CHAIN DUPITEM ITITMF;
EXCEPT DTL1;


EXEC SQL Fetch CheckDups into :DUPITEM,:DUPCOUNT;

ENDDO;

EXEC SQL Close CheckDups;

So code reviews would definitely get your shop more modernized.

Please note .... I'm a 20+'er

Tell Bill W. and Sally M. I said Hi. Great people. Note ... I like the consultant also.




-----Original Message-----
From: rpg400-l-bounces@xxxxxxxxxxxx [mailto:rpg400-l-bounces@xxxxxxxxxxxx] On Behalf Of RNewton@xxxxxxxxxxxxxxxxx
Sent: Wednesday, November 16, 2011 9:48 AM
To: RPG programming on the IBM i / System i
Subject: Re: Do you do formal code reviews?

Thanks everyone for their input. I see the majority here are smaller shops
it or one man consulting firms so I can see why there are not any code
reviews needed. Our needs are a little different however.

We are implementing coding standards in our shop and will be enforcing
those standards with regular code reviews. Our shop has 30+ RPG
developers, each with their own styles and techniques varying from
techniques considered old 15 years ago to some guys counting the days
until 7.1 gets installed to get DB2 ALIAS in externally described data
structures.

We have ever expanding presentation technologies that our business logic
in RPG must be able to support. If we do not have code reviews, we will
continue to see business logic tied up in interactive programs and not
pulled out to service programs for reuse from other interactive
applications, web pages, web services, stored procedures, desktop
applications, etc...

Modernizing a shop this size without code reviews and shop standards for
techniques and syntax styling would be impossible (as we have shown over
the years).

Thanks,
Robert Newton
Estes IT
System Architect
804-353-1900 x2256
rnewton@xxxxxxxxxxxxxxxxx



From:
"Norm Dennis" <nhdennis@xxxxxxxxxxx>
To:
rpg400-l@xxxxxxxxxxxx
Date:
11/15/2011 10:36 PM
Subject:
Re: Do you do formal code reviews?
Sent by:
rpg400-l-bounces@xxxxxxxxxxxx



Haven't even heard formal code reviews mentioned since the early 80's.

Norm Dennis



----- Reply message -----
From: Rick.Chevalier@xxxxxxxxxxxxxxx
Date: Wed, Nov 16, 2011 07:07
Subject: Do you do formal code reviews?
To: <rpg400-l@xxxxxxxxxxxx>

We have talked about doing it but the idea never seems to get much
traction.


-----Original Message-----
From: rpg400-l-bounces@xxxxxxxxxxxx [mailto:rpg400-l-
bounces@xxxxxxxxxxxx] On Behalf Of RNewton@xxxxxxxxxxxxxxxxx
Sent: Tuesday, November 15, 2011 3:13 PM
To: RPG programming on the IBM i / System i
Subject: Re: Do you do formal code reviews?

I take it no one here does code reviews in their shop?

Robert Newton
Estes IT
System Architect
804-353-1900 x2256
rnewton@xxxxxxxxxxxxxxxxx



From:
RNewton@xxxxxxxxxxxxxxxxx
To:
RPG programming on the IBM i / System i <rpg400-l@xxxxxxxxxxxx>
Date:
11/11/2011 08:55 AM
Subject:
Do you do formal code reviews?
Sent by:
rpg400-l-bounces@xxxxxxxxxxxx



How does your shop implement code reviews for ensuring quality and
enforcing coding standards?

Do you have software to manage the process and what are the metrics that
you collect?

Thanks,
Robert Newton
Estes IT
System Architect
804-353-1900 x2256
rnewton@xxxxxxxxxxxxxxxxx



For 80 Years ? Delivering Solutions that Exceed Expectations.



This communication and any transmitted documents are intended to be
confidential. If there is a problem with this transmission, please
contact

the sender. If the reader of this message is not the intended recipient,
or the employee or agent responsible to deliver it to the intended
recipient, you are hereby notified that any dissemination, distribution
or

copying of this communication is strictly prohibited.

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







For 80 Years ? Delivering Solutions that Exceed Expectations.



This communication and any transmitted documents are intended to be
confidential. If there is a problem with this transmission, please
contact
the sender. If the reader of this message is not the intended recipient,
or the employee or agent responsible to deliver it to the intended
recipient, you are hereby notified that any dissemination, distribution
or
copying of this communication is strictly prohibited.

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



Privileged and Confidential. This e-mail, and any attachments there to,
is intended only for use by the addressee(s) named herein and may contain
privileged or confidential information. If you have received this e-mail
in error, please notify me immediately by a return e-mail and delete this
e-mail. You are hereby notified that any dissemination, distribution or
copying of this e-mail and/or any attachments thereto, is strictly
prohibited.

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.