Tauschprogramm C problem

[Michael H. (m0ddmylife)]

Hallo, habe ein Programm zum tauschen zweier "Sätze" geschrieben.
Nur habe ich leider das Problem das ich nicht weis warum beim 
Tauschergebnis der erste Buchstabe des 1.Satzes IMMER ein "H" ist... 
warum?

danke im vorraus
Michael

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

void tausch(char *satz1, char *satz2)
{   
    char hilf;
    int i;
    
    for(i=0;i<=80;i++)
    {
       hilf  = *satz1;
      *satz1 = *satz2;
      *satz2 = hilf;
       
      satz1++;
      satz2++;    
      } 
}


int main(void)
{ char satz1[80]="";
  char satz2[80]="";
  printf("\t Satzgewurste\n\n");
  printf("Bitte Satz 1 Eingeben:");
  gets(satz1);
  fflush;
  printf("Bitte Satz 2 Eingeben:");
  gets(satz2);
  fflush;
  printf("\n\nEingegebener Satz 1: %s \nEingegebener Satz 2: %s\n", satz1, satz2);
  tausch(&satz1, &satz2);
  printf("\n\nVerdrehter Satz 1: %s \nVerdrehter Satz 2: %s\n\n\n", satz1, satz2);
  
  system("PAUSE");  
  return 0;
}

[Hmm (Gast)]

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?

[Thomas R. (Gast)]

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]="";

[Dumdi D. (dumdidum)]

Ersetz mal in
    for(i=0;i<=80;i++)

<= durch ein <

[Michael H. (m0ddmylife)]

Thomas R. schrieb:
> mach die Arrays mal um 1 grösser.
>
> char satz1[81]="";
> char satz2[81]="";

Danke, das hat geholfen!

[icke (Gast)]

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

[Thomas R. (Gast)]

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

[Michael H. (m0ddmylife)]

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.

wird gemacht! ;)

[Hmm (Gast)]

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

[Michael H. (m0ddmylife)]

Jap. mit den 80 Kopierten zeichen hat er sich eines aus einem "fremden" 
speicherbereich geholt...

[Norbert (Gast)]

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

gets(satz1);
gets(satz2);

jeweils eine implizite Null Terminierung machen.

Wenn man also bei

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:

void tausch(char *satz1, char *satz2)
{...}
und dann
tausch(&satz1, &satz2);

das ist übel und sollte:

tausch(satz1, satz2);

heißen.
Mit Pointern treibt man kein Schindluder!

[... (Gast)]

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.

[Michael H. (m0ddmylife)]

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

[Hmm (Gast)]

>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.

[Norbert (Gast)]

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:

Norbert:/tmp$ gcc x.c
x.c: In function ‘main’:
x.c:32: warning: passing argument 1 of ‘tausch’ from incompatible pointer type
x.c:4: note: expected ‘char *’ but argument is of type ‘char (*)[80]’
x.c:32: warning: passing argument 2 of ‘tausch’ from incompatible pointer type
x.c:4: note: expected ‘char *’ but argument is of type ‘char (*)[80]’
/tmp/ccWBJ5HR.o: In function `main':
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.

[Vn N. (wefwef_s)]

Michael Hakenkötter schrieb:
> gets(satz1);
>   fflush;

Michael Hakenkötter schrieb:
> system("PAUSE");

Wo hast du sowas denn her? Ich würde mir definitiv einen anderen 
Lehrer/anderes Buch/anderes Tutorial suchen.

[SummerWilli (Gast)]

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

[Rolf M. (rmagnus)]

Michael Hakenkötter schrieb:
> Jap. mit den 80 Kopierten zeichen hat er sich eines aus einem "fremden"
> speicherbereich geholt...

Nein. Das Problem ist, daß du 81 Zeichen kopierst und nicht 80.

PS: Der Compiler müßte bei deinem Code übrigens mehrere Warnungen 
erzeugen. Du solltest diese verstehen und entfernen.

[Vn N. (wefwef_s)]

Michael S. schrieb im Beitrag #3118171:
> DEV-CPP wird doch nicht mehr gepflegt und soll noch noch Bugs enthalten.

Mittlerweile wird es wieder gepflegt, wenn auch nicht von Bloodshed 
(Blodshed bietet immer noch eine veraltete Version an).