#include <iostream>
#include <string>

using std::cout;
using std::endl;
using std::cin;
using std::string;

int main()
{
   string option1("Hallo Welt!");
   string option2("Hallo Welt!");
   if (option1 == option2)
      cout << "Die Strings sind gleich!" << endl;  // lib stil
   const char *coption1("Hallo Welt!");   
   const char *coption2("Hallo W2lt!");
   int erg(0);
   for (int vergleich(0); vergleich != strlen(coption1)+1 && vergleich !=   strlen(coption2)+1;
        vergleich++) {
      if (*coption1 == *coption2) {
         coption1++; coption2++;
         erg = 5;
      }
      else {
      erg = 3;
      break;
      }
   }
   if (erg == 5)
   cout << "Beide Strings sind gleich!" << endl;
   if (erg == 3)
   cout << "Die Strings sind ungleich!" << endl;  //cstring variante
   system("PAUSE");
   return 0;
}

Warum gibt das Programm aus das die Strings gleich sind obwohl
sie das nicht sind? Wenn ich bei "Hallo Welt" nur bei Hallo
was ändere dann wird ausgegeben das es ungleich ist.
Aber nach dem Leerzeichen sagt es immer das die Strings gleich sind.
Obwohl sie das nicht sind.
Vielen Dank
Flashgott

EDIT: Ja ich weiß das es eine Funktion gibt, um cstrings zu vergleichen.
Aber dieses mal brauche ich es so.

const char *coption1("Hallo Welt!");   
const char *coption2("Hallo W2lt!");

und nicht so

char *coption1("Hallo Welt!");   
char *coption2("Hallo W2lt!");

Flashgott schrieb:

ah wollte noch Fragen warum man hier ein const davor setzten sollte

const char *coption1("Hallo Welt!");

und nicht so

char *coption1("Hallo Welt!");

Dazu ganz schnell: Mit diesem Statement wird lediglich ein Pointer bereitgestellt, der auf einen Speicherbereich verweist, in dem "Hallo Welt" steht. Dieser Speicherbereich ist konstant (sprich: Der C++-Standard garantiert definiertes Verhalten nur, solange man den Inhalt dieses Speichers nicht verändert).
Als C-Altlast wird leider an dieser Stelle kein const gefordert (IIRC), aber es ist trotzdem hochgradig ratsam, das zu tun, um z.B. eine unbeabsichtigte Änderung zu hindern.

Um Längen besser ist da schon Dein ursprünglicher Ansatz mit

Flashgott schrieb:

string option1("Hallo Welt!");
   string option2("Hallo Welt!");

Nun zu Deiner ersten:
WENN Du schon mit C-Strings arbeitest und str-Funktionen nutzt (wie strlen()) .... warum dann nicht auch strncmp() ?
(die strn..-Funktionen sind sowieso besser als die "ohne n").

Gruß,

Simon2.

Edit: Und wie kommst du überhaupt auf so merkwürdige Werte wie 6 für "gleich" und 3 für "verschieden"?

Hast du mal im Debugger durchgesehen, was während der Schleife eigentlich passiert?

Edit: Und wie kommst du überhaupt auf so merkwürdige Werte wie 6 für "gleich" und 3 für "verschieden"?

hm du meinst sicher 5 für "gleich".
Das sind nur 2 zufällige Werte um es zu vergleichen.
Ist doch egal ob ich jetzt 1 2 3 oder 4 nehme?
Mit dem Debugger habe ich noch keine Erfahrungen.

bool equal = true;
for (...) {
    if ( /*strings an der aktuellen Stelle ungleich*/ ) {
        equal = false;
        break;
    }
}
if ( equal )
    ....
else
    ....

CStoll schrieb:

Hast du mal im Debugger durchgesehen, was während der Schleife eigentlich passiert?

Edit: Und wie kommst du überhaupt auf so merkwürdige Werte wie 6 für "gleich" und 3 für "verschieden"?

hm du meinst sicher 5 für "gleich".
Das sind nur 2 zufällige Werte um es zu vergleichen.
Ist doch egal ob ich jetzt 1 2 3 oder 4 nehme?

Ja, ich meinte 5. Und es ist zumindest ungewohnt - normal verwendet man für eine Ja/Nein-Aussage nicht zwei willkürliche Zahlenwerte, sondern einen bool (true oder false).

Mit dem Debugger habe ich noch keine Erfahrungen.

Dann solltest du das mal machen (btw, was für einen Compiler verwendest du eigentlich?)

Für den Anfang kannst du auch vor der if()-Anweisung eine Ausgabe einfügen, damit du den Überblick behältst:

cout<<vergleich<<". Vergleich: "<<*coption1<<"-"<<*coption2<<endl;

if (*coption1 == *coption2) {
         cout << *coption1 << endl;
         coption1++; coption2++;
         erg = 5;
      }

Es wird nur

H
a
l
l
o

ausgegeben

und wenn ich es so wie du mache

cout<<vergleich<<". Vergleich: "<<*coption1<<"-"<<*coption2<<endl;

wird nur H-H A-A L-L O-O -
ausgegeben

Wozu brauchst du überhaupt die Variable 'vergleich'? Entweder du verwendest sie als Index bei den späteren Vergleichen - oder du nutzt die String-Literale direkt, um das Ende festzustellen:

int erg = true;
for(int vergleich=0;vergleich!=strlen(coption1)+1&&vergleich!=strlen(coption2);++vergleich)
{
  if(coption1[vergleich]!=coption2[vergleich])
  {
    erg=false;
    break;
  }
}

//oder

int erg = true;
for (; *coption1!='\0' && *coption2!='\0';)
{
  if (*coption1 == *coption2)
  {
    coption1++; coption2++;
  }
  else
  {
    erg = false;
    break;
  }
}

#include <iostream>
#include <string>

using std::cout;
using std::endl;
using std::cin;
using std::string;

int main()
{
  const char *lies1 = 0;
  string string1;
  cin >> string1;
  for (int i = 0; i < string1.size(); i++) {
     *lies = string1[i];
     lies++, 
  }
  cout << "Es wurde ";
  for (int i = 0; i != strlen(lies1)+1; i++)
     cout << *lies;
  cout << "eingeben!" << endl;
  system("PAUSE");
  return 0;
}

Das programm soll statt strings cstrings verwenden,
um etwas einzulesen und das dann auszugeben.
Also mache ich erst cin >> in einen normalen string
und dann tausche ich es in einen cstring um.
Kann es aber nicht kompilieren!
no match for operator * in lies *lies = string1[i];

Zweitens: In einen "const char*" darfst du nichts reinschreiben - hier brauchst du einen nicht-konstanten "char*".

Drittens: Du hast noch keinen Speicher angefordert, wo du die String-Daten reinschreiben könntest - das dürfte dir im laufenden Programm einen SegFault entgegenwerfen.

Viertens: Eventuell solltest du dich etwas intensiver mit Pointer-Arithmetik auseinandersetzen.

PS: Und fünftens: Wenn es dir nicht um den Lerneffekt geht - bleib lieber bei std::string.

char lies[100];
cin>>lies;
cout<<"Sie haben "<<lies<<" eingegeben\n";

ich muss als Übung ein Programm schreiben,...

OK, das erklärt's ... und damit ist es halt unumgänglich (ist ja schonmal erfreulich, dass in dieser Übung auch gleich std::string vorgestellt wird).

Flashgott schrieb:

Aber wenn man den Wert, also "Hallo Welt"
ändern will, muss man ja das const weglassen.

Nein!!!
Wenn Du "Hallo Welt" ändern möchtest, darfst Du es nicht in den konstanten Speicher legen, sondern selbst Speicher dafür bereitstellen ... z.B. über ein Array:

char option1[100] = "Hallo Welt!"; // erzeugt ein Array von char-s und belegt den Anfang mit "Hallo Welt" (inkl. '\0'

In den darfst Du dann nach Lust&Laune reinschreiben ... aber eben höchstens 100 Zeichen (inkl. abschließender '\0').

Gruß,

Simon2.