Nur PaaS

Best Practices für die Überprüfung des Codes für Adobe Commerce

Letzte Aktualisierung: 18. Dezember 2024

Erstellt für:

  • Experte
  • Entwickler

Das Hauptziel der Code-Überprüfung besteht darin zu überprüfen, ob die implementierte Funktionalität die Anforderungen aus Leistungs-, Architektur- und Sicherheitssicht optimal erfüllt.

Außerdem soll durch die Code-Überprüfung sichergestellt werden, dass der Code den Best Practices für die Adobe Commerce-Entwicklung entspricht, leicht zu verstehen und zu pflegen ist und den Codierungsstandards entspricht. Die meisten Codierungsstandards sollten automatisch von entsprechenden Tools überprüft werden.

Empfohlener Code-Überprüfungsprozess

Im Großen und Ganzen sollte der Code-Überprüfungsprozess die folgenden Schritte umfassen:

  1. Checkout-Funktionsverzweigung in der lokalen Entwicklungsumgebung.
  2. Wenn es eine Weile her ist, seit der Code zur Überprüfung übermittelt wurde, führen Sie die neuesten Änderungen aus der Zielverzweigung (normalerweise die QA-Verzweigung) zusammen und übertragen Sie Aktualisierungen auf die Remote-Funktionsverzweigung (falls vorhanden).
  3. Führen Sie eine allgemeine Überprüfung durch, um zu überprüfen, ob der Code alle Anforderungen implementiert. Wenn es größere Diskrepanzen gibt, wenden Sie sich zur Klärung an den Entwickler.
  4. Das Ausführen des Codes ist optional, aber wenn Sie Zweifel haben, dass der Code erwartungsgemäß funktioniert, oder wenn er die Effizienz des Prozesses erleichtert, testen Sie, ob die implementierte Funktionalität für die wichtigsten Nutzungsszenarien wie erwartet funktioniert. Wenn es größere Probleme gibt, beenden Sie den Überprüfungsprozess und öffnen Sie das Ticket erneut mit entsprechenden Kommentaren.
  5. Führen Sie eine gründliche Überprüfung durch, um zu überprüfen, ob der Code alle Anforderungen implementiert. Wenn Probleme auftreten, fügen Sie der Pull-Anfrage entsprechende Kommentare hinzu und öffnen Sie das Ticket erneut.
  6. Wenn alles gut aussieht, führen Sie die Pull-Anfrage zusammen (oder übergeben Sie sie an die nächste Code-Prüfungsebene, falls eine eingerichtet wurde).

Beachten Sie außerdem die folgenden Punkte bei der Implementierung von Code-Überprüfungsprozessen:

  • Die Code-Überprüfung ist ein primäres Tool für die Kommunikation und den Wissenstransfer innerhalb eines Teams. Selbst wenn eine Pull-Anfrage keine größeren Probleme enthält und die erforderliche Geschäftslogik implementiert, können Sie sie als Gelegenheit nutzen, nach der Zusammenführung weitere Verbesserungen vorzuschlagen.

  • Im Durchschnitt sollte die Überprüfung von Code nicht mehr als 10 % bis 20 % des Entwicklungsaufwands in Anspruch nehmen. Das setzt voraus, dass das Entwicklungsteam aus erfahrenen Ingenieuren besteht, die gut zusammenarbeiten. Wenn die Überprüfung des Codes länger dauert, kann dies das Projektbudget und den Zeitrahmen beeinflussen.

  • Beheben Sie Probleme mit übermäßigem Aufwand für Code-Überprüfungen, indem Sie die Grundursache identifizieren. Normalerweise liegt es daran, dass entweder Anforderungen schlecht in Entwicklungs-Tickets übersetzt werden (aufgrund von Kommunikationsproblemen, schlechten Benutzergeschichten) oder es sich um ein Coaching-Problem handelt. Im ersten Fall wird als Risikominderung empfohlen, sicherzustellen, dass die Tickets über genügend Informationen verfügen, damit die Team-Mitglieder die Anforderungen effizient erfüllen können. Im zweiten Fall müssen Sie möglicherweise die Team-Struktur anpassen, um mehr erfahrene Ingenieure einzubeziehen oder eine Genehmigung außerhalb von Mentoring und Coaching zu erhalten.

Betroffene Produkte und Versionen

Alle unterstütztenvon:

  • Adobe Commerce auf Cloud-Infrastruktur
  • Adobe Commerce On-Premises

Worauf bei der Code-Überprüfung geachtet werden muss

Stil

Der Stil kann automatisch durch Ausführen der PhpStorm-Inspektion getestet werden (siehe unten).

Stellen Sie sicher, PHPMD und PHPCS zu konfigurieren und das Coding Standard-Tool über die CLI (auch unten) auszuführen. Es gibt einige Überschneidungen, aber beide verfügen auch über eindeutige Tests.

Konvention und Struktur

Die Überprüfungen der Konvention und Struktur werden manuell durchgeführt.

  • Ist die Klassenfunktionalität auf eine einzige Verantwortung beschränkt?
  • Ist die Verzeichnisstruktur sinnvoll?
  • Wird auf der richtigen Ebene ausgeführt (Server, Client, CSS, JS, Datenbank, Framework, Infrastruktur).
  • Ist die Versionierung korrekt?
  • Sieht der Code unkonventionell aus oder ist er so, als ob er versucht, ein Problem auf die falsche Weise zu umgehen?
  • Wird die Namenskonvention für den Modul-, Paket- und Repository-Namen korrekt angewendet?
  • Stellen Sie sicher, dass globale CSS-Stile sorgfältig angewendet und nicht überbeansprucht werden.

Vollständigkeit

Die Überprüfung auf Vollständigkeit erfolgt manuell.

  • Kann der Code durch Konfiguration aktiviert oder deaktiviert werden und verhält sich der gesamte erforderliche Code erwartungsgemäß?
  • Sind alle Konfigurationen vorhanden, die im Ticket erwähnt werden? Überprüfen Sie Umfang, Datentyp, Validierung, Übersetzung und Standardwerte.
  • Wird die Konfiguration immer auf der niedrigstmöglichen Ebene abgerufen (Store-Ansichtsebene, Website-Ebene oder globale Ebene)? Der Konfigurationsabruf muss mit der Definition des Umfangs in der system.xml übereinstimmen.
  • Werden alle Pfade im Flussdiagramm der technischen Spezifikation abgedeckt? Werden alle anderen technischen Spezifikationen berücksichtigt?
  • Sind für die neue Funktion ACLs definiert?
  • Sind PHP-Dokumente klar? Sind Commit-Nachrichten klar?
  • Ist irgendein Code auskommentiert oder wird Debug-Code angezeigt?

Leistung

Die Überprüfung der Leistung erfolgt manuell, was im Zweifelsfall durch die Ausführung von Code unterstützt werden kann.

  • Werden Abfragen in einer Schleife ausgeführt? Diese Schleife kann sich außerhalb der bearbeiteten Dateien befinden.
  • Können Sie cachable="false" Attribute erkennen? Werden sie korrekt angewandt?

Sicherheit

Die Überprüfung auf Sicherheit erfolgt manuell, was durch die Textsuche unterstützt werden kann. Ein Teil der Sicherheitsprüfung wird durch automatisierte Tests durchgeführt.

  • Werden bei Bedarf Ausnahmen protokolliert? Werden die richtigen Ausnahmetypen verwendet?
  • Können around Plug-ins vermieden werden?
  • Gibt das Plug-in die richtigen Datentypen zurück?
  • Können Sie Rohdaten-SQL-Abfragen finden, die mit der Datenbankabstraktionsebene erstellt werden sollen?
  • Wird ein neuer Datentyp einem beliebigen Benutzer, Administrator oder Frontend bereitgestellt? Ist diese Anfälligkeit ein Sicherheitsrisiko?
  • Werden benutzergenerierte Daten validiert? Alles, was vom Browser kommt, wird als benutzergeneriert betrachtet, einschließlich Cookie-Werte und Server-Kopfzeilen.

Datenschutz und DSGVO

Die Überprüfungen für Datenschutz und DSGVO werden manuell durchgeführt.

  • Verarbeitet der Code Kundendaten oder E-Mails? Achten Sie besonders darauf.
  • Wenn dieser Code in einer Schleife ausgeführt werden kann, kann er dann Kundendaten von einem Schleifenzyklus zum anderen durchsickern lassen?
  • Risikoindikatoren sind Importe, Cron-Aufträge, Transaktions-E-Mails und Batch-Warteschlangen-Handler.
  • Sicherstellen, dass Benutzerdaten in Schleifen isoliert werden. Adobe empfiehlt die Verwendung von Fabriken oder Repositorys, um Modelle im Schleifenzyklus zu erstellen, auf die außerhalb der Schleife nicht zugegriffen werden kann.