Autor Thema: ROLLO ins SVN  (Gelesen 425 mal)

Offline KernSani

  • Developer
  • Hero Member
  • ****
  • Beiträge: 2644
ROLLO ins SVN
« am: 13 Januar 2019, 14:32:53 »
Hallo zusammen,

ich habe mich in letzter Zeit wieder verstärkt im ROLLO-Modul engagiert und würde es demnächst ins SVN einchecken wollen - Mag irgendwer mal drüber schauen ob noch irgendwelche groben Schnitzer auffallen?

Danke,

Oli
RasPi: RFXTRX, HM, zigbee2mqtt, mySensors, JeeLink, miLight, squeezbox, Alexa, Siri, ...

Offline CoolTux

  • Developer
  • Hero Member
  • ****
  • Beiträge: 18385
Antw:ROLLO ins SVN
« Antwort #1 am: 13 Januar 2019, 14:49:00 »
Hallo,

Ich habe nur mal eben kurz geschaut und es sind nur Formalitäten

Das direkte schreiben in Hash sollte bei FHEM Standardroutinen vermieden werden.
$attr{$name}{"rl_secondsDown"}
Hier lieber mit CommandAttr arbeiten und mit AttrVal abfragen ob schon gesetzt wurde.

Auch hier nicht direkt in den Hash STATE schreiben wegen bekannter Probleme
$hash->{STATE} = "inactive";
Lieber
readingsSingleUpdate( $hash, "state", "Initialized", 1 );

Hier wäre meine Empfehlung
CommandDeleteReading statt readingsDeletezu verwenden


Die Funktion fhem() sollte in einem Modul nichts zu suchen haben. Das ist User Krempel. ( persönliche Meinung ) und noch nie in einem Modul gesehen.
fhem("attr $name devStateIcon $iconHomeKit")
              if ( ( $aVal eq "HomeKit" ) && ( ( $iconAktuell eq $iconNormal ) || ( $iconAktuell eq "kein" ) ) );
            fhem("attr $name devStateIcon $iconNormal")
              if ( ( $aVal eq "normal" ) && ( ( $iconAktuell eq $iconHomeKit ) || ( $iconAktuell eq "kein" ) ) );

Das war erstmal das was ich so auf den schnellen Blick gesehen habe.

Mein persönlicher Favorit wäre natürlich noch ein packages daraus zu machen  ;D


Die Commandref sieht soweit ok aus. Kann man aber ruhig noch mal durch commandref_join jagen.


Grüße
Du musst nicht wissen wie es geht! Du musst nur wissen wo es steht, wie es geht.
Support me to buy new test hardware for development: https://www.paypal.me/MOldenburg
Mein GitHub: https://github.com/LeonGaultier
kein Support für cfg Editierer

Offline KernSani

  • Developer
  • Hero Member
  • ****
  • Beiträge: 2644
Antw:ROLLO ins SVN
« Antwort #2 am: 13 Januar 2019, 15:04:13 »
Thanks Leon...

Gute Punkte, das sind "Altlasten", die ich bisher nicht angefasst habe... schau ich nochmal drüber.

Was meinst du mit
Zitat
Mein persönlicher Favorit wäre natürlich noch ein packages daraus zu machen  ;D
???

Grüße,

Oli
RasPi: RFXTRX, HM, zigbee2mqtt, mySensors, JeeLink, miLight, squeezbox, Alexa, Siri, ...

Offline CoolTux

  • Developer
  • Hero Member
  • ****
  • Beiträge: 18385
Antw:ROLLO ins SVN
« Antwort #3 am: 13 Januar 2019, 15:16:37 »
Schau mal im Web nach Perl packages. Ist im Grunde sowas wie Klassen in C und Java.
Abgeschlossener Bereich. Du musst dann nicht mehr mit Präfix bei den Funktionen arbeiten.
Ich finde das ganz Klasse.
Einzig Du musst dafür sorgen daß jede Funktion welche Du aus der main aufrufen willst entweder importiert werden oder Du die Funktion über den Klassenaufruf aufrufst.
Z.b. main::CommandDelete

Ich habe bereits einige meiner Module entsprechend umgebaut. Kannst gerne reinschauen und mich dann einfach anschreiben.
Muss ja auch nicht heute oder morgen sein  :)


Grüße
Du musst nicht wissen wie es geht! Du musst nur wissen wo es steht, wie es geht.
Support me to buy new test hardware for development: https://www.paypal.me/MOldenburg
Mein GitHub: https://github.com/LeonGaultier
kein Support für cfg Editierer

Offline KernSani

  • Developer
  • Hero Member
  • ****
  • Beiträge: 2644
Antw:ROLLO ins SVN
« Antwort #4 am: 13 Januar 2019, 20:17:08 »
Sodele, Cooltux Empfehlungen sind umgesetzt (außer der Idee mit den packages ;-)). Wer nochmal checken mag: https://github.com/RettungsTim/fhem-rollo
RasPi: RFXTRX, HM, zigbee2mqtt, mySensors, JeeLink, miLight, squeezbox, Alexa, Siri, ...

Offline CoolTux

  • Developer
  • Hero Member
  • ****
  • Beiträge: 18385
Antw:ROLLO ins SVN
« Antwort #5 am: 13 Januar 2019, 21:06:24 »
Sieht sehr gut aus.
Die Codeformatierung kommt mir irgendwie bekannt vor  ;D


Grüße
Du musst nicht wissen wie es geht! Du musst nur wissen wo es steht, wie es geht.
Support me to buy new test hardware for development: https://www.paypal.me/MOldenburg
Mein GitHub: https://github.com/LeonGaultier
kein Support für cfg Editierer

Offline KernSani

  • Developer
  • Hero Member
  • ****
  • Beiträge: 2644
Antw:ROLLO ins SVN
« Antwort #6 am: 13 Januar 2019, 22:19:22 »
Sieht sehr gut aus.
Die Codeformatierung kommt mir irgendwie bekannt vor  ;D
Yep, da ich dein Shutters-Modul gerade eh offen hatte, um mir packages anzuschauen hab ich mir CommandAttr geklaut ;-) Übrigens - wenn du Ideen hast, wie man die Integration von AutoShutters und ROLLO verbessern kann: Immer her damit (ich habe mich noch nicht daran gewagt, meine Rollladen-Steuerung anzufassen, nutze das AutoShutters-Modul daher nicht)
 
RasPi: RFXTRX, HM, zigbee2mqtt, mySensors, JeeLink, miLight, squeezbox, Alexa, Siri, ...

Offline CoolTux

  • Developer
  • Hero Member
  • ****
  • Beiträge: 18385
Antw:ROLLO ins SVN
« Antwort #7 am: 13 Januar 2019, 22:22:35 »
Yep, da ich dein Shutters-Modul gerade eh offen hatte, um mir packages anzuschauen hab ich mir CommandAttr geklaut ;-) Übrigens - wenn du Ideen hast, wie man die Integration von AutoShutters und ROLLO verbessern kann: Immer her damit (ich habe mich noch nicht daran gewagt, meine Rollladen-Steuerung anzufassen, nutze das AutoShutters-Modul daher nicht)

Das mit den Attributen meinte ich weniger, eher die Codeformatierung und den Stil. Benutzt Du auch perltidy?
Du musst nicht wissen wie es geht! Du musst nur wissen wo es steht, wie es geht.
Support me to buy new test hardware for development: https://www.paypal.me/MOldenburg
Mein GitHub: https://github.com/LeonGaultier
kein Support für cfg Editierer

Offline KernSani

  • Developer
  • Hero Member
  • ****
  • Beiträge: 2644
Antw:ROLLO ins SVN
« Antwort #8 am: 13 Januar 2019, 22:27:28 »
Benutzt Du auch perltidy?
Genau :-) Habe nur noch keinen guten Weg gefunden, wie ich das HTML für die commandref auch vernünftig formatiert bekomme...
RasPi: RFXTRX, HM, zigbee2mqtt, mySensors, JeeLink, miLight, squeezbox, Alexa, Siri, ...

Offline CoolTux

  • Developer
  • Hero Member
  • ****
  • Beiträge: 18385
Antw:ROLLO ins SVN
« Antwort #9 am: 13 Januar 2019, 22:28:51 »
Genau :-) Habe nur noch keinen guten Weg gefunden, wie ich das HTML für die commandref auch vernünftig formatiert bekomme...
Ach da mache ich ausser einrücken gar nichts weiter.


Grüße
Du musst nicht wissen wie es geht! Du musst nur wissen wo es steht, wie es geht.
Support me to buy new test hardware for development: https://www.paypal.me/MOldenburg
Mein GitHub: https://github.com/LeonGaultier
kein Support für cfg Editierer

Offline betateilchen

  • Developer
  • Hero Member
  • ****
  • Beiträge: 15519
  • s/fhem\.cfg/configDB/g
Antw:ROLLO ins SVN
« Antwort #10 am: 14 Januar 2019, 09:04:40 »
Habe nur noch keinen guten Weg gefunden, wie ich das HTML für die commandref auch vernünftig formatiert bekomme...

Mach Dir nicht zuviel Gedanken, alles was Du da (unnötig) an HTML Tags investierst, muss ich im schlimmsten Fall bei "help" wieder weg-regexen.
-----------------------
Unaufgeforderte Anfragen per email werden von mir nicht beantwortet. Dafür ist das Forum da.
-----------------------
Nächster Hamburg-Stammtisch: 22.03.2019 - 18:30 Uhr im Baseler Hof

Offline CoolTux

  • Developer
  • Hero Member
  • ****
  • Beiträge: 18385
Antw:ROLLO ins SVN
« Antwort #11 am: 14 Januar 2019, 09:40:22 »
Mach Dir nicht zuviel Gedanken, alles was Du da (unnötig) an HTML Tags investierst, muss ich im schlimmsten Fall bei "help" wieder weg-regexen.

 ;D  ;D
Du musst nicht wissen wie es geht! Du musst nur wissen wo es steht, wie es geht.
Support me to buy new test hardware for development: https://www.paypal.me/MOldenburg
Mein GitHub: https://github.com/LeonGaultier
kein Support für cfg Editierer

Offline nils_

  • Sr. Member
  • ****
  • Beiträge: 986
Antw:ROLLO ins SVN
« Antwort #12 am: 14 Januar 2019, 10:33:53 »
Mach Dir nicht zuviel Gedanken, alles was Du da (unnötig) an HTML Tags investierst, muss ich im schlimmsten Fall bei "help" wieder weg-regexen.

und ich dachte du machst das gerne  8) :-X :-X
viele Wege in FHEM es gibt!

 

decade-submarginal