search

enbility / eebus-go

EEBUS protocol implementation in go
https://enbility.net
MIT License
59 stars 21 forks source link

Refactor enum types #1

Closed DerAndereAndi closed 2 years ago

DerAndereAndi commented 2 years ago

This way we never would need to cast for these

DerAndereAndi commented 2 years ago

@taimo42 was denkst du hierzu?

taimo42 commented 2 years ago

Top! Auf jeden Fall eine Vereinfachung. Können wir noch die doppelten 'Type' entfernen? Also z.B. statt DeviceTypeTypeChargingStation nur DeviceTypeChargingStation?

DerAndereAndi commented 2 years ago

@taimo42 Hmm... laut XSD heisst der type DeviceTypeType, dann sollten die dazugehörigen Varianten imho auch so beginnen. Ansonsten passt die Zuordnung nicht mehr.

In networkmanagment.go Zeile 141 sieht man warum das so gemacht ist:

  DeviceType  *DeviceTypeType  `json:"deviceType,omitempty"`

Dann wird klar was ein type ist, und was ein field.

Würde ich also so lassen.

taimo42 commented 2 years ago

Ok, macht Sinn