while-Schleife vorzeitig verlassen

... schrieb:
> Was denkst Du, was hier passiert?

Ich weiß zwar nicht, was Du damit meinst, aber solange der Hubanschlag 
nicht erreicht ist, wird die berechnete Hubstellung angefahren.

... schrieb:
> Im Moment kenne ich nur den Codeschnipsel.

deshalb als Anhang mal das gesamte Hauptprogramm, auch wenn es 
vielleicht nicht allen Anforderungen entsprechen sollte.
Es enthält zusammen mit *.h Dateien u.a.:
Aufzeichnung von Daten auf SD-Karte
Uhrzeit
Temperaturmessung
Anzeige auf Flüssigkristallanzeige

Vielleicht könnte jemand die Abruchbedingungen an der richtigen Stelle 
einfügen.

mit break
oder if (ADW_P6_3 > 24)... //steht für Stromstärke am Anschlag ZU
hatte ich es versucht --> Programm reagierte nicht mehr
deshalb mein Hilferuf

wolle g. schrieb:
> Vielleicht könnte jemand die Abruchbedingungen an der richtigen Stelle
> einfügen.

ich schau mal wo platz ist ...
gehts noch?!


mt

Tim T. schrieb:
> Wo kommt eigentlich hub her? Sehe dafür keine Deklaration... Und
> wenn
> sollte da ein volatile bei stehen.
Deklaration ist in DS10-g0107-LMK62-SD.h als volatile int  enthalten

Deinen Vorschlag habe ich wie folgt eingebaut:

void ventil_schliessen_regler(void)
      {
       ZU=1;  //zähler zählt herunter
       while(hub >(ventilstellung_alt-umdrehung_zu ))
       P3OUT &=~BIT0; // ZU-fahren
       if ( ADW_P6_3 > 24) break;      //hier Zeile 248
       P3OUT |=BIT0;  // ZU-fahren HALT
       ZU=0;
       }

Fehlermeldung: DS10-g0122-PID.c:248: break statement not within loop or 
switch

Tim T. schrieb:
> als so minderbegabt

oder einfach nur talentfrei! ... ist hier doch die mehrheit.
das du und andere sich hier reinhängen wundert mich.


mt

Prozessor Dokter Schmunz schrieb:
> Also bleib devot und lies,
> was wir Männer schreiben.

n1
Ymmd

/regards

Wahrscheinlich war ich diesmal zur falschen Zeit am falschen Ort, denn 
normalerweise wird hier im Forum zumeist konkret auf  Fragen 
eingegangen.
Unabhängig von seinen "sinnlosen Kommentaren" führten die 
Programmschnipsel von Tim T. letztendlich zum Erfolg.
Auch wenn es "nur" die Klammern waren.
So was sieht nun mal ein Laie nicht, auch wenn es "lächerlich" 
erscheinen soll.
Vielen Dank.
Hier die aus meiner Sicht geänderte entscheidende Funktion.

void ventil_schliessen_regler(void)
      {
       ZU=1;                      //zähler zählt herunter
       while(hub >(ventilstellung_alt-umdrehung_zu ))
       {
       ADW_lesen();               // Stromaufnahmemessung übernehmen
       if ( ADW_P6_3 > 80) break; // ADW_6_3=Kanal 6.3; Stromaufnahme
       // wird am Anschlag überschritten; while-Schleife wird verlassen
       P3OUT &=~BIT0;             // ZU-fahren
       }
       P3OUT |=BIT0;              // ZU-fahren HALT
       ZU=0;
       }

äxl schrieb:
>> fragt ein Programmierlaie
>
> Ging ja wieder heiss her hier, gestern abend.
> Dafür, dass er 'n Laie ist auf dem Gebiet, geht das doch. Etwas holperig
> und ungelenk vielleicht.
> Aber trotzdem muss man hier nicht soo arrogant reagieren. Schade...

Ich stimme dir zu äxl. Es ist schade, traurig und dumm. Jeder ist Mal 
Laie und Mal Fachmann auf einem Gebiet. Mit etwas mehr Verständnis 
füreinander würd's hier glatt noch Spass machen ... . Wahrscheinlich zu 
gut für einige.

Tim T. schrieb:
>Wobei aber ADW_P6_3 WÄHREND der Schleife nicht aktualisiert wird,

Hat das programmiertechnische Gründe oder liegt es an der Arbeitsweise 
des MSP430F1611?
Ich hatte eigentlich gedacht, dass der AD-wandler die Abtastung der 
Kanäle mit
ADC12CTL0 |= ADC12SC+ ENC;
ständig wiederholt  und dass mit
ADW_P6_3 = ADC12MEM3;
der Speicher ADC12MEM3 abgefragt wird und damit ADW_P6_3 auch beim 
Durchlaufen der while-Schleife aktuell ist.

Tim T. schrieb:
> Wobei aber ADW_P6_3 WÄHREND der Schleife nicht aktualisiert wird, also
> entweder ist die Stromaufnahme schon vor der Schleife zu hoch und damit
> als Abbruchbedingung vorhanden oder es wirkt sich in der Schleife auch
> nicht aus...

Das aktualisieren passiert neben der Main() im interrupt.

while-Schleife vorzeitig verlassen:
unbedingt fristgerecht kündigen, sonst muss man ein weiteres Jahr drin 
bleiben.

Roland E. schrieb:
>Das aktualisieren passiert neben der Main() im interrupt.
Könnte das hier so gemeint sein? (aus  SLAU049E)

Repeat-Sequence-of-Channels Mode
A sequence of channels is sampled and converted repeatedly. The ADC
results are written to the conversion memories starting with the 
ADC12MEMx defined by the CSTARTADDx bits. The sequence ends after the 
measurement
of the channel with a set EOS bit and the next trigger signal re-starts 
the
sequence. Figure 17−9 shows the repeat-sequence-of-channels mode.

wolle g. schrieb:
>> ADW_P6_3 = ADC12MEM3;

>Das ist ne Zuweisung, mehr nicht. Und wann wird die in der Schleife
>gemacht? Richtig, garnicht.
Gibt es eine Lösung, um an den aktuellen Wert von ADW_P6_3  innerhalb 
der while-Schleife zu kommen?

wolle g. schrieb:
> ... schrieb:
>> Im Moment kenne ich nur den Codeschnipsel.
>
> deshalb als Anhang mal das gesamte Hauptprogramm, auch wenn es
> vielleicht nicht allen Anforderungen entsprechen sollte.

Nein, das ist nicht das gesamte Programm.
* Was verbirgt die includierte C-Datei namens "DS10-gP12-SD_MAINTI.c" ?
* Sind da vielleciht die Funktionen temp_messung_starten() und 
FKA_einrichten() definiert?

Ausserdem: Die pauseX() Funktionen werden wenn du Glück oder Pech hast, 
vom Compiler komplett wegoptimiert und bewirken allen das gleich, 
nämlich NIX. Du hast einen Timer - benutzt den!

Eric B. schrieb:
> Ausserdem: Die pauseX() Funktionen werden wenn du Glück oder Pech hast,
> vom Compiler komplett wegoptimiert und bewirken allen das gleich,
> nämlich NIX. Du hast einen Timer - benutzt den!

Das hat aber nichts mit dem Thema while ... zu tun.
Aber wenn man evtl. mein Programm verbessern möchte, dann bitte mit 
konkreten Zeilen (einschließlich Klammern usw.)

>Du hast einen Timer - benutzt den!
Damit kann ich z.B. in diesem Fall gar nichts anfangen.

Nach der oben gezeigten Korrektur läuft man Programm, wobei jetzt noch 
offen ist, welcher Wert der Stromaufnahme für den Abbruch der Schleife 
verwendet wird. Während des Lauf der while-Schleife oder vor Beginn der 
Schleife.
M.M. nach müsste es der ADW_P6_3 Wert sein, der während des 
Schleifenlaufs vom AD_Wandler (ADCM12EM3) gemessen wird.

Nach kurzem Überflügen des DB:
ADCM12EM3 scheint das Register im ADC zu sein, in dem der gewandelte 
Wert hinterlegt wird. Einfach dort abholen:

1

    if ( ADCM12EM3 > 80) break; // ADW_6_3=Kanal 6.3; Stromaufnahme

Das Register dürfte "volatile" deklariert sein, also der Zugriff darauf 
auch aggressiv optimierende Compiler überlebt.

Tim T. schrieb:
>Und jetzt hab ich keine Lust mehr...
Nicht so schnell aufgeben!

> Die VARIABLE ADW_P6_3 muss der Wert aus dem REGISTER
> ADCM12EM3 zugewiesen werden ( ADW_P6_3 = ADCM12EM3 )

Wahrscheinlich hast Du es übersehen.
In der Funktion:    int ADW_lesen(void)
werden  die ADW-Register gelesen.
Das soll jetzt kein Vorwurf sein, denn mein Programm lädt vermutlich 
nicht zum Studium ein.

>Wobei sich da natürlich dann die Frage stellt, warum du nicht direkt das 
>REGISTER ADCM12EM3 testest (z.B. if ( ADCM12EM3 > 80 ) break;) !?
Die Frage ist berechtigt. Hier bin ich dem Holzweg gelandet. Vor 
gewisser Zeit hatte ich etwas zur Funktionsweise des AD-Wandlers falsch 
gedeutet.

Prozessor Dokter Schmunz schrieb:
> Quatsch. Das sind Weicheierschleifen.
>
> Männerschleifen werden SO geschrieben:While(1) {
>   if(Abbruchbedingung)
>      {break;}
> }

Echt lustig, dass du Männerschleifen und Rheinwerk (Jürgen Wolf) in 
einem Beitrag gemeins erwähnst.

Ich sage nicht, dass du falsch liegst, aber es kommt auf das Problem 
drauf an, man kann vieles nicht einfach über ein Kamm scheren.
Manchmal ist es Sinnvoller beides oder doch nur das eine zu 
machen...oder vllt. doch eine

do
{
/* */
} while (x);

Deine while(1) ist nämlich in deinem Link nicht zu finden...
DORT steht die Abbruchbedingung auch im Kopf der while().

Naja, sei es drum.

Und nun geht es seid 2 Tagen und hab in allen Beiträgen keine Anständige 
Antwort gesehen, die dem TO als Lösung dienen könnte...

Adam P. schrieb:
> in allen Beiträgen keine Anständige
> Antwort gesehen, die dem TO als Antwort dienen könnte...

Wie kommst Du darauf? Mit Hilfe von Tim T. wurde das Problem im Prinzip 
gelöst.
siehe meinen Beitrag 05.03.2019 13:48

wolle g. schrieb:
> Wie kommst Du darauf? Mit Hilfe von Tim T. wurde das Problem im Prinzip
> gelöst.
> siehe meinen Beitrag 05.03.2019 13:48

Ja OK,

Ich bin davon ausgegangen, dass du die Klammern nicht gesetzt hast, da 
du wolltest das die while nur die nächste Anweisung einbezieht.
( Hatte den geposteten C Code nicht gesehen) lediglich dein ersten 
Code-Schnipsel.

Tim T. schrieb:
>aber sie wird nirgendwo aufgerufen,

Ich denke, im Programmschnipsel vom 05.03.2019 13:48 wird mit
ADW_lesen();  ADC12MEM3 abgerufen.

void ventil_schliessen_regler(void)
      {
       ZU=1;                      //zähler zählt herunter
       while(hub >(ventilstellung_alt-umdrehung_zu ))
       {
       ADW_lesen();               // Stromaufnahmemessung übernehmen
       if ( ADW_P6_3 > 80) break; // ADW_6_3=Kanal 6.3; Stromaufnahme
       // wird am Anschlag überschritten; while-Schleife wird verlassen
       P3OUT &=~BIT0;             // ZU-fahren
       }
       P3OUT |=BIT0;              // ZU-fahren HALT
       ZU=0;
       }


Mit  ADW_P6_3 = ADC12MEM3;
enthält m. E. ADW_P6_3 das Messergebnis, welches in ADC12MEM3 aktuell 
steht.

>Jetzt bin ich aber wirklich raus,

Nee, nee; Das ist erst möglich, wenn Du mich überzeugt hast.