Malloc - Fehler

[Daniel S. (dani2304)]

Hallo,
ich habe ein Problem mit der dynamischen Speicherverwaltung. Es 
reserviert mehr Speicherplatz als ich vorhabe

#include <stdio.h>
#include <stdlib.h>
#define GROESSE_FELD 4
#define DIM_ZEILE GROESSE_FELD * 2+1
#define DIM_SPALTE GROESSE_FELD * 2+1


void spielGeruest(char **spielfeld){
int i=0, j=0;

for (i=0;i<(DIM_ZEILE); i++){
    for(j=0;j<(DIM_SPALTE);j++){
        if(!(i%2)){
            if(!(j%2)){
                spielfeld[i][j]='|';
        }
        else{
            spielfeld[i][j]='-';

        }
        }
        else{
            if(!(j%2)){
                spielfeld [i][j]='|';
            }
            else{
                spielfeld [i][j]='x';
            }
        }
    }
}
}

void ausgabeSpielfeld(char **spielfeld){

}
int main(){
    char **Feld;
    int i;
    Feld=(char**)malloc(sizeof(char*)* DIM_ZEILE);
    for(i=0;i<DIM_SPALTE;i++){
        Feld[i]=(char*)malloc(sizeof(char)* DIM_SPALTE);
}
spielGeruest(Feld);
printf("%s",*(Feld+1));
}

Zusätzlich zum String gibt er mir noch mehr Zeichen aus.

Warum?

MfG

[AVerr (Gast)]

Fehlende Nullterminierung

[Rufus Τ. F. (rufus)]

Daniel S. schrieb:
> Feld[i]=(char*)malloc(sizeof(char)* DIM_SPALTE);

Das macht nicht das, was Du erwartest.

> sizeof(char)* DIM_SPALTE

Wieviel mag das sein?

> #define GROESSE_FELD 4
> #define DIM_SPALTE GROESSE_FELD * 2+1

Also:

> sizeof (char)  4  2 + 1


Durch die Verwendung von Klammern im #define-Ausdruck ließe sich das 
Problem beseitigen.


Davon abgesehen

> Feld=(char**)malloc(sizeof(char*)* DIM_ZEILE);
> for(i=0;i<DIM_SPALTE;i++){
>   Feld[i]=(char*)malloc(sizeof(char)* DIM_SPALTE);

ist hier das Schleifenabbruchkriterium falsch, da sollte nicht 
DIM_SPALTE, sondern DIM_ZEILE stehen. Das macht hier zwar nichts, da 
beide gleich groß sind, aber wenn sie es nicht mehr sind (weil die 
Feldgrößen geändert werden), knallts.


Im übrigen wird C-Sourcecode durchaus lesbarer, wenn man an sinnvollen 
Stellen Leerzeichen zur Trennung verwendet:

> Feld = (char **) malloc(sizeof (char *) * DIM_ZEILE);
> for (i = 0; i < DIM_SPALTE; i++)
> {
>   Feld[i] = (char *) malloc(sizeof (char) * DIM_SPALTE);

Die beiden Typecasts sind in C übrigens überflüssig; malloc gibt void* 
zurück, das lässt sich beliebigen Pointertypen zuweisen.

[Rolf M. (rmagnus)]

Und sizeof(char) ist per Definition immer 1.
Generell bietet es sich aber an, bei Malloc einfach das Zeigerziel des 
Ergebnisses für die Größe herzunehmen:

Feld = malloc(DIM_ZEILE * sizeof *Feld);

Feld[i] = malloc(DIM_SPALTE * sizeof **Feld);

So sind sämtliche Größen der Typen bei den mallocs nur noch davon 
abhängig, wie Feld definiert ist statt "hart codiert". Solltest du 
später mal auf die Idee kommen, aus den chars doch shorts oder gar 
structs zu machen, passt der Code ohne Änderung.

[Kaj (Gast)]

Daniel S. schrieb:
> Feld=(char**)malloc(sizeof(char*)* DIM_ZEILE);
In C soll man den Rueckgabewert von malloc nicht casten!

[tictactoe (Gast)]

Das hier
> #define DIM_ZEILE GROESSE_FELD * 2+1

und das hier
>     Feld=(char**)malloc(sizeof(char*)* DIM_ZEILE);

passt nicht gut zusammen.

Man schreibt die Klammern im #define (und nicht dort, wo die Konstante 
benutzt wird) und zwar genau deshalb, damit der Wert nicht den 
Vorrangregeln von Operatoren gestört wird:

sizeof(char*)* GROESSE_FELD * 2+1

ist nämlich was ganz anderes als

sizeof(char*)* (GROESSE_FELD * 2+1)

[Daniel A. (daniel-a)]

Tip: Man kann in c auch solche Dinge tun:

#include <stdio.h>

int main(){
  printf("%u\n",sizeof(int[5][2]));
  return 0;
}

Oder sowas:

#include <stdio.h>

int main(){
  const int a=5;
  const int b=2;
  printf("%u\n",sizeof(int[a][b]));
  return 0;
}

[Daniel A. (daniel-a)]

Noch ein Trick, den ich gerne anwende:

#include <stdio.h>

void func( const int x, const int y, int buf[y][x] ){
  buf[1][3] = 42;
}

int main(){
  const int x=5, y=2;
  int buf[y][x];
  func(x,y,buf);
  printf("%d\n",buf[1][3]);
  return 0;
}

Die Vorteile hier sind, dass man sich nirgens verrechnen kann und es 
kein malloc gibt das fehlschlagen könnte. Ein nachteil ist, dass es yu 
einem Stackoverflow kommen kann, wenn buf zu gross ist.

[Daniel S. (dani2304)]

Okay perfekt. Vielen Dank schon einmal für eure umfassende Hilfe :). 
Habe den Großteil schon ausgebessert.

Eine Frage hätte ich jedoch noch. Und zwar will ich in einem String den 
Wert einer Variablen abspeichern. Wie geht das?

Kenne nur die Möglichkeiten mit "..." und '...' konstante Werte 
abzuspeichern. Aber sowas wie str1[0][0]=x;
geht nicht ....

[Markus L. (rollerblade)]

Guckst Du nach sprintf.