Mehrdimensionales char Array

[Welle 🧐 S. (w3llschmidt)]

Hallo C-Freunde ...

ich möchte aus drei Datein jeweils die einzelnen Werte auslesen und in
ein  mehrdimensionales char Array schreiben ... 3 Zeilen, 64 Strings.

  1 2 3 4 5 6 7 8 ... (n)
1
2       X
3

um auf diese Strings dann direkt zu referenzieren (sensorid[2][4] = X).

Was meint ihr ->

int ds2482_sysfs_init(void) {

  char sensorid[3][64][17];  

  for (i=1; i<4; i++) 
  {
    char *fn;
    sprintf ( fn, "/sys/bus/w1/devices/w1_bus_master%d/w1_master_slaves", i );  
    FILE *fp = fopen( fn, "r");

      int count = 1;
            
      while ( fgets ( sensorid[i][count], sizeof(sensorid), fp ) != 0 )
      {
      count++;
      }
        
  fclose(fp);
  }

// zum Testen
syslog( LOG_INFO, "%s", sensorid[1][1]);
syslog( LOG_INFO, "%s", sensorid[1][2]);
syslog( LOG_INFO, "%s", sensorid[2][1]);

return ( EXIT_SUCCESS );
}

[Martin L. (sbond)]

Hi,

Ich denke du müsstest dein Array etwas größer machen, denn 
sensorid[3][...][...] geht ja von sensorid[0] bis sensorid[2] und nicht 
von 1 bis 3.

Der sizeof Befehl ist etwas ungeeignet, da er wahrscheinlich die 
Gesamtgröße des Arrays ausgibt. Der Wert von 16 darf aber nicht 
überschritten werden, da du deine Stringlänge auf 17 definiert hast.

syslog kenne ich nicht. Ich vertraue mal darauf, dass du den Befehl 
kennst ;)

#define Dateien = 3
#define Werte = 64
#define Zeichen = 16


int ds2482_sysfs_init(void) {

  char sensorid[Dateien+1][Werte+1][Zeichen+1];  

  for (i=1; i<=Dateien; i++) 
  {
    char *fn;
    sprintf ( fn, "/sys/bus/w1/devices/w1_bus_master%d/w1_master_slaves", i );  
    FILE *fp = fopen( fn, "r");

      int count = 1;
            
      while ( fgets ( sensorid[i][count], Zeichen, fp ) != 0 )
      {
      if (count > Werte) return ( EXIT_ERROR ); // wenn die Datei deine Array-größe sprengt
      count++;
      }
        
  fclose(fp);
  }

// zum Testen
syslog( LOG_INFO, "%s", sensorid[1][1]);
syslog( LOG_INFO, "%s", sensorid[1][2]);
syslog( LOG_INFO, "%s", sensorid[2][1]);

return ( EXIT_SUCCESS );
}

[g457 (Gast)]

> char *fn;
        ^^^
> sprintf(fn, "/sys/bus/w1/devices/w1_bus_master%d/w1_master_slaves", i);
          ^^
∗hüstel∗

[Helfender (Gast)]

> fgets ( sensorid[i][count], sizeof(sensorid), fp )

sizeof(sensorid) ist die Größe des ganzen Arrays,
nicht nur eines einzelnen Eintrages.

Ich denke, Du müsstest (neben den anderen oberen Ratschlägen)
hier etwas in der Art

fgets ( sensorid[i][count], sizeof(sensorid[i][count]), fp )

verwenden.

[Welle 🧐 S. (w3llschmidt)]

Hi Martin ...

Martin L. schrieb:
> Hi,
>
> Ich denke du müsstest dein Array etwas größer machen, denn
> sensorid[3][...][...] geht ja von sensorid[0] bis sensorid[2] und nicht
> von 1 bis 3.
>
> Der sizeof Befehl ist etwas ungeeignet, da er wahrscheinlich die
> Gesamtgröße des Arrays ausgibt. Der Wert von 16 darf aber nicht
> überschritten werden, da du deine Stringlänge auf 17 definiert hast.

Ja, die IDs sind 15+/0+/n

syslog( LOG_INFO, "%s", sensorid[i][count] );

Dec 15 13:47:44 rpi 1wirevz: 28-0000042b0406
Dec 15 13:47:44 rpi 1wirevz: 28-0000042b410e
Dec 15 13:47:44 rpi 1wirevz: not found.
Dec 15 13:47:44 rpi 1wirevz: not found.

[Welle 🧐 S. (w3llschmidt)]

g457 schrieb:
>> char *fn;
>         ^^^
>> sprintf(fn, "/sys/bus/w1/devices/w1_bus_master%d/w1_master_slaves", i);
>           ^^
> ∗hüstel∗

Hi g457 ... das passt nicht?

[Helfender (Gast)]

Mit

>> char *fn;

legst Du nur Speicher für einen Pointer (z.B. 4 Bytes auf 32 Bit System)
an. Dann schreibst Du aber viele Bytes hinein...

Du brauchst etwas wie

char fn[128];

[AVerr (Gast)]

Henrik Wellschmidt schrieb:
> Hi g457 ... das passt nicht?

Wo zeigt fn denn hin ?
Richtig, irgendwohin ... nur in (mit großer Wahrscheinlichkeit) keinen 
initialisierten Bereich.
Da fehlt noch ein malloc Aufruf.

[Welle 🧐 S. (w3llschmidt)]

Helfender schrieb:
> Mit
>
>>> char *fn;
>
> legst Du nur Speicher für einen Pointer (z.B. 4 Bytes auf 32 Bit System)
> an. Dann schreibst Du aber viele Bytes hinein...
>
> Du brauchst etwas wie
>
> char fn[128];

Ups, danke!

  char fn[64];
  sprintf(fn, "/sys/bus/w1/devices/w1_bus_master1/%s/w1_slave", sensorid );
  FILE *fp = fopen ( fn, "r" );