C++ Nybörjare..

Vad menar du med loopar om? Jag kan inte C++ själv men tror inte du kan sätta din variabel "deposit" till något annan än heltal eftersom den är en integer..

Den kommer fastna i första while loopen så länge deposit är skilt ifrån 50, 100 eller 500. Så om du skriver in 0m453 vilket är en sträng så kommer strömmen tolka det som noll eftersom den kommer stanna vid m eftersom det inte är en siffra (rätta mig om jag har fel). Eller menar du att den även fastnar när du skriver in korrekt summa?

Sen behöver du inte kolla om deposit är lika med 50,100 eller 500 efteråt då du redan ser till att den är det i första while loopen.

samt för rad brytning räcker bara std::cout<<std::endl;
Du behöver inte skicka in en tom sträng.

"while (deposit != 50 && deposit!=100 && deposit!=500) //Insats är inte 50,100,500 (Ny insats)
{
cout<< "Please try again!"<<endl;
deposit = 0;
cin>> deposit;
}"
ledsen att jag inte citerar, ska inte den vara en if sats istället?

Om man har en if sats där istället så kommer den bara köra checken en gång och sen gå vidare. Om man nu vill garantera att deposit endast får ha värderna 50,100 eller 500 så funkar en while sats bättre.

Mitt tips till trådskaparen är att koppla på en debugger och stega igenom programmet och medhjälp av den inspektera variablerna och se vad för värden varje variabel har. Eller iaf skriv ut lite debug utskrifter typ

// Debug print out. 
#define print_var(str,x) std::cout<<"[Debug]: "<<str<<" "<<#x<<" = "<<x<<std::endl;

print_var("before loop", deposit); // will print out "[Debug]: before loop deposit = <some value>"
while (deposit != 50 && deposit!=100 && deposit!=500) //Insats är inte 50,100,500 (Ny insats)
{
print_var("during loop", deposit);
cout<< "Please try again!"<<endl;
deposit = 0;
cin>> deposit;
}
print_var("after loop", deposit);
#undef print_var;

Det har absolut ingenting med trådskaparens While-loopar eller If-satser. Felet här är att input-bufferten (cin) används fel.

Eftersom mitt svar blir ganska långt, delar jag upp det i att förklara problemet för att sedan ge tips på en lösning.

Problem

Börjar vi från början så ser det ut så här.

cout<<"WELCOME TO MEANMACHINE!"<<endl;
cout<<"(Press 'ENTER' to continue)"<<endl;
cin.get();

cin.get() kommer endast plocka ett tecken som finns i inputbufferten (cin). Trycker man nu på Enter kommer den förse bufferten med en newline eller linebreak '\n', vilket cin.get() gladeligen plockar upp. Skriver man däremot t.ex. "0m434" så kommer detta också accepteras och gå vidare. Skillnaden nu är att i inputbufferten ligger det "0m434\n", cin.get() kommer endast ta första tecknet, i det här fallet "0" och lämna återstående i bufferten "m434\n".

När vi sen kommer ner till första inmatningen från användaren stöter vi på problem. cin>>deposit kommer läsa första ordet (alla tecken fram till mellanslag eller radbrytning) som finns i inputbufferten. Eftersom våran inputbuffer redan innehåller "m434\n" så kommer cin förstås ta detta. Vilket gör att användaren inte kommer få någon chans att själv mata in ett värde.

cout <<"Please make a deposit, 50, 100 or 500."<<endl; //Här gör du din insättning.
cin >> deposit;

Nu har cin läst "m434", den läser inte \n eftersom detta användes som en indikation för att tala om var inmatningen tog slut. Efter detta kommer den tilldela detta värdet till en int, som inte kan hantera något annat än siffror. Vad händer då? Jo, våran int-variabel blir förstås korrupt. När vi sedan går vidare till jämförelsen med en korrupt int-variabel.

while (deposit != 50 && deposit!=100 && deposit!=500) //Insats är inte 50,100,500 (Ny insats)
{
     cout<< "Please try again!"<<endl;
     deposit = 0;
     cin>> deposit;
}

Så kommer jämförelsen gå sönder eftersom variablen som vi försöker jämföra med är trasig. Vilket sätter loopen ur spel och låter den gå i all evighet.

Lösning

Hur löser vi det då detta? Väldigt enkelt, läs all användarinmatning till strängar och konvertera sedan det på ett säkert sätt till en int. En sträng som tillåter de flesta tecknen som finns har inte ont av ifall användaren trycker på hela tangentbordet samtidigt.

Börjar vi om från början med lite nya exempel kan det se ut så här:

int balance = 0; //Variabel för saldo
int deposit = 0; //Variabel för insättning av pengar
int bet; //Variabel för bet
string userInput = ""; // Variabel för användarens inmatning
    
cout<<"WELCOME TO MEANMACHINE!"<<endl;
cout<<"(Press 'ENTER' to continue)"<<endl;
getline(cin, userInput);

Det som är nytt här är strängen userInput och getline() istället för get(). getline tillskillnad från get läser hela raden istället för ett tecken. Vilket gör att det inte spelar någon roll ifall användaren hade skrivit en hel uppsats innan denne hade tryckt på enter.

Ifall användaren har matat in något lustigt, t.ex. "0m434" så kommer detta ligga i userInput-variabeln så länge.

Väl vidare i koden kommer vi till nästa del, där användaren ska ange en mängd för sin deposit.

userInput = ""; // Töm variablen från det skräp som kan ha plockats upp tidigare.
cout <<"Please make a deposit, 50, 100 or 500."<<endl; //Här gör du din insättning.
cin>>userInput; // Läs ett värde (till userInput)  

// Här kommer det roliga, omvandla din string till en int.
deposit = atoi(userInput.c_str());
    
// Jämför sedan deposit som vanligt.
while (deposit != 50 && deposit!=100 && deposit!=500) //Insats är inte 50,100,500 (Ny insats)
{
    cout<< "Please try again!"<<endl;
    cin>>userInput; // Läs in det nya värdet till userInput igen
    deposit = atoi(userInput.c_str()); // Konvertera userInput till en int och tilldela deposit denna
}

Detta gör att användaren kan skriva vad som helst, sedan sköter atoi en säker omvandlning från string till int. Är strängen fylld med bokstäver, t.ex. "abc319ddad" kommer detta konverteras till värdet 0. Är det däremot ett korrekt heltalsvärde t.ex. "148" så koverteras detta till värdet 148.

Nu bör alltså problemet som uppstod när du skrev in bokstäver istället för siffror vara löst. Förhoppningsvis har jag täckt upp för principen, så med lite skicklighet bör du kunna klura ut vad du behöver för ändringar i resten av koden.

Jag hoppas det här hjälper dig en bit på vägen i alla fall.

"int balance = 0; //Variabel för saldo
int deposit = 0; //Variabel för insättning av pengar
int bet; //Variabel för bet"

* Onödiga kommentarer
* Gör det finare med mer mellanrum mellan koden och kommentarerna
* Gör det finare med mellanrum mellan // och kommentaren
(programmering är ganska mentalt arbete, så håll programmet fint)

"cout<<"WELCOME TO MEANMACHINE!"<<endl;
cout<<"(Press 'ENTER' to continue)"<<endl;"

Du kan förkorta det till "cout << "WELCOME TO MEANMACHINE!\n(Press 'ENTER' to continue)" << endl;

"cin.get();"

Och hur ser den här funktionen ut? Ser inte du som att du använder ett library som definierar den.

"cout <<"Please make a deposit, 50, 100 or 500."<<endl; //Här gör du din insättning."

Åter igen är koden ganska tydlig så kommentaren är onödig.

Jag vill inte vara respektlös, men koden är ganska jobbig att läsa igenom. Några delar förstår jag inte alls, som:
"cout<<""<<endl;

cout <<"Please make a bet! (1-5000)"<<endl; //Insats"

Som jag förstår det är "cout<<""<<endl;" bara där för att göra en ny rad. Detta är onödigt. Du kan förkorta ner det till:
"cout <<"\nPlease make a bet! (1-5000)"<<endl; //Insats"

Det finns tydligen medlemmar här som lägger ner tid åt att fixa ditt program. Men för din egen skull bör du överge programmet och börja om på nytt.

Programmet har bra struktur, men det verkar mest som att du härmar det du har sett i andra koden istället för att använda dig av programmeringskunskaper.

Så därför undrar jag hur du lär dig programmering. Kollar du på Youtube?
För C++ skulle jag rekommendera Bjorne Stroustrups "Programming: Practice and Principles using C++".
Den är mycket vänlig för de som inte har programmerat förut. Boken lär dig hur du skriver korta, effektiva och simpla koder.

Glad att du har en bra inställning, men jag skulle fortfarande rekonmendera boken. Som jag skrev i mitt förra inlägg är den väldigt vänlig för nybörjare (den är lätt att förstå).

Skickades från m.sweclockers.com

Har skrivit om koden lite. Något du måste använda dig av är en input-loop, alltså while (cin >> variabel)

EDIT: Har lagt till std_lib_facilities, ett library med funktioner. Du behöver dock inte använda den.

Lägg märke till hur den här delen av koden har ändrats:

cin >> deposit;

while (deposit != 50 && deposit!=100 && deposit!=500) //Insats är inte 50,100,500 (Ny insats)
{
cout<< "Please try again!"<<endl;
deposit = 0;
cin>> deposit;
}

#include "../../std_lib_facilities.h"

int main ()
{
int balance = 0;
int deposit = 0;
int bet;

cout<<"WELCOME TO MEANMACHINE!\n(Press 'ENTER' to continue)"<<endl;
cin.get();

cout <<"Please make a deposit of 50, 100 or 500."<<endl;
cin >> deposit;

while (deposit != 50 && deposit!=100 && deposit!=500) //Insats är inte 50,100,500 (Ny insats)
{
cout<< "Please try again!" << endl;
deposit = 0;
cin >> deposit;
}

if (deposit == 50 || deposit ==100 || deposit == 500) //Din insats är korrekt.
{
balance += deposit;
cout <<"Your deposit of ("<<deposit<<") suceeded, your balance is ("<<balance<<").\n(Press 'ENTER' to continue)"<<endl;

cin.get();
}

cout <<"\nPlease make a bet! (1-5000)"<<endl; //Insats

while(cin >> bet)
{

if (bet > balance) //Insats större än saldo (Nytt bet).
{
cout<<"Invalid bet, your balance is ("<<balance<<")"<<endl;
cin>>bet;
}

if (bet > 5000 || bet < 1 ) //Insats större än 5000 eller mindre än 1 (Nytt bet).
{
cout << "The amount you entered isn't valid, please try again"<<endl;
cin >> bet;
}

if (bet <= 5000 && deposit >= 1)
{
balance -= bet;
cout<<"Your bet is ("<<bet<<") and your balance is now ("<<balance<<')'<<endl;
}

}
}