Bei dem Spiel gehts darum, eine Zahl (nZielzahl) zu erreichen, indem man zu einer Zahl (nZahl, logisch) immer eine Zahl zwischen 1 und der höchsten erlaubten Dazuzählzahl (nmaxE) addiert. Spieler und Programm geben abwechselnd ihre Zahl (nSeing und Ceing) ein. Wer mit seiner Eingabe auf die Zielzahl kommt, hat gewonnen - klar. Der Trick bei dem Spiel ist, mit seiner Eingabe auf bestimmte Zahlen (nSZ für Schlüsselzahl) zu kommen.

Beispiel nZielzahl = 20, nmaxE = 2
Wenn ich mir die 17 sichere, habe ich gewonnen - egal, was der andere dazuzählt (1 oder 2), er kommt nicht auf 20, ich nach seiner Eingabe schon, da die Zahl dann 18 oder 19 ist. Ich habe gewonnen, wenn ich die 17 habe und diesen Trick kenne.Genauso kann man dann weitermachen - wenn ich mir die 14 sichere, komme ich auf die 17 und habe gewonnen. Immer wenn ich nmaxE+1 abziehe, erhalte ich eine neue Schlüsselzahl (20 - 17 - 14 - 11 - 8 - 5 - 2) . Macht keinen Spaß, wenn man dieses System kennt, aber ich wollte den Computer so stark wie möglich werden lassen. Also lasse ich den Computer (wenn der Gegner auf keine Schlüsselzahl gekommen ist - das else!) seine Eingabe so berechnen, dass er auf die nächste nSZ kommt. Und ich glaube, da liegt der Fehler - die Computereingabe hält sich nicht an die maximale Eingabe, um an die nächste Schlüsselzahl zu kommen. Vielleicht kann mir da jemand helfen!

#include <cstdio>
#include <cstdlib>
#include <iostream>
using namespace std;

int main(int nNumberofArgs, char* pszArgs[])
{
int nZielzahl;
int nmaxE;          //die höchste Eingabezahl
int nCEing;         //Computereingabe
int nSEing;         //Spielereingabe
int nSZArray[50];   //Array mit "Schlüsselzahlen"
int nSZ;            //Aktuelle Schlüsselzahl
int ni;             //Arrayzähler
int nZahl;          //Die Zahl

                    //hier beginnen die Einstellungen...

cout <<"Anleitung...die maximale Eingabe, die zur Zahl addiert wird,\ndarf maximal ein Drittel der zu erreichenden Zahl sein."
    << "\n\nGib die zu erreichende Zahl ein (maximal 100, mindestens 10) :\t\t";
cin >> nZielzahl;
if ((nZielzahl > 100) || (nZielzahl < 10) )
{
    cout << "\n\nSelber schuld. ";
    system("PAUSE");
    return 0;
}

cout <<"\n...und jetzt die hoechste Zahl, die zur Zahl addiert werden darf:\t";
cin >> nmaxE;
if ((nmaxE > (nZielzahl / 3)) || (nZielzahl < 2))
{
    cout << "\n\nSelber schuld. ";
    system("PAUSE");
    return 0; 
}
nSZ = nZielzahl;
while (nSZ > (nmaxE + 1))   //Bis zur kleinsten Schlüsselzahl herunterzählen
{
    nSZ -= (nmaxE + 1);
}
for (ni = 0;nSZ < nZielzahl; ni++)
{
    nSZArray[ni] = nSZ;
    nSZ += (nmaxE + 1);
}
ni = 0;
nZahl = 0;
                                        //...und hier enden sie. Das Spiel beginnt 
                                        //mit einer Eingabe des Spielers.

cout << "\n\nUnd los gehts. Du beginnst.\n\n";
while (1)
{
    cout << "\nDu:\t";
    cin >> nSEing;
    if (nSEing < 1 || nSEing > nmaxE)       //die nächsten beiden ifs testen, ob die Eingabe OK ist.
    {
        cout << "\n\nNicht innerhalb des erlaubten Bereichs!. ";
        system("PAUSE");
        return 0; 
    }
    nZahl += nSEing;
    cout << "\tZahl:\t" << nZahl;
    if (nZahl == nZielzahl)
    {
        cout << "\n\nDu hast gewonnen!\n\n";
        system("PAUSE");
        return 0; 
    }
    if (nSZArray[ni] < nZahl)               //Arrayüberprüfung
    {
        ni++;
    }
    cout << "\t\t" << nSZArray[ni];
    if (nZahl == nSZArray[ni])
    {
        nCEing = nSEing;
        ni++;
    }                                        //Gewinnkursüberprüfung (wenn nZahl == nSZArray[ni])
    else
    {
        nCEing = (nSZArray[ni] - nSEing);
        if (nZahl == nZielzahl)
             {
             cout << "Ich habe gewonnen!\n"
             << "\nIch:\t" << nCEing << "\tZahl:\t" << nZahl;
             system("PAUSE");
             return 0; 
             } 
    }
    nZahl += nCEing;
    cout << "\nIch:\t" << nCEing << "\tZahl:\t" << nZahl;
}

system("PAUSE");
return 0; 
}

Langer Code - vielleicht etwas umständlich. Ich bin totaler Anfänger, also versucht's mir bitte in einfachen Worten zu erklären.

Ich kann dir nur einen Tipp geben, lerne, objektorientiert zu programmieren, und DANN wage dich an solche Spiele.

C++ hat doch extra die OOP dazubekommen, warum sollte man sie dann nicht einsetzen? (Dazu noch ist objekt orientieren extrem nützlich und eigentlich einfach, damit kannst du dein Spiel viel besser kontrollieren)

MfG

Niko

Objekt orientiert hat man mehr Übersicht, weniger Fehler und kann leichter und gezielter korrigieren.

Mich wundert es, dass da überhaupt noch irgendetwas funktioniert.

Ich kann dir nur einen Tipp geben, lerne, objektorientiert zu programmieren, und DANN wage dich an solche Spiele.

C++ hat doch extra die OOP dazubekommen, warum sollte man sie dann nicht einsetzen? (Dazu noch ist objekt orientieren extrem nützlich und eigentlich einfach, damit kannst du dein Spiel viel besser kontrollieren)

Ich finde, es spricht nichts dagegen, das als Anfänger auch mal ohne OOP zu versuchen. Die Gedanken die sich DaNiko gemacht hat, muss er sich ja auch bei der OOP-Programmierung machen. Da er aber Anfänger ist, ist es verständlich, dass er noch nicht mit Klassen etc. in Berührung gekommen ist, aber trotzdem schon etwas programmieren will.
Später wenn er dann OOP lernt, kann er ja auf diese Gedanken schon aufbauen.

So, jetzt aber zum Problem:
DaNiko, im ersten Teil ist eigentlich alles soweit in Ordnung, aber ab der Stelle, wo dann der Computer dran ist, hast du ein paar Logikfehler drin.

Erstmal das hier:

if (nSZArray[ni] < nZahl)               //Arrayüberprüfung
{
    ni++;
}

Damit willst du ja prüfen, ob die aktuelle Schlüsselzahl schon erreicht wurde, um dann ggf. die nächste Schlüsselzahl zu verwenden. Du prüfst in deiner Bedingung aber nur, ob die Schlüsselzahl kleiner als die aktuelle Zahl ist.
Was ist aber, wenn sie gleich der Zahl ist? Du änderst die Schlüsselzahl in diesem Fall nicht. Da man aber mindestens eine 1 eingeben muss, ist es dem Computer nicht möglich, auf die aktuelle Schlüsselzahl zu kommen.
Du musst also prüfen, ob die Schlüsselzahl kleiner oder gleich der aktuellen Zahl ist (<=).

Damit fällt auch diese Abfrage weg:

if (nZahl == nSZArray[ni])
{
    nCEing = nSEing;
    ni++;
}

Ich weiß ehrlich gesagt nicht, was du damit bezwecken wolltest.

Normalerweise würde ja jetzt der else-Block folgen, da das if aber gerade weggefallen ist, wird aus diesem Block eine ganz normale Anweisungsfolge.

Es geht also weiter mit:

nCEing = (nSZArray[ni] - nSEing);

Wenn ich das richtig erkannt habe, willst du hier berechnen, welche Zahl der Computer wählen muss, um auf die nächste Schlüsselzahl zu kommen.
Da hast du nun wieder einen Logikfehler drin:
Angenommen, die aktuelle Zahl ist 14 und die nächste Schlüsselzahl ist 16.
Was macht man um auf die nötige Zahl (also in diesem Fall 2) zu kommen? Richtig man subtrahiert die zwei Zahlen.
Also muss es so heißen: (die Klammern kannst du übrigens weglassen)

nCEing = nSZArray[ni] - nZahl;

Weiter gehts:
Bei dir kommt jetzt die Überprüfung, ob der Computer die Zahl erreicht hat, was aber ziemlich unsinnig ist, da er ja noch gar kein "Zug" gemacht hat.
Du musst also folgende Anweisungen vor die Überprüfung schreiben:

nZahl += nCEing;
cout << "\nIch:\t" << nCEing << "\tZahl:\t" << nZahl;

Jetzt bist du fast fertig.
Wenn du das Programm ausführst, wirst du aber erkennen, dass der Computer bis zum letzten "Zug" alles richtig macht, dann aber eine völlig unsinnige Zahl wählt.
Das liegt an der Methode, wie diese "Eingabezahl" berechnet wird.

nCEing = (nSZArray[ni] - nSEing); // s.o.

Wenn 40 z.b. die Zielzahl ist und die aktuelle Zahl 38, dann müsste der Computer ja die Zahl 2 wählen.
Die Methode oben funktioniert aber nicht, weil die Zielzahl selbst nicht im SZArray gespeichert ist. Um das zu ändern änderst du einfach deinen Code zum Initialisieren der Schlüsselzahlen um:

for (ni = 0;nSZ < nZielzahl; ni++)
{
    nSZArray[ni] = nSZ;
    nSZ += (nmaxE + 1);
}

Hier einfach aus dem < ein <= machen, und er nimmt auch die Zielzahl in dein Array mit auf.

So, jetzt müsste es eigentlich klappen.

Achja, noch kurz was zu deinem Stil: der ist stellenweise ein bisschen unübersichtlich. Du kannst ruhig ab und zu ne Leerzeile dazwischen machen.
Außerdem ist es eigentlich unnötig, eine Integer-Variable mit einem n zu beginnen, das verschlechtert imho die Lesbarkeit auch noch mal.

PS: an die, die grade eben noch gepostet haben (ceplusplus, Leon++):

DaNiko schrieb:

Ich bin totaler Anfänger

Wie gesagt, von einem "totalen Anfänger" würde ich noch nicht erwarten, dass er mit Klassen programmiert.

-predator- schrieb:

Damit fällt auch diese Abfrage weg:

if (nZahl == nSZArray[ni])
{
    nCEing = nSEing;
    ni++;
}

Ich weiß ehrlich gesagt nicht, was du damit bezwecken wolltest.

Falls der Spieler eine Schlüsselzahl erreicht, lasse ich das Programm einfach dieselbe Eingabe wie der Spieler machen, um zu verhindern, dass er seine Eingabe mit

nCEing = nSZArray[ni] - nZahl;

berechnet, da die nächste SZ (Schlüsselzahl) nmaxE+1 von der Zahl weg ist. Der Computer würde schummeln. Die Eingabe des Spielers ist schon auf Schummelei geprüft, deshalb wiederhole ich sie einfach.

Nochmal danke - der eifrig an seinen OOP - Kenntnissen feilende Niko