dr.e.

Hallo HStev,

Eine direkte ABhängigkeit wäre dann gegeben, wenn die Connection den Command kennen muss um ihn ausführen zu können. Dass der Inhalt eines Commands irgendwann an die Connection gerät ist klar, nur sollte das die Connection-Klasse regeln. So kannst du beliebige Commands einhängen.

Nach längerem Nachdenken würde ich fast sagen, dass noch eine dritte Klasse ins Spiel gebracht werden sollte, die das Ganze steuert. Connection und Command sind in diesem Fall dann passiv und werden von einer Networking-Komponente gesteuert, der man eine beliebige Connection und einen beliebigen Command geben kann. So wärst du noch unabhängiger und könntest z.B. nicht nur HTTP, sondern z.B. auch FTP addressieren.

Back to topic: Zeig mir mal ein UML, daran kann man sicher einfacher diskutieren.

__________________
Viele Grüße,
Dr.E.

~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
1. Think about software design before you start to write code!
2. Discuss and review it together with experts!
3. Choose good tools (-> Adventure PHP Framework (APF))!
4. Write clean and reusable software only!
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Mister Ad

HStev

Achso verstehe du meinst also eine Klasse die den reinen Socket und dessen Funktionen bereitstellt. Eine Klasse die die Communikation handelt und die jeweiligen Unterklassen für die Commands ... hinzu würden dann noch die entsprechenden "Besucher" kommen die dann entweder die Daten speichern oder darstellen.

Naja in diesen Fall wird es bei einen Protokoll bleiben, es handelt sich hierbei um das svdrp Protokoll des VDR's in der Version 1.5 es kommt höchstens zu einer Erweiterung oder Änderung der Befehle deshalb will ich das ganze Klassen basiert haben.

SVDRP - VDR Wiki

__________________
Gewisse Dinge behält man besser für sich, z.B. das man gewisse Dinge für sich behält.

HStev

So richtig fit bin ich in UML noch nicht ich arbeite mich grad erst ein.
Aber ich hoffe das es einigermaßen verständlich ist.

so hab ich mir das bisher gedacht, da PHP Klassen als Referenz übergibt brauche ich im Prinzip den Handler nicht.



ps: ist übrigens das 1. Projekt wo ich eine sinnvolle Verwendung des destructors gefunden habe, der Destructor sendet in diesen Fall noch ein QUIT an den Server damit der Server nicht ins Timeout läuft und die Connection sauber geschlossen wird.

__________________
Gewisse Dinge behält man besser für sich, z.B. das man gewisse Dinge für sich behält.

dr.e.

Hallo HStev,

hast du mal ein Code-Beispiel dazu?

Was ich nicht ganz nachvollziehen kann:
* Warum hat Connection::execute() ein Argument-Array dabei? Warum bildest du das nicht über den Command ab?
* Warum führtst du kein Answer-Objekt ein? Einen String zurückzubekommen ist problematisch, da dieser wieder geparst werden muss.

__________________
Viele Grüße,
Dr.E.

~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
1. Think about software design before you start to write code!
2. Discuss and review it together with experts!
3. Choose good tools (-> Adventure PHP Framework (APF))!
4. Write clean and reusable software only!
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

HStev

ist mittlerweile schon recht umfangreich welchen Teil willst genau sehen

Da bin ich mir halt noch nicht so ganz im klaren wie ich das lösen soll.
Einige der Befehle erwarten 0-3 Parameter. Und es ist auch eigentlich kein Array sonderen soll eine Variable Parameter-Liste sein und ich frag die Parameter daher mit func_get_args ab und gib die dann nur direkt weiter an das Command.

Wie meinst du das? Die Antwort für jeden Befehl sieht anders und muss anders geparsed bzw. verarbeitet werden.
Die Daten die ankommen sind im Prinzip
Einzeilige Antwort:
3 stelliger Code gefolgt von einen Datenstring und Carage Return Line Feed
Mehrzeilige Antwort:
3 stelliger Code - Zeilennummer Datenstring Carage Return
Letzte Zeile wird abgeschlossen mit CR LF

Und der Datenstring ist selber auch noch mal unterteilt je nach Befehl zb. der Befehl LSTC holt alle Kanale und gibt deren Daten nach den Schema aus wie sie in der channels.conf angegeben sind.
Channels.conf - VDR Wiki

Das würde dann im Prinzip bedeuten das ich für jeden Befehl noch ein AnswerObjekt einführen muss was aber eigentlich sinnlos wäre weil dann bräucht das Command-Object nicht weil beim senden ist nicht viel zu beachten.

__________________
Gewisse Dinge behält man besser für sich, z.B. das man gewisse Dinge für sich behält.

dr.e.

Hallo HStev,

Ich meinte nur ein Anwendungsbeispiel. Die Implementierung selbst ist erst mal zweitrangig, die Benutzung (API) ist in unserer Diskussion interessant.

Das ist IMHO nicht gut. Die Parameter sollen dem Command mitgegeben werden. Weiß die Connection davon bist du vom Design nicht mehr sauber.

Anders gefragt: es gibt nie den Fall, dass mehrzeilige oder komplexe Daten zurückgeliefert werden? Falls ja, macht ein Answer-Objekt Sinn, falls nein, gebe ich dir Recht.

__________________
Viele Grüße,
Dr.E.

~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
1. Think about software design before you start to write code!
2. Discuss and review it together with experts!
3. Choose good tools (-> Adventure PHP Framework (APF))!
4. Write clean and reusable software only!
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

HStev

Doch hab ich ja gesagt es hängt vom Befehl der LSTC liefert ein mehrzeiliges Ergebnis aber mit eigener Struktur. Wie gesagt wenn ich ein Answer Objekt einführe dann hätte das Command nur noch eine Methode die vielleicht 3-4 Zeilen Code enthält.

__________________
Gewisse Dinge behält man besser für sich, z.B. das man gewisse Dinge für sich behält.

dr.e.

Hallo HStev,

Das glaube ich zwar jetzt nicht ganz, aber in diesem Fall musst du entscheiden, ob du eine saubere Datenstruktur haben möchtest oder nicht. Ich wäre für Answer-Objekt.

__________________
Viele Grüße,
Dr.E.

~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
1. Think about software design before you start to write code!
2. Discuss and review it together with experts!
3. Choose good tools (-> Adventure PHP Framework (APF))!
4. Write clean and reusable software only!
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~