Code Inspector -> Eigene Prüfung

Alles rund um die Sprache ABAP®: Funktionsbausteine, Listen, ALV
10 Beiträge • Seite 1 von 1
10 Beiträge Seite 1 von 1

Code Inspector -> Eigene Prüfung

Beitrag von SaskuAc (Specialist / 321 / 37 / 43 ) »
Hallo Zusammen,

gerade bin ich dabei, da ich etwas Zeit nebenbei habe, eine Prüfung für den Code Inspector zu schreiben. Nun möchte ich noch ein paar Prüfungen einbauen.
Speziell geht es mir darum, dass die Entwickler "clean" programmieren sollen. Heißt in einer einheitlichen Form, so dass es schön lesbar ist. Ein Aspekt der dabei schon sehr gut unterstützt, ist der Pretty Printer.
Für die Prüfung von "Clean Code" möchte ich jetzt einfach prüfen ob der Entwickler den Pretty Printer verwendet hat.

gibt es also eine Möglichkeit, zu Prüfen, über einen FUBA, Methode, etc. ob ein Pretty Print ausgeführt worden ist?

Vielen Dank.

gesponsert
Stellenangebote auf ABAPforum.com schalten
kostenfrei für Ausbildungsberufe und Werksstudenten


Re: Code Inspector -> Eigene Prüfung

Beitrag von black_adept (Top Expert / 3943 / 105 / 886 ) »
Nein - und das ist auch gut so!

Folgende Benutzer bedankten sich beim Autor black_adept für den Beitrag:
Daniel

live long and prosper
Stefan Schmöcker

email: stefan@schmoecker.de

Re: Code Inspector -> Eigene Prüfung

Beitrag von SaskuAc (Specialist / 321 / 37 / 43 ) »
black_adept hat geschrieben:Nein - und das ist auch gut so!
Wieso?

Re: Code Inspector -> Eigene Prüfung

Beitrag von Tron (Top Expert / 1327 / 35 / 331 ) »
Moin.
Rezept zum erweitern des Codeinspector
https://blogs.sap.com/2006/11/02/code-i ... new-check/

Ich würde aber auch keine Erweiterung für eine Pretty Print Prüfung vornehmen,
sondern die Quellcodes lesen, Pretty print im Speicher durchführen und mit
CALL FUNCTION 'RS_CMP_COMPUTE_DELTA'
die Unterschiede ermitteln.
Aber Vorsicht, es gibt Ecken, wo der Pretty print nicht funktioniert.
(z.B. In user-Exits mit Include-Coding, oder in Modifikationen.)

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

Re: Code Inspector -> Eigene Prüfung

Beitrag von a-dead-trousers (Top Expert / 4271 / 213 / 1140 ) »
SaskuAc hat geschrieben:
black_adept hat geschrieben:Nein - und das ist auch gut so!
Wieso?
Ich würde mal behaupten, weil der Pretty-Print benutzerbezogen einstellbar ist und daher bei der Prüfung uU unterschiedliche Varianten verglichen werden.
(siehe auch Antwort von Jens bezüglich RS_CMP_COMPUTE_DELTA)
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

Re: Code Inspector -> Eigene Prüfung

Beitrag von black_adept (Top Expert / 3943 / 105 / 886 ) »
SaskuAc hat geschrieben:
black_adept hat geschrieben:Nein - und das ist auch gut so!
Wieso?
Weil SAP von Zeit zu Zeit den PP überarbeitet - und wenn dann Codestrecken im Transport sind, obwohl an ihnen keine Änderungen vorgenommen wurden ( bzw. diese wieder zurückgebaut wurden, sie aber damit auch im Transport stehen ) meckert die Prüfung . Desweiteren gibt es manchmal gute Gründe eine andere Formatierung als die vom PP zu verwenden - einfach weil es dann schöner aussieht.
Wenn man so weit geht einem Entwickler zu unterstellen Code abzugeben, der durch den PP lesbarer würde, kann man auch gleich vorschreiben Datendeklarationen oder WHEN-Anweisungen in alphabetischer Reihenfolge durchzuführen...

Folgende Benutzer bedankten sich beim Autor black_adept für den Beitrag:
Daniel

live long and prosper
Stefan Schmöcker

email: stefan@schmoecker.de

Re: Code Inspector -> Eigene Prüfung

Beitrag von ewx (Top Expert / 4784 / 294 / 628 ) »
http://www.tricktresor.de/blog/series/c ... erweitern/

Speziell zu Pretty Printer fällt mir auch nur der Ansatz von Jens ein.

Re: Code Inspector -> Eigene Prüfung

Beitrag von SaskuAc (Specialist / 321 / 37 / 43 ) »
Also ich weiß, wie ich eine eigene Prüfung schreibe, ist ja nicht meine erste :) ( danke dennoch! - achja, durch Tricktresor habe ich es auch ursprünglich gelernt, verfolge den block sehr gerne ;) )

Okay, zur Info, ich hätte sowieso nur den geänderten Code getestet, dafür habe ich meine Methoden :) - aber die Bedenken sind natürlich war, daran habe ich nicht gedacht, dass theoretisch jeder seine eigene Einstellungen für den PP hat ..

Das der PP auch häufiger nicht Ideal ist, ist mir auch bewusst, allerdings müssen insbesondere die Entwickler die von unserem Uralt ERP System ( auf dem sie COBOL programmieren ) auf SAP wechseln lernen dass man den Code formatiert.. ich habe programme bei denen gesehen, mein gott .. da wird einem ( wenn's um Formatierung geht ) schlecht.. deswegen sollte das zumindest eingebaut werden.

Was man eventuell einbauen könnte, wäre eine Prüfung ob der User die Rechte zum Entwickeln schon über einen gewissen Zeitraum ( z. B. 3 Monate ) hat. - Wenn ja, soll die Source nicht auf PP geprüft werden.. könnte eine Idee sein... aber da muss ich mir nochmal Gedanken machen und das abklären, was da am besten ist.

Re: Code Inspector -> Eigene Prüfung

Beitrag von black_adept (Top Expert / 3943 / 105 / 886 ) »
@SaskuAc: Wenn dein Problem die Dinos sind: Macht lieber am Anfang einen monatlichen Codereview und schnapp dir halt die Deliquenten, die tatsächlich völlig unleserlichen Code abgeben, der durch den PP hübscher wird. Bin zwar im Grunde kein Fan des Prangers - aber wenn's hilft....
live long and prosper
Stefan Schmöcker

email: stefan@schmoecker.de

Re: Code Inspector -> Eigene Prüfung

Beitrag von ewx (Top Expert / 4784 / 294 / 628 ) »
Pranger ist nie gut!
Aufklärung und Problembesprechung sollte man ohne Pranger machen.
Entweder im 4-Augen-Gespräch oder anonymisierter Code in größerer Runde.
Da ist es dann hilfreich, wenn man wirklich gute Beispiele hat.
Mit "Das sieht einfach besser aus" kann man niemanden überzeugen.

Ich habe mir z.B. angewöhnt immer alle nicht verwendeten Parameter aus der Mustererzeugung komplett zu löschen (Aufruf von Funktionsbausteinen oder Methoden).
Es nervt mich tierisch, wenn diese sinnlosen, auskommentierten Zeilen im Quelltext sind. (die im Übrigen das Verhalten des PP negativ beeinflussen).
Andere (viele andere) scheint es nicht zu stören...

Seite 1 von 1

Vergleichbare Themen

7
Antw.
3094
Views
Code Inspector
von Norbert_Aeq » 10.04.2012 15:54 • Verfasst in ABAP Objects®
0
Antw.
934
Views
Code Inspector - Eingabeparameter nicht möglich
von SaskuAc » 22.05.2018 14:59 • Verfasst in ABAP® Core
2
Antw.
1469
Views
Code Inspector mit Prüfvariante S4HANA_READINESS_1909 spinnt
von PeterPaletti » 02.03.2021 12:25 • Verfasst in ABAP® Core
4
Antw.
2613
Views
Code Inspector erweitern durch neue Checks
von SteveOh » 20.08.2010 11:29 • Verfasst in ABAP® für Anfänger
0
Antw.
821
Views
Prüfung
von 1337 » 19.11.2013 09:53 • Verfasst in ABAP® für Anfänger

Aktuelle Forenbeiträge

Zwischensumme Adobe Forms
vor 3 Tagen von Lucyalison 1 / 64
Interne Tabelle
vor 5 Tagen von black_adept 2 / 133
MaLo-Checker in ABAP
vor einer Woche von A6272 6 / 254

Newsletter Anmeldung

Keine Beiträge verpassen! Wöchentlich versenden wir lesenwerte Beiträge aus unserer Community.
Die letzte Ausgabe findest du hier.
Details zum Versandverfahren und zu Ihren Widerrufsmöglichkeiten findest du in unserer Datenschutzerklärung.

Aktuelle Forenbeiträge

Zwischensumme Adobe Forms
vor 3 Tagen von Lucyalison 1 / 64
Interne Tabelle
vor 5 Tagen von black_adept 2 / 133
MaLo-Checker in ABAP
vor einer Woche von A6272 6 / 254

Unbeantwortete Forenbeiträge

Zwischensumme Adobe Forms
vor 3 Tagen von Lucyalison 1 / 64
Group Items auf einer Filterbar
vor einer Woche von Bright4.5 1 / 107
tRFC Transaktionen SM58
vor 4 Wochen von A6272 1 / 140