search

LFF5644 / site-spotifyApiService

let control the spotify api with my web page and see my now playing song global
https://lff.l3p3.de/server/spotify/spotify.api
0 stars 0 forks source link

Typenfehler #3

Closed L3P3 closed 1 year ago

L3P3 commented 1 year ago

Die Konfiguration ist ein object, aber false ist ein boolean. Diese Variable hat wegen dieser Zeile einen dynamischen Typen, V8 muss also extra prüfen und speichern, ob da gerade ein Objekt oder ein Boolean drin gespeichert ist. Du meinstest vermutlich das null-Objekt.

https://github.com/LFF5644/site-spotifyApiService/blob/eeb84909112d9358fca8f6746bd73284a2c7eaa6/spotify.service.js#L55

Die Variable darüber sollte dann auch mit null initialisiert werden, weil es sonst sogar noch schlimmer ist: Die Variable kann ein String, ein Objekt oder ein Boolean sein! AUA!

LFF5644 commented 1 year ago

ja das kann gut sein ABER ich kann so um so einfacher im code schneller Prüfen https://github.com/LFF5644/site-spotifyApiService/blob/eeb84909112d9358fca8f6746bd73284a2c7eaa6/spotify.service.js#L57

LFF5644 commented 1 year ago

ich finde das jz nicht als fehler sondern ok denn es machen viele leute so!