Probleme mit C Code und hilfe bei änderung

[Nadine W. (nadine_w)]

Hallo,

ich bin nicht so bewandt was C angeht aber bisher hat alles geklappt. 
Nunja. ich habe nun Probleme mit Strings.

Ich habe mit folgende Funktion gebastelt:

#include "main.h"
#include <avr/io.h>
#include <avr/interrupt.h>
#include <stdint.h>
#include "mylcd.h"
#include <avr/interrupt.h>
#include <stdint.h>
#include <inttypes.h>
#include <avr/pgmspace.h>
#include <stdlib.h>
#include <stdio.h>
#include <avr/eeprom.h>

int display_run = 0;
extern char temp_a;
extern char temp_b;
extern char temp_c;
extern int stunde;
extern int minute;
char Buffer[20];

char temp_log_data_1_time[10];
char temp_log_data_2_time[10];
char temp_log_data_3_time[10];
char temp_log_data_4_time[10];
char temp_log_data_5_time[10];

char temp_log_data_1_temp1;
char temp_log_data_1_temp2;
char temp_log_data_1_temp3;

char temp_log_data_2_temp1;
char temp_log_data_2_temp2;
char temp_log_data_2_temp3;

char temp_log_data_3_temp1;
char temp_log_data_3_temp2;
char temp_log_data_3_temp3;

char temp_log_data_4_temp1;
char temp_log_data_4_temp2;
char temp_log_data_4_temp3;

char temp_log_data_5_temp1;
char temp_log_data_5_temp2;
char temp_log_data_5_temp3;

void write_temp_log(void) {

  *temp_log_data_5_time = *temp_log_data_4_time;
  temp_log_data_5_temp1 = temp_log_data_4_temp1;
  temp_log_data_5_temp2 = temp_log_data_4_temp2;
  temp_log_data_5_temp3 = temp_log_data_4_temp3;

  *temp_log_data_4_time = *temp_log_data_3_time;
  temp_log_data_4_temp1 = temp_log_data_3_temp1;
  temp_log_data_4_temp2 = temp_log_data_3_temp2;
  temp_log_data_4_temp3 = temp_log_data_3_temp3;

  *temp_log_data_3_time = *temp_log_data_2_time;
  temp_log_data_3_temp1 = temp_log_data_2_temp1;
  temp_log_data_3_temp2 = temp_log_data_2_temp2;
  temp_log_data_3_temp3 = temp_log_data_2_temp3;

  *temp_log_data_2_time = *temp_log_data_1_time;
  temp_log_data_2_temp1 = temp_log_data_1_temp1;
  temp_log_data_2_temp2 = temp_log_data_1_temp2;
  temp_log_data_2_temp3 = temp_log_data_1_temp3;

  sprintf(temp_log_data_1_time, "foo");
  temp_log_data_1_temp1 = temp_a;
  temp_log_data_1_temp2 = temp_b;
  temp_log_data_1_temp3 = temp_c;
}

In der main führe ich folgendes aus:

.....
  while(1)
  {
    write_temp_log();

    read_temps();

    itoa(temp_log_data_1_time,Buffer,10);
    lcd_set_cursor(7,LINE3);
          lcd_puts(small_font,Buffer);
.....

Auf dem Display erhalte ich in LINE3 anstatt "foo" 102 angezeigt...

ich möchte allerdings folgendes:

sprintf(temp_log_data_1_time, "%d:%d", stunde, minute);

wenn ich nur

sprintf(temp_log_data_1_time, "%d", minute);

oder

sprintf(temp_log_data_1_time, "%d", stunde);

mache dann bekomme ich in LINE3 auf dem Display die Stunde oder die 
Minute angezeigt.... in

temp_log_data_1_time

soll aber die Zeit in Format SS:MM stehen, also z.b. 17:10

wenn ich nun versuche das ganze direkt da rein zu schreiben, also

temp_log_data_1_time = "17:10";

bzw

*temp_log_data_1_time = "17:10";

dann bekomme ich als Ausgabe auf dem Display "48" angezeigt.

Was mache ich falsch?

Dann ist das ganze ja ziemlich lang.. kann an das nicht irgendwie ein 
ein Array packen?

Also das z.b.

LOG_DATA[0][0] = UHRZEIT
LOG_DATA[0][1] = TEMPERATUR1
LOG_DATA[0][2] = TEMPERATUR2
LOG_DATA[0][3] = TEMPERATUR3

LOG_DATA[1][0] = 2te UHRZEIT
LOG_DATA[1][1] = 2te TEMPERATUR1
LOG_DATA[1][2] = 2te TEMPERATUR2
LOG_DATA[1][3] = 2te TEMPERATUR3

usw...

Ich danke euch vielmals.

P.S.:

Falls wichtig, beim compilieren bekomme ich keine Fehler oder Warnings..

make all
avr-gcc -c -Os -std=gnu99 -mmcu=atmega128 main.c -o main.o
avr-gcc -c -Os -std=gnu99 -mmcu=atmega128 mylcd.c -o mylcd.o
avr-gcc -c -Os -std=gnu99 -mmcu=atmega128 gen.c -o gen.o
avr-gcc -c -Os -std=gnu99 -mmcu=atmega128 ds18S20.c -o ds18S20.o
avr-gcc -c -Os -std=gnu99 -mmcu=atmega128 display.c -o display.o
avr-gcc -c -Os -std=gnu99 -mmcu=atmega128 read_temps.c -o read_temps.o
avr-gcc  -mmcu=atmega128 main.o mylcd.o gen.o ds18S20.o display.o read_temps.o -o main.elf
avr-objcopy -O ihex -j .data -j .text main.elf main.hex
avr-objcopy -O ihex -j .eeprom --change-section-lma .eeprom=1 main.elf main_eeprom.hex

[Rufus Τ. F. (rufus)]

Nadine W. schrieb:
> *temp_log_data_5_time = *temp_log_data_4_time;

Das macht recht sicher nicht das, was Du erzielen willst.

> Auf dem Display erhalte ich in LINE3 anstatt "foo" 102 angezeigt...

Wenn Du itoa aufrufst, und dem als Quelle einen Puffer übergibst, in dem 
"foo" steht, ...

Das ist wirr.

> Dann ist das ganze ja ziemlich lang.. kann an das nicht irgendwie ein
> ein Array packen?

Du könntest die zusammenhängenden Daten in eine Struktur packen, und 
ein Array vom Typ dieser Struktur anlegen.

[zitter_ned_aso (Gast)]

Nadine W. schrieb:
> char temp_log_data_4_time[10];
> char temp_log_data_5_time[10];
> ....
> ....
> ....
>   *temp_log_data_5_time = *temp_log_data_4_time;

Was  bedeutet das?




Nadine W. schrieb:
> char temp_log_data_1_time[10];
> char temp_log_data_2_time[10];
> char temp_log_data_3_time[10];
> char temp_log_data_4_time[10];
> char temp_log_data_5_time[10];

wäre

char temp_log_data{5][10];

nicht besser?

[Nadine W. (nadine_w)]

ich will das ganze im timer laufen lassen alle 30min sodass ich mir 
immer die letzten 5 werte ansehen kann....

und der soll dann eben immer einen "weiter rücken" also eintrag 1 nach 
2, 2 nach 3, 3 nach 4 und den neusten eben in 1

[zitter_ned_aso (Gast)]

Nadine W. schrieb:
> wenn ich nun versuche das ganze direkt da rein zu schreiben, also
> temp_log_data_1_time = "17:10";
>
> bzw
> *temp_log_data_1_time = "17:10";
>
> dann bekomme ich als Ausgabe auf dem Display "48" angezeigt.

Das soll ja ein String sein.

Nadine W. schrieb:
> char temp_log_data_1_time[10];


Also z.B. mit

strcpy(temp_log_blablabla, "17:10");

arbeiten

[Vn N. (wefwef_s)]

Du suchst einen Ringbuffer: 
https://www.mikrocontroller.net/articles/FIFO
Und Strukturen: https://www.mikrocontroller.net/articles/Strukturen_in_C
Du willst auch statt sprintf viel lieber itoa() and friends: 
https://www.mikrocontroller.net/articles/FAQ#sprintf.28.29

Ach, eigentlich willst du ein C-Buch und das Wiki durcharbeiten.

[zitter_ned_aso (Gast)]

Nadine W. schrieb:
> und der soll dann eben immer einen "weiter rücken" also eintrag 1 nach
> 2, 2 nach 3, 3 nach 4 und den neusten eben in 1

eine Liste wäre wohl dafür geeignet. Z.B. die neuen Werte vorne 
einfügen. Dann muss man nichst "schieben".

[Nadine W. (nadine_w)]

zitter_ned_aso schrieb:
> Also z.B. mitstrcpy(temp_log_blablabla, "17:10");
> arbeiten

Alles auch schon versucht

gen.c: In function ‘write_temp_log’:
gen.c:71:3: warning: implicit declaration of function ‘strcpy’ 
[-Wimplicit-function-declaration]
   strcpy(*temp_log_data_1_time, "17:10");
   ^


oder

gen.c: In function ‘write_temp_log’:
gen.c:71:3: warning: implicit declaration of function ‘strcpy’ 
[-Wimplicit-function-declaration]
   strcpy(temp_log_data_1_time, "17:10");
   ^

[zitter_ned_aso (Gast)]

für strcpy(...) braucht man eine entsprechende Header-Datei.

#include <string.h>

Nadine W. schrieb:
> strcpy(*temp_log_data_1_time, "17:10");

warum temp_log.... mit Stern?

Hier ein kostenloses Buch zu C:

http://openbook.rheinwerk-verlag.de/c_von_a_bis_z/

[S. R. (svenska)]

zitter_ned_aso schrieb:
> für strcpy(...) braucht man eine entsprechende Header-Datei.

Und doppelt einbinden bringt für die existierenden Header auch nichts.

[zitter_ned_aso (Gast)]

dann halt nicht inkludieren

[leo (Gast)]

Nadine W. schrieb:
> gen.c:71:3: warning: implicit declaration of function ‘strcpy’
> [-Wimplicit-function-declaration]
>    strcpy(temp_log_data_1_time, "17:10");
>    ^

Dir wurde ja schon gesagt, dass du eine Headerdatei brauchst. Lerne auf 
die Fehlermeldungen und Warnungen zu ragieren.

E.g. im Terminal:

$ man strcpy

Bzw. auch im Browser
https://www.google.com/search?q=man+strcpy

Dort findest du u.a. auch die Header:

#include <string.h>

char *strcpy(char *dest, const char *src);
...

HTH
leo

[Nadine W. (nadine_w)]

ja.. habe ich gerade auch alles getestet...

ich habe jetzt alles durch.. ich bekommen es auch OHNE Warnings und 
Fehler...

aber die Ausgabe ist kein Text... sondern aus 17:10 was er anzeigen soll 
zeigt er auf einmal 12336 an...

Also ich bekomme eine ausgabe... nur anstatt "foo" eben 102... 102 wäre 
"f" aber "o" und "o" fehlt usw usw...

da ist einfach irgendwo der Wurm drinne....

Ein

temp_log_data_1_time = 35;

das geht auch... dann sehe ich 35 auf dem Display... oder aber

temp_log_data_1_time = 9999;

dann sehe ich 9999 auf dem Display....

aber

temp_log_data_1_time = "foo";

wird zu

102 statt foo

.......

[Rufus Τ. F. (rufus)]

Nadine W. schrieb:
> Also ich bekomme eine ausgabe... nur anstatt "foo" eben 102... 102 wäre
> "f" aber "o" und "o" fehlt usw usw...

Kein Wunder, itoa wertet nur einen übergebenen Wert aus. Daß hinter dem 
Wert noch andere stehen, interessiert itoa nicht.

> da ist einfach irgendwo der Wurm drinne....

Lies. Ein. C-Buch.

[Dirk B. (dirkb2)]

Rufus Τ. F. schrieb:

> Lies. Ein. C-Buch.

Arbeite es durch. Nur lesen reicht meist nicht.

[Dirk B. (dirkb2)]

Nadine W. schrieb:
> ich habe jetzt alles durch.. ich bekommen es auch OHNE Warnings und
> Fehler...

Ist immer noch der Aufruf von itoa drin?
wie sieht der aus?

[zitter_ned_aso (Gast)]

Nadine W. schrieb:
> temp_log_data_1_time = 9999;
>


bei char-Datentyp?

[Nadine W. (nadine_w)]

zitter_ned_aso schrieb:
> Nadine W. schrieb:
>> temp_log_data_1_time = 9999;
>>
>
>
> bei char-Datentyp?

Ja alles genau so wie der Code ist... Ich mache einfach

temp_log_data_1_time = 9999;

Und dann habe ich auf dem Display 9999 als Ausgabe....

Das Display ist ein ks0108 wenn das weiter hilft.

Aber wenn ich in der main.c

itoa(temp_log_data_1_temp1,Buffer,10);
lcd_set_cursor(7,LINE3);
lcd_puts(small_font,Buffer);

Also die Temperaturen von temp1 bis temp3 durchgehen bekomme ich die 
korrekten Werte dargestellt...

[Rufus Τ. F. (rufus)]

Nadine W. schrieb:
> Ja alles genau so wie der Code ist... Ich mache einfach

Das ist mehr als unwahrscheinlich. Eine char-Variable kann numerische 
Werte im Bereich von 0 bis 255 aufnehmen, wenn sie ohne Vorzeichen 
interpretiert wird, oder aber Werte zwischen -128 und 127, wenn sie mit 
Vorzeichen interpretiert wird.

[Dirk B. (dirkb2)]

Nadine W. schrieb:
> Ja alles genau so wie der Code ist... Ich mache einfach
> temp_log_data_1_time = 9999;

temp_log_data_1_time ist doch ein char-Array.

Einem Array kann man so keinen Wert zuweisen.

Wenn es ein char-Pointer wäre, sollte es eine Warnung geben.

[Lass es besser sein (Gast)]

Was hilft ihr solchen Leuten eigentlich, wenn sie nicht mal die 
grundregelsten Grundlagen der Grundlagen mitbringen und schon ein 
Raumschiffe steuern wollen, wenn sie nicht mal wissen was ein char ist?

Euch kann man echt nimmer helfen, sorry.

[foobar (Gast)]

> char Buffer[20];
> char temp_log_data_1_time[10];
> char temp_log_data_1_temp1;
>
> itoa(temp_log_data_1_temp1, Buffer, 10);
> lcd_puts(small_font, Buffer);

Ich denke, du bist etwas verwirrt und deine beschissene Namensvergabe 
hilft da auch nicht gerade.

itoa wandelt eine (int-)Zahl nach Ascii, erster Parameter ist die Zahl, 
zweiter der Puffer, in den er die Ascii-Zeichen reinschreibt, dritter 
die Basis, in der die Umwandlung durchgeführt werden soll.  Also, 
temp_log_data_1_temp1 ist Input, Buffer ist Output.

lcd_puts gibt einen Text aus, erster Parameter ist die Fontgröße, 
zweiter der Puffer mit dem Text.

> sprintf(temp_log_data_1_time, "%d:%d", stunde, minute);

Dies schreibt den formatierten Text in den Puffer temp_log_data_1_time. 
Dann muß auch ein 'lcd_puts(small_font, temp_log_data_1_time);' folgen. 
Alternativ kannst du auch mit 'strcpy(Buffer, temp_log_data_1_time);' 
die Uhrzeit in den Buffer kopieren und den lcd_puts mit Buffer machen.

Tipp: Mach dir Strukturen, z.B.:

struct temp {
    uint8_t stunde, minute;
    int8_t temp1, temp2, temp3;
};

#define LOGSIZE 5
struct temp log[LOGSIZE];

void write_temp_log()
{
    for (uint8_t i = LOGSIZE-1; i > 0; --i)
        log[i] = log[i-1];
    //oder: memmove(log+1, log+0, (LOGSIZE-1)*sizeof(*log));
}

...
    while(1)
    {
        char buf[32];

        write_temp_log();
        read_temps();  // fuellt 'log[0]'

        for (uint8_t i = 0; i < LOGSIZE; ++i)
        {
            sprintf(buf, "%02u:%02u %3d %3d %3d",
                           log[i].stunde, log[i].minute,
                           log[i].temp1, log[i].temp2, log[i].temp3);
            lcd_set_cursor(0, LINE2 + i);
            lcd_puts(small_font, buf);
        }
    }

[... (Gast)]

Ich nehme das mal etwas auseinander.

char temp_log_data_1_time[10];
temp_log_data_1_time = "foo";

temp_log_data_1_time ist ein Zeiger auf das erste Element eines 
Char-Arrays.
Dieser sollte auf einem AVR 16-Bit lang sein.

itoa wird Dir nur diese Zeigeradresse in einen String wandeln.

Natürlich kannst Du diesen Zeiger auch selber setzen:

temp_log_data_1_time = 9999;

Das ist aber kein String. "17:10" wirst Du so nicht ablegen können.

[... (Gast)]

... schrieb:
> itoa wird Dir nur diese Zeigeradresse in einen String wandeln.

Ich meine natürlich, so wie Du es benutzt.

[... (Gast)]

Nadine W. schrieb:
> *temp_log_data_5_time = *temp_log_data_4_time;

Und damit kopierst Du nur das erste Zeichen des CHAR-Arrays.

[Dirk B. (dirkb2)]

... schrieb:
> temp_log_data_1_time ist ein Zeiger auf das erste Element eines
> Char-Arrays.

Nein. Das ist die Adresse von dem ersten Element des Arrays

> Natürlich kannst Du diesen Zeiger auch selber setzen:

Nein.
Eine Arrayadresse kann man nicht ändern.

Ein Array ist kein Pointer. Ein Pointer ist kein Array.

[Stefan F. (Gast)]

Dirk B. schrieb:
>> temp_log_data_1_time ist ein Zeiger auf das erste Element eines
>> Char-Arrays.
> Nein. Das ist die Adresse von dem ersten Element des Arrays

Ist das nicht das Gleiche?

[Rufus Τ. F. (rufus)]

Stefanus F. schrieb:
> Ist das nicht das Gleiche?



Nein, in diesem Fall ist es das nicht. Zwar verhält es sich wie ein 
Zeiger auf das erste Element eines Arrays, und sobald man es einer 
Funktion übergibt, ist es einer, aber so, wie Du es da hingeschrieben 
hast, ist es kein Zeiger, sondern tatsächlich ein Array.

Den Unterschied zeigt Dir der sizeof -Operator:

void test(char[] p)
{
  printf("sizeof = %d\n", sizeof p);
}

void main(void)
{
  char array[10];

  printf("sizeof = %d\n", sizeof array);

  test(array);
}

[Nadine W. (nadine_w)]

ein

sprintf(temp_log_data_1_time, "%s:%s", stunde, minute);

Also %d mit %s tauschen brauchte mich zum Erfolg...

[Dirk B. (dirkb2)]

Stefanus F. schrieb:
> Dirk B. schrieb:
>>> temp_log_data_1_time ist ein Zeiger auf das erste Element eines
>>> Char-Arrays.
>> Nein. Das ist die Adresse von dem ersten Element des Arrays
>
> Ist das nicht das Gleiche?

Nein.
Eine Arrayadresse kann man nicht ändern.
Einen Zeiger schon.

temp_log_data_1_time++; geht beim Array nicht.

temp_log_data_1_time = Null; geht auch nicht.

Ein Array ist kein Pointer. Ein Pointer ist kein Array.

[Dirk B. (dirkb2)]

Nadine W. schrieb:
> ein
>
>

> sprintf(temp_log_data_1_time, "%s:%s", stunde, minute);
>

>
> Also %d mit %s tauschen brauchte mich zum Erfolg...

Muhaha.

Ein Integer ist kein Pointer.

Ohne Warnungen zu compilieren bedeutet nicht, die Warnungen 
abzuschalten, sondern die Ursache der Warnungen zu beheben.

[stdc (Gast)]

Rufus Τ. F. schrieb:
> Den Unterschied zeigt Dir der sizeof -Operator:
>
>

> void test(char[] p)
> {
>   printf("sizeof = %d\n", sizeof p);
> }
> 
> void main(void)
> {
>   char array[10];
> 
>   printf("sizeof = %d\n", sizeof array);
> 
>   test(array);
> }
>

Ich verstehe, was du sagen möchtest. Aber ich glaube, dass deine 
Begründung falsch ist/der Vergleich hinkt:

char[]

 als Typ für einen Parameter ist lediglich syntaktischer Zucker und wird 
identisch zu

char*

 behandelt. Du kannst auch

char p[10]

 als Parametertyp verwenden, doch sizeof ist immer noch die Größe eines 
pointers (-Wall wirft hier sogar eine Warnung):

$ avr-gcc -O3 -Wall -o foo foo.c 
foo.c: In function ‘fun’:
foo.c:7:26: warning: ‘sizeof’ on array function parameter ‘p’ will return size of ‘char *’ [-Wsizeof-array-argument]
     printf("%d\n", sizeof(p));
                          ^
foo.c:5:10: note: declared here
 fun(char p[10])
          ^
$ avr-gcc -dumpversion
5.4.0

[stdc (Gast)]

stdc schrieb:
> Ich verstehe, was du sagen möchtest. Aber ich glaube, dass deine
> Begründung falsch ist/der Vergleich hinkt:

Will sagen: Auch wenn die Syntax suggeriert, man könne ein Array als 
Parameter übergeben, geht es trotzdem nicht.

[Nop (Gast)]

stdc schrieb:

> Will sagen: Auch wenn die Syntax suggeriert, man könne ein Array als
> Parameter übergeben, geht es trotzdem nicht.

Richtig. Man benutzt das nur, wenn die Funktion ein Array fester Größe 
verwendet, damit das schon anhand des Prototypen für den aufrufenden 
Programmierer ersichtlich ist. Also zur Doku.

[Rufus Τ. F. (rufus)]

stdc schrieb:
> Aber ich glaube, dass deine Begründung falsch ist/der Vergleich hinkt:

Nein, denn genau das ist der Punkt, auf den ich hinauswill.

Dirk hat aber schon das wesentliche geschrieben:

>> Ist das nicht das Gleiche?
>
> Nein.
> Eine Arrayadresse kann man nicht ändern.
> Einen Zeiger schon.
>
> temp_log_data_1_time++; geht beim Array nicht.
>
> temp_log_data_1_time = Null; geht auch nicht.
>
> Ein Array ist kein Pointer. Ein Pointer ist kein Array.