Loop Exit Sy-Subrc

Re: Loop Exit Sy-Subrc

Der Subrc sagt gar nichts darüber aus, ob employee leer ist, sondern ob mindestens eine Zeile gefunden werden konnte, bei der employee leer ist. Wenn er keine findet, dann führt er den Inhalt des LOOP überhaupt nicht aus, sondern gibt Dir direkt SY-SUBRC = 4 zurück.

Ich vermute, dass Dein Problem darin besteht, dass Du irgendwann ganz oben im Orgbaum angekommen bist. Dort findest Du keine besetzte Planstelle, rufst Dich erneut rekursiv auf, obwohl es nach oben nicht mehr weiter geht, findest wieder nichts usw.

Nach meiner Erfahrung taugt Rekursion sowieso nur etwas, wenn man den Orgbaum abwärts suchen möchte, weil die Rekursion einem dann hilft, in alle Teilzweige hineinzukommen. Sucht man hingegen aufwärts, dann ist das eine lineare Sache, da jede Orgeinheit ja nur (maximal) eine übergeordnete Orgeinheit haben kann. Da ist man mit einer WHILE-Schleife besser bedient (WHILE SY-SUBRC = 0 und am Ende dann SY-SUBRC <> 0 werfen, wenn keine höhere Orgeinheit mehr gefunden wird).

Davon abgesehen möchte ich noch anmerken, dass mir nicht gefällt, dass Du hier überhaupt einen LOOP genommen hast. Für den sehe ich keinerlei Veranlassung, da Du nicht auf Ungleichheit prüfst. Ein READ TABLE ... TRANSPORTING NO FIELDS würde Deinen Zweck mindestens genauso gut erfüllen, wäre performanter und obendrein besser lesbar. Hast Du mindestens Release 7.40, dann könnest Du mit einem LINE_EXISTS( ) das ganze superelegant schreiben:

IF LINE_EXISTS( ch_responsibles[ employee <> '00000000' ] ). " Unter der Maßgabe, dass employee eine Personalnummer ist, denn dann ist '00000000' der Initialwert
* Mindestens 1 besetzte Planstelle vorhanden
DELETE ch_responsibles WHERE employee IS INITIAL.
ELSE.
* KEINE besetzte Planstelle vorhanden, suche aufwärts -> Rekursivaufruf

Das EXIT verändert den SY-SUBRC nicht. Wenn die LOOP-Schleife durchlaufen worden ist, wird SY-SUBRC auf 0 stehen.

Bei weiteren Fragen poste doch bitte etwas mehr Code, sonst ist das immer so ein Rätselraten, was da passiert. Du beklagst Dich über eine Endlosschleife und lässt in Deinem Codeausschnitt die Endlosschleife selbst weg. Das ist nicht hilfreich.

Re: Loop Exit Sy-Subrc

Noch schicker, hast recht. Wobei ich, wenn ich er wäre, dennoch verstehen wollen würde, wo mein Denkfehler gelegen hat. Ein LOOP AT mit EXIT drin zum Finden einer einzelnen Zeile ist ja nicht unüblich - und notwendig, sobald man Bedingungen drin hat, bei denen eine Prüfung auf Gleichheit nicht ausreicht.

Re: Loop Exit Sy-Subrc

Okay ich hatte hier einfach einen Denkfehler bezgl. des TRANSPORTING NO FIELDS ... habe das selber noch nie verwendet, daher kannte ich das nicht.

Und das Coding selber stammt nicht von mir, das ist von einem Kollegen, ansonsten weiß ich natürlich, dass ich hier keinen Loop verwendet hätte sondern wie ihr schon gesagt habt, direkt den Delete und dann dort auf sy-subrc prüfen. Allerdings kann ich das Coding aktuell nicht anfassen, da es in anderen größeren Changes drinnen ist - in die ich nicht rein darf..

Zur Abbruchbedingung - es kann sein, dass hier im Entwicklungssystem ( denn nur da tritt der Fehler auf ) Datenfehler im Orgmanagement ( das inzwischen seit 4 Jahren nicht mehr vom Produktiv kopiert worden ist ... ) vorhanden sind und es deswegen zu einer Endlosschleife kommt.. im Quality und Produktivsystem tritt das nicht auf.

Was mich noch mehr wundert und wahrscheinlich die Frage nach dem Coding irrelevant macht ist, warum es bei meinem Kollegen funktioniert, der die PA20 ( oder 30 ) ganz normal aufruft, bei mir aber nicht .. er hat da absolut keine Probleme ...

Sind für die PA20 / PA30 irgendwelche Userparameter vorhanden oder etwas derartiges, dass die Art und Weise wie man auf die Transaktion zugreift ( denn Berechtigungen - also Rollen und Profile sind identisch ) bestimmt? Das ist das einzige was ich mir vorstellen kann ...

Re: Loop Exit Sy-Subrc

Danke, das Problem hat sich allerdings gerade eben in Luft aufgelöst..

Das Programm hat aufgrund eines Customizingfehlers von einem Kollegen bezgl. strukturellen Berechtigungen ( die fürs MSS gedacht sind ) den Code aufgerufen .. den zusammenhang verstehe ich auch noch nicht wirklich, aber naja .. behoben ist behoben.

Da war dann allerdings noch das Problem mit der Endlosschleife .. diese wurde durchlaufen, weil sich mein Kollege für eben das MSS Reporting auf die Stelle des Personalchefs und seine aktuelle Stelle gleichzeitig gesetzt hat. Daraus ist dann dieser Loop entstanden.


Danke euch dennoch für die Hilfe!
Gruß!

Re: Loop Exit Sy-Subrc

DeathAndPain hat geschrieben:

Da war dann allerdings noch das Problem mit der Endlosschleife .. diese wurde durchlaufen, weil sich mein Kollege für eben das MSS Reporting auf die Stelle des Personalchefs und seine aktuelle Stelle gleichzeitig gesetzt hat. Daraus ist dann dieser Loop entstanden.

Das ist aber trotzdem Mist, denn dass eine Person mehrere Planstellen besetzt, ist ja absolut legitim (weswegen SAP auch den jeweiligen Besetzungsprozentsatz führt). Selbst wenn ihr davon in eurem Haus keinen Gebrauch macht, würde ich den Code so schreiben, dass er damit umgehen kann (wer weiß, was in Zukunft noch kommt).

Es lag nicht daran, dass er mehrere Planstellen hatte, das ist bei uns tatsächlich üblich ..
Er hat sich hier dann allerdings zum Chef von seinem Chef gemacht. Dadurch wurde dann wieder sein chef gesucht, dann wurde er wieder gefunden .. dann wurde wieder der chef gesucht .. dann wieder er .. usw.

DeathAndPain hat geschrieben:

Das Programm hat aufgrund eines Customizingfehlers von einem Kollegen bezgl. strukturellen Berechtigungen ( die fürs MSS gedacht sind ) den Code aufgerufen

Strukturelle Berechtigungen sind häufig in HCM-bezogene Probleme verwickelt. Trotzdem liebe ich sie, denn man kann eine Menge damit machen, und die strukturellen Berechtigungen als solche sind ja nicht schuld am jeweiligen Problem, sondern ein Fehler im Umgang damit.

Da hast du natürlich recht .. wobei ich noch nicht wirklich 100%ig hinter diese gestiegen bin ..
Sie waren hier auch nicht das Problem .. sondern eben, wie du sagtest, der Umgang damit .. ^^