Problem mit Doppelpointer (für fortgeschrittene)

Greekboy

es wird nichts mit access violation ausgegeben, aber das Programm stürzt einfach ab, und das lässt mich vermuten, dass es eine access violation sein muss.

das hässliche ist, dass ich devc++ verwende und dass das debugging nicht geht. kennt ihr eine andere IDE, die freeware und besser ist? (mit guter debugging funktion?)

simon.gysi

Visual Studio 2008 Express Edition.

hellihjb

Du uebergibst eine Kopie der Adresse auf die *"char *Inhalt" zeigt an Deine funktion "getrow".
Alles was Du dann in "getrow" damit anstellst, hat keinen Einfluss auf das tatsaechliche "Inhalt".

Vorschlag:

char** getrow(char Str[], int AnzahlZeilen, int AnzahlZeichen)
{
  char **Inhalt;
  ...
  return Inhalt;
}

oder:

getrow(String, &Inhalt, Zeilen, ZeichenProZeile);
...
void getrow(char Str[], char ***Inhalt, int AnzahlZeilen, int AnzahlZeichen) 
{
  *Inhalt = (char **)malloc(AnzahlZeilen * sizeof(char *)); 
  ...
}

darueber hinaus solltest du sicherstellen, dass hier:

Inhalt[i][j] = Str[ZeichenNr];

0<=i<AnzahlZeilen und 0<=j<AnzahlZeichen gilt.

Greekboy

Hab deinen 2. Codevorschlag veruscht.

demnach müsste das allokieren der 2. Dimension so lauten:

for(i = 0; i < AnzahlZeilen; i++) 
    {
		**(Inhalt + i) = (char *)malloc(AnzahlZeichen * sizeof(char));
	}

da stürzt aber das programm ab, nachdem i den wert 3 erreicht... hab ich da nen fehler gemacht?

für den 2. Vorschlag hab ich die schleife so ergänzt, zur sicherheit:
for( i = 0, ZeichenNr = 0; Str[ZeichenNr] != '\0' || i < AnzahlZeilen; i++, ZeichenNr++)

Übrigens: Danke für die tolle Hilfe, die hier geleistet wird!

Greekboy

hab mal das ganze Programm hochgeladen ins web, damit es auch mal ausprobiert werden kann...
so findet man evtl. leichter den fehler...

http://www.loaditup.de/179493.html

Greekboy

hat keiner einen neuen Verbesserungsvorschlag

MFK

In deiner getstr-Funktion holst du zu wenig speicher für Str:

Str = malloc( sizeof(laenge) );

Das sizeof ist hier falsch. In laenge steht, wie lang die Datei ist. sizeof( laenge ) ergibt aber nur die Größe einer size_t-Variablen.

In getrow greifst du falsch auf Inhalt zu:

**(Inhalt + i)

Damit inkrementierst du Inhalt selbst, den Zeiger auf dein char**. Dort steht aber nichts. Du musst das inkrementieren, auf das Inhalt zeigt:

*((*Inhalt) + i) = malloc(AnzahlZeichen * sizeof(char));

Und hier stolperst du über die Operatorreihenfolge:

*Inhalt[i][j] = Str[ZeichenNr];

[] bindet stärker als *. Du musst also klammern:

(*Inhalt)[i][j] = Str[ZeichenNr];

Greekboy

ok, es geht.

im 1. allokieren, hattest du einen kleinen fehler noch drin gehabt (1 * zu viel) (auf mein verschulden vlt.)

das mit getstr stimmt, aber komischerweise funktionierts auch, wenns falsch ist..

danke!

Greekboy

und noch ne frage zum Verständniss:

In "Inhalt" steht die Adresse der Variablen.

Also kann ich **(Inahlt + i) schreiben, da ich ja lediglich die Adresse um i erhöhe.

genauso wie *((*Inhalt)+i) den Inhalt von *Inhalt um i erhöht (also auch die Adresse um i) und dann dereferenziert wird.

Greekboy

Jetzt wollte ich die Zeilenlänge variabel machen, und dafür die einzelnen Zeilenlängen in einem dynamischen Array speichern, welches einer Funktion per Call-by-Reference übergeben wird, doch da kommt folgende Fehlermeldung in der Funktion: "assignment makes pointer from integer without a cast"

void ReadDocLength(int *AnzahlZeilen, int **AnzahlZeichen, char Str[])
{			
		int ZeichenNr = 0;
		char Zaehler;
		int i = 0;

		/* Anzahl der Zeichen lesen und in "AnzahlZeichen" speichern */

	for(ZeichenNr = 0, i = 0, *(AnzahlZeichen+0) = 0, Zaehler = 1; Str[ZeichenNr] != '\0'; ZeichenNr++)
	{
	    *(AnzahlZeichen+i) = Zaehler ;  // DIESE ZEILE HAT DEN FEHLER
		Zaehler++;

printf("\nAnzahlZeichen[i]: %i", *(AnzahlZeichen+i));

		if(Str[ZeichenNr] == '\n')
		{
			i++;
			Zaehler = 0;
		}	
	}	

system("pause");