Daten auf EEPROM schreiben und Zeit dazu addieren

Du musst erst die entsprechende Speicherstelle auslesen, dann die 
geloggte Nutzungsdauer dem ausgelesenen Wert hinzuaddieren und dann das 
Ergebnis zurück an die entsprechende Speicherstelle schreiben. Hoffe das 
hilft dir zum Anstoß.

Für die Neunmalklugen:
Und ja, sowas würde man eher weniger auf ein EEPROM speichern aber hier 
gehts ja um den Lerneffekt ;)

Dan N. schrieb:

> Erstellen Sie nun zum Speichern einer einzelnen Gabelstaplernutzung eine

Klingt nach Hausaufgaben.

> Funktion in C++ mit dem Namen addiere_Nutzungsdauer() mit folgenden

WOW!!! C++. Das nenn ich mal Luxus ;-)

> Anforderungen:
> - Die Funktion hat keinen Rückgabewert
> - Der Funktion werden zwei Byte-Parameter übergeben:
>     - RFID-ID (=Adresse im EEPROM)
>     - Ganzzahliger Wert für die Nutzungsdauer in Minuten (1 bis 255)
> - Die Nutzungsdauer soll stets zu dem Wert im EEPROM hinzugezählt
> werden.

Trivial

> void addiere_Nutzungsdauer (unsigned int EEAdresse, unsigned char
> EEDaten)

Warum nennst du deine Parameter nicht entsprechend. Daß die Daten im 
EEPROM landen, hat damit nix zu tun.

> {
>    //Schreiben
>    while(EECR & (1<<EEWE);
>    EEAR = EEAdresse;
>    EEDR = EEDaten;
>    EECR |= (1<<EEMWE);
>    EECR |= (1<<EEWE);
>
>    //an dieser Stelle weiß ich nicht, wie ich die Nutzungsdauer
> dazuaddieren soll?

Im Ernst? Vor allem, warum schreibst du ZUERST auf en EEPROM?

>
>    //Auslesen und zurücksetzen
>    while(EECR & (1<<EEWE);
>    EEAR = EEAdresse;
>    EECR |= (1<<EERE);
>    cout<< "Nutzungsdauer: "<< EEDaten;
>    EECR &= ~(1<<EEAR)|(1<<EEDR);
> }

> Ich hoffe, dass der Code soweit stimmt, wenn nicht bitte korrigieren.

Die Hoffnung stirbt zuletzt. Uuuup, schon ist sie tot!

> Die Lücke bitte ergänzen...

Nö.

Beitrag "Einheitlicher Umgang mit faulen Schülern etc.?"

M. K. schrieb:
> Du musst erst die entsprechende Speicherstelle auslesen, dann die
> geloggte Nutzungsdauer dem ausgelesenen Wert hinzuaddieren und dann das
> Ergebnis zurück an die entsprechende Speicherstelle schreiben. Hoffe das
> hilft dir zum Anstoß.

Danke für den Anstoß...ich versuche es mal:

void addiere_Nutzungsdauer (unsigned int ID, unsigned char Dauer)
{
   //Schreiben
   while(EECR & (1<<EEWE);
   EEAR = ID;
   EEDR = Dauer;
   EECR |= (1<<EEMWE);
   EECR |= (1<<EEWE);

   //Auslesen
   while(EECR & (1<<EEWE);
   EEDR = Dauer;
   EECR |= (1<<EERE);

   //Addieren
   Dauer = Dauer + ???;
//Was soll ich hier dazuaddieren? Ich hab ja keine Variable mit der 
eingelesenen Nutzdauer? Hast du vielleicht wieder einen Anstoß für mich?

//Schreiben
   while(EECR & (1<<EEWE);
   EEDR = Dauer;
   EECR |= (1<<EEMWE);
   EECR |= (1<<EEWE);
   cout<< "Nutzungsdauer: "<< Dauer;
   EECR &= ~(1<<EEAR)|(1<<EEDR);
}

@Falk B.:
Ich erwarte nicht, dass jemand für mich die komplette Lösung 
hinschreibt. Ich bin froh wenn jemand hilft und ich von selbst drauf 
komme, schließlich will ich dazu lernen. Ich schreibe bald eine Prüfung 
in Mikrocontollertechnik. Du hast recht...evtl hab ich mich falsch 
ausgedrückt, aber auf solche Kommentare kann ich trotzdem verzichten. 
Danke trotzdem für deine Mühe.

Da wirst Du wohl innerhalb deiner Routine eine lokale Variable 
definieren müssen, diese mit dem alten EEPROM-Wert befüllen, die 
Nutzungsdauer dazu addieren und das ganze wieder im EEPROM abspeichern 
müssen.

Wolfgang R. schrieb:
> Da wirst Du wohl innerhalb deiner Routine eine lokale Variable
> definieren müssen, diese mit dem alten EEPROM-Wert befüllen, die
> Nutzungsdauer dazu addieren und das ganze wieder im EEPROM abspeichern
> müssen.

ja so in etwa hätte ich mir das auch gedacht, aber laut aufgabe soll ich 
ja nur die "Funktion" schreiben. Verstehe ich das richtig, wenn ich 
innerhalb meiner "Funktion" per cin-Befehl einen wert für die 
nutzungsdauer einlese?
Ich dachte nämlich, dass ich dies im "int main"-Bereich tun müsste und 
den wert dann als übergabeparameter an die funktion übergeben muss.
Aber da ich ja nur dir funktion schreiben soll, stehe ich irgendwie am 
schlauch...

Die lokale Variable in der Funktion interessiert das Main Programm 
überhaupt nicht, du kannst ja prinzipiell auch eine Funktion nach dem 
Motto "schreibe EEPROM(gelesenes EEPROM + Nutzungsdauer) erstellen, da 
taucht die lokale Variable nicht explizit auf...

Du kannst in deiner Funktion immer lokale Variablen für die nötigen 
Daten anlegen...

Noch ein Versuch:

void addiere_Nutzungsdauer (unsigned int ID, unsigned char Dauer)
{
   int x;

   //Schreiben
   while(EECR & (1<<EEWE);
   EEAR = ID;
   EEDR = x;
   EECR |= (1<<EEMWE);
   EECR |= (1<<EEWE);

   //Auslesen
   while(EECR & (1<<EEWE);
   EEDR = x;
   EECR |= (1<<EERE);

   //Addieren
   x = x + Dauer;

   //Schreiben
   while(EECR & (1<<EEWE);
   EEDR = x;
   EECR |= (1<<EEMWE);
   EECR |= (1<<EEWE);
   cout<< "Nutzungsdauer: "<< x;
   EECR &= ~(1<<EEAR)|(1<<EEDR);
}

Was sagt ihr dazu? könnte das sol klappen?

EEDR = x; sollte wohl andersrum beim Lesen...

und ein Ganzzahlwert sollte hier wohl eher uint sein.

Wolfgang R. schrieb:
> Die lokale Variable in der Funktion interessiert das Main Programm
> überhaupt nicht, du kannst ja prinzipiell auch eine Funktion nach dem
> Motto "schreibe EEPROM(gelesenes EEPROM + Nutzungsdauer) erstellen, da
> taucht die lokale Variable nicht explizit auf...
>
> Du kannst in deiner Funktion immer lokale Variablen für die nötigen
> Daten anlegen...

Ach ja stimmt...ich hatte einen Denkfehler! Sorry. Ich hab erst 
angefangen zu programmieren.

Dan N. schrieb:
> cout<< "Nutzungsdauer: "<< x;

Ausgaben würde ich nicht in Unterfunktionen machen. Gib nur den Wert 
zurück und mach die Ausgaben im Main.

Dan N. schrieb:
> EECR &= ~(1<<EEAR)|(1<<EEDR);

Was soll das machen und wozu?

Eine gute Idee wäre es, auch endlich mal den AVR-Typ zu nennen.

Wolfgang R. schrieb:
> EEDR = x; sollte wohl andersrum beim Lesen...

wie meinst du das?


>
> und ein Ganzzahlwert sollte hier wohl eher uint sein.

Ok. also ändere ich char auf int

Du musst schon ein wenig selber denken...

die Verwendung von char, int uint (unsigned int) etc. ist Dir 
überlassen, du musst die Datentypen kennen, um zu verstehen, was damit 
alles passieren kann. Char ist o.k bei zahlen bis 255.

Wenn ich ein Datenregister in eine Variable auslese, dann muss ich 
schreiben

variable = Register;

Wenn ich ein Register aus einer Variable befülle, dann schreibe ich

Register = variable;

Also:

Ziel = Quelle;

Peter D. schrieb:
> Dan N. schrieb:
>> cout<< "Nutzungsdauer: "<< x;
>
> Ausgaben würde ich nicht in Unterfunktionen machen. Gib nur den Wert
> zurück und mach die Ausgaben im Main.

In der Aufgabe steht, dass die Funktion keinen Rückgabewert haben soll. 
Kann man dann trotzdem mit "return" etwas zurückgeben? Wenn ja, wie geht 
das dann?

>
> Dan N. schrieb:
>> EECR &= ~(1<<EEAR)|(1<<EEDR);
>
> Was soll das machen und wozu?

Dieser Befehl setzt die beiden Bits wieder auf Null, so wie es gefordert 
wird.
>
> Eine gute Idee wäre es, auch endlich mal den AVR-Typ zu nennen.

Atmega 16

Dan N. schrieb:
> Noch ein Versuch:
>
> void addiere_Nutzungsdauer (unsigned int ID, unsigned char Dauer)
> {

Willst du uns eigentlich veralbern? Warum um alles in der Welt willst du 
den EPRROM zuerst beschreiben? Hat du mal versucht, den logischen Ablauf 
normal ohne Programmiersprache zu beschreiben? Dir ist doch nicht mal 
der Ablauf klar!

Dan N. schrieb:
> Kann man dann trotzdem mit "return" etwas zurückgeben? Wenn ja, wie geht
> das dann?

Mit return... ;-)

int myfunction(a, b)
{
  int c = a+b;
  return(c);
}

Wolfgang R. schrieb:

> Wenn ich ein Register aus einer Variable befülle, dann schreibe ich
>
> Register = variable;
>
> Also:
>
> Ziel = Quelle;

Ach so...jetzt verstehe ich es langsam. Danke das hat schon mal geholfen

Wolfgang R. schrieb:
> Dan N. schrieb:
>> Kann man dann trotzdem mit "return" etwas zurückgeben? Wenn ja, wie geht
>> das dann?
>
> Mit return... ;-)
>
> int myfunction(a, b)
> {
>   int c = a+b;
>   return(c);
> }

ok, aber geht das denn, wenn oben in der Funktion "void" steht?

Dan N. schrieb:
> ok, aber geht das denn, wenn oben in der Funktion "void" steht?

Natürlich nicht. eine void-Funktion erwartet keinen Rückgabewert, 
deswegen ist sie ja auch vom Typ void.

Deswegen habe ich die Funktion ja auch vom Typ int deklariert in meinem 
Beispiel!

Dir fehlen die einfachsten Grundlagen von C! Kauf Dir ein Buch für 
Anfänger oder schau Dir im Internet entsprechende Beispiele gut an!

Hier hast Du auf jeden Fall genug Antworten auf Deine Frage bekommen, 
der Rest sollte Selbststudium sein, sonst ist der Lerneffekt fast null.

Dan N. schrieb:
> Dieser Befehl setzt die beiden Bits wieder auf Null, so wie es gefordert
> wird.

In welchem AVR-Typ sind EEAR und EEDR Bits?

Im Datenblatt wird nichst gefordert (siehe auch Beispielcode).

void addiere_Nutzungsdauer (unsigned int ID, unsigned char Dauer)
{
   int x;

   x = Dauer;

   //Schreiben
   while(EECR & (1<<EEWE);
   EEAR = ID;
   EEDR = x;
   EECR |= (1<<EEMWE);
   EECR |= (1<<EEWE);

   //Auslesen
   while(EECR & (1<<EEWE);
   x = EEDR;   //hab es umgedreht,aber in meine Unterlagen stehts 
andersrum
   EECR |= (1<<EERE);

   //Addieren
   x = x + Dauer;

   //Schreiben
   while(EECR & (1<<EEWE);
   EEDR = x;
   EECR |= (1<<EEMWE);
   EECR |= (1<<EEWE);
   return x;
   EEAR = 0x00;
   EEDR = 0x00;
}

Dan N. schrieb:
> Wolfgang R. schrieb:
>
>> Wenn ich ein Register aus einer Variable befülle, dann schreibe ich
>>
>> Register = variable;
>>
>> Also:
>>
>> Ziel = Quelle;
>
> Ach so...jetzt verstehe ich es langsam. Danke das hat schon mal geholfen

Ich auch. Du hast NULL Plan von Programmierung und C und kopierst 
hirnlos irgendwelchen Scheiß zusammen. Tolles Konzept. Wirst bestimmt 
bald Bachelor.

> Ups, jetzt wirst Du höre, C lernen zu müssen!
>
> Void heist soviel wie nichts. Also Keine Daten.
>
> Aber man kann es ja mal versuchen und sich die Compilermeldungen
> ansehen.

Das weiß ich selbst, dass VOID "nichts" heißt, deswegen frag ich ja, 
weil Wofgang geschrieben hat, dass ich mit "return" zurückgeben 
soll...aber meiner meinung nach geht das nicht

Dan N. schrieb:
> In der Aufgabe steht, dass die Funktion keinen Rückgabewert haben soll.

Es ist guter Programmierstil, wenn man Funktion und Anzeige voneinander 
trennt. Denn das wird schnell ein Chaos, wenn jede Funktion wild auf das 
Display schreibt.


Dan N. schrieb:
> Kann man dann trotzdem mit "return" etwas zurückgeben? Wenn ja, wie geht
> das dann?

Steht in jedem C-Tutorial.

Peter D. schrieb:
> Dan N. schrieb:
>> Dieser Befehl setzt die beiden Bits wieder auf Null, so wie es gefordert
>> wird.
>
> In welchem AVR-Typ sind EEAR und EEDR Bits?
>
> Im Datenblatt wird nichst gefordert (siehe auch Beispielcode).

Sorry, da hab ich mich vertan...

Wolfgang R. schrieb:
> Dan N. schrieb:
>> ok, aber geht das denn, wenn oben in der Funktion "void" steht?
>
> Natürlich nicht. eine void-Funktion erwartet keinen Rückgabewert,
> deswegen ist sie ja auch vom Typ void.
>
> Deswegen habe ich die Funktion ja auch vom Typ int deklariert in meinem
> Beispiel!

Ja das verstehe ich ja, aber das darf ich in meinem Fall laut Angabe 
nicht, deswegen verwirrt mich das und bekomme von gewissen Personen 
unschöne Kommentare wenn ich nachfrage :-)

Peter D. schrieb:
> Dan N. schrieb:
>> In der Aufgabe steht, dass die Funktion keinen Rückgabewert haben soll.
>
> Es ist guter Programmierstil, wenn man Funktion und Anzeige voneinander
> trennt. Denn das wird schnell ein Chaos, wenn jede Funktion wild auf das
> Display schreibt.
>
>
> Dan N. schrieb:
>> Kann man dann trotzdem mit "return" etwas zurückgeben? Wenn ja, wie geht
>> das dann?
>
> Steht in jedem C-Tutorial.

Nochmal für alle: ich persönlich würde es auch nicht so schreiben, aber 
laut Angabe darf die Funktion keinen Rückgabewert haben!

Danke an Alle :-)

Irgendwie werde ich schon dahinter kommen...

Da hast du vollkommen recht!