Ansi-C 4d Array

[Dominic K. (domi1997)]

Guten Tag zusammen
Ich versuche gerade ein 4 dimensionales Array Aufzustellen. Nun will bei 
mir nicht in den Kopf wie ich dies nun darstellen muss.
Ich interpretiere es wie folgt, doch korrigiert mich bitte wenn ich 
falsch liege:

static unsigned char SwitchTimes[20][2][4][2]=
Die 20 steht für die Anzahl Zeilen.
Die Zwei für die Anzahl Spalten.
Die 4 Für die Anzahl Ziffern die in dem Feld sein können.
Die 2te Zwei für die Anzahl Stapel die existieren.

Also wenn ich auf die 10. Zeile in der 2. Spalte die 3. ziffer auf dem 
ersten Stapel will müsste das wie folgt aussehen:

x = SwitchTimes[10][2][3][1];

Doch entweder stimmt dies nicht oder meine Aufstellung der Tabellen 
werte ist falsch. Ich stelle euch meine Definition und Deklaration der 
Tabelle in den Anhang.

[Kirsch (Gast)]

ob eindimensionel oder mehr dimensionel

die indexis gehen immer von 0 bis n-1

                    |
....................V
x = SwitchTimes[10][2][3][1];

[Erich (Gast)]

Dominic K. schrieb:
> Ich versuche gerade ein 4 dimensionales Array Aufzustellen.

SwitchTimes[320]
wäre einfacher.

Gruss

[Rufus Τ. F. (rufus)]

Dominic K. schrieb:
> SwitchTimes[20][2][4][2]

Das bedeutet folgende legale Index-Werte:

SwitchTimes[0 - 19][0 - 1][0 - 3][0 - 1]

[43222342 (Gast)]

wie wärs mit einem struct das dir das ganze lesbarerer gestaltet

sonst bekommt man schon beim schreiben der eigentlich gewollten 
funktionen einen knoten im kopf

[Dominic K. (domi1997)]

Die Tabelle soll später für eine Funktion Schaltzeiten für zwei Relais 
zur Verfügung stellen. 20 Zeilen weil ich 20 Schaltpunkte will. 2 
Spalten für ein und Aus Zeit. 4 Werte weil ich Tag, Stunde, Minute und 
Sekunde Speichern will.Und zwei Stapel weil die 2 Relais eigene Zeiten 
haben. Daher denke ich ist eine 4d Tabelle das beste.

[Dominic K. (domi1997)]

Kirsch schrieb:
> ob eindimensionel oder mehr dimensionel
>
> die indexis gehen immer von 0 bis n-1
>
>                     |
> ....................V
> x = SwitchTimes[10][2][3][1];

Okey das wusste ich noch hatte es aber gerade nicht im Kopf beim 
Beispiel schreiben aber danke

[Dominic K. (domi1997)]

Hat jemand vlt gerade ein übersichtliches Beispiel einer 4d Tabelle zur 
hand? Am besten eine bei der Alle werte in den eckigen Klammern anderst 
sind um besser zu erkennen was für was steht.

[Kirsch (Gast)]

Dominic K. schrieb:
> Hat jemand vlt gerade ein übersichtliches Beispiel einer 4d Tabelle zur
> hand? Am besten eine bei der Alle werte in den eckigen Klammern anderst
> sind um besser zu erkennen was für was steht.

Ein N-Dimensonalles Array ist doch nur eine Struktur um Daten zu 
speichen, was die einzelne dimension bedautet ist doch dir selbst 
überlassen.

[Yalu X. (yalu) (Moderator)]

Deklaration und Initialisierung passen nicht zusammen. Gehen wir mal von
der Initialisierung aus:

Sie besteht aus Daten für  2   Relais.
Für jedes Relais gibt es   2   Blöcke (je einer für Ein- und Auschaltzeiten).
Jeder Block besteht aus   20   Zeiten (Schaltpunkten)
Jede Zeit besteht aus      4   Werten (Tag, Stunde, Minute und Sekunde)

Genau in dieser Reihenfolge (also vom Groben zum Feinen hin) müssen auch
die Größenangaben in der Deklaration auftauchen:

unsigned char SwitchTimes[2][2][20][4]=

Dann fehlt in Zeile 48 noch ein Komma.

Ich würde aber mindestens die Zeiten nicht als Array, sondern als
Struktur deklarieren, bspw. so:

struct zeit {
  unsigned char tag, stunde, minute, sekunde;
};

struct einaus {
  struct zeit ein, aus;
};

#define ANZ_RELAIS 2
#define ANZ_SCHALTPUNKTE 20

struct einaus schaltzeiten[ANZ_RELAIS][ANZ_SCHALTPUNKTE];

[Eric B. (beric)]

Dominic K. schrieb:
> Die Tabelle soll später für eine Funktion Schaltzeiten für zwei Relais
> zur Verfügung stellen. 20 Zeilen weil ich 20 Schaltpunkte will. 2
> Spalten für ein und Aus Zeit. 4 Werte weil ich Tag, Stunde, Minute und
> Sekunde Speichern will.Und zwei Stapel weil die 2 Relais eigene Zeiten
> haben. Daher denke ich ist eine 4d Tabelle das beste.

typedef struct{
  uint8_t day;
  uint8_t hour;
  uint8_t minute;
  unit8_t second;
} Timestamp;

#define SUNDAY 0
...
#define SATURDAY 6

typedef struct {
  Timestamp on;
  Timestamp off;
} Switchpoint;

#define NOF_SWITCHPOINTS 20

Switchpoint relais1[NOF_SWICHTPOINTS];
Switchpoint relais2[NOF_SWITCHPOINTS];

relais1[0].on = {SUNDAY, 0, 15, 0 }; // first switchpoint ON is Sunday 0:15'00"

Sieht für mich besser aus als einen undurchsichtige 4D Tabelle


// Yalu war eine halbe Minute schneller :-D

[Peter D. (peda)]

Dominic K. schrieb:
> 20 Zeilen weil ich 20 Schaltpunkte will. 2
> Spalten für ein und Aus Zeit. 4 Werte weil ich Tag, Stunde, Minute und
> Sekunde Speichern will.Und zwei Stapel weil die 2 Relais eigene Zeiten
> haben. Daher denke ich ist eine 4d Tabelle das beste.

Bei wären das:

uint32_t SwitchTimes[2][40];

Die [2] sind die Relaisnummer.
Die [40] sind die 40 Zeiten, wovon die geraden einschalten und die 
ungeraden aus.
Und uint32_t sind die Sekunden (z.B. seit 2000).

[Eric B. (beric)]

Übrigens:

Dominic K. schrieb:
> 20 Zeilen weil ich 20 Schaltpunkte will. 2
> Spalten für ein und Aus Zeit.

Das sind dann insgesamt 40 Schaltpunkte: 20x AN, 20x AUS.
Ist das so gewollt?

[Dr. Sommer (Gast)]

Mal so am Rande - warum muss es ANSI-C sein, eine 29 Jahre alte 
Sprachversion? Für nahezu jede Plattform sollte es mittlerweile einen 
Compiler für C99 oder gar C11 geben.

[43222342 (Gast)]

Dominic K. schrieb:
> ie Tabelle soll später für eine Funktion Schaltzeiten für zwei Relais
> zur Verfügung stellen. 20 Zeilen weil ich 20 Schaltpunkte will. 2
> Spalten für ein und Aus Zeit. 4 Werte weil ich Tag, Stunde, Minute und
> Sekunde Speichern will.Und zwei Stapel weil die 2 Relais eigene Zeiten
> haben. Daher denke ich ist eine 4d Tabelle das beste.


wer sagt das da eine 4D tabelle am besten ist?

2D .. ok aber aslles darüber hinaus verwirrt und mach nur knoten im hirn

[43222342 (Gast)]

mach das mit den structs und du bist in 2h fertig.

[Dominic K. (domi1997)]

Dr. Sommer schrieb:
> Mal so am Rande - warum muss es ANSI-C sein, eine 29 Jahre alte
> Sprachversion? Für nahezu jede Plattform sollte es mittlerweile einen
> Compiler für C99 oder gar C11 geben.

Weil ich Ansi-C gelernt habe und dadurch es am besten verstehen werde. 
Und halt weil es sicher auf meinem Mikroprozessor läuft

[Dominic K. (domi1997)]

> wer sagt das da eine 4D tabelle am besten ist?
>
> 2D .. ok aber aslles darüber hinaus verwirrt und mach nur knoten im hirn

Weil ich dan 2 mal die selbe Funktion in den Code nehmen müsste

[Dirk B. (dirkb2)]

Dominic K. schrieb:
> Weil ich dan 2 mal die selbe Funktion in den Code nehmen müsste

Wöfür wäre(n) die dann?

[Dominic K. (domi1997)]

Die Funktion ist ein Algorithmus der eine neue Zeit die neu gespeichert 
wird mit allen bereits gespeicherten Zeiten vergleicht ob diese sich 
nicht irgendwie überschneiden. es handelt sich um ziemlich viel code was 
doppel wäre...

[43222342 (Gast)]

?
das musst du mal nächer erklären warum doppelt?
und warum das durchrollen der einträge viel code ist?

Viel code verstehe ich eher was ab 20000zeilen ..

Die sache mit den relais ist eher was auf 1-2 DIN A4 seiten.

[Dominic K. (domi1997)]

Eric B. schrieb:
>
> #define NOF_SWITCHPOINTS 20
>
> Switchpoint relais1[NOF_SWICHTPOINTS];
> Switchpoint relais2[NOF_SWITCHPOINTS];
>
> relais1[0].on = {SUNDAY, 0, 15, 0 }; // first switchpoint ON is Sunday
> 0:15'00"
> [/c]
> Sieht für mich besser aus als einen undurchsichtige 4D Tabelle

Sieht für interessant aus bin nur gerade nicht sicher wie ich da die 
Werte wieder auslesen kann. Angenommen ich will die Minute von Relais1 
von der 3. Zeit stimmt dan der folgende Code
x = relais1[3].on->2
Das x ist hier nur ein Platzhalter.

[Dominic K. (domi1997)]

43222342 schrieb:
> ?
> das musst du mal nächer erklären warum doppelt?
> und warum das durchrollen der einträge viel code ist?
>
> Viel code verstehe ich eher was ab 20000zeilen ..
>
> Die sache mit den relais ist eher was auf 1-2 DIN A4 seiten.

Okey dan ist das Einstellungssache. ist natürlich nur ein kleiner 
Teilausschnitt des ganzen. Und mich verwirrt das eher wenn code doppelt 
vorkommt.

[43222342 (Gast)]

typedef struct{
  uint8_t day;
  uint8_t hour;
  uint8_t minute;
  unit8_t second;
} Timestamp;

typedef struct {
  Timestamp on;
  Timestamp off;
} Switchpoint;

#define NOF_SWITCHPOINTS 20
#define NOF_RELAIS       2

Switchpoint relais[NOF_RELAIS][NOF_SWICHTPOINTS];

// werte aus Switchpoint 3 
// relais 1
uint8_t min = relais[0][2].minute ;

// relais 2
uint8_t min = relais[1][2].minute ;

da man aber nur an oder aus braucht...
kann man speicher sparen und das sogar zusammenfassen

typedef enum{
  RELAIS_OFF,
  RELAIS_ON
}relais_state_e;

typedef struct{
  uint8_t          day;
  uint8_t          hour;
  uint8_t          minute;
  unit8_t          second;
  relais_state_e   state;
} Switchpoint;

#define NOF_SWITCHPOINTS 20
#define NOF_RELAIS       2

Switchpoint relais[NOF_RELAIS][NOF_SWICHTPOINTS];

[Teo D. (teoderix)]

Yalu X. schrieb:
> Ich würde aber mindestens die Zeiten nicht als Array, sondern als

Intern nur mit Sekunden arbeiten, und nur für das HID aufbereiten.

[43222342 (Gast)]

43222342 schrieb:
> Switchpoint relais[NOF_RELAIS][NOF_SWICHTPOINTS];
>
> // werte aus Switchpoint 3
> // relais 1
> uint8_t min = relais[0][2].minute ;
>
> // relais 2
> uint8_t min = relais[1][2].minute ;

korrektur

 // on werte aus Switchpoint 3
 // relais 1
 uint8_t min = relais[0][2].on.minute ;
 // relais 2
 uint8_t min = relais[1][2].on.minute ;

 // off werte aus Switchpoint 3
 // relais 1
 uint8_t min = relais[0][2].off.minute ;
 // relais 2
 uint8_t min = relais[1][2].off.minute ;

[43222342 (Gast)]

Teo D. schrieb:
>> Ich würde aber mindestens die Zeiten nicht als Array, sondern als
>
> Intern nur mit Sekunden arbeiten, und nur für das HID aufbereiten.

macht die sache noch einfacher :-)

[Dominic K. (domi1997)]

Werde den Code implementieren und ein Test programm schreiben. Bin 
optimistisch das dieser funktioniert. Danke viel mals an alle

[Dominic K. (domi1997)]

#define NOF_SWITCHPOINTS 20
#define NOF_RELAIS       2
typedef struct{
  unsigned char day;
  unsigned char hour;
  unsigned char minute;
  unsigned char second;
} Timestamp;

typedef struct {
  Timestamp on;
  Timestamp off;
} Switchpoint;
static Switchpoint relais[NOF_RELAIS][NOF_SWITCHPOINTS];

void TS_Setup(void)
{
  unsigned char i;
  while (i<20)
  {
//     relais[0][i].on = {255,255,255,255};
//     relais[1][i].on = {255,255,255,255};
    i++;
  }
  write_zahl(3,3,relais[0][2].on.minute,3,0,0);
}
Irgendwas mach ich noch falsch den Es kommen pro auskommentierte Linie 
ein { Klammer Fehler sobald ich diese normal einsetze.
Und ja ein For wäre besser geeignet gewesen aber ich wollte die Funktion 
nur 1 mal aufrufen und sicher sein das überall 255er drin stehen.

[43222342 (Gast)]

memset( relais , 255, (sizeof(Switchpoint)*NOF_SWITCHPOINTS)*NOF_RELAIS 
);

[Kirsch (Gast)]

Dominic K. schrieb:
> relais[0][i].on = {255,255,255,255};
> //     relais[1][i].on = {255,255,255,255};

Dieser Syntax geht nur bei der Initialisierung einer Struktur/Array, 
aber nicht bei einer späteren Zuweisung.

muss du leider alles einzeln machen
relais[0][i].on[0] = 255;
relais[0][i].on[1] = 255;
relais[0][i].on[2] = 255;
relais[0][i].on[3] = 255;

[43222342 (Gast)]

// alle ON einträge mit 0xff
for( i = 0; i< NOF_SWITCHPOINTS ; i++ ){
     memset( &relais[0][i].on , 0xff , sizeof( Timestamp )) ;
}

// alle OFF einträge mit 0
for( i = 0; i< NOF_SWITCHPOINTS ; i++ ){
     memset( &relais[0][i].off , 0 , sizeof( Timestamp )) ;
}

[43222342 (Gast)]

43222342 schrieb:
> // alle ON einträge mit 0xff
> for( i = 0; i< NOF_SWITCHPOINTS ; i++ ){
>      memset( &relais[0][i].on , 0xff , sizeof( Timestamp )) ;
> }
>
> // alle OFF einträge mit 0
> for( i = 0; i< NOF_SWITCHPOINTS ; i++ ){
>      memset( &relais[0][i].off , 0 , sizeof( Timestamp )) ;
> }

releais 2 vergessen

// alle ON einträge mit 0xff
for( i = 0; i< NOF_SWITCHPOINTS ; i++ ){
    memset( &relais[0][i].on , 0xff , sizeof( Timestamp )) ;
    memset( &relais[1][i].on , 0xff , sizeof( Timestamp )) ;
}

// alle OFF einträge mit 0
for( i = 0; i< NOF_SWITCHPOINTS ; i++ ){
    memset( &relais[0][i].off , 0 , sizeof( Timestamp )) ;
    memset( &relais[1][i].off , 0 , sizeof( Timestamp )) ;
}


oder bei mehereren relais

for( i = 0; i< NOF_RELAIS ; i++ ){
  for( k = 0; k< NOF_SWITCHPOINTS ; k++ ){
    memset( &relais[i][k].on , 0xff , sizeof( Timestamp )) ;
  }
}

[Dominic K. (domi1997)]

Die Funktion um alle zu überschreiben geht nach dem includieren der 
string.h library aber der befehl um auf eine einzelne zeit 
einzuschreiben geht nicht:
relais[0][1].on[0] = 255;
Bringt denn error: subscripted value is neither array nor pointer nor 
vector

[43222342 (Gast)]

relais[0][1].on.hour = 10;
relais[0][1].on.minute = 30;
relais[0][1].on.second = 00;
relais[0][1].on.day = 0;

[Dominic K. (domi1997)]

43222342 schrieb:
> relais[0][1].on.hour = 10;
> relais[0][1].on.minute = 30;
> relais[0][1].on.second = 00;
> relais[0][1].on.day = 0;

Okey klar demfall ist die Methode von Kirsch unpassend danke.

[Yalu X. (yalu) (Moderator)]

Dominic K. schrieb:
>      relais[0][i].on = {255,255,255,255};

Das geht nur in C++.

In C (aber nicht in ANSI C) geht es mit Compound-Literals:

      relais[0][i].on = (Timestamp){255,255,255,255};

In ANSI-C kannst du wie in deinem ersten Beispiel das gesamte Array
direkt bei seiner Definition initialisieren (s. Anhang).

Da du aber C99-Kommentare (mit //) benutzen kannst, ist dein Compiler
vielleicht gar nicht so altmodisch wie du denkst, so dass du nicht an
die ANSI-Restriktionen (von 1989) gebunden bist.

[43222342 (Gast)]

das mit

Timestamp var = {0};

geht auch nur bedingt..
bin damit schon auf die nase gefallen.

das funzt erstmal auf den ersten blick
und vereinfacht ein memset()
aber je nach optimierer ist das auch schonmal irgendwo intern verendet