Interruptroutine atmega C++

[Frank (Gast)]

Hallo,
ich möchte einen Timerinterrupt für meinen AVR in C++ erstellen.
Habe mir 
https://www.mikrocontroller.net/articles/AVR_Interrupt_Routinen_mit_C%2B%2B 
durchgelesen und es versucht umzusetzen.
Allerdings bekomme ich beim Kompilieren einen Error
undefined reference to `timer::timerInterrupt::timerInterrupt(int, 
timer*)'
Das heisst ja, dass der kompiler das nicht finden kann, das ist aber in 
timer.h vorhanden.

Was habe ich bei meiner Umsetzung falsch gemacht?

Frank

[Wilhelm M. (wimalopaan)]

Da fehlt die Implementierung der Elementfunktion (en).

Ausserdem ist Dein Beispiel nicht vollständig.

[Ausführlichprogrammierer (Gast)]

uint32_t damitichdiesevariableauchversteheheisstsiecounter32 = 0;

[Matthias T. (mati123)]

Hier meine Lösung für ATtiny 2313:

#include <stdint.h>
#include <avr/io.h>
#include <avr/interrupt.h>
#include <util/atomic.h>

volatile uint32_t epoch=0; // globaler Zeitstempel in 1/10 sec. nach Start

void timer_init(void)
{
  // Timer 0 konfigurieren
  TCCR0A = (1<<WGM01); // CTC Modus
  TCCR0B |= (1<<CS00) | (1<<CS01)  ; // Prescaler 64, d.h bei jedem 64en Takt wird hochgezählt.
  /* Ich will alle 1/100 sec  einen Timerüberlauf
  F_CPU / 64 * 0,01 = 156,2 */
  OCR0A = 155;

  // Compare Interrupt erlauben
  TIMSK |= (1<<OCIE0A);

  // Global Interrupts aktivieren
  sei();

}

int main(void)
{
  timer_init();
        while (1) {
    ATOMIC_BLOCK(ATOMIC_RESTORESTATE) {
        sek_aktuell=epoch / 10;
// auf epoch nur vor Interrupts geschützt zugreifen
                 }
// mache irgendetwas mit den sekunden
  }
}

/*
Der Compare Interrupt Handler
wird aufgerufen, wenn
TCNT0 = OCR0A = 156-1
ist (156 Schritte), d.h. genau alle 10 ms
*/
ISR (TIMER0_COMPA_vect)
{
  static uint8_t dezisekunden;
  dezisekunden++;
  if(dezisekunden > 9) {
    // alle 10 dezisec, d.h. alle 100 ms zählt epoch hoch.
    epoch++;
    dezisekunden = 0;
  }
}

[Ausführlichprogrammierer (Gast)]

Matthias T. schrieb:
> Hier meine Lösung für ATtiny 2313:

Und was hat das mit Klassen-Programmierung und C++ zu tun?

[Wilhelm M. (wimalopaan)]

Ausführlichprogrammierer schrieb:
> Matthias T. schrieb:
>> Hier meine Lösung für ATtiny 2313:
>
> Und was hat das mit Klassen-Programmierung und C++ zu tun?

Ich kenne ja ne Menge von Programmierparadigmen, (s.a.

https://de.wikipedia.org/wiki/Programmierparadigma

), aber was ist "Klassenprogrammierung"?

Du meintest wohl OOP, wobei man in C++ ja nicht OOP machen muss(!).

[Matthias T. (mati123)]

Ausführlichprogrammierer schrieb:

> Und was hat das mit Klassen-Programmierung und C++ zu tun?
Nichts. Ein funktionierendes C-Beispiel sollte aber auch für C++ helfen 
...

[Wilhelm M. (wimalopaan)]

Matthias T. schrieb:
> Ausführlichprogrammierer schrieb:
>
>> Und was hat das mit Klassen-Programmierung und C++ zu tun?
> Nichts. Ein funktionierendes C-Beispiel sollte aber auch für C++ helfen
> ...

Das löst aber nicht sein Problem, dass er die ODR verletzt hat!

[Ausführlichprogrammierer (Gast)]

Wilhelm M. schrieb:
> aber was ist "Klassenprogrammierung"?

Korinthenkacker scheinen es nicht zu verstehen, andere schon.

[Wilhelm M. (wimalopaan)]

Ausführlichprogrammierer schrieb:
> Wilhelm M. schrieb:
>> aber was ist "Klassenprogrammierung"?
>
> Korinthenkacker scheinen es nicht zu verstehen, andere schon.

Es geht hier nicht um Erbsenzählerei, sondern darum, dass man sich in 
einer (technischen) Disziplin am prägnantesten mit den richtigen Termini 
unterhält. Das ist hier bei der Programmierung so, oder in der 
E-Technik, oder ...

Dieses Forum soll doch Hilfe zur Selbsthilfe geben! Deswegen sollte der 
TO auch die richtigen Bergriffe verwenden. Sonst wird er beim googlen ja 
auch nichts finden. Er sollte sich also mal auf den Weg machen und nun 
auch ODR nachschlagen.

[Ausführlichprogrammierer (Gast)]

Wilhelm M. schrieb:
> dass er die ODR verletzt hat!

https://de.wikipedia.org/wiki/ODR

Ich als Korinthenkacker ... also ich nehm mal <Octal Data Rate>.

[Wilhelm M. (wimalopaan)]

Ausführlichprogrammierer schrieb:
> Wilhelm M. schrieb:
>> dass er die ODR verletzt hat!
>
> https://de.wikipedia.org/wiki/ODR
>
> Ich als Korinthenkacker ... also ich nehm mal <Octal Data Rate>.

Ging es nicht um C++?

[Frank (Gast)]

Ausführlichprogrammierer schrieb:
> Matthias T. schrieb:
>> Hier meine Lösung für ATtiny 2313:
>
> Und was hat das mit Klassen-Programmierung und C++ zu tun?

In C bekomme ich die Timer auch zum laufen. Allerdings wollte ich das 
ganze anders machen um mein C++ zu verbessern.

Wilhelm M. schrieb:
> Ausserdem ist Dein Beispiel nicht vollständig.

Was fehlt denn noch? Die main.cpp sieht noch ziemlich kahl aus. diese 
ist noch leer

#include "system.h" // hier ist nur define F_CPU
#include <util/delay.h>
#include <avr/io.h>
#include "Timer/timer.h"
#include "Interrupt/interrupt.h"

int main(void)
{
  while(1){

  }
  
}

interrupt.cpp

#include "interrupt.h"
#include "../Timer/timer.h"

void interrupt::handler1() {
  if(owner[0])
  owner[0]->serviceRoutine();
}

void interrupt::record(int interruptNumber, interrupt *i) {
  owner[interruptNumber - 1] = i;
}

int timer.cpp habe ich nocht

void timer::timerInterrupt::serviceRoutine() {
  if(ownerTimer != 0)
  ++ownerTimer->i;
}

 hinzugefügt.

timer.h hat noch bekommen

#include <avr/interrupt.h>
#include <avr/io.h>
#include <stdint.h>

Jetzt bekomme ich für timer.h den Error
expected class-name before '{' token
-> class timerInterrupt : public interrupt {

[Wilhelm M. (wimalopaan)]

Frank schrieb:

> Jetzt bekomme ich für timer.h den Error
> expected class-name before '{' token
> -> class timerInterrupt : public interrupt {


Kann es sein, dass Du die Header-datei mit "interrupt" nicht includiert 
hast?

[Wilhelm M. (wimalopaan)]

Wilhelm M. schrieb:
> Frank schrieb:
>
>> Jetzt bekomme ich für timer.h den Error
>> expected class-name before '{' token
>> -> class timerInterrupt : public interrupt {
>
>
> Kann es sein, dass Du die Header-datei mit "interrupt" nicht includiert
> hast?

 ... und am besten ein Minimalbeispiel (in einer Datei) und vollständige 
Fehlermeldungen. Dann ist es mit weniger Aufwand nachzuvollziehen ...

[Ausführlichprogrammierer (Gast)]

Wilhelm M. schrieb:
> ... und am besten ein Minimalbeispiel (in einer Datei) und vollständige
> Fehlermeldungen.

Volle Zustimmung. Kann man alles zur Überprüfung der Funktionsfähigkeit
in eine einzige Datei packen.

UndmitkurzenprägnantenIdentifiernlässtessichnochvielleichterverstehen.

[Ausführlichprogrammierer (Gast)]

Frank schrieb:
> int timer.cpp habe ich nocht
> void timer::timerInterrupt::serviceRoutine()
> {
>   if(ownerTimer != 0)
>   ++ownerTimer->i;
> }
>  hinzugefügt.

Du meinst also dieses Chaos inklusive Salamitaktik soll man
vernünftig nachvollziehen können?

Du machst es dir leicht, aber nicht denen die dir vielleicht
helfen wollen.

[gelegenheitsprogrammierer (Gast)]

#ifndef TIMER_H_
#define TIMER_H_

#include "../Interrupt/interrupt.h"

class timer {
  
  public:
  timer(volatile uint8_t &timerCounterControlRegister, uint8_t tccrNewState, volatile uint8_t &timerInterruptMaskRegister, uint8_t timskNewState, int interruptNumber);
  ~timer();
  
  volatile uint32_t i;

  class timerInterrupt : public interrupt {
    timer *ownerTimer;
    void serviceRoutine();
    
    public:
    timerInterrupt(int interruptNumber, timer *ownerTimer);
  } nestedTimerInterrupt;
    
  friend timerInterrupt;
  
};



#endif

Hier wird die Klasse timerInterrupt innerhalb von der Klassendeklaration 
timer deklariert. Das alleine sieht schon komisch aus. Dazu wird in 
timerInterrupt noch ein Pointer auf ein Objekt vom Typ timer deklariert. 
Deswegen meckert dein Kompiler.

Die Deklaration von timerInterrupt gehört hier nicht hin.

[Wilhelm M. (wimalopaan)]

gelegenheitsprogrammierer schrieb:

>
> Die Deklaration von timerInterrupt gehört hier nicht hin.

Nested classes kann man schon machen.

Ich warte erst mal auf das Minimalbeispiel, denn gegenüber der ersten 
Version am Anfang hat der TO ja auch noch was geändert ...

[Frank (Gast)]

Habe jetzt mal beide (timer und interrupt)in eine datei gemacht.
leider bekomme ich immer noch den Fehler
expected class-name before '{' token

Line 13
-> class timerInterrupt : public interrupt {

[Frank (Gast)]

gelegenheitsprogrammierer schrieb:
> Hier wird die Klasse timerInterrupt innerhalb von der Klassendeklaration
> timer deklariert. Das alleine sieht schon komisch aus. Dazu wird in
> timerInterrupt noch ein Pointer auf ein Objekt vom Typ timer deklariert.
> Deswegen meckert dein Kompiler.
>
> Die Deklaration von timerInterrupt gehört hier nicht hin.

Das ist doof, da ich mir diese Seite als Vorlage genommen habe
https://www.mikrocontroller.net/articles/AVR_Interrupt_Routinen_mit_C%2B%2B#Interrupt_Routine_in_einer_geschachtelten_Klasse
und es hier genauso gemacht worden ist

[Wilhelm M. (wimalopaan)]

Der C++ Compiler ist im wesentlichen ein one-pass Compiler, d.h. er geht 
von oben nach unten vor.  An der Stelle, wo der Typ interrupt gebraucht 
wird, ist er noch gar nicht deklariert. Er kennt also den Typ nicht.

Deswegen muss die Deklaration der BasisKlasse vor der abgeleiteten 
Klasse stehen (oder inkludiert werden).

[Sheeva P. (sheevaplug)]

Frank schrieb:
> Habe jetzt mal beide (timer und interrupt)in eine datei gemacht.
> leider bekomme ich immer noch den Fehler
> expected class-name before '{' token
>
> Line 13
> -> class timerInterrupt : public interrupt {

In Deiner Datei timerinterrupt.h deklarierst Du zunächst die Klasse 
"timer" und danach die Klasse "interrupt". In der Definition der Klasse 
"timer" willst Du aber eine nested class "timerInterrupt" definieren, 
die von der an dieser Stelle noch unbekannten Klasse "timer" erbt.

Verschieb' die Deklaration von "interrupt" einfach mal nach oben vor die 
Deklaration von "timer", dann sollte es klappen -- vorausgesetzt, Du 
hast keine weiteren Fehler eingebaut.

[Kai (Gast)]

Wilhelm M. schrieb:
> Deswegen muss die Deklaration der BasisKlasse vor der abgeleiteten
> Klasse stehen (oder inkludiert werden).

Das stimmt. Das hätte mir eigentlich auch auffallen müssen.
Habe ich auch geändert, jedeoch kommt dann der nächste Fehler:

undefinied reference to 'interrupt::owner'

void interrupt::handler1() {
    if(owner[0])
    owner[0]->serviceRoutine();
}
[code]

das kommt daher, dass dieser noch niergends initialisiert ist.
Auf der Seite steht:
Das owner-Array wird - auch hier der Vollständigkeit des Codes halber - auf Null initialisiert.
interrupt *interrupt::owner[] = {0};

aber wo mache ich das genau?
in der main-funktion?

[code]
#include "TimerInterrupt/timerinterrupt.h"

int main(void)
{
  
  timer timer0(TCCR0B,
  TCCR0B | (1 << CS00) & ~(1 << CS01) | (1 << CS02),
  TIMSK0,
  TIMSK0 | (1 << TOIE0),
  TIMER0_OVF_vect_num);

  interrupt *interrupt::owner[] = {0};

  while(1)
  {
  }
}

bekomme so aber die Fehler:
-suggest parantheses around arithmetic in operand of '|'[-Wparentheses]
-qualified-id in declaration before '[' token
-expected initializer before '[' token

[Wilhelm M. (wimalopaan)]

Kai schrieb:
> Wilhelm M. schrieb:
>> Deswegen muss die Deklaration der BasisKlasse vor der abgeleiteten
>> Klasse stehen (oder inkludiert werden).
>
> Das stimmt. Das hätte mir eigentlich auch auffallen müssen.
> Habe ich auch geändert, jedeoch kommt dann der nächste Fehler:
>
> undefinied reference to 'interrupt::owner'
>
> [code]
> void interrupt::handler1() {
>     if(owner[0])
>     owner[0]->serviceRoutine();
> }
> [code]
>
> das kommt daher, dass dieser noch niergends initialisiert ist.

Es scheint mit, dass Deine C++-Kenntnisse nur rudimentär sind ...

owner ist ein static-member der Klasse. Das ist dann nur eine 
Deklaration - die Definition muss in einer Implementierungsdatei 
erfolgen -> ODR-Rule!

Wenn Dir die Grundlagen nicht klar sind, so tust Du Dir mit C++ auf dem 
uC keinen Gefallen. Lerne C++ auf dem PC, und dann kannst Du umsteigen 
damit auf den uC. Man programmiert auf dem uC in C++ zusätzlich auch 
noch sehr viel anders als mit C++ auf dem PC ...