JavaScript Code Quality in 10 einfachen Schritten steigern.

JavaScript Code Quality in 10 einfachen Schritten steigern.

Die Codequalität von Javascript und CSS Dateien rückt gerade bei größeren Projekten in den Vordergrund. Riesiger und schwer pflegbarer Code wird hier zum Albtraum jedes Entwicklers, welcher sich in Zukunft mit dem Projekt beschäftigen muss. Umso wichtiger ist es deshalb, bereits bei der Entwicklung auf eine einheitliche Codebasis zu achten. Wie du Fehler vorbeugen kannst und die Codequalität mit Hilfe von Lintern steigerst, erfährst du in diesem Artikel.

1. Was sind Linter?

Code Linting ist eine Möglichkeit zur Verbesserung der Codequalität. Der Begriff “Linter” stammt vom Tool “Lint”, welches ursprünglich C-Quellcode analysierte. Bereits damals war das Ziel den Quellcode möglichst automatisch zu analysieren und Optimierungen zu finden. Im Laufe der Zeit wurden dann zusätzliche Features hinzugefügt.

Heutzutage gibt es Linter für fast alle Programmiersprachen. JavaScript Linter wie JSHint oder ESLint spüren potentielle Fehler sowie schwer wartbaren Code auf. Sie erkennen häufige Fehler im Quellcode und stellen sicher, dass Best Practices eingehalten werden. Die automatische Erkennung von Fehlern bereits vor der manuellen Codeüberprüfung durch den Reviewer spart dadurch Zeit und Geld.

Außerhalb von JavaScript werden in der Webentwicklung zudem Linter für CSS oder HTML entwickelt und eingesetzt. 

2. Warum du Linter nutzen solltest

  • Einheitlicher Coding Style bei der Arbeit im Team
  • Grobe Bugs frühzeitig erkennen und vorbeugen
  • Best Practices im Team durchsetzen

3. Warum wir ESLint nutzen:

ESLint zeichnet sich vorallem durch seine Konfigurierbarkeit aus. Presets helfen ESLint schnell mit einer Basiskonfiguration zu versehen. Trotz Preset können die Regeln mithilfe der .ESLintrc Konfigurationsdatei weiter auf Ihre Team Bedürfnisse angepasst werden. 

ESLint zum Projekt hinzufügen

4. NPM Setup:

Im Projektordner NPM installieren:

$ npm init

Die Package.json Datei sollte nun in etwa so aussehen

{ "name": "my-eslint-project", "version": "1.0.0", "description": "", "main": "index.js", "scripts": { "test": "echo \"Error: no test specified\" && exit 1" }, "author": "", "license": "ISC" }

5. ESLint als Develop Dependency istallieren:

$ npm i eslint -D

6. ESLint Konfigurationsdatei erstellen:

$ touch .eslintrc

7. ESLint Basiskonfiguration:

In der .eslintrc kannst du nun ESLint konfigurieren.

Standardmäßig wird das Default Preset eslint:recommended genutzt.

Über die ignorePatterns ist es zudem möglich, einzelne Ordner zu ignorieren

Die Config sollte in etwa so aussehen:

{ "env": { "browser": true, "es6": true }, "extends": ["eslint:recommended"], "ignorePatterns": [ “build/assets/javascripts/legacy/**/*.js”, ] }

8. NPM Task hinzufügen:

Über “Scripts” kann nun das Test Script gegen “lint” getauscht werden. Hierfür wird am Anfang der Name des Tasks definiert und dann der Befehl. 

Über npm run lint kann nun ESLint automatisch über die Konsole ausgeführt werden.

{ "name": "eslint-test", "version": "1.0.0", "description": "", "main": "index.js", "scripts": { "lint": "eslint 'path/to/javascript/file.js'" }, "author": "", "license": "ISC", "devDependencies": { "eslint": "^7.32.0" } }

9. Optional: Airbnb Preset installieren:

Es bietet sich optional die Möglichkeit ein zusätzliches ESLint Preset von Airbnb (ja wirklich das Airbnb, das wir alle kennen) zu installieren. Hierfür müssen zusätzliche Dependencies installiert werden. Dies geht ganz einfach über folgenden Befehl:

$ npx install-peerdeps --dev ESLint-config-airbnb-base

Im nächsten Schritt muss das Preset dann noch inkludiert werden. Hierfür wird airbnb-base in das Extends Array der .eslintrc einfügt.

{ "extends": [ "eslint:recommended", "airbnb-base" ], "rules": [ "no-alert": "off" ], }

10. Optional: Überschreiben von Regeln

ESLint Regeln können über zwei verschiedene Wege überschrieben / deaktiviert werden.

Deaktivieren über Inline Kommentar:

Komplette Zeile deaktivieren:

var myTest = “hello world” // eslint-disable-line

Einzelne Regeln deaktivieren

alert(“hello world); // eslint-disable-line no-alert

Deaktivieren über Konfiguration
{ ... "extends": [ "eslint:recommended", "airbnb-base" ], "rules": { "no-alert": "off" } }

Schlusswort

Nicht jeder Linter und Preset ist für jeden Einsatzzweck geeignet. Deshalb ist es wichtig, verschiedene Varianten zu testen. Wir nutzen bei in2code eine Vielzahl an Lintern für verschiedene Einsatzzwecke. Von ESLint bis hin zu PHP Fixern helfen und diese Tools unsere Codequalität hochzuhalten.

TYPO3: Unbenutzte Dateien im fileadmin finden

Ihr wollt unbenutzte oder verwaiste Dateien im fileadmin oder einer anderen Storage löschen? Leider gibt es hier nichts direkt vom Core. Aber ein kleiner Command in eurem Sitepackage hilft schnell...

Zum Beitrag

TYPO3: Redakteure mit individuellen user_upload Folder

Vielleicht kennt ihr die Kundenanforderung? Redakteure sollen Videos über den Button "Add media by URL" hinzufügen können. Aber die Dateien sollen sich dann nicht in fileadmin/user_upload/ sondern in...

Zum Beitrag

TYPO3: Seiten im Mixed Mode finden

Mixed Mode bezeichnet man in TYPO3 übersetzte Seiten, auf denen sich Seiteninhalte befinden, die nur zum Teil eine Relation auf entsprechende Seiteninhalte in der Hauptsprache haben. Dies wird im...

Zum Beitrag

Extbase Extensions: An Erweiterbarkeit denken mit data, site und language

Heute mal eine kleine Bitte an die TYPO3-Extension-Autoren da draußen: Achtet auf Erweiterbarkeit eurer Extensions. Dies fördert dann auch die Verbreitung der entsprechenden Plugins.

Zum Beitrag

SQL: Zeige alle Tabellen absteigend nach Größe sortiert

Ich brauche in letzter Zeit häufiger den SQL-Befehl, um herauszufinden, welche Tabellen in der TYPO3-Datenbank am größten sind. Ich habe das Snippet einmal veröffentlicht.

Zum Beitrag

TYPO3 12 mit CKEditor 5: Stile als Einfachauswahl

Wenn man im RTE in TYPO3 einen Link setzt, kann es sein, dass man zwischen verschiedenen Link-Klassen auswählen muss, um beispielsweise Buttons im Frontend zu erzeugen. Neu ist in TYPO3 12 dass man...

Zum Beitrag