Diskussionsissue

[tderflinger]

Hallo,

wie besprochen hier ein Issue meiner Findings:

• Nutzung von let/const anstatt var
• Nutzung von .toString anstatt new String()
• ES6 Linter strenger einstellen
• Bevorzugung forEach / for of / for in anstatt konventionelles for (let x; x < 10;x++)
• Funktionsparameter keine Werte zuweisen
• Komponenten feingranularer schneiden, nicht zu viel pro Komponente, lieber mehrere Komponenten, auch Kindkomponenten

Grüsse,
-Thomas

[xdoo]

@tderflinger bitte in den Code referenzieren. So können wir nicht alle Punkte nachvollziehen.

[xdoo]

@tderflinger

Komponenten feingranularer schneiden, nicht zu viel pro Komponente, lieber mehrere Komponenten, auch Kindkomponenten

Bitte Beispiele benennen und einen Schnitt für diese Beispiele vorschlagen.

[rowe42]

@tderflinger @a52team Kann hier bitte nochmal jemand reagieren auf unsere Anfrage oben?

[tderflinger]

Zum Beispiel in animad-app.html könnte man noch eine Komponente aus folgenden Zeilen machen:

                        <template is="dom-if" if="{{animalsViewAllowed}}">
                        <a class="paper-item-link" name="animals-view" href="#[[rootPath]]animals-view/" tabindex="-1">
                            <paper-icon-item>
                               <iron-icon icon="animad-icons:animals-view" slot="item-icon"></iron-icon>
                                [[$t('animals')]]
                          </paper-icon-item>
                        </a>
                        </template>

Danach:

<template is="dom-if" if="{{animalsViewAllowed}}"> <animal-icon ... /> </template>

Meiner Meinung sollte man lieber mehrere Komponenten generieren, was die Leserlichkeit erhöht (allerding auch etwas mehr Code).