ahueber / iperf-gui Goto Github PK

Unter Mac OS X verhalten sich die Fenster sehr eigenartig. Einerseits habe ich die volle Fensterbreite (was auf dem Raspberry Sinn macht), andrerseits verschwinden die Fenster aber bei gewissen Aktionen (z.B. wenn ich das Numpad schliesse). Zur Abhilfe lasse ich mir die Windows mittels s->show() anzeigen. Hier sollte noch geprüft werden, ob das Handling der QWidgets korrekt ist.

Das Numpad benötigt meiner Meinung nach eine Filterung bzw. Validierung der Eingabe. Im Moment können völlig sinnlose Werte eingegeben werden. Eine mögliche Filterung/Validierung könnte die folgenden Aspekte berücksichtigen:

• auf Punkte prüfen
• auf Anzahl Segmente prüfen
• auf Dezimalwert der einzelnen Segmente prüfen (min. 0, max. 255)
• auf max. Anzahl Zeichen prüfen (15 inkl. Punkte)

Bei Klick auf "Server" stürzt das Programm (Branch iperf3-interface) ab.

Fehlermeldung von Qt:
Starte /Users/thomas/temp/iperf-GUI/build-iperf-Desktop_Qt_5_6_0_clang_64bit-Debug/iperf.app/Contents/MacOS/iperf...
ASSERT: "!isEmpty()" in file /Applications/Qt5.6.0/5.6/clang_64/lib/QtCore.framework/Headers/qlist.h, line 335
Das Programm ist abgestürzt.
/Users/thomas/temp/iperf-GUI/build-iperf-Desktop_Qt_5_6_0_clang_64bit-Debug/iperf.app/Contents/MacOS/iperf ist abgestürzt

Wenn ich auf eine Sliderposition klicke, ohne den Slider zu packen und zu verschieben, wird kein Wert gesetzt. Das sieht für mich so aus, als ob ein SIGNAL nicht implementiert ist.

Labels und Buttons sind noch uneinheitlich benannt. Ich schlage vor, dass wir Deutsch als Standard vorsehen. Wir können uns aber auch auf Englisch einigen. Es muss "nur" einheitlich sein.

Folgendes sollte geändert werden:

welcomescreen.cpp:

• Exit --> Beenden

server.cpp:

• Start --> Starten
• Close --> Schliessen

client.cpp:

• Start --> Starten
• Close --> Schliessen

numpad.cpp:

• Done --> (Vorschlag?)

Und ebenfalls auch das Menu sollte angepasst werden:

• Hide iperf
• Hide others
• Show all
• Quit iperf

Es fehlen Programm-Icons. Für Raspberry Pi möglicherweise nicht notwendig, aber etwas unschön für andere Umgebungen.

Im Client-Modus fehlen Defaultwerte für Laufzeit und Bandbreite. Generell sollten Defaultwerte wo möglich gesetzt sein. Idealerweise löst man dies über QSettings (kein Kompilieren notwendig bei Ändern von Settings).