Go Down

Topic: [Solved]Subroutine (Read 2 times) previous topic - next topic

Citystars

Dec 22, 2013, 08:34 pm Last Edit: Dec 22, 2013, 08:45 pm by Citystars Reason: 1
Geachte,

Ik wil alles in een subroutine steken omdat alles onder void loop nogal veel is.
Dit is mijn voorbeeld code:

Code: [Select]

void setup(){}
void loop()
{  
blink();
}
void blink()
{
"code hier"
}


Het probleem is nu in de arduino software gaat dit allemaal wel maar ik programmeer in atmel studio 6.
Hier geeft hij de error: 'blink' was not declared in this scope.

Opgelost door in Arduino.h regel toe te voegen void blink(void);
Hierdoor herkent hij de regel

RobvdVeer

Ouwe truck: definieer je blink() vóór je hem aanroept. Zomaar een suggestie, ik ken atmel studio niet.
Rob
Follow my progress on http://simplicate.weebly.com

robtillaart


Het is geen goed idee om de aanpassing in Arduino.h te doen,

je kunt het veel beter in de source file zelf doen, of 2e optie een lokale .h maken

Code: [Select]

// PROTOTYPES OF FUNCTIONS
void blink();

// ARDUINO SKETCH STARTS HERE
void setup()
{
}

// INFINITE LOOP
void loop()
{   
  blink();
}

void blink()
{
  // "code hier"
}


of 2e optie een lokale .h maken

Code: [Select]


#include "blink.h" // bevat de blink code..

// ARDUINO SKETCH STARTS HERE
void setup()
{
}

// INFINITE LOOP
void loop()
{   
  blink();
}
Rob Tillaart

Nederlandse sectie - http://arduino.cc/forum/index.php/board,77.0.html -
(Please do not PM for private consultancy)

Citystars

Bedankt!

Ik heb dan nog een vraag.
Ik wil een output voor een bepaalde tijd hoog laten staan tenzij ik een puls krijg van een bepaalde input ( of constant) zodat deze uitgang laag word. De puls verkrijgen heb ik al maar weet totaal niet hoe eraan te beginnen.

rikky

Een timertje dus.
Ik heb ook de grootste moeite met die timertjes.
Ik had vorig jaar uit frustratie een tuintimertje gemaakt.
Die is echter nog nooit in de praktijk uitgetest.

Maar als ik m zo eens even over lees, is ie wel lekker simpel.

Misschien kan je er wat mee.

Bijgesloten.


MAS3

Quote from: Citystars

Ik wil een output voor een bepaalde tijd hoog laten staan tenzij ik een puls krijg van een bepaalde input ( of constant) zodat deze uitgang laag word.


Hoi.
Het kan niet vaak genoeg gezegd worden:
Deel je probleem op in kleinere, wel oplosbare problemen.
Zo behoud je het overzicht en wanneer je die kleinere problemen hebt opgelost, heb je je grote probleem ook opgelost.

  • Output moet standaard hoog zijn.
  • Tenzij er een bepaalde tijd is verlopen.
  • Of tenzij er een inputwaarde geldig is.


Nu oplossen:
Je moet altijd bijhouden wat je aan het doen bent, en wat je al gedaan hebt.
Daarom werk je met  variabelen.
Die variabele kun je gebruiken om een bewerking op te doen (je kan er bijvoorbeeld mee rekenen, of m ergens mee vergelijken).
Uiteindelijk gebruik je de waarde van de variabele om je uitgang te sturen.


  • Maak de variabele en geef die een waarde die correspondeert met HOOG.
    Maak een tweede variabele waarin de vorige toestand word opgeslagen, zodat je kunt bijhouden of er iets veranderd is.

  • Houd de tijd bij.
    Dat doe je zoals in het voorbeeld van "Blink without delay".
    Ook hiervoor gebruik je meerdere variabelen.
    Kijk of er voldoende tijd verstreken is om iets te doen.
    Hiervoor moet je ook weten wat de huidige toestand is.
    Doe wat er gedaan moet worden (dat kan dus ook "niets" zijn)

  • Kijk of de inputwaarde geldig is.
    Doe wat er gedaan moet worden (dat kan dus ook "niets" zijn)




Zoals je ziet kan er hier ook nog aan de efficiëntie gewerkt worden maar dat zit er bewust in...
Have a look at "blink without delay".
Did you connect the grounds ?
Je kunt hier ook in het Nederlands terecht: http://arduino.cc/forum/index.php/board,77.0.html

rikky

if (years>7) { years = 0; decades++; }
wordt natuurlijk
if (years>9) { years = 0; decades++; }

En zo zitten er nog wat foutjes in.:-)

rikky

Blinkwithoutdelay, met alle respect, werkt niet nar mijn idee.
Want die loopt na een dag of 50 stuk.
Op de ..weet ik hoe het heet.. overflow ofzoiets.
Blinkwithoutdelay moet je weggooien.
Scheelt een paar weken vogelen.


rikky

Of eigenlijk is het dat je de max millies value erin moet sleutelen.
Het is een schande dat dat nog steeds niet in de "officiele" arduino software is ingesleuteld.

#define MAX_MILLIS_VALUE 34359738
etc.

Met alle respect.
Happy new year.

robtillaart


Blinkwithoutdelay, met alle respect, werkt niet nar mijn idee.
Want die loopt na een dag of 50 stuk.
Op de ..weet ik hoe het heet.. overflow ofzoiets.
Blinkwithoutdelay moet je weggooien.
Scheelt een paar weken vogelen.

integendeel

Blink without delay loopt prima, maar enkel omdat er aftrekken gebruikt wordt heeft de overflow geen gevolgen.

Dit is makkelijk aan te tonen met een korte schets die de waarde van miellies  laat overflowen en daar een andere van aftrekt.

Code: [Select]
void setup()
{
  Serial.begin(115200);
  Serial.println("Start ");
 
  uint32_t x = 0xFFFFFFF0;
  uint32_t miellies = 0xFFFFFFF0;
  Serial.print(x);
 
  while (miellies - x < 50)
  {
    Serial.print(miellies);
    Serial.print(" - ");
    Serial.print(x);
    Serial.print(" = ");
    Serial.println(miellies - x);
   
    miellies++;
  }
}

void loop()
{
}


natuurlijk is miellies een gesimuleerde millis();

Kortom Blink without delay is een prima techniek voor scheduling.

Het pattern is

Code: [Select]

void loop()
{
  if (millis() - previousTime >= threshold)   // soms ook  millis() - previousTime > threshold
  {
     previousTime += threshold;                    // previousTime = millis();   kan ook, net iets ander gedrag, voor meeste sketches niet relevant.

     // doe de code die 1 keer per threshold milliseconden gedaan moet worden
  }
 
  // schedule andere zaken.
}



Rob Tillaart

Nederlandse sectie - http://arduino.cc/forum/index.php/board,77.0.html -
(Please do not PM for private consultancy)

Citystars

#10
Dec 24, 2013, 12:52 am Last Edit: Dec 24, 2013, 01:05 am by Citystars Reason: 1
Wegens de tips het kunnen klaren.
Paar uurtjes men hoofd erop gebroken maar toch de oplossing gevonden ;)!

Code: [Select]

int relaisState = LOW;
long previousMillis = 0;
long interval = 20000;

void setup()
{
pinMode(contact, INPUT_PULLUP);
pinMode(relais, OUTPUT);
}

Void loop()
{
deurcontact = digitalRead(contact);
unsigned long currentMillis = millis();

if(buttonPushCounter % 2 == 1)
{
relaisState = LOW;
previousMillis = currentMillis;
}

if (deurcontact == HIGH && buttonPushCounter % 2 == 0)
{
if(currentMillis - previousMillis > interval) {

if (relaisState == HIGH);
{
relaisState = LOW;
}
}
else
{
relaisState = HIGH;
}
}

digitalWrite(relais,relaisState);
}


Voor de mensen die nieuwsgierig zijn.
Kan zijn dat er dingen weggelaten zijn.
Dit stuk code eruit gehaalt.

Bedankt!

rikky

Oh, nou at ga ik dan eens bekijken binnenkort.
Bedankt in ieder geval voor de uitleg.

rikky

Nou, ik ben blijkbaar niet de enige die problemen heeft (gehad) met de timerfunctie van blinkwithout delay.
Als je een klein applicatietje maakt, dat in de praktijk 24 op 24 uur moet werken, jaar in jaar uit, voldoet ie niet, of je moet heel erg uitkijken, hoe je m schrijft.
En als je je arduinotje dan ergens achter hebt geplakt, en je moet er helemaal heen kruipen met je laptop, om het 1 en ander te veranderen, (bijvoorbeeld resetten voor de volgende 50 dagen), wordt je daar best wel sjacherijnig van.
millis loopt over na 50 dagen en begint weer terug op nul.
Dus pas op hoe je dat opvangt, en druk vooral niet op je knopje (of watever) tijdens dat ie overloopt.
Afhankelijk van je timer kan dat een seconde, uur of dag beslaan, dat je niet op je knopje mag drukken.

In het geval van Citystar neem ik aan dat ie zijn deur het hele jaar door bewaakt wil hebben.

Er is inmiddels nogalwat voor op het internet te vinden gelukkig.
onder vele anderen:
http://www.forward.com.au/pfod/ArduinoProgramming/TimingDelaysInArduino.html

Misschien een nieuw topic?
Of deze gewoon houden?

robtillaart

#13
Dec 25, 2013, 02:33 pm Last Edit: Dec 25, 2013, 02:36 pm by robtillaart Reason: 1
Quote
Nou, ik ben blijkbaar niet de enige die problemen heeft (gehad) met de timerfunctie van blinkwithout delay.
Als je een klein applicatietje maakt, dat in de praktijk 24 op 24 uur moet werken, jaar in jaar uit, voldoet ie niet, of je moet heel erg uitkijken, hoe je m schrijft.

Er zijn mischien wel duizenden mensen dieproblemen met  blink without delay gehad hebben.
Het komt wekelijks voor op het forum, mischien wel vaker.

En ja, voor alle applicaties geldt dat je uit moet kijken hoe je hem schrijft.  Het algoritme om hoe op de juiste manier met de overflow van de millis() en micros() klok om te gaan is al jaren bekend, maar het is minder intuitief dat aftrekken wel werkt waar optellen faalt.

Er is mij op dit moment één probleem bekend met millis(), en dat is NIET de overflow. millis() heeft de eigenschap dat hij een getal kan overslaan, oorzaak, de ophoging van de interne teller gebeurt tegelijk met een foutcorrectie. DIt is een bekend probleem waarvoor geen goede oplossing bestaat zover ik weet. [ Een goede oplossing heeft vrijwel dezelfde code grootte en snelheid als de huidige millis().]
De enige manier om hiermee om te gaan is altijd ongelijkheden (> en <  en >= en <= )  te gebruiken als je met millis() of micros() en tellers in het algemeen werkt. Als je namelijk == gebruikt en hij springt net met 2 omhoog (of je bent gewoon een paar millis te laat) dan faalt de IF.


Quote
Dus pas op hoe je dat opvangt, en druk vooral niet op je knopje (of watever) tijdens dat ie overloopt.
Afhankelijk van je timer kan dat een seconde, uur of dag beslaan, dat je niet op je knopje mag drukken.


Kun je een voorbeeld sketch posten waarin dit misgaat? Ik ben heel benieuwd.


Van de link die jij geeft
If you are just interested in a repeating timer then see http://arduino.cc/en/Tutorial/BlinkWithoutDelay for a simple reliable repeating timer which does not count down. The code below works because even if currentMillis overflows back to a small number, currentMillis - previousMillis still gives the correct result.

void loop() {
 unsigned long currentMillis = millis();
 if(currentMillis - previousMillis> interval) {
   // save the last time
   previousMillis = currentMillis;
   // do stuff here each interval (interval -- an unsigned long)
   ...
 }


Er staat wel een fout op de pagina, hier wordt == gebruikt met millis(). - zie boven)
// the loop routine runs over and over again forever:
void loop() {
 if (millis() == timeout) { // NOTE: the == only toggle led when millis() equals timeout
   // time to toggle the Led
   timeout += 1000;
   if (digitalRead(led)) {
     digitalWrite(led, LOW); // turn the LED off by making the voltage LOW
   } else {
     digitalWrite(led, HIGH); // turn the LED on (HIGH is the voltage level)
   }
 }
}
This code works because when timeout + 1000 overflows, millis() will not equal it until millis() also overflows.

Deze code werkt niet als millis() net met 2 verspringt over de waarde van timeout heen,
Millis() zal ~50 dagen moeten ophogen voordat ie weer een kans heeft om gelijk aan timeout te zijn.

Een mijn insziens betere uitleg van hoe om te gaan met millis() - http://www.gammon.com.au/forum/?id=11411 -

Rob Tillaart

Nederlandse sectie - http://arduino.cc/forum/index.php/board,77.0.html -
(Please do not PM for private consultancy)

rikky

Quote
Kun je een voorbeeld sketch posten waarin dit misgaat? Ik ben heel benieuwd.


Daar maar eens mee beginnen.
http://forum.arduino.cc/index.php?topic=168476.0
Dat was naar aanleiding van:
http://forum.arduino.cc/index.php?topic=141106.new;topicseen#new

Ik heb nu dus inmiddels wel geleerd dat ook digitalread() een interne delay heeft, en zo nog wel wat opdrachten, waardoor ik inmiddels begin te bevroeden, wat er allemaal nog zo meer fout kan gaan.
Ik ga het eens even allemaal bestuderen.


Go Up