UDF V6R1

[KingofKning]

Hallo *all,

ich bin auf dem Weg eine UDF zu erstellen die mir für einen Eingabewert einen HEX Wert zurückliefert.
Leider ist es so das die Eingabewerte zu groß sind da der Rückgabewert max FFFF sein darf.
Deswegen erstelle ich mir eine Tabelle wo ich die Eingabewerte reinschreiben will, die Anzahl der Sätze zähle und dann den Wert auch wegschreibe. (Für spätere Abfragen)

Leider scheitere ich schon am Anfang.
Beim ersten Aufruf ist es ja so, das die Tabelle leer ist. Also würde ich ein einfaches select count(*) machen und den Wert zurückgeben. Das will er aber nicht.



CREATE FUNCTION rptrade/zaehler /*10.10.19 14:34*/
( DEBITOR dec(08) ) RETURNS
CHAR(04) LANGUAGE SQL READS SQL DATA BEGIN
DECLARE RETURNVAL CHAR ( 04 ) NOT NULL DEFAULT '0000' ;
set returnval =
case when select count(*) from dat020 = 0 then 'XXXX'
else '0001'
end;
RETURN LTRIM(RETURNVAL);
END;

Für Hinweise dankbar. Werde mal sehen wo ich vernünftige Doku für die UDFs finde

GG 4251

[dschroeder]

Ich denke, du musst die Anzahl erst in eine Variable schieben:

DECLARE anzahl decimal(10);
select count(*) into anzahl from dat020;
set returnval = case when anzahl = 0 then 'XXXX' else '0001'

Gruß,
Dieter

[dschroeder]

Noch eine kleine Optimierung: Du willst ja eigentlich gar nicht wissen, wie viele Sätze in der Datei sind, sondern nur, ob es mehr als 0 sind. Deshalb würde ein "fetch first row only" verhindern, dass alle gezählt werden:
select count(*) into anzahl from dat020 fetch first row only;

[KingofKning]

Hallo,
werde ich morgen mal ausprobieren.

BTW. In der Tabelle sind hinterher Max. 200 Sätze, deswegen passt der Satz „Noch eine kleine Optimierung...“ ;-)

[B.Hauser]

Versuch mal bei dem CASE die SELECT-Anweisung in eine Klapper zu packen.

Birgitta

[KingofKning]

Danke, das hat schon mal funktioniert.

Habe mir das Redbook runtergeladen https://www.redbooks.ibm.com/redbooks/pdfs/sg248326.pdf

Muss jetzt mal schauen wo ich das Teil für V6 bekomme. Werde mich vermutlich nochmals melden ;-)

GG 4250

[Fuerchau]

Wie Birgitta sagte, skalare Subselects sind grundsätzlich in Klapper ähm... Klammern zu setzen.

Fetch first 1 rows only optimiert hier gar nichts, denn dies wirkt grundsätzlich nur auf das Ergebnis und nicht auf den Count oder einen Where.
Je nach Index oder sonstigen Rahmenbedingungen kann ebenso ein Tablescan erfolgen.

Der Count(*) ohne Where ist sowieso bereits SQL-Optimiert.

Daher folgende Vereinfachung:

CREATE FUNCTION rptrade/zaehler /*10.10.19 14:34*/
( DEBITOR dec(08) ) RETURNS
CHAR(04) LANGUAGE SQL READS SQL DATA BEGIN
return
case when (select coalesce( count(*), 0) from dat020) = 0 then 'XXXX'
else '0001'
end;
END;

Der Coalesce ist erforderlich, da ein Subselect auch schon mal NULL zurückliefert wenn keine Daten vorhanden sind.

Allerdings frage ich mich noch, wofür du den Eingangsparameter DEBITOR benötigst. Fehler der ggf. im Count?

[dschroeder]

Klar, du hat recht. Bei count bringt fetch first row only nichts. War mein Denkfehler. Wir lösen die Problemstellung, zu wissen, ob bestimmte Einträge vorhanden sind, trotzdem mit fetch first row only, aber nicht mit count. Ich denke, dass es dann etwas bringt:


dcl-s vorhanden char(1) inz('N');
exec sql select 'J' into :vorhanden from datei where ... fetch first row only;

[KingofKning]

Hallo *all,

mir fehlt jetzt nur noch die Konvertierung von hex nach decimal. Finde es auf Anhieb nicht wie der cast aussehen muß. Weil ohne fällt das Teil ab 000A auf die Nase.

Wie kann man eigentlich solche UDFs debuggen?

Ansonsten sieht meine UDF so aus.



CREATE FUNCTION rptrade/zaehler /*11.10.19 10:51*/
( DEBITOR dec(08) ) RETURNS
CHAR(04)
LANGUAGE SQL MODIFIES SQL DATA ALLOW DEBUG MODE
DISALLOW PARALLEL
BEGIN
DECLARE RETURNVAL CHAR (04) NOT NULL DEFAULT '0000' ;
DECLARE i_zaehler decimal(04) DEFAULT 0;

select d020werta into i_zaehler
from dat020 where d020key = debitor;
if i_zaehler = 0 then
select d020werta into i_zaehler
from dat020 order by d020werta desc
fetch first row only;
set i_zaehler = i_zaehler + 1;
insert into dat020 (d020key, d020werta)
values(debitor, substr(hex(int(i_zaehler)), 5, 4));
end if;
set returnval = substr(hex(int(i_zaehler)), 5, 4);
RETURN LTRIM(RETURNVAL);
END

[Fuerchau]

Wie wäre es mit

select 'J' into :vorhanden from datei where exists (...)
Schließlich enthält der Exists implizit den Fetch first und ist für den Einzelsatzzugriff optimiert.