Hi alle zusammen.
Das ist vielleicht merkwürdig, aber ich würde gerne eure Meinung über mein erstes (funktionierendes) Progamm erfahren.
Es geht um Datenerfassung einer Messdose und um die gleichzeitige Steuerung einer Achse für verschiedene Abzugsversuche.
Dazu muss ich sagen, das ich vorher nicht viel Erfahrung hatte und auch noch keine Sub-VI`s eingebaut hab.
Also, eure ehrliche Meinung ist gefragt. Immer raus damit...

Und danke noch mal für die zahlreiche Unterstützung eurerseits.
Steve
Hi,
was schön ist: Du ziehst gerade Linien! So sieht es sauber aus!
ABER: Hast du schon mal was von SubVIs gehört? Alle deine Funktionen (Beschleunigugn, etc.) könntest du jeweils in ein solches packen und dadurch das ganze deutlich übersichtlicher, weil kompakter gestalten. Empfehlung: Nie größer als eine Bildschirmgröße coden (ca. 1280x1024).
Was macht die While-Schleife in der While-Schleife? Wie funktioniert das?
Die Wartezeit (10ms) in der oberen Schleife solltest du am besten an die Eventstruktur anschließen!
Die Flat Sequence ist unnötig, weil sich die Reihenfolge durch die Verbindungsdrähte ergibt! Ich vermute, das hast du mehr wegen der Übersichtlichkeit gemacht. Das kann dann bei SubVIs entfallen, wenn diese z.B. durch den Resource Name miteinander verbunden sind!
Dein VI hätte man hübsch als State Machine programmieren können, dann wäre es VIEL übersichtlicher. Beispiele dazu findest du en masse im Forum!
Über die Funktionalität kann ich nichts sagen!
Sei mir bitte nicht böse, aber du wolltest ja die Wahrheit hören
Gruss
Achim
Hallo,
ich finde es sieht gar nicht schlecht aus.
Eine State-Machine wäre aber ein guter Ratschlag für das nächste Projekt.
Gruß Markus
' schrieb:Hi alle zusammen.
Das ist vielleicht merkwürdig, aber ich würde gerne eure Meinung über mein erstes (funktionierendes) Progamm erfahren.
Es geht um Datenerfassung einer Messdose und um die gleichzeitige Steuerung einer Achse für verschiedene Abzugsversuche.
Dazu muss ich sagen, das ich vorher nicht viel Erfahrung hatte und auch noch keine Sub-VI`s eingebaut hab.
Also, eure ehrliche Meinung ist gefragt. Immer raus damit... 
Und danke noch mal für die zahlreiche Unterstützung eurerseits.
Steve
Also ich habe nur einen ganz flüchtigen Blick darauf geworfen, verstehe also im Moment nichts.
Nur ein Nebensache fiel mir sofort auf.
Die Verwendung von Sequenzen ist nur für Notfälle gedacht, die viele bessere Alternative, und dafür ist LV konzipiert, ist die Programmsteuerung über den Datenfluß. Da hast fast überall im Programm eine einwandfrei Datenflußsteuerung mittels Fehlerstrang und Task-Identifier - was Dich aber überhaupt nicht davon abhält, überflüssiger- und schädlicherweise (Was die Ausführungszeit betrifft) zusätzlich Sequenzen anzuwenden. Es sieht so aus, als traust Du der LV-Datenflußsteuerung nicht so richtig über den Weg und möchtest auf Nummer sicher gehen.
Prüfe kritisch, was davon wegfallen kann. Im Groben sieht es so aus: die meisten "Trennwände" in den Großen Sequenzen können wegfallen, und nur die kleinen Sequenzen innerhalb der großen Sequenzen haben ihre Berechtigung.
' schrieb:Hi alle zusammen.
Das ist vielleicht merkwürdig, aber ich würde gerne eure Meinung über mein erstes (funktionierendes) Progamm erfahren.
Es geht um Datenerfassung einer Messdose und um die gleichzeitige Steuerung einer Achse für verschiedene Abzugsversuche.
Dazu muss ich sagen, das ich vorher nicht viel Erfahrung hatte und auch noch keine Sub-VI`s eingebaut hab.
Also, eure ehrliche Meinung ist gefragt. Immer raus damit... 
Und danke noch mal für die zahlreiche Unterstützung eurerseits.
Steve
Ehrliche Meinung??
Also:
Solange das Blockdiagramm des VI nicht auf eine Bildschirmseite passt, ist es kein gutes VI.
(Hängt natürlich von der Bildschirmauflösung ab. Aber auf 1280x1024 sollte es schon passen. Meine VIs versiuche ich immer auf 1024x768 unterzubringen)
Und bitte:
keine Sequenzen benutzen, das zählt nicht (aber state machine ist erlaubt; wat issene state machine? Musst Du hier im Forum oder per google suchen...)
SubVIs sind dagegen nicht nur zu empfeheln, sondern ein Muss! Dein VI enthält leider keins...
Nach dieser schlimmen Kritik:
im Vergleich zu meinen LabVIEW-Anfängen ist das ganze Klasse! Hut ab!