Ausertung Materialstamm
7 Beiträge
•
Seite 1 von 1
7 Beiträge
Seite 1 von 1
Hallo zusammen
Ich habe ein Programm erstellt, dass Auswertungen über den kompletten Materialstamm macht.
Nun weiss ich nicht, wo Verbesserungen nötig sind.
Ich lade das Programm mal hier hoch, danke für eure Rückmeldungen.
PS: Bitte nicht zu viel Änderungen auf einmal, ich möchte ja etwas lernen und nicht von Informationen erschlagen werden
Ich habe ein Programm erstellt, dass Auswertungen über den kompletten Materialstamm macht.
Nun weiss ich nicht, wo Verbesserungen nötig sind.
Ich lade das Programm mal hier hoch, danke für eure Rückmeldungen.
PS: Bitte nicht zu viel Änderungen auf einmal, ich möchte ja etwas lernen und nicht von Informationen erschlagen werden
Du hast keine ausreichende Berechtigung, um die Dateianhänge dieses Beitrags anzusehen.
gesponsert ⓘ
Stellenangebote auf ABAPforum.com schalten
kostenfrei für Ausbildungsberufe und Werksstudenten
kostenfrei für Ausbildungsberufe und Werksstudenten
Moin.
So mal mein erster Eindruck.
1.) TABLES - Deklarationen solltest Du nicht mehr verwenden.
2.) LIKE ist auch Obsolet. Nimm lieber TYPE.
3.)
Es fällt auf, daß Du viele Types deklarierst hast. Ab einer gewissen Menge von Feldern verwende ich lieber DDIC-Strukturen (SE11).
Das hat den Charme, das die "Sache" Pflegbarer ist. (Und die Überschriften für ein eventuelles ALV Grid sind dann auch präsent).
lg Jens
So mal mein erster Eindruck.
1.) TABLES - Deklarationen solltest Du nicht mehr verwenden.
2.) LIKE ist auch Obsolet. Nimm lieber TYPE.
3.)
Es fällt auf, daß Du viele Types deklarierst hast. Ab einer gewissen Menge von Feldern verwende ich lieber DDIC-Strukturen (SE11).
Das hat den Charme, das die "Sache" Pflegbarer ist. (Und die Überschriften für ein eventuelles ALV Grid sind dann auch präsent).
lg Jens
<:: XING-Gruppe Tricktresor::>
Die deutsche Rechtschreibung ist Freeware, du darfst sie kostenlos nutzen –
Aber sie ist nicht Open Source, d. h. du darfst sie nicht verändern oder in veränderter Form veröffentlichen.
Die deutsche Rechtschreibung ist Freeware, du darfst sie kostenlos nutzen –
Aber sie ist nicht Open Source, d. h. du darfst sie nicht verändern oder in veränderter Form veröffentlichen.
Beitrag
von black_adept (Top Expert / 4089 / 127 /
940
)
»
Hallo Christof,
ein erster Ansatzpunkt wäre einfach mal den Extended-Syntaxcheck aufzurufen, auf den Button "Activate all checks" zu drücken und sehen, dass die Anzahl der Meckereien, die SAP da ausgibt reduziert werden. Ich häng mal den Syntaxcheck an, den es auf meinem 7.4er System gibt - da ist einiges was man abarbeiten könnte.
ein erster Ansatzpunkt wäre einfach mal den Extended-Syntaxcheck aufzurufen, auf den Button "Activate all checks" zu drücken und sehen, dass die Anzahl der Meckereien, die SAP da ausgibt reduziert werden. Ich häng mal den Syntaxcheck an, den es auf meinem 7.4er System gibt - da ist einiges was man abarbeiten könnte.
Du hast keine ausreichende Berechtigung, um die Dateianhänge dieses Beitrags anzusehen.
Beitrag
von a-dead-trousers (Top Expert / 4396 / 223 /
1182
)
»
Führ das Programm mal über die SAT (Laufzeitanalyse) aus und schau wo die meiste (Netto-)Zeit verloren geht.
lg ADT
lg ADT
Theory is when you know something, but it doesn't work.
Practice is when something works, but you don't know why.
Programmers combine theory and practice: Nothing works and they don't know why.
ECC: 6.18
Basis: 7.50
Practice is when something works, but you don't know why.
Programmers combine theory and practice: Nothing works and they don't know why.
ECC: 6.18
Basis: 7.50
Beitrag
von black_adept (Top Expert / 4089 / 127 /
940
)
»
Wenn du für einen Sack voll Materialien alle eine Stock-Requirement-Analyse von SAP zurückgeben lässt wundern mich lange Laufzeiten nicht sonderlich. Aber ich wüsste auch nicht wie du das umgehen willst.christof hat geschrieben:Könnte irgendwie die Laufzeit drastisch verringert werden?
Je nach angelgegten Orgebenen wird die Stock-Requirement-Analyse für ein Material mehrfach aufgerufen: Anzahl VKorgs * Anzahl Bewertungskreise.black_adept hat geschrieben:Wenn du für einen Sack voll Materialien alle eine Stock-Requirement-Analyse von SAP zurückgeben lässt wundern mich lange Laufzeiten nicht sonderlich. Aber ich wüsste auch nicht wie du das umgehen willst.
Auch die vielen einzelnen SELECTS in den FORMS sind nicht optimal, da könnte man mit geschickten FOR ALL ENTRIES und READ BINARY auch noch ein wenig Performance raus hohlen.
SELECT * mit INTO CORRESPONDING: Wenn nicht alle gelesenen Felder verwendet werden. Wieso nicht die gewollten Felder explizit angeben? Das verbessert mMn auch die Lesbarkeit des Selects.
FIELD-SYMBOLES beim LOOP verwenden, spart das spätere MODIFY .
READ_DATA wird bei jedem PBO durchlaufen, sollte aber auch nur einmal durchlaufen werden analog dem CREATE go_custom_container.
Wieso ein Dynpro, wenn dann doch ein SALV als "Full-Screen" aufgerufen wird? Der SALV bringt das Dynpro doch schon mit.
Zur verwendeten Notation:
Wenn schon eine Notation genutzt wird, sollte sie auch aussagekräftig sein:
s_werks ist zwar eine SelectionScreenVariable, aber keine SelOpt sondern ein Parameter.
go_ vs. gr_ vs. gcl_: Wann wird was verwendet?
ls_ für globale Daten.
Sprechende Namen für Deklarationen verwenden: itab01, itab02, itab03, itab04 ist nichts sagend ohne ewig zu scrollen.
Gruß Hendrik
Seite 1 von 1
Vergleichbare Themen
3
Antw.
Antw.
3626
Views
Views
Mußfeld Materialstamm
von Christl » 01.10.2007 14:17 • Verfasst in Material Management & Produktionsplanung
von Christl » 01.10.2007 14:17 • Verfasst in Material Management & Produktionsplanung
4
Antw.
Antw.
12598
Views
Views
Periodenverschieber Materialstamm
von jh0294 » 05.02.2007 16:36 • Verfasst in Material Management & Produktionsplanung
von jh0294 » 05.02.2007 16:36 • Verfasst in Material Management & Produktionsplanung
Frage die ABAP Experten
Über diesen Beitrag
Unterstütze die Community und teile den Beitrag für mehr Leser und Austausch
Aktuelle Forenbeiträge
Newsletter Anmeldung
Keine Beiträge verpassen! Wöchentlich versenden wir lesenwerte Beiträge aus unserer Community.Die letzte Ausgabe findest du hier.