Erstes eigenes VI: Bitte um Feedback

Danke, Jens!

Anzeige

(27.11.2018 13:28 )GerdW schrieb:  - Das Frontpanel muss nicht auf "maximiert" gestellt werden. Vor allem bei den paar FP-Elementen…
- Ab und zu mal AutoCleanup benutzen…
- Ich bin kein Freund des DAQAssistenten. Dafür gibt es einfache DAQmx-Funktionen.
- Sehr viele lokale Variablen - IMHO zuviele für ein solche kleines VI.
- Du verwendest eine Wartezeit in der Hauptschleife, obwohl DAQmx das Schleifentiming vorgeben sollte.
- Das State-Enum ist nicht typdefiniert.
- Das BuildArray gehört direkt vor den zweiten DAQAssi, das musst du nicht 4 oder 5mal im Code verstreuen!
- Die Registerkarte sollte keine äußere Case-Struktur umschalten, sondern als Bedingung einen State der inneren Case-Struktur aufrufen.
- SelectedPath gehört in ein Schieberegister, so wie alle anderen internen "Variablen" der Statemachine auch…
- Verwende einen Latch-Schaltmodus für deine Buttons, statt sie über lokale Variablen wieder zurückzusetzen.
- Das VI kann nicht sauber beendet werden (nur gewaltsames Abbrechen ist möglich).
- Warum sind die Ausgabewerte vom Typ DBL, obwohl der DAQAssi U32 erwartet?

Habe mein Programm nun entsprechend den Vorschlägen modifiziert. Könntet ihr nochmal drüberschauen? Ich habe aktuell das Problem, dass das Umschalten der Legende beim Diagramm von "Spannung" auf "Weg" nicht funkioniert. Es wird immer "Spannung" angezeigt bzw. wenn ich den String "Spannung" in etwas anderes abändere, zeigt das Diagramm immer noch "Spannung" an.

Mir ist auch nicht klar, was du, Gerd, mit der Anmerkung meintest, von wegen das FP sei auf maximiert gestellt.

Ansonsten tut das Programm scheinbar was es soll. Wenn ich den Dialog zum Speichern der Datei abbreche, bricht das VI aber mit einem Fehler ab, obwohl ich eigentlich nur wieder in den Init State wollte.

(03.12.2018 11:55 )_Markus_ schrieb:  Habe mein Programm nun entsprechend den Vorschlägen modifiziert. Könntet ihr nochmal drüberschauen? Ich habe aktuell das Problem, dass das Umschalten der Legende beim Diagramm von "Spannung" auf "Weg" nicht funkioniert. Es wird immer "Spannung" angezeigt bzw. wenn ich den String "Spannung" in etwas anderes abändere, zeigt das Diagramm immer noch "Spannung" an.

Hallo Markus,

ergänze in dem Eigenschaftsknoten wo du den Plotname ändern willst oberhalb Plotname noch die Eigenschaft "Aktiver Plot" und erstelle dazu die Konstante 0, dann wird immer der erste Plotname in der Legende geändert wie du es gerne hättest.

Viele Grüße,
Ralf