Arduino -osnovna šola - 01.12.21 ob 20:00

[VolkD]

Kje pa imaš definirano spremenljivko _FadeValue ?

Ta spremenljivka se nahaja v objektu LEDFadeCls in ni potrebe, da bi bila vsem vidna.

Torej podobno kot _FadeTime.

[skat]

Za danes bo dovolj! Moram it plužit sneg, ga je že kakih 10cm.

[valter]

Uh sem se kar namučil.
LedFade.h

Koda: Izberi vse

#ifndef __LEDFADE_H__
#define __LEDFADE_H__

#include <stdint.h>
#include <Arduino.h>

#include "LED.h"

class LEDFadeCls : public LEDCls
{
   public:
      LEDFadeCls(uint8_t pin);
     
      void Fade(uint32_t time);
     
      virtual void Loop() override;
     
   private:
      uint32_t _FadeTime;
     uint32_t   _FadeValue;
     uint32_t   _FadeDir;
     uint32_t  Pin _;
};




#endif

Brez napak

[Kroko]

Izgeda da ti v LEDFade.h manjka _FadeValue.
Še glede napak, ki jih javlja arduino - vedno poglejte samo prvo izpisano napako. Ostale so večinoma posledice prve.

Drugače pa - To, kar smo naredili do sedaj je daleč najtežja stvar za osvojit. Vse ostalo bo veliko bolj preprosto. Je pa to osnova. Prva stopnica, ki pa je res zelo visoka.
Tako da ne obupat.
Kdor misli obupat ali pa ima kakšnekoli težave naj pride k meni v kabinet in če imal le čas bom pomagal.
https://meet.jit.si/kroko

[valter]

Kaj ni nobenega junaka?
Me matra ta fade .. nikakor ne zalavfa.
v mainu.cpp

Koda: Izberi vse

#include <stdint.h>
#include <Arduino.h>

#include "pins.h"
#include "LED.h"
#include "LEDBlink.h"
#include "LEDFade.h"

#include "main.h"

LEDBlinkCls _LED_R(PIN_LED_R);
LEDFadeCls  _LED_G(PIN_LED_G);
LEDFadeCls _LED_B(PIN_LED_B);

void setup()
{
   Serial.begin(115200);
   
   _LED_R.begin();
   _LED_G.begin();
   _LED_B.begin();
 
  _LED_R.Blink(500);
  _LED_G.Fade(500);
  _LED_B.Fade(500);
 
      
   
}

void loop()
{
   _LED_R.Loop();
   _LED_G.Loop();
   _LED_B.Loop();
}


Blink dela , fade pa ne ??'

[Kroko]

@skat - vse kar vemo je, da je nekaj narobe s spremenljivko _FadeValue . Ali je ta definirana tam, kjer mora biti? Inicializacijo vidim tudi da imaš zakomentirano, kar ni OK.

[VolkD]

Kaj ni nobenega junaka?

Kako da ne,....

Na https://meet.jit.si/kroko je video kako moja LED dela.

[valter]

Matr 5 minut z Darkotom in je "proradilo".
Napaka pa je bila , da sem delal v 2 mapah in logično vse pobrkal.
Zahvala gre Krokotu in seveda Darkotu.
Niso samo podpičja važna pri C++
V bistvu sem vse napravil prav sem pa LEDFade.cpp dal v napačno mapo..
Seveda če bi bil pri arduinotu pozoren , bi opazil da manjka LEDFade.cpp.
Za spodbudo ostalim:
C-ja nisem nikoli uporabljal .Še več sem ga prav sovražil.
Nanj sem prešel šele s pojavom arduinota (bolj copy-paste)
Sedaj pa se je spremenilo (s Krokotom) Sploh ni bav bav...
Zato priporočam še ostalim da se potrudijo.
Res super Hvala vsem sodelujočim.

[mirkatmir]

Napake ne javi nobene, gori pa neprestano rdeča led. Nekje bluzim, se vidi koliko je ura.

LEDFade.h:

Koda: Izberi vse

#ifndef __LEDFade_h__
#define __LEDFade_h__

#include <stdint.h>
#include <Arduino.h>

#include "LED.h"

class LEDFadeCls : public LEDCls
{
    public:
        LEDFadeCls(uint8_t pin);

        void Fade(uint32_t time);

        virtual void Loop() override;
        //void Loop();

    private:
        uint32_t _FadeTime;
        uint8_t _FadeValue;
        uint8_t _FadeDir;
        uint8_t _Pin;

};

#endif


in LEDFade.cpp

Koda: Izberi vse

#include "DelayUtils.h"
#include "LEDFade.h"

LEDFadeCls::LEDFadeCls(uint8_t pin) :
  LEDCls(pin),
  _FadeTime(0),
  _FadeValue(0),
  _FadeDir(0)
 
{   
}

void LEDFadeCls::Fade(uint32_t time)
{
   _FadeTime = time * 1000 / 255;
    _FadeValue = 0;
    _FadeDir = +1;
   _Time = DelayUtils::DelayInitUS();
}

void LEDFadeCls::Loop()
{
    if (DelayUtils::DelayCheckUS(_Time, _FadeTime))
    {
        if (_FadeValue == 0)
            _FadeDir = +1;
        else if (_FadeValue == 255)
            _FadeDir = -1;
        _FadeValue += _FadeDir;
        analogWrite(_Pin, _FadeValue);
    }
}



[VolkD]

Za začetek,...
Za _Pin je vseeno ali potrebujemo predznak ali ne. Vrednosti tega polje tako in tako nikoli niso negativne . Prav tako ne potrebujemo vrednosti nad 127.

Kaj pa _FadeDir ?

[VolkD]

Ko zgornje rešiš, .... kam boš shranil pri tem novem objektu pin, torej to kar kot prameter pošlješ v objekt.

[VolkD]

Za začetek,...
Za _Pin je vseeno ali potrebujemo predznak ali ne. Vrednosti tega polje tako in tako nikoli niso negativne . Prav tako ne potrebujemo vrednosti nad 127.

Kaj pa _FadeDir ?

Zanimivo. Dela vseeno tudi če v spremenljivko, ki je unsigned vpišemo -1.

[Kroko]

Seveda dela. Gre zgolj za interpretacijo bitov. Sodobni računalniki za pretvorbo bitov v števila uporabljajo dvojiški komplement.
Tule je več branja na to temo:
https://en.wikipedia.org/wiki/Two%27s_complement
https://sl.wikipedia.org/wiki/Dvoji%C5%A1ki_komplement

Ste pa ravnokar dobili novo domačo nalogo. Preberite tole:
https://anzeljg.github.io/rin2/book2/2202/index.html
in rešite naloge na koncu. Če kaj ni jasno vprašajte učitelja Krokota

[VolkD]

Seveda dela. Gre zgolj za interpretacijo bitov.

Ja, seveda gre za interpretacijo, saj pri 0 ali pri +1 naj nebi bilo problemov. Ampak kaj če v unsigned spremenljivko vpišemo 128, potem jo pa uporabimo kot signed ?

[Kroko]

Pretvarjanju med različnimi tipi se reče "casting".
Pretvarjanju tipov ne da bi se izgubil pomen se reče "implicit casting". Primer:

Koda: Izberi vse

uint8_t a = 10;
uint16_t b;
b = a; //implicit cast


Drugi primer bi bil takole:

Koda: Izberi vse

uint16_t a = 10;
uint8_t b;
b = a; //implicit cast - ampak to dobimo warning, da lahko pride do izgube podatkov. Žal arduino warningov ne pokaže


Da to rešimo naredimo "explicit cast". S tem točno povemo, da to res želimo narediti:

Koda: Izberi vse

uint16_t a = 10;
uint8_t b;
b = (uint8_t )a; //explicit cast ("c" zapis)


oziroma povsem enako z malce drugačnim zapisom:

Koda: Izberi vse

uint16_t a = 10;
uint8_t b;
b = uint8_t (a); //explicit cast ("functional" zapis)


Povsem enako je s signed/unsigned. Če to mešamo in implicitno castamo se imamo samo za "zahvalit" Arduino šlampariji, ki skrije vse warninge, da je nekaj hudo narobe.

[skat]

Tudi meni je uspelo sprogramirati Fade delovanje. Malo s pomočjo sina profi programerja, ki je takoj videl, da so imena različna, čeprav samo v Velike/male črke ali s podčrtaji

Ampak način razmišljanja in elemente takega programiranja moram pa še prežvečit in prebavit, da pride v kri.

[valter]

Ampak način razmišljanja in elemente takega programiranja moram pa še prežvečit in prebavit, da pride v kri.

Se podpišem .
To je bilo izi , saj je Kroko vse napravil. Ampak sedaj tudi C ni več tako zagulen.

[skat]

Še en link z zapiski o C++ : https://e6.ijs.si/~jslak/c++/index.html

[transalp]

Kroko, prosim te za pojasnilo - tule je moj delujoči LEDFade.cpp:

Koda: Izberi vse

#include "DelayUtils.h"
#include "LEDFade.h"

LEDFadeCls::LEDFadeCls(uint8_t pin):
  LEDCls(pin),
  _FadeTime(0),
  _FadeValue(0),
  _FadeDir(0)
{
   
}

void LEDFadeCls::Fade(uint32_t time)
{
   _FadeTime = time * 1000 / 255;
   
   _FadeValue = 0;
   _FadeDir = +1;
   
   _Time = DelayUtils::DelayInitUS();
}

void LEDFadeCls::Loop() //override;
{
   if (DelayUtils::DelayCheckUS(_Time, _FadeTime))
   {
               
      if (_FadeValue == 0)
         _FadeDir = +1;
      else if (_FadeValue == 255)
         _FadeDir = -1;
            
      _FadeValue += _FadeDir;
            
      Set(_FadeValue);
      // analogWrite(_Pin, _FadeValue); //ne dela in javi, da je "const uint8_t LEDCls::_Pin private in this context"
   }
}


Za output _FadeValue na izhodni pin sem uporabil "Set" iz "LEDCls". Če poskusim z "analogWrite" na "_Pin" (zakomentirano v vrstici 37), javi, da je "private in this context".

[gumby]

...Žal arduino warningov ne pokaže...

Tole se mi zdi precej narobe... sploh pri začetnikih ne bi smel tega delat.
Jaz imam še vedno vklopljeno, da warninge obravnava, kot napake. Te zelo hitro nauči, da "lepo" programiraš

[Kroko]

Kroko, prosim te za pojasnilo...

_Pin lastnost (property) je privatna lastnost razreda LEDCls. Nihče drug te lastnosti ne vidi.
Če želiš da izpeljan razred (LEDFadeCls) vidi to lastnost, jo moraš deklarirati kot "protected:".
Če želiš za vsi vidijo neko lastnost jo moraš deklarirati kot "public:".

Pa še to. To niso spremenljivke. _Pin, _BlinkTime, _FadeValue in ostalo so lastnosti razreda in ne spremenljivke. Navadite se na to tako gledati, in bo bolj preprosto.

Na primer, imamo razred "oseba". Osebe imajo "ime" in "priimek". Ti lastnosti poznajo vsi. Oseba ima tudi nek "hobi", po pa poznajo samo njeni otroci. Ima tudi "pin" številko ki pa je nihče ne pozna.

Koda: Izberi vse

class OsebaCls
{
  public:
    string Ime;
    string Priimek;
  protected:
    string Hobi;
  private:
    string Pin;
}

[transalp]

Hvala lepa. Bil sem zelo površen in nisem pogledal vsebine LED.h ...

[Kroko]

Kaj pa je bolj prav? Naj v "LEDFadeCls" uporabimo "Set" iz "LEDCls" ali direktno "analogWrite"?

Razmislite. In napišite kaj se vam zdi bolj prav in zakaj.

[transalp]

Meni se zdi bolj prav uporabiti "Set" iz "LEDCls" ...

[S55RU]

Nekako mi je uspelo napisati delujoč Fade, vsaj mislim da je pravilno delujoč. Kako bi se dalo FadeValue izpisati na serijski monitor, da bi človek lahko debugiral pravilno delovanje?
Malo sem si izpisal še komentarje, da se ne bom naslednjič lovil.

LEDFade.cpp

Koda: Izberi vse

#include "DelayUtils.h"
#include "LEDFade.h"

//konstruktor
LEDFadeCls::LEDFadeCls(uint8_t pin):
  LEDCls(pin), //kličemo kar konstruktor starša
  _FadeTime(0) //inicializacija (damo na 0)
{
   
}

void LEDFadeCls::Fade (uint32_t time)
{
   _FadeTime = time * 1000 / 255;
   
   _FadeValue=0;
   _FadeDir=+1;
   
   _Time=DelayUtils::DelayInitUS();
}

void LEDFadeCls::Loop() //override lastnosti iz LEDCls
{
   if (DelayUtils::DelayCheckUS (_Time, _FadeTime))
   {
      if (_FadeValue == 0)
            _FadeDir = +1;
      else if (_FadeDir ==255)
            _FadeDir =-1;
               
         _FadeValue +=_FadeDir;
                  
         Set(_FadeValue); //pošlji nazaj v Set, ki je lastnost v LED.h
   }
}

Ter LEDFade.h

Koda: Izberi vse


#ifndef __LEDFADE_H__
#define __LEDFADE_H__

#include <stdint.h>
#include <Arduino.h>

#include "LED.h"  //obvezno includat.h starša, če hočemo uporabljati njegove lastnosti

//razred LEDFade izhaja iz razreda LED (public-> ker hočemo vse lastnosti razreda videti navzven)
class LEDFadeCls : public LEDCls
{
   public:
      LEDFadeCls(uint8_t pin); //konstruktor
      
      void Fade(uint32_t time);
      
      //virtualna funkcija, preglasimo njeno obnašanje, ki je definirano v starševskem .h
      virtual void Loop() override;
      
   private:
      uint32_t _FadeTime;
      uint8_t _FadeValue;
      int8_t _FadeDir;
};

#endif // __LEDFADE_H__

[Kroko]

Na prvi uč izgleda ok.

Na serijski minitor pa izpisuješ z

Koda: Izberi vse

Serial.print(_FadeValue);

Ne pozabi v main.cpp, v Setup() najprej narediti

Koda: Izberi vse

Serial.begin(115200);

[S55RU]

Hvala. Sem razmišljal malo drugače in poskušal uvoziti _FadeValue v main.cpp in od tam printati, pa sem imel same težave in sem obupal.

Rezultat je sledeč, da gre od 0 do 255, nato skoči nazaj na 0 in spet proti 255. Torej le v eno smer. Ne bi moralo povečevati do 255, nato obrniti smeri in zmanjševati proti 0? Torej 0, 1, 2, ..., 254, 255, 254,..., 1, 0, 1,...?

Screenshot_2.png

[S55RU]

Mi je uspelo najti napako. Zatipk v LEDFade.cpp (popravek označen z bold-popravek bold v codi seveda ne dela). Namesto _FadeValue==255 sem imel _FadeDir==255. Sedaj dela OK:

Screenshot_3.png

Koda: Izberi vse

#include "DelayUtils.h"
#include "LEDFade.h"

//konstruktor
LEDFadeCls::LEDFadeCls(uint8_t pin):
  LEDCls(pin), //kličemo kar konstruktor starša
  _FadeTime(0) //inicializacija (damo na 0)
{
   
}

void LEDFadeCls::Fade (uint32_t time)
{
   _FadeTime = time * 1000 / 255;
   
   _FadeValue=0;
   _FadeDir=+1;
   
   _Time=DelayUtils::DelayInitUS();
}

void LEDFadeCls::Loop() //override lastnosti iz LEDCls
{
   if (DelayUtils::DelayCheckUS (_Time, _FadeTime))
   {
      if (_FadeValue == 0)
            _FadeDir = +1;
      else if (_FadeValue ==255)
            _FadeDir =-1;
               
         _FadeValue +=_FadeDir;
                  
         Set(_FadeValue); //pošlji nazaj v Set, ki je lastnost v LED.h
         Serial.print(_FadeValue);
         Serial.print('\n');
         delay(10);
   }
}



[tomazp]

Hvala. Sem razmišljal malo drugače in poskušal uvoziti _FadeValue v main.cpp in od tam printati, pa sem imel same težave in sem obupal.

Sedaj imaš _FadeValue deklariran kot privat lastnost in je zato vidna samo razredu. Prestavi jo v public in bo vidna tudi iz main.cpp. Od tam jo boš lahko natisnil s Serial.print.

[tomazp]

Morda bo komu koristno. Ko sem šel ponovno čez posnetke sem si naredil nekaj koristnih zaznamkov, ki morajo biti v "flashu", da se ne pozabijo. V EEram je običajno treba na začetku večkrat vpisati zadeve, ker jih drugače pozabi

[valter]

Super ,zelo koristno

[RaInfFiMa]

Verjetno bomo takrat prešli kar na ESP32. WiFi in modri zob se mi zdijo kul Pa multicore procesor...
Bom še malo naštudiral kako to izvesti. Vi pa do takrat kakšen evro v štumf. Elektronika ni poceni. Hitro je 20-30 EUR, sploh če bomo imeli kakšno fino periferijo. TFT, kakšen senzor in pa JTAG debugiranje so skoraj nuja. Če ste za seveda.

+1