Re: SQL - just looking for a little guidance. -- RPG400-L

Mike,

You might get more responses if you were to provide some additional
information. Right now, we don't know what the cardinality is between
RTUPDWF and the other two files. Is it one to many, or many to many? In
order to construct decent SQL statements, one needs to understand the
tables, and the relationships between them. So let's see the primary/foreign
keys that you have, and maybe we can suggest something.


John Taylor
Canada

----- Original Message -----
From: "Smith, Mike" <Mike_Smith@RGCResources.com>
To: <RPG400-L@midrange.com>
Sent: Friday, May 04, 2001 13:27
Subject: RE: SQL - just looking for a little guidance.


> the 2 tests are just something that i copied from a book.  i thought it a
> bit strange myself, but i figured if thats the way it was done in the
book,
> it should be good enough for me.
>
>
> -----Original Message-----
> From: R. Bruce Hoffman, Jr. [mailto:rbruceh@attglobal.net]
> Sent: Friday, May 04, 2001 3:16 PM
> To: RPG400-L@midrange.com
> Subject: Re: SQL - just looking for a little guidance.
>
>
> Not a comment on the SQL, but why do you first test not equal to 100 and
> then test equal to zeros on the SQLCODE?
>
> If you are looking for efficiency, those tests don't make a whole lot of
> sense.
>
> Also be aware that since your loops are based on the SQLCODE being 100,
that
> another serious error returning something other than 100, will cause the
> program to loop.
>
> ===========================================================
> R. Bruce Hoffman, Jr.
>  -- IBM Certified Specialist - AS/400 Administrator
>  -- IBM Certified Specialist - RPG IV Developer
>
> "America is the land that fought for freedom and then
>   began passing laws to get rid of it."
>
>      - Alfred E. Neuman
>
> -----Original Message-----
> From: Smith, Mike <Mike_Smith@RGCResources.com>
> To: RPG400-L@midrange.com <RPG400-L@midrange.com>
> Date: Friday, May 04, 2001 9:46 AM
> Subject: SQL - just looking for a little guidance.
>
>
> >I would just like for you to look at this program, and tell me if it
should
> >have been done differently.
> >It doesn't do a whole lot, but i would like to make it as efficient as
> >possible.
> >Should i have done a multirow fetch?
> >could i have done a join on the update and made the whole thing work in 1
> or
> >2 sql statements.?
> >
> >I'm just trying to figure out how to do things better.
> >
> >Oour shop is exploring SQL and trying to use it as much as possible and
i'm
> >the guy that gets to experiment with it.  So any help is appreciated.
> >
> >I have this program that updates 2 files based on the data from the first
> >file
> >  * DATA STRUCTURES
> > D UPRMDS       E DS                  EXTNAME(UPRM) INZ
> > D UCBSDS        E DS                  EXTNAME(UCBS) INZ
> > D RTUPDWFDS E DS                  EXTNAME(RTUPDWF) INZ
> >
> >C/EXEC SQL
> >C+  Declare WRKCURS CURSOR FOR
> >C+  SELECT * FROM RTUPDWF
> >C/END-EXEC
> >
> > * OPEN THE CURSOR - ROUTE CHANGE WORK FILE
> >
> >C/EXEC SQL
> >C+  OPEN WRKCURS
> >C/END-EXEC
> >
> > * READ THROUGH THE FILE UNTIL EOF
> >
> >C                   DOU       SQLCOD = 100
> >C/EXEC SQL
> >C+  FETCH WRKCURS INTO :RTUPDWFDS
> >C/END-EXEC
> >
> >
> > C                   IF        SQLCOD <> 100
> > C                   IF        SQLCOD =  00000
> >
> >  * GET ALL RECORDS IN UPRM FOR THE ROUTE SELECTED IN RTUPDWF
> >
> > C/EXEC SQL
> > C+  Declare PRMCURS CURSOR FOR
> > C+  SELECT * FROM UPRM
> > C+  WHERE  UPRTE = :RTORT
> > C/END-EXEC
> >
> >  * OPEN THE CURSOR - PREMISE FILE
> >
> > C/EXEC SQL
> > C+  OPEN PRMCURS
> > C/END-EXEC
> >
> >  * READ THROUGH THE FILE UNTIL EOF
> >
> > C                   DOU       SQLCOD = 100
> >
> > C/EXEC SQL
> > C+  FETCH PRMCURS INTO :UPRMDS
> > C/END-EXEC
> >
> > C                   IF        SQLCOD <> 100
> > C                   IF        SQLCOD =  00000
> > C                   EXSR      $UPDATEPRM
> > C                   ENDIF
> > C                   ENDIF
> > C                   ENDDO
> >
> > C                   EXSR      $CLOSEPRM
> >
> >  * GET ALL RECORDS IN UCBS FOR THE ROUTE SELECTED IN RTUPDWF
> >
> > C/EXEC SQL
> > C+  Declare CBSCURS CURSOR FOR
> > C+  SELECT * FROM UCBS
> > C+  WHERE  USORT = :RTORT
> > C/END-EXEC
> >  * OPEN THE CURSOR - ACCOUNT BILLED SERVICES FILE
> >
> > C/EXEC SQL
> > C+  OPEN CBSCURS
> > C/END-EXEC
> >
> >  * READ THROUGH THE FILE UNTIL EOF
> >
> > C                   DOU       SQLCOD = 100
> > C/EXEC SQL
> > C+  FETCH CBSCURS INTO :UCBSDS
> > C/END-EXEC
> >
> > C                   IF        SQLCOD <> 100
> > C                   IF        SQLCOD =  00000
> > C                   EXSR      $UPDATECBS
> > C                   ENDIF
> > C                   ENDIF
> > C                   ENDDO
> >
> > C                   EXSR      $CLOSECBS
> >
> > C                   ENDIF
> > C                   ENDIF
> >                      ENDDO
> > C                   EXSR      $CLOSEWRK
> >                      EVAL      *INLR = *ON
> > *================================================================
> >  *  PROGRAM SUBROUTINES
> >  *----------------------------------------------------------------
> > *****************************************************************
> >C     $UPDATEPRM    BEGSR
> >C/EXEC SQL
> >C+                  UPDATE UPRM SET
> >C+                     UPRTE = :RTNRT
> >C/END-EXEC
> >
> > C                   ENDSR
> > *****************************************************************
> >                                                                RTU
> > C     $UPDATECBS    BEGSR
> > C/EXEC SQL
> > C+                  UPDATE UCBS SET
> > C+                     USORT = :RTNRT
> > C/END-EXEC
> >
> > C                   ENDSR
> > *****************************************************************
> > C     $CLOSEWRK     BEGSR
> > C/EXEC SQL
> >  + CLOSE WRKCURS
> > C/END-EXEC
> > C                   ENDSR
> > *****************************************************************
> > C     $CLOSEPRM     BEGSR
> > C/EXEC SQL
> >  + CLOSE PRMCURS
> > C/END-EXEC
> > C                   ENDSR
> > *****************************************************************
> >
> > *************************************
> >C     $CLOSECBS     BEGSR
> >C/EXEC SQL
> > + CLOSE CBSCURS
> >C/END-EXEC
> >C                   ENDSR
> > *===================================
> >
> >
> >Michael Smith
> >
> >+---
> >| This is the RPG/400 Mailing List!
> >| To submit a new message, send your mail to RPG400-L@midrange.com.
> >| To subscribe to this list send email to RPG400-L-SUB@midrange.com.
> >| To unsubscribe from this list send email to
RPG400-L-UNSUB@midrange.com.
> >| Questions should be directed to the list owner/operator:
> david@midrange.com
> >+---
>
> +---
> | This is the RPG/400 Mailing List!
> | To submit a new message, send your mail to RPG400-L@midrange.com.
> | To subscribe to this list send email to RPG400-L-SUB@midrange.com.
> | To unsubscribe from this list send email to RPG400-L-UNSUB@midrange.com.
> | Questions should be directed to the list owner/operator:
> david@midrange.com
> +---
> +---
> | This is the RPG/400 Mailing List!
> | To submit a new message, send your mail to RPG400-L@midrange.com.
> | To subscribe to this list send email to RPG400-L-SUB@midrange.com.
> | To unsubscribe from this list send email to RPG400-L-UNSUB@midrange.com.
> | Questions should be directed to the list owner/operator:
david@midrange.com
> +---

+---
| This is the RPG/400 Mailing List!
| To submit a new message, send your mail to RPG400-L@midrange.com.
| To subscribe to this list send email to RPG400-L-SUB@midrange.com.
| To unsubscribe from this list send email to RPG400-L-UNSUB@midrange.com.
| Questions should be directed to the list owner/operator: david@midrange.com
+---