Probleme bei einer alten Prüfungsaufgabe

vandread

Vielen Dank für die ganzen Tipps!

Ich habe den Code etwas angepasst:

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

char* grosseSchrift(char *c);

int main (void){
	char *s = "Herr Der Ringe\0";
	char *newS;

	newS = grosseSchrift(s);

	printf("*s = %s \n\n", s);
	printf("*newS = %s \n\n", newS);

	return 0;
}

char* grosseSchrift(char *c) {
	int i = 0;
	char *s= (char*)malloc(strlen(c) * sizeof(*c));
	char *tmp = s;

	while (*c != '\0') {
		*tmp = toupper(*c);
		tmp++;
		c++;
	}

	*tmp = '\0';

	return s;
}

Allerdings bin ich mir jetzt wegen dem free() nicht sicher,
also ich weiß jetzt nicht wo ich das genau setzen soll...

Es in der Funktion zu setzen würde doch keinen Sinn machen oder?
Ich würde ja somit meinen String den ich gerade dort geschrieben habe wieder "löschen" also frei geben...

Also würde sich einfach in der main nach dem printf ein free(newS) setzen...
Macht das Sinn?

CStoll

vandread schrieb:

Es in der Funktion zu setzen würde doch keinen Sinn machen oder?
Ich würde ja somit meinen String den ich gerade dort geschrieben habe wieder "löschen" also frei geben...

Gut erkannt

Also würde sich einfach in der main nach dem printf ein free(newS) setzen...
Macht das Sinn?

Ja, genau dort gehört das free() hin - nachdem du den zurückgegebenen String nicht mehr benötigst, räumst du ihn wieder auf.

Du forderst immer noch zuwenig Speicher an.

Der Leerstring "" besteht aus einem Zeichen ('\0') hat aber die Länge 0

vandread

@CStoll
danke! (:

@DirkB
also muss ich noch ein byte dazu holen?!
also:
char *s= (char*)malloc(strlen(c) * sizeof(*c) + 1);

Wenn du schon mit sizeof(*c) arbeitest, bin ich auch pingelig:

char *s= (char*)malloc( (strlen(c) + 1 ) * sizeof(*c));

Bei deiner Formel gehst du davon aus, dass ein *c ein Byte groß ist. Und dann kannst du auch auf den Ausdruck verzichten.

vandread

Alles klar tut mir Leid (:

Hier also der fertige Code:

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

char* grosseSchrift(char *c);

int main (void){
	char *s = "Herr Der Ringe\0";
	char *newS;

	newS = grosseSchrift(s);

	printf("*s = %s \n\n", s);
	printf("*newS = %s \n\n", newS);

	free(newS);

	return 0;
}

char* grosseSchrift(char *c) {
	int i = 0;
	char *s= (char*)malloc((strlen(c) + 1) * sizeof(*c));
	char *tmp = s;

	while (*c != '\0') {
		*tmp = toupper(*c);
		tmp++;
		c++;
	}

	*tmp = '\0';

	return s;
}

Prima!

Noch ein paar Anmerkungen:
Bei char *s = "Herr Der Ringe\0"; brauchstdu das \0 am Ende nicht.
Das wird bei Stringliteralen (Text zwischen ") automatisch vom Compiler eingefügt.

Und du kannst den Warnungen vom Compiler noch nachgehen.
Z.B. Variable i deklariert aber nie benutzt.

vandread

Ups das i stammt noch aus einigen Versuchen, das muss natürlich weg...

Gilt das mit \0 eigentlich auch bei Arrays?
Also bei einem Pointer bin ich mir sicher das es automatisch gesetzt wird,
aber wird es beispielsweise bei:
char s[] = "Das ist ein Text";
auch gesetzt?

DrakoXP

ja

Das gilt für "Stringliterale".
Stringliterale sind Texte zwischen Anführungszeichen.
In deinem Beispiel eben gerade ist das "Das ist ein Text" ein Stringliteral.