#include <iostream>
#include <cstring>

using namespace std;

int main()
{
    char carray[30];
    int eingabe;
    int leben = 10;

    do
    {
        cout << "Druecken sie '1' fuer ein neues Spiel!" << endl;
        cout << "Druecken sie '2' um das Programm zu beenden!" << endl;
        cin >> eingabe;

        if(eingabe == 1)
        {
            cout << "Geben sie ein Wort ein! (Bitte nur Grossbuchstaben): ";
            cin.getline(carray, sizeof(carray));
            strncat(carray, '\0', 1);
            for(int i = 0; i < strlen(carray); i++)
            {
                cout << "-";
            }
        }
    }
    while(eingabe != 2);
    cout << "Programm wird beendet..." << endl;
    return 0;
}

Wie ihr sehen könnt ist es noch nicht ganz fertig!

Schonmal danke an euch!

Wie auch immer: Dies ist ganz offensichtlich keine Spielerei mit C-Strings um sie mal gesehen zu haben sondern ein echtes Programm an dem du bastelst. Daher die berechtigte Frage: Warum C-Strings? std::string ist schließlich in jeder Hinsicht überlegen.

Ich bin noch(wie du wahrscheinlich bemerkt hast) blutiger Anfänger und bin noch nicht mit "normalen" Strings vertraut. Aber wenn du meinst, warte ich noch etwas mit meinem Programm und befasse mich mit strings.

Du verzichtest nicht auf strings, sondern benutzt nur die einfachere, sicherere und nicht zuletzt relevantere Variante und bastelst nicht mit C-Strings rum, denn das kommt, wie du siehst nicht gut.

btw:
Schau mal hier hat jemand gerade letztens eine nette Hangman Version programmiert:
http://www.c-plusplus.net/forum/viewtopic-var-t-is-268133.html

Ich habs mal "C++-mäßig" umgeschrieben.
Ausserdem hast du eine schöne Endlos-Schleife programmiert. Darum die Variable end_loop.

mfg bernd

#include <iostream>
#include <string>

using namespace std;

int main()
{
    string wort;
    int eingabe;
    int leben = 10;
    bool end_loop = false;

    do
    {
        cout << "Druecken sie '1' fuer ein neues Spiel!" << endl;
        cout << "Druecken sie '2' um das Programm zu beenden!" << endl;
        cin >> eingabe;

        if(eingabe == 1)
        {
            cout << "Geben sie ein Wort ein! (Bitte nur Grossbuchstaben): ";
            cin >> eingabe;
        }
        else if(eingabe == 2)
        {
          end_loop = true;
        }
    }
    while(end_loop == true);
    cout << "Programm wird beendet..." << endl;
    return 0;
}

@bauerb: Wohl eher andersrum mit dem end_loop. Jetzt läuft das Spiel nämlich nur so lange wie man "Spiel beenden" wählt .

Die Originalversion vom Honky hatte eine Endlosschleife produziert, wenn man zuerst "1" (Wort eingeben) gewählt hat. Die Variable loop_end hab ich reingeschrieben, um das Problem zu beheben. Jetzt läuft es so, wie es, meiner Meinung nach, sein sollte. Man kann beliebig oft ein neues Wort eigeben (Menüpunkt 1) und wenn man keine Lust mehr hat, das Programm mit Menüpunkt 2 beenden.

Ja klar... Im eiffer des Gefechtes...

#include <iostream>
#include <string>

using namespace std;

int main()
{
    string wort;
    int eingabe;
    int leben = 10;
    bool end_loop = false;

    do
    {
        cout << "Druecken sie '1' fuer ein neues Spiel!" << endl;
        cout << "Druecken sie '2' um das Programm zu beenden!" << endl;
        cin >> eingabe;

        if(eingabe == 1)
        {
            cout << "Geben sie ein Wort ein! (Bitte nur Grossbuchstaben): ";
            cin >> wort;
        }
        else if(eingabe == 2)
        {
          end_loop = true;
        }
    }
    while(end_loop == false);
    cout << "Programm wird beendet..." << endl;
    return 0;
}

Ausserdem hast du eine schöne Endlos-Schleife programmiert.

?

strncat(carray, '\0', 1);

daran schuld ist. Ich habe es auch mit folgendem Probiert:

carray[sizeof(carray)-1] = '\0';

doch dann scheint der C-String trotzdem kein Terminierungszeichen zu enthalten, da dann strlen ins unendliche zählt.
Und ich meine als angehender (Hobby-)Programmierer sollte man sich doch mit C-Strings genau so gut aukennen wie mit den C++ Strings. Oder nich?

1. strncat funktioniert so nicht. Guck dir die Doku nochmal an. Hier mal ein paar Warnungen die mein Compiler zu dem Code hat:

test.cc: In function 'int main()':
test.cc:22: warning: null argument where non-null required (argument 2)
test.cc:22: warning: null argument where non-null required (argument 2)
test.cc:23: warning: comparison between signed and unsigned integer expressions
test.cc:10: warning: unused variable 'leben'

2. getline funktioniert so nicht. Guck dir die Doku nochmal an.

The ending null character that signals the end of a c-string is automatically appended to s after the data extracted.

Honky schrieb:

Und ich meine als angehender (Hobby-)Programmierer sollte man sich doch mit C-Strings genau so gut aukennen wie mit den C++ Strings. Oder nich?

3. Wozu? Das alles hat man in C++ überhaupt nicht nötig. Guck dir die Doku nochmal an. Ich erwarte auch nicht, dass mein Automechaniker weiß wie man einem Pferd ein Hufeisen verpasst.

OK, guter vergleich^^

ich hoff ja immernoch, dass iwann mal nen neuer toller vergleich kommt - langsam wirds langsam mit der pferd/auto-geschichte

Liegt das etwa an meiner IDE? (benutze codeBlocks)

Wie übergangen? was meinst du damit?

Nur so nebenbei solange du es einfach ausführst kann es nicht an der IDE liegen weil diese dir
unter die Arme greift in Sachen Projekt Zusammenstellung und Aufruf des Compilers. Und bei guten
noch etwas Syntax-Highligting und IntelSense aber mehr auch nicht.

Im Debugging Modus könnte es sein, das dir die IDE das falsche anzeigt, aber ich kann mich nicht
erinnern das ich das jemals Probleme mit CodeBlocks hatte und ich benutzt es schon seit langem.

Mfg marco

Naja, das Problem mit der Terminierung hab ich jetz begriffen, nur hab ich jetzt noch ein Problem. Meine for-Schleife wird einfach übergangen??? Warum?

Aktuellen Code zeigen!

Liegt das etwa an meiner IDE? (benutze codeBlocks)

Nein.

#include <iostream>
#include <cstring>

using namespace std;

int main()
{

    char* carray[30];
    int eingabe;
    bool end_loop = true;

    do
    {
        cout << "Druecken sie '1' fuer ein neues Spiel!" << endl;
        cout << "Druecken sie '2' um das Programm zu beenden!" << endl;
        cin >> eingabe;

        if(eingabe == 1)
        {
            cout << "Geben sie ein Wort ein! (Bitte nur Grossbuchstaben): ";
            cin.getline(carray, sizeof(carray));

            for(signed int i = 0; i < strlen(carray); i++)
            {
                cout << "-";
            }
        }

        else
        {
            end_loop = false;
        }

    }
        while(end_loop == true);
        cout << "Programm wird beendet..." << endl;
        return 0;
}

Ich glaube ich habe mich falsch ausgedrückt, die for-schleife wird nicht übergangen, ich habe es jedoch geschafft irgendwo eine endlosschleife einzubauen. Wenn ich das nun Compiliere und 1 drücke wird erstmal ausgegeben:
"Geben sie ein Wort ein! (Bitte nur Grossbuchstaben): " Direkt dahinter werden dann wieder die anweisungen drücke 1 oder 2 gestellt. (Warum)
Gebe ich dann ein Wort/Buchstabe ein lauft das Programm unendlich durch.

Als erstes:

Du brauchst der Anweisung nicht zu sagen, das sie "Wahr" sein soll, du kannst also einfach

while(end_loop);

schreiben.

So wie die Schleife jetzt aufgebaut ist, würdest du das Programm beenden, wenn du bei der ersten Eingabe keine 1 gemacht hast. Ist das Sinn und Zweck?
Ich würde das

end_loop = false

in die "true" Bedingung setzen, würde zumindest meiner Meinung nach mehr Sinn ergeben

würde dann in etwa so aussehen:

#include <iostream>
#include <cstring>

using namespace std;

int main()
{

    char* carray[30];
    int eingabe;
    bool end_loop = true;

    do
    {
        cout << "Druecken sie '1' fuer ein neues Spiel!" << endl;
        cout << "Druecken sie '2' um das Programm zu beenden!" << endl;
        cin >> eingabe;

        if(eingabe == 1)
        {
            cout << "Geben sie ein Wort ein! (Bitte nur Grossbuchstaben): ";
            cin.getline(carray, sizeof(carray));

            for(signed int i = 0; i < strlen(carray); i++)
            {
                cout << "-";
            }
            end_loop = false;
        }
    } while(end_loop);
        cout << "Programm wird beendet..." << endl;
        return 0;
}

Nur mein Problem bleibt.
Ich komme nicht zur eingabe des Wortes.

Und ich kann's nnoch nicht einmal compilieren. Denn es ist einfach totaler Mist und das müsste dir dein Compiler auch sagen:

char* carray[30];

Das ist ein Array von char*. Das ist kein C-String! Entsprechend gehen auch alle späteren Stringoperationen schief:

test.cc: In function ‘int main()’:
test.cc:22: [b]error: no matching function for call to ‘std::basic_istream<char, std::char_traits<char> >::getline(char* [30], long unsigned int)’[/b]
/usr/include/c++/4.4/istream:593: note: candidates are: std::basic_istream<_CharT, _Traits>& std::basic_istream<_CharT, _Traits>::getline(_CharT*, std::streamsize, _CharT) [with _CharT = char, _Traits = std::char_traits<char>]
/usr/include/c++/4.4/istream:405: note:                 std::basic_istream<_CharT, _Traits>& std::basic_istream<_CharT, _Traits>::getline(_CharT*, std::streamsize) [with _CharT = char, _Traits = std::char_traits<char>]
test.cc:24: [b]error: cannot convert ‘char**’ to ‘const char*’ for argument ‘1’ to ‘size_t strlen(const char*)’
[/b]

Das alles würdest du dir sparen, wenn du endlich std::string benutzen würdest. Das ist irgendwie als wenn du zum Arzt gehst und jammerst, dass dein Kopf weh tut, wenn du vor die Wand läufst. Da kann man eben nur antworten, dass du aufhören sollst absichtlich vor die Wand zu laufen.

edit: So ungefähr wolltest du es wohl haben. Man könnte das ganze natürlich noch besser machen, indem man auf Fehleingaben reagiert, aber wir wollen für den Anfang mal nicht übertreiben:

#include <iostream>
#include <string>

using namespace std;

int main()
{

  int eingabe;  

  do
    {
      cout << "Druecken sie '1' fuer ein neues Spiel!" << endl;
      cout << "Druecken sie '2' um das Programm zu beenden!" << endl;
      cin >> eingabe;

      if(eingabe == 1)
        {
          cout << "Geben sie ein Wort ein! (Bitte nur Grossbuchstaben): ";
          string str;
          cin >> str;   // So leicht kann das sein. 

          for(unsigned int i = 0; i < str.size(); i++) // Warum signed, wenn i doch ganz klar nur positiv sein kann?
            {
              cout << "-";
            }
          cout<<endl;   // Neue Zeile nach den ----, das sieht hübscher aus.
        }
    } while(eingabe!=2);   // Wozu das Gefrickel mit end_loop?
  cout << "Programm wird beendet..." << endl;
  return 0;
}

Und das mit dem end_loop wurde mir von anderen vorgeschlagen. Zuvor hatte ich auch die abfrage mit

while(eingabe !=2);

Und zu den C-Strings: Wenn es wirklich so schlimm ist, dass ich C-Strings benutze, dann lass ich es eben bleiben. Ich dachte nur es müsste auch mit diesen gehen.

Trotzdem danke

PS: Was benutzt du denn für ne IDE

Und zu den C-Strings: Wenn es wirklich so schlimm ist, dass ich C-Strings benutze, dann lass ich es eben bleiben. Ich dachte nur es müsste auch mit diesen gehen.

Klar geht es, aber es ist einiges umständlicher, als mit std::string .