[Programmiererstammtisch] "Zum ächzenden Compiler"

[alpha civ]

Und was war jetzt das Problem mit insert? Weil einfügen tust du mit deiner Lösung nichts.

[EpicFail]

Also ja dann habe ich mich wohl falsch ausgedrückt dies bezüglich

Insert funktioniert dann auch mit dem dereferenzieren, zumindest meckert der Compiler nicht.

	auto it = (*test)[0].begin();
	(*test)[0].insert(it, b);

[Mr. X]

Sieht doch gut aus. Ich finde deine <vector<vector>>-Konstruktion interessant. Brauchst du tatsächlich einen Vektor beliebiger Länge, gefüllt mit Vektoren beliebiger Länge?

[Gobl]

Hi,
ich stehe irgendwie total aufm Schlauch. Wir sollen ein Programm schreiben, dass einen String einliest, auf den Buchstaben char c überprüft und den restlichen Teil des Strings im Doppelpointer **rest an die Stelle *rest speichern.
Ich hab jetzt über eine Stunde blöd rumprobiert, aber das Programm stürzt jedes mal ab... Habs auch schon ohne den Doppelpointer probiert, aber irgendwie bekomm ich das einfach nicht hin
Kann mir irgendjemand sagen, was falsch ist?

#include <stdio.h>
#include <stdlib.h>

int split(char * w, char c, char **rest);

int main(void){
	char c = 'a';
	char *w = "Hallo";
	char **rest;
	int i;
	
	printf("*w: %s\n", w);
	i = split(w, c, rest);
	printf("%i\n", i);

	if(i == 1){
		printf("**rest: %s\n", rest);
	}

	return 0;	
}

int split(char * w, char c, char **rest) {
	int i = 0;
	while (w[i] != c && w[i] != '\0')
		++i;
	if (w[i] == c) {
		w[i] = '\0';
		if(rest != NULL){
			rest = &w[i + 1];
		}
	
		return 1;
	}
	else
		return 0;
}

Edit: char**rest wird natürlich deklariert

[Zulan]

Hi,
ich stehe irgendwie total aufm Schlauch.

Als Erstes, kompiliere dein Programm mal mit Warnungen, z.B. -Wall -Wextra bei gcc/clang... Dann solltest du schon mal weiter kommen.

[Gobl]

Ich befürchte Hilfe zur Selbsthilfe bringt nichts mehr

Hab jetzt die zwei Warnungen mit aufgenommen und es wurde nur angemerkt, dass in printf("**rest: %s\n", rest); rest falsch ist und **rest uninitialisiert verwendet wird. Erstes hab ich ausgebessert, fürs zweite finde ich schonmal keine passende Lösung. Alles was ich finde sind immer nur Pointer auf bestehende Matrizen, aber hier soll ja gar nichts wirklich initialisiert werden

[Schlumpf]

Du versucht sie pointer adresse zu printen und und nicht den String, da rest ein Doppelpointer ist.

[Gobl]

Du versucht sie pointer adresse zu printen und und nicht den String, da rest ein Doppelpointer ist.

printf("**rest: %s\n", *rest); Hatte ich verbessert, wie geschrieben. Das war das einzige das ich anpassen konnte.
Der Fehler muss aber schon in der split-Funktion auftreten, das die erste print-Anweisung noch ausgegeben werden kann, aber dann das Programm abstürzt, wenn split() aufgerufen wird

[fuchs87]

&w[i+1] ergibt als Ergebnis die Adresse einer char-Variablen.

[Strat]

Wie kriegt du das Programm überhaupt zum Laufen ? Mein Compiler weigert sich zu kompilieren wegen Zeile 30

[Gobl]

&w[i+1] ergibt als Ergebnis die Adresse einer char-Variablen.

Aber ich möchte ja die Adresse speichern? Ich sehe nicht was falsch ist. Vielleicht ist es total offensichtlich, aber so wie ich mich da mittlerweile reingesteigert habe, werd ich das nicht mehr sehen. Kann denn jemand mir bitte sagen, was dort zu verbessern ist?

@Strat: Keine Ahnung, es läuft einfach

[Schlumpf]

Du versuchst einem pointer auf einen char pointer die Adresse des chars zu geben. w[i+1] dereferenziert das array und dann referenzierst du ihn nur einmal weswegen rest da ein normaler char pointer sein müsste und nicht ein pointer auf einen char pointer.

[Gobl]

Das ist aber diese tolle Aufgabenstellung hier^^ Wir müssen einen Doppelzeiger verwenden

Edit: Hier mal die Aufgabenstellung:
a) (**, 4 Minuten)
Betrachten Sie die folgende Funktion aus Kapitel 13 der Vorlesung:
char * split(char * w, char c) {
int i = 0;
while (w[i] != c && w[i] != '\0')
++i;
if (w[i] == c) {
w[i] = '\0';
return &w[i + 1];
}
else
return NULL;
}
Implementieren Sie eine abgeanderte Version dieser Funktion mit dem Prototypen
int split(char *w, char c, char **rest), die
 1 zuruckgibt, falls c in w vorkommt,
 0 zuruckgibt, falls c in w nicht vorkommt,
 die Adresse des abgeteilten zweiten Teils von w nicht zuruckgibt, sondern an der Stelle
*rest speichert, falls fur rest nicht NULL ubergeben wurde.

[Ramkhamhaeng]

Oben fehlt bei dir der Stern vor rest (= &w[i+1] oder auch w+(i+1))


Edit: Im printf weiter oben ebenfalls.
Edit2: Wobei in main noch mehr schief gegangen ist, wie Quietsch mittlerweile schrieb. Das dortige rest darf so nicht übergeben werden, sondern die Adresse auf seine Position.
Also statt char **rest; ... ; split(..., rest); muss es char *foobar; ...; split(..., &foobar); sein.

[fuchs87]

#include <stdio.h>
#include <stdlib.h>

int split(char * w, char c, char **rest);

int main(void){
	char c = 'a';
	char *w = "Hallo";
	char *rest;
	int i;

	printf("*w: %s\n", w);
	i = split(w, c, &rest);
	printf("%i\n", i);

	if (i == 1){
		printf("**rest: %s\n", rest);
	}

	return 0;
}

int split(char * w, char c, char **rest) {
	int i = 0;
	while (w[i] != c && w[i] != '\0')
		++i;
	if (w[i] == c) {
		if (rest != NULL){
			 *rest = &w[i + 1];
		}

		return 1;
	}
	else
		return 0;
}