Wieso geht das nicht

[Anfänger 7]

Hab das jetzt mal so gemacht, geht aber immer noch nicht richtig:

cout<<"Sortiert"<<endl;    
int z=0;
for (int k=i-1;k>0;k--)
{
for (int j=0;j<k;j++)
{if (n[j].autor<n[j+1].autor)
  { 
   swap (n[j],n[j+1]);

                             /*                          
  h=n[j+1];
  n[j+1]=n[j];
  n[j]=h;

cout<<"--------"<<endl;
cout<<n[z].autor<<endl;
cout<<n[j].autor<<endl;
 */
}//IF  
}
}

[leo aka qsch]

ist die Anzahl der Elemente sicher richtig??
also in deinem Fall die Variable "i"

[Anfänger 7]

Ja, i= die Zählvariabel di beim ersten schleifen durchlauf mit läuft:

while (x==1)
    {n[i].id=i+1;
    cout<<"Titel: "<<endl; 
cin>>n[i].titel;
//scanf("%s",n[i].titel);    
    cout<<"Autor: "<<endl;
    cin>>n[i].autor;

      cout<<"Verlag: "<<endl;
   cin>>n[i].verlag;

      cout<<"Preis: "<<endl;
    cin>>n[i].preis;

    cout<<"Noch einen Datensatz hinzufügen? 0 = Nein , 1= Ja"<<endl;
    cin>>x;
    if (x==1) {i++; //system("CLEAR");
    } 
     }

[chrigi]

if (n[j].autor<n[z].autor)

hast du schon überlegt, was genau du hier vergleichst?

[Anfänger 7]

Dachte das bedeutet folgendes:

Wenn n[j].autor alphabetisch nach n[j+1].autor kommt.

Stimmt das nicht?

Aber wenn ich das ">" schreibe gehts auch nicht richtig...

[chrigi]

du vergleichst 2 variablen des typs const char*, also 2 adressen.
mach dich mal über strcmp(...) schlau!

[Anfänger 7]

Ja du hast recht ist mir jetzt auch aufgefallen.
Habs jetzt so umgeändert:

struct buch{
       int id;
       string autor, titel, verlag;
       float preis;       
       };

Geht aber immer noch nicht.

[chrigi]

for (int k=i-1;k>0;k--)
{
for (int j=0;j<k;j++)
{

die zweite schleife wird nie betretten...

[Anfänger7]

das stimmt nicht ganz,

irgendwie sortiert er da schon was.

Wie würde das richtig kommen?

[leo aka qsch]

die zwei Schleifen sind richtig,
aber wie chirgi schon sagte das vergleichen der Objekte stimmt nicht
habe das leider nicht gesehen hab etwas ueberhastet daruebergelesen

du musst die Objekte mit strcmp vergleichen bzw wenn du die std::string Klasse verwendest kannst du einfach denn Operator verwenden

und das austauschen muss natuerlich dann auch anders gemacht werden
bei cstrings zb: mit strcpy o memcpy

ok?

[chrigi]

hatte nicht gemerkt, dass die schleifen bereits korrigiert wurden... (und dabei habe ich die bereits korrigierte version gequotet! )

[abc-Gast]

Was bedeutet std::???

[Gier]

Kann mier mal jemand die Bedeutung von
"using namespace std;" erklären?

[terraner]

Was bedeutet std::???

Gier schrieb:

Kann mier mal jemand die Bedeutung von
"using namespace std;" erklären?

namespaces

mfg