Problem mit while und for-Schleife

[Byte_Breaker]

Hallo ihr,

hier ist wieder mal ein Fehler drin, den ich nicht finde

Es sollen die Werte innerhalb des Array's s geordnet werden.

Zugegebennermaßen ist dies nicht wirklich sinnvoll, aber es soll mir zum Verständnis dienen. Ich lerne gerade C.

Hier der Quelltext.

Der Fehlerteufel wird sich wahrscheinlich irgendwo in der while oder der for-Schleife eingeschlichen haben. Aber ich bekomme echt nicht raus was da falsch läuft...

#include <stdio.h>

#include <conio.h>


char s[100]="Hier steht ein Text\0";

char *zeiger;

int change(int a, int ;

int i;

int j;

int rueck;

int lauf;

int temp=0;


main()

{

clrscr();



/* Dient nur zur Ermittlung, wie viele Felder das Array hat 

(und druckt die Feldnummern aus): */


printf("%s\n\n",s);

printf("Dieser String beinhaltet\n");

for(i=0; s[i]!='\0'; i++)

{printf("%s %d","  ",i);};

printf(" diese Felder");




/* Hier sollen die Werte geordnet werden */

j=0;

rueck=0;

while(rueck==0) 

/* wird die Funktion change in der for-Schleife aufgerufen, gibt change den Wert Null zurück 

damit while wieder durchlaufen wird */

{

for(rueck=1; j<=i-1; j++)

   {

	if (strcmp(s[j], s[j+1]) > 0)

	{

	rueck=change(j, j+1); /* Die Funktion change soll die Werte vertauschen, falls ungeordnet, 

gibt den Rückgabewert Null, damit while-Schleife neu durchlaufen wird */

	}

   };

}




printf("\n\n%s\n\n",s);



getch();

}


int change(int a, int 

{

char k[30];

strcpy(k,s[a]); strcpy(s[a],s[b]); strcpy(s[b],k); /* soll Felder im Array vertauschen */

return(0);

}


[/code]

[Klotzkopp]

Willst du mehrere Strings alphabetisch sortieren, oder willst du die einzelnen Zeichen in einem einzigen String sortieren? So, wie dein Code aussieht, vermischst du beides.

Wenn du die einzelnen Zeichen sortieren willst, darfst du nicht die Stringfunktionen (strcmp, strcpy) benutzen, weil die mit Zeichenketten arbeiten.

Wenn du ganze Strings sortieren willst, solltest du mehr als einen haben

Abgesehen davon musst du j innerhalb der while-Schleife zurücksetzen.

P.S.:

Das \0 in deinem Stringliteral ist überflüssig. Stringliterale werden automatisch nullterminiert.

[Byte_Breaker]

Danke, jetzt funktionierts.

Das waren die beiden Fehler.

[Sythus]

Wenn du die einzelnen Zeichen sortieren willst, darfst du nicht die Stringfunktionen (strcmp, strcpy) benutzen, weil die mit Zeichenketten arbeiten.

Und welche kann man dann benutzen? Versuche den Code nämlich auch zum laufen zu bringen - so als Übung ^^

Grüße

[Klotzkopp]

Und welche kann man dann benutzen?

Man braucht gar keine Funktionen. Man kann direkt mit dem Arrayoperator [] auf die Elemente zugreifen.

[Sythus]

Also einfach ne schleife die die einzelnen stellen im array miteinander vergleicht und ne weitere anweisung die die dann sortiert?

Das vergleichen ist nicht das problem, aber wie sortieren? Vll gleiche teile in ein neues array?

[Klotzkopp]

Die Schleife mit dem Vergleich ist doch schon im Programm drin. Du musst nur die Vergleichsbedingung und die Implementierung der change-Funktion ändern.

[Sythus]

puhhhh ^^ hier wird einem was um die Ohren gehaun xD

naja ich geb nich auf vll krieg ichs noch hin xD