"While-loop"...."break", en dan?

[Dorus1404 Profiel Privébericht]

while (pinin == LOW) // wachten tot pinin high wordt *** pinin moet je lezen. hij komt nooit voorbij deze opdracht
digitalWrite(pinuit, HIGH); // donker = 5 volt stuurt relais Aan

Ik heb de while loop zo begrepen dat zolang de conditie (pinin == low) true is de loop eindeloos doorgaat, maar in mijn geval krijgt pinin, als het donker wordt, bijna 5 volt en dan wordt de conditie false en wordt de volgende regel gelezen. Klopt dit of niet?
Indien het niet goed is geschreven, hoe zou ik dan het moeten wachten op het hoog worden van pinin?

[nicoverduin Profiel PrivéberichtWebsite]

Zoals ik al eerder zei...... bekijk de voorbeelden van een digitale io. En als jij een ldr of zo gebruikt dan moet je een analoge read doen....

[nicoverduin Profiel PrivéberichtWebsite]

En pinin zelf wordt nooit LOW. Daar staat 5 hard gecodeerd in

[shooter Profiel Privébericht]

geloof nico maar die heeft gelijk,zoek uit wat een digitalRead doet ofwel het reultaat moet naar een variabele.
een ingang kun je niet writen (nou ja het kan wel maar wordt gebruikt om een ingang te pushen.

[Dorus1404 Profiel Privébericht]

Mannen bedankt! Dank zij o.a. jullie hulp heb ik nu een goed werkend programmaatje dat precies doet wat ik wil. Het is weliswaar stap voor stap tot stand gekomen, maar aldoende leert men (vooral van fouten) en nu werkt het goed. Ik vraag me alleen af of ik de meest simpele vorm heb gevonden. Ik heb n.l. zo'n vaag vermoeden dat de code nog compacter zou kunnen, maar ik ben vooralsnog tevreden met het resultaat. Voor de nieuwsgierigen hier het eindresultaat:

cpp code

/*
   Avondtimer

   Een avondtimer, schakelt een relais in bij schemer (externe schakeling)
   van schemer tot bedtijd. Verlengt tussen 21 jun en 21 dec de avondtijd
   elke dag met 2 minuten en na 21 dec verkort elke dag met 2 minuten.
   De waarde van de variabele "d" kan worden aangepast aan de begindatum.
*/

int pinin = 5;  // Krijgt 5 volt van schemerschakeling bij invallende duisternis
int pinuit = 6;  // stuurt een vermogenstransistor met relais. Hoog is Aan
int vala = 0;
int valb = 0;

unsigned long d = 25200000; // donkere tijd in mSec; schemering tot bedtijd.
long x = 120000; // extra + of - tijd (tot 21-12 +, daarna -)

void setup()
{
  pinMode (pinin, INPUT);
  pinMode (pinuit, OUTPUT);
}
void loop()
{
  digitalRead (pinin); // Pin 5 = 5 / 0 volt
  vala = digitalRead(pinin); // 0 volt bij licht en 5 volt in donkere tijd
  if (vala == LOW)
    digitalWrite (pinuit, LOW);
  else
    digitalWrite (pinuit, HIGH); // donker = 5 volt stuurt relais Aan
  if (vala == HIGH)
    delay(d);  // schemer tot bedtijd
  if (vala == HIGH)
    d = d + x;  // dagelijkse verlenging of verkorting avondtijd
  if (d >= 31560000)  // 29 okt - 21 dec = 53(x) + 25.200.000
    x = -120000;  // x wordt neg na 21 dec
  digitalWrite (pinuit, LOW); // relais Uit bij bedtijd
  if (vala == HIGH) // Voorkomt "delay" overdag, zodat pin 5 weer op tijd wordt geactiveerd
    delay(50400000); // Wachten tot ca. 14:00 uur volgende dag. Pin 5 = LOW
}

Dit resultaat moedigt aan om weer andere projectjes te gaan proberen.

Nogmaals dank!

[nicoverduin Profiel PrivéberichtWebsite]

Zo had ie ook gekun:

cpp code

/*
 Avondtimer
 
 Een avondtimer, schakelt een relais in bij schemer (externe schakeling)
 van schemer tot bedtijd. Verlengt tussen 21 jun en 21 dec de avondtijd
 elke dag met 2 minuten en na 21 dec verkort elke dag met 2 minuten.
 De waarde van de variabele "d" kan worden aangepast aan de begindatum.
 */

int pinin   = 5;  // Krijgt 5 volt van schemerschakeling bij invallende duisternis
int pinuit  = 6;  // stuurt een vermogenstransistor met relais. Hoog is Aan
int vala    = 0;

unsigned long d = 25200000; // donkere tijd in mSec; schemering tot bedtijd.
long          x = 120000;   // extra + of - tijd (tot 21-12 +, daarna -)

void setup() {
  pinMode(pinin, INPUT);
  pinMode(pinuit, OUTPUT);
}

void loop() {
  vala = digitalRead(pinin);    // 0 volt bij licht en 5 volt in donkere tijd
  if (vala == LOW)
    digitalWrite(pinuit, LOW);
  else {
    digitalWrite(pinuit, HIGH); // donker = 5 volt stuurt relais Aan
    delay(d);                   // schemer tot bedtijd
    d = d + x;                  // dagelijkse verlenging of verkorting avondtijd
    if (d >= 31560000)          // 29 okt - 21 dec = 53(x) + 25.200.000
        x = x * -1;            // x wordt neg na 21 dec
    digitalWrite(pinuit, LOW);  // relais Uit bij bedtijd
    delay(50400000);            // Wachten tot ca. 14:00 uur volgende dag. Pin 5 = LOW
  }
}

[Dorus1404 Profiel Privébericht]

Ja, ziet er inderdaad beter uit. Ga ik aan denken als ik schrijf.

[shooter Profiel Privébericht]

ipv vala zet er neer wat het werkelijk betekent dat is een stuk beter leesbaar.
en stap voor stap is prima hoor dat in elkaar drukken van code is wel leuk, maar stamt uit de 70 jaren toen wel alles schreven in 1k geheugen.

[nicoverduin Profiel PrivéberichtWebsite]

ipv vala zet er neer wat het werkelijk betekent dat is een stuk beter leesbaar.
en stap voor stap is prima hoor dat in elkaar drukken van code is wel leuk, maar stamt uit de 70 jaren toen wel alles schreven in 1k geheugen.

Helaas Paul maar is nu nog net zo actueel. Studenten moeten nog steeds kijken of de code efficiënter kan. En bij Cortex Assembly praten we zelfs over minimaliseren van cpu cycles en memory gebruik. Dus ik zou niet stellen dat het uit de jaren 70 is en dus achterhaald.
Over variabele naamgeving helemaal mee eens.