[C] dynamische Arrays in Funktionen

[DerPaul (Gast)]

Hallo Leute ich brauche mal eure Hilfe...

Ich schreibe bin dabei ein Programm zu schreiben, bei dem ich Werte über 
die Tastatur einlese und in ein Array schreibe. Das Array soll dynamisch 
sein! Wenn ich einen neuen Wert über die Tastatur einlese soll das Array 
um ein Element vergrößert werden...

Leider habe ich etwas Probleme! Wie schreibe ich die Werte in das 
Element? Wie schaffe ich es die mit realloc() erstellten pointer adresse 
wieder zurück zu geben?

So weit habe ich das jetzt, funktioniert aber nicht richtig...

Vielleicht kann mir jemand ein paar Lösungsansätze geben...

//main
int main(void)
{
  double *werteX = NULL;
  double *werteY = NULL;

  werteX = calloc(ARRAYSIZE, sizeof(double));
  werteY = calloc(ARRAYSIZE, sizeof(double));
  
    
  hauptmenue(&werteX, &werteY);

    
}

//Hauptmenü
void hauptmenue(double **werteX, double **werteY){
  int auswahl, anzahl = 0;

  do{
      
    printf("\n"
      "Berechnung einer Ausgleichsgeraden\n"
      "==================================\n"
      "(1) Daten aufnehmen\n"
      "(2) Daten ausgeben\n"
      "(3) Daten korrigieren\n"
      "(4) Daten speichern\n"
      "(5) Daten laden\n"
      "(6) Ausgleichsgerade ermitteln\n"
      "(0) Ende\n"
      "\n"
      "Treffen Sie eine Wahl: "    
      );

    scanf("%i", &auswahl);
    while (getchar() != '\n');
  
    switch (auswahl){
    case 1: anzahl = menue1(&werteX, &werteY); break; 
    case 2: menue2(werteX, werteY, anzahl); break;
    case 3: menue3(werteX, werteY, anzahl); break;
    case 4: menue4(werteX, werteY, anzahl); break;
    case 5: anzahl = menue5(werteX, werteY); break;
    case 6: menue6(werteX, werteY, anzahl); break;
    default: break; 
    }

  } while (auswahl != 0);
  
}


//Menü Daten aufnehmen 
int menue1(double **werteX, double **werteY){
  int counter = 0;

  double *test1, *test2;
  

  printf("\n"
    "(1) Daten aufnehmen: (Ende ersten Punkt noch einmal eingeben)\n"
    "____________________\n"
    "\n");

  do{  
    //X-Koordinate einlesen  
    printf("%i. Punkt: x: ", counter + 1);
    scanf("%lf", &werteX[counter]);
    while (getchar() != '\n');

    //Y-Koordinate einlesen  
    printf("%i. Punkt: y: ", counter + 1);
    scanf("%lf", &werteY[counter]);
    while (getchar() != '\n');

    counter++;

    if (counter >= ARRAYSIZE){
      test1 = realloc(*werteX, (counter + 1) * sizeof(double));
      test2 = realloc(*werteY, (counter + 1) * sizeof(double));

      if (test1 != NULL && test2 != NULL){
        
        *werteX = test1;
        *werteY = test2;
      }
      
    }
    
  } while (counter <= 1 || werteX[0] != werteX[counter - 1] || werteY[0] != werteY[counter - 1]);

  return counter - 1;
}

[Bitflüsterer (Gast)]

>So weit habe ich das jetzt, funktioniert aber nicht richtig...

So? Was genau funktioniert denn nicht richtig? Woran stellst Du das 
fest?

Aber, klar ist, dass Du dem Arrays erst einen Wert zufügst, und dann 
seine Grösse erhöhst. Nimm mal an, Du ruft die Funktion das erstemal 
auf. Wohin wird der Wert, den Du mit scanf liest, denn gespeichert? Da 
existiert ja noch garkein Array!

[Bitflüsterer (Gast)]

Ups. Da steht ja ein alloc. OK. Das Problem tritt also die ersten 
ARRAYSIZE noch nicht auf. Aber beim ARRAYSIZE + 1 dann.

[Bitflüsterer (Gast)]

Sorry. Irgendwie habe ich mich nochmal versehen. Im Code wird gibts ja 
das realloc wenn der counter >= ARRAYSIZE wird.

Hm. Dann beschreibe mal wie der Fehler genau aussieht. Es sollte mit dem 
Code kein Problem geben.

[Bitflüsterer (Gast)]

Blöde lange Leitung.

Das hier wird das Problem sein.

... (... **werteY ...)
scanf("%lf", &werteY[counter]);

Da bildest Du ein weiteres Mal die Address von WerteY. Du erhälst also:

double ***WertY

Die Array-Index-Klammern dereferenzieren wieder eine Ebene. Und Du 
erhälst.

double **WertY

Da ist aber garkein Speicherplatz für doubles, sondern erstmal nur 
wieder ein Zeiger (auf einen Zeiger).

Vielmehr musst Du also

scanf("%lf", (*werteY)[counter]);

schreiben.

Tut mir leid wegen meiner falschen Antworten.

[DerPaul (Gast)]

Danke für die schnelle Antwort!!

Also ich habe es so probiert,...

scanf("%lf", (*werteY)[counter]);

Aber dann bekomme ich einen Speicherzugriffsfehler...

Wenn ich

scanf("%lf", &(*werteY)[counter]);

Schreibt er immerhin das erste Element, aber im 2. Durchgang schreibt 
der nicht mehr! Bei einer wiederholten eingabe der ersten beiden Werte 
soll er aus der Schleife rausspringen - macht er aber nicht. Im Debugger 
ändert sich der Wert für werteX und werteY auch nicht...

[Bitflüsterer (Gast)]

Ich fürchte ich habe heute einen Knoten im Gehirn. Ich halte mich lieber 
raus. Sorry.

[DerPaul (Gast)]

Schade,... wäre dir trotzdem sehr dankbar wenn du versuchst zu 
helfen...! Hast ja um Welten mehr Ahnung davon als ich...

Und wenn nicht,.. Trotzdem Vielen Dank!!!

[Dirk B. (dirkb2)]

@Bitflüsterer
Es ist nicht dein Gehirn, sondern der Auruf von menue1.
Die Typen passen nicht zusammen.

void hauptmenue(double **werteX, double **werteY){
...
   case 1: anzahl = menue1(&werteX, &werteY); break; 
...
}

int menue1(double **werteX, double **werteY){
....