Jeśli chodzi o vue, praktycznie wzorcowo napisany kod według dokumentacji. Dobra robota! Kilka rzeczy złapałem
1. Niepotrzebnie tworzysz gettery. Jeśli potrzebujesz odczytać dane ze state'u, odczytaj je bezpośrednio. Gettery to takie computed dla stora
2. Nie nadpisuj natywnych obiektów (Error)
import Error from '@/components/Error';
3. Miejscami brakuje scoped w css-ach, Warto dodawać, żeby później się nie denderwować
4. Niepotrzebnie rzutujesz na booleana
:light="!!error && darkMode"
operator sam sobie zrzutuje
5. Eslint albo IDE powinno Ci wychwytywać takie "Błędy"
let answer = [];
for (const index in this.answer) {
if (!this.answer[index][0]) answer[index] = this.text[index];
else answer[index] = this.answer[index][0];
}
this.error = this.session.check(answer.join(' '));
const answer, zamiast let
6. Nie musisz wywoływać metod w templatkach, wystarczy, że podasz referencje
<v-btn icon dark @click="dismiss()">
@click="dismiss">
7. Preferowany rodzaj komunikacji z parentem to $emit
props: {
dismiss: Function,
}
Zwiększa to czytelność i reużywalność komponentów
8. Nie rozumiem po co tutaj
const state = {
courseList: [],
courses: {},
};
hash table i zwykla lista, imo dużo zamieszania jest w tym module