Boleanska uttryck c++

#include <string>
main(){
string pick = number, red, black;
		int bettingNumber;
		cout << "Would you like to bet on a number or a color? Type, 'red', 'black' or ' number' to make your pick.\n";
		cin >> pick;
		while (pick != number, red, black) {
			cout << "please type in number, red or black to make your choice";
			cin >> pick;
return 0;
}

1. Du deklarerar red och black på raden "string pick = number, red, black;", däremot deklareras aldrig number.
2. Vet inte hur mycket det hjälper men ett boolskt uttryck är ju helt enkelt ett uttryck som har ett boolskt värde, dvs sant eller falskt. Det kan ju vara t.ex. en variabel av lämplig typ eller en jämförelse.

Jag tror ju dock snarare att det är det här skrivandet av saker med kommatecken mellan som är någon missuppfattning av hur någonting fungerar.

Om det är en skoluppgift så kommer du inte få så mycket specifik hjälp

Var ett tag sen jag pilla med c++ men ska försöka peka dig i en riktning...

Kör färgen som string och number som int, även om det fungerar med string på tal ocskå, men i kommande uppgifter så är det bra om du förstår skillnaden då det blir bökigt att utföra beräkningar med strängar =). Eftersom du matar in en siffra så är det bättre att ta emot den som int. Du nu angett dem som variablar dvs pick = number, red, black vilket inte funkar du skulle kunna skriva pick "number, red, black" så kommer pick innehålla texten: number, red, black. Men du vill ha dem separat.

Det du vill göra är att deklarera number som integer (hanterar heltal) red och black som strings.
Sedan jämföra dem en å en. jag gissar att din while-loop inte fungerar heller?

Du behöver tänka till igen Som det ser ut nu så har du inte riktigt koll på vad string är eller integer.

Edit: var lite för snabb.

Gör lite enkelt Pseudokod (typ flödesschema) kan hjälpa en att få till det ibland.

Ex och spark i en riktning.

#include <string>
main(){
string pick = ""; //dessa initierar /deklarerar pick som en sträng med "tomt" innehåll redo att användas
string red = "";
string black = "";
int bettingNumber = 0; // bra att ge variabeln ett värde, inget måste.

//vill du ha allt på en rad kan du ex skriva: string black="", string red="" etc
		
		cout << "Would you like to bet on a number or a color? Type, 'red', 'black' or ' number' to make your pick.\n";
		cin >> pick;

		while (pick != number, red, black) {   //här jämför du om strängen pick != red eller pick != black eller number > 0. På siffrar ska du inte citattecken.
			cout << "please type in number, red or black to make your choice";
			cin >> pick;
//sedan skickar du in en cout här för att se innehållet på pick
return 0;
}

@Bizzzzznes:
om du ska använda integer så behöver du göra om en del. Det går att använda strängar för att jämföra siffor. Men blir svårt att jämföra varje siffra för sig i din loop. Du måste jämföra siffra 1,2,3...etc. Om du istället inmatningen som en integer så kan du bara kolla om en siffra är inmatat med number > 0. > = större än samt < mindre än. Dvs om en siffra större än 0 är inmatad så fortsätt.
Är inte säker på om man kan skriva number > 0 < 30 eller om man behöver skriva number > 0 && number <= 30 (för att se om siffra mellan 1 och 30 är inmatad.

Gör varje steg för sig och använt cout för att se vad variablarna innehåller. Om du inte använder ex visual studio då du kan använda debugläge för att se innehållet i variablarna i realtid.

std::string initieras till tom sträng i defaultkonstruktorn, att skriva = "" är alltså onödigt.

Misstänkte det, men ville vara tydlig mot TS ifall. Så han förstår vad jag menade.

@Bizzzzznes:

Vad har du programmerat för språk förut? Och vad vill du egentligen göra i ditt villkor?

Detta kommer bli lite rörigt, men i C++ finns operator "," (komma). Den säger att först skall man göra det som står till vänster om kommat och sedan det som står till höger om kommat. Resultatet av komma-uttrycket blir det som står till höger om kommat. Precis som när du skriver "v = uttryck1 + uttryck2" kan du skriva "v = uttryck1 , uttryck2". Kompilatorn kommer då räkna ut uttryck1 och sedan uttryck2 och tilldela resultatet av uttryck2 till v. Operator "," är mest användbar om man vill trixa och göra flera saker i exempelvis ett ?:-uttryck.

När du skriver

while (pick != number, red, black)

har du skrivit just ett komma-uttryck i villkoret. I ditt fall betyder det att kompilatorn kommer räkna ut pick != number, strunta i resultatet, jämföra om strängen red != 0, strunta i det och sedan testa black != 0 och låta resultatet av det styra while-loopen.

Du ville sannolikt göra ett test med flera deluttryck:

while (pick != number && pick != "red" && pick != "black")

Detta löser dock inte den problematik som @fragwolf tog upp. Den första delen av testet borde kolla om pick är ett godkänt nummer, men nu kollar den bara om pick skiljer sig från number (vad det nu är).

Du kanske vill göra något i stil med:

  while (true)
  {
    cout << "Would you like to bet on a number or a color? Type, 'red', 'black' or ' number' to make your pick.\n";
    cin >> pick

    if (pick == "red") break;
    if (pick == "black") break;
    int number = std::stoi(pick);
    if (number >= 0 && number <= 36) break;
  }

Felhanteringen är inte den bästa och du kommer nog vilja komma ihåg vad användaren gjorde för val, men exemplet visar på ett annat sätt att angripa det krångliga loop-villkoret. En enklare lösning vore att säga att -1 betyder rött och -2 betyder svart. Då kan du jobba bara med heltal och då blir problemet mycket enklare.

int number = 1 || 2 || 3 || ....

string pick = number, red, black;
		int bettingNumber;
		cout << "Would you like to bet on a number or a color? Type, 'red', 'black' or ' number' to make your pick.\n";
		cin >> pick;
		while (pick != number, red, black) { //Som jag har förstått det ska jag ändra denna raden till = while (pick !=number && !=red && !=black).
			cout << "please type in number, red or black to make your choice";
			cin >> pick;
		}

		if (cin >> number) {
			cout << "Pick a number between 1 and 36: ";
			cin >> bettingNumber; cout << "\n";
			while (bettingNumber > 36 || bettingNumber < 1) {
				cout << "That number is not allowed, please pick a number between 1 and 36: ";
				cin >> bettingNumber; cout << "\n";
			}
		}
		else if (cin >> red) {
			cout << "You've chosen to play on the color red. Good luck!\n";
		}
		else (cin >> black); {
			cout << "You've chose to play on the color black. Good luck!\n";
		}

Hmm, var skall vi börja. Du tänker rätt men när det skall översättas till C++ kommer du lite snett.

Punkt 1 går fint

När vi kommer till punkt två blir det lite mer spännande. Om du skriver exempelvis

string pick, number, red, black;

kommer du deklarera fyra variabler: pick, number, red och black av typen string som alla innehåller den tomma strängen "". När du sedan läser in ett värde och testar pick mot red

  cin >> pick;
  if (pick != red) ...

kommer du jämföra innehållet i variablerna pick och red. Pick innehåller då det som användaren matade in och red innehåller det senaste värde som du tilldelade red, dvs den tomma strängen. Du har två alternativ: antingen initierar du dina variabler så red innehåller strängen "red"

string pick, number("number"), red("red"), black("black");

eller så jämför du pick med strängen "red" istället för variabeln red:

  if (pick != "red") ...

Lite kommentarer angående resten av koden: För att while-loopen skall kompilera måste du fylla i pick på alla ställena:

while (pick !=number && pick !=red && pick != black).

Precis som du påpekar måste du testa på pick i if-statserna efter loopen. Att skriva

if (cin >> number) ...

är visserligen korrekt C++ (man kommer ta if-satsen om man lyckades läsa in ett heltal från cin) men då kommer det krävas att man matar in något som inte är ett heltal för att komma till villkoret för red och där kommer det man matade in hamna i red. Det kommer inte bete sig som du vill att det skall funka.

		else if (pick == red) {
			cout << "You've chosen to play on the color red. Good luck!\n";
		}
		else (pick == black); {
			cout << "You've chose to play on the color black. Good luck!\n";
		}

Du har glömt if på sista else if-satsen och satt ett semikolon efter villkoret. Det betyder att efter sista else står det ett komplett uttryck. Efter det står det { cout << "You've chose to play on the color black. Good luck!\n"; }. Detta statement kommer alltid att exekveras, oavsett vilket val du gjorde.

Men, men... Lärare och handledare får väl inte använda internet för att läsa i allmänna forum på nätet ?

@Bizzzzznes: Hoppas du inte behöver mer hjälp...