Tauschprogramm C problem

Nun, dann analysiere mal das Problem.

Da tauchen Zeichen auf, die Du nicht eingegeben hast. Woher?
Du kopierst aus dem anderen String! Seltsam!

Was steht denn da vorher drin? Na, das was Du ausgibst.

Das auch, aber was gibst Du aus? Wirklich den kompletten Inhalt der 
Strings?
Nein.

Du gibst mit printf aus. Was gibt printf aus? Die Zeichen bis.... Ja, 
bis wohin?

Dann kopierst Du. Was kopierst Du? Was in den Strings steht. Was steht 
in den Strings? Siehe den vorherigen Punkt?

Wieviel kopierst Du? Na alle Zeichen. Wieviele sind das? Wieviele 
Kopieroperationen führst Du aus?

Michael Hakenkötter schrieb:
> int main(void)
> { char satz1[80]="";
>   char satz2[80]="";

mach die Arrays mal um 1 grösser.

char satz1[81]="";
char satz2[81]="";

Deine tauschschleife ist um eins zu lang!
Du greifst ausserhalb des Arrays auf den Speicher zu.

Trotzdem ist das nicht schön programmiert!
Fazit:
weiterlernen und dran denken, dass ein String immer noch eine Null am 
Ende braucht.

@  Michael Hakenkötter
Hast Du denn auch verstanden, was das Problem dabei ist?

Thomas R. schrieb:
> Trotzdem ist das nicht schön programmiert!
> Fazit:
> weiterlernen und dran denken, dass ein String immer noch eine Null am
> Ende braucht.

Ich lehne mich mal sehr weit aus dem Fenster und behaupte das

1

gets(satz1);

2

gets(satz2);

jeweils eine implizite Null Terminierung machen.

Wenn man also bei

1

char satz1[80];

nicht mehr als 79 Zeichen eingibt, haben wir da kein Problem.
Die Schleife muß natürlich wie bereits erwähnt korrigiert werden.

AAABER:

1

void tausch(char *satz1, char *satz2)

2

{...}

3

und dann

4

tausch(&satz1, &satz2);

das ist übel und sollte:

1

tausch(satz1, satz2);

heißen.
Mit Pointern treibt man kein Schindluder!

Mit welchem Compiler compilierst du das Programm? Wenn es gcc ist, dann 
verwende unbedingt die Option -Wall und am besten noch -Werror. (Alle 
Warnungen anschalten und Warnungen als Fehler betrachten.) Dann passiert 
sowas wie mit den falschen Zeigern nicht.

>Ich arbeite mit Dev-Cpp.. der hat keine warnung mehr rausgeschmissen...

Genau deswegen sollst Du ja die Warnungen einschalten. Wenn es trotz 
eingeschalteter Warnungen keine gibt, dann schmeiss den Compiler weg. Er 
wäre schlicht Schrott.

Michael Hakenkötter schrieb:
> Ich arbeite mit Dev-Cpp.. der hat keine warnung mehr rausgeschmissen...

Der basiert doch auf gcc (Mingw).

Das interessiert mich jetzt mal, Augenblick...
Hier unter Linux:

1

Norbert:/tmp$ gcc x.c

2

x.c: In function ‘main’:

3

x.c:32: warning: passing argument 1 of ‘tausch’ from incompatible pointer type

4

x.c:4: note: expected ‘char *’ but argument is of type ‘char (*)[80]’

5

x.c:32: warning: passing argument 2 of ‘tausch’ from incompatible pointer type

6

x.c:4: note: expected ‘char *’ but argument is of type ‘char (*)[80]’

7

/tmp/ccWBJ5HR.o: In function `main':

8

x.c:(.text+0xc0): warning: the `gets' function is dangerous and should not be used.

und da sind die Warnungen noch nicht mal explizit eingeschaltet.

>weiterlernen und dran denken, dass ein String immer noch eine Null am
>Ende braucht.
Hat aber mit dem Problem hier überhaupt nichts zu tun.