search

ioBroker / ioBroker.ekey

Connect ekey to ioBroker
MIT License
8 stars 2 forks source link

Update main.js #4

Closed Maximilian4711 closed 5 years ago

Maximilian4711 commented 5 years ago

Leerzeichen/blank/space als Separator führt zum von mir gemeldeten Fehler. Name darf z.B. bei Adelstiteln ein blank enthalten. Am Ende eines Namens kann bei Eingabe in ekey-Multi-Zentrale auch irrtümlich ein blank angefügt werden; durch das kleine Display ist das kaum zu erkennen. Korrektur eines falsch eingegebenen Namens ist nicht möglich, löschen & Neueingabe aufwändig. Daher macht es Sinn, blank als Separator rauszunehmen. Semikolon als Separator wäre sinnvoller Ersatz, da bei Namenseingabe definitiv falsch und gut erkennbar.

Maximilian4711 commented 5 years ago

Sinnvoll wäre auch, die Gesamtzahl der nach split erzeugten values[]zu zählen, sodass eine aussagekräftige Fehlermeldung im Log auf die falsche Trennung des Parameterstrings hinweist.

GermanBluefox commented 5 years ago

Ich bin noch nicht schlüssig.... Ist delimiter bey Ekey einstellbar?

Maximilian4711 commented 5 years ago

Ja, mit dem Konfigurationstool für den CV Lan kann man nicht nur IP-Adresse, Gateway, Mode (Home, Multi, Raw/Rare) usw. einstellen, sondern auch das Delimiterzeichen. Default ist der Unterstrich _ . Also wäre es am geschicktesten, im Adapter den Delimiter mit Unterstrich vorzubelegen und dem Anwender die Eingabe eines von ihm in den CV Lan eingegebenen Delimiter zu ermöglichen. Dieser (dann einzige) Delimiter kann dann in der split-Funktion verwendet werden.

Apollon77 commented 5 years ago

Das klingt in meinen Augen sinnvoll so muss der Adapter nicht raten.