Kapitel 6 -> Weshalb wird das vote Array kopiert...

[duschata]

wenn die Referenz auf das Objekt eh da ist?

registerChoice(choice) {
        choice.count++;
        this.setState({vote: this.state.vote});
    }

obiger Code hat das gleiche Verhalten.

Ferner irritieren mich die mehrmalige Verwendung von
const {vote} = this.state;
in den Funktionen. Hier frage ich mich wieso jedes mal eine neue lokale Referenz zu einem Wert, der sowieso in der Klasse existiert erzeugt wird.
Vielleicht nur Kosmetik, aber wäre vote einmal im Konstruktor als Instanzvariable gesetzt, wüsste jeder bei der Verwendung von "this.vote" innerhalb der Klasse was gemeint sein sollte.

[nilshartmann]

Hallo,

Du hast recht - dein Code würde auch funktionieren. Wir kopieren das Array, damit das ursprüngliche Array unverändert (immutable) bleibt.

In React wird aber grundsätzlich versucht, mit unveränderlichen (immutable) Objekten zu arbeiten (es gibt sogar eine eigene Bibliothek dafür: https://facebook.github.io/immutable-js/) um Fehler zu vermeiden oder auch um Code zu optimieren (Objekte müssen z.B. nicht "tief" verglichen werden, sondern ihre Referenz kann einfach verglichen werden, um zu sehen, ob sie sich verändert haben).

Die Verwendung von const { vote } = this.state ist Geschmackssache. Wir haben versucht, am Beginn jeder Funktion die Werte, die wir aus dem State und aus den Properties innerhalb der Funktion benötigen in einer lokalen Variable abzulegen, damit man schnell erkennen kann, welche Teile des States/der Properties in der Funktion benötigt werden. Man kann natürlich stattdessen jederzeit mit this.state.xyz / this.props.xyz auf die benötigten Informationen zugreifen.

Das Ablegen von Teilen des States als Instanzvariable sollte man hingegen nicht tun. Zum einen stehen die Informationen ja im State (und dort müssen sie rein, da sie ja "Render-relevant" sind), d.h. man würde sie redundant halten. Und wenn man mit immutable Objekten arbeitet, ist das ohnehin keine Option, weil sich die Referenz ja nach jedem Ändern der Daten ändert und dann auch this.vote angepasst werden müsste.

Beantwortet das deine Fragen? Sonst gerne nochmal nachhaken. Die Fragen zu Immutable nehme ich mal in unsere Liste mit Verbesserungen für eine neue Auflage des Buches auf.

[duschata]

sorry, eineinhalb Jahre später lese ich gerade das Buch ein zweites Mal und ich weiß nicht, ob es eine Neuauflage des Buchs gibt. Ich stolpere genau wieder über die Stelle und seniere darüber, was mich so irritiert. Wenn für das Rendering die Objekte "immutable" sein müssen, müsste der Code konsequenter Weise auch dich nicht veränderten Choices kopieren, oder?

const newVote = {
            ...vote,
            choices:
                vote.choices.map(choice =>
                    choice.id !== choiceToIncrement.id
                        ? {
                            ...choice
                        }
                        : {
                            ...choice,
                            count: choice.count + 1
                        }
                )
        };