was ist an diesem c++ script falsch

[jookerxxx]

#include <iostream>
#include <windows.h>
#include <string>
#include <stdio.h>
#include <cstdio>

using namespace std;
int main(){
string frage1,aussage;

cout<<"hey was geht? \n alles was beine hat oda?(j/n)\n"; //frage1
cin>>frage1;
if (frage1 == "j" || "J")
{
aussage="das stimmt nicht!!!\n was ist mit Tischbeinen?\n";
}
if (frage1 == "n" || "N")
{
aussage=" aha......\n also krabbelste?\n";
}




cout<<aussage;
Sleep(5000);
}

dazu:
das ist der anfang von einem programm für meinen freund
wo man nur mit ja und nein antworten darf.
ps:ich weiß ihr hättet den drang zu sagen:
bevor wir dir helfen such dir n besseres thema als tischbeine^^
aba bitte.........
ich weiß es..........
also pls haltet euch zurück^^

das eigentliche problem:

er gibt bei beiden möglichkeiten aus:
aha......\n also krabbelste?\n"

[noctem]

if (frage1 == "j" || "J")

if(frage1=="j" || frage1=="J")

So müsste es aussehen.

Für weitere Postings bitte beachten:
-Fehler posten, die der Compiler ausgibt.
-Code-Tags verwenden:

[osiris]

frage1 == "j" || "J"

wenn schon

frage1 == "j" || frage1 == "J"

das gleiche mit

frage1 == "n" || "N"

EDIT: zu spät ...

[Guimaster]

Semikolons nicht vergessen am ende !!

[jookerxxx]

danke für die neue hilfe dann kann ich heute noch weiter scripten.
das mit mach ich auch.
beim nächsten ma
evtl. kann diese thema offen gelassen werden denn ich werde bestimmt noch mehr fragen haben
und dann brauch ich net imma neue themen zu eröffnen.

[osiris]

wo fehlen semikolons?

[M$A!]

Benutz doch für zeichen lieber chars anstatt strings

[blackberry]

Benutz doch für zeichen lieber chars anstatt strings

Davon ist C++-Anfängern eher abzuraten, da diese dann selbst auf den Speicher aufpassen müssen.
(ich gehe davon aus, dass du mit "chars" char-Vektoren meinst )

@jookerxxx
C++ ist keine Skript Sprache -> es heißt nicht "in C++ skripten".

Zusätzlich noch ein bisschen was zum optimieren:

#include <stdio.h>
#include <cstdio>

"stdio.h" (C) und "cstdio" (C++) erweisen sich als äquivalent, wobei, wie bereits angedeutet, "cstdio" die C++ Version von dem C Header "stdio.h" (in manchen Implementierungen macht cstdio auch nichts anderes als stdio.h einzubinden).
Somit ist einer Einbindung beider Header auf einmal *nicht falsch*, aber Redundant.
Weiterhin benutzt du keine Funktionen, Typen, ..., die in diesen Headern vereinbart wurden -> die Einbindung von "stdio.h" bzw. "cstdio" in deinem Programm ist etwas irreführend.

Zudem wird "cstdio" bereits (über Umwege) von "iostream" includiert:
- iostream
-- istream
--- ios
---- cstdio
(Quelle: glibc++ 4.2.4)

Ansonsten vielleicht die Antworten ("das stimmt nicht!!!\n was ist mit Tischbeinen?\", " aha......\n also krabbelste?\n") gleich über cout ausgeben und "string aussage" eliminieren (Jedes string Objekt verbraucht extra Speicher).


mfG. BlackBerry

[AlterHacker]

#include <iostream>
using namespace std;
int main(){
    string frage1;
    cout<<"hey was geht? \n alles was beine hat oda?(j/n)\n";   //frage1
    cin>>frage1;
    if (frage1 == "j" || frage1 == "J") cout << "das stimmt nicht!!!\n was ist mit Tischbeinen?\n";
    if (frage1 == "n" || frage1 == "N") cout << " aha......\n also krabbelste?\n";
      Sleep(5000);   
   return 0;
}

So sollte es besser sein

[.wired]

#include <iostream>
#include <conio.h>
#include <windows.h>
using namespace std;
int main(){
	char eingabe;
    cout<<"hey was geht? \n alles was beine hat oda?(j/n)\n";   //frage1
	eingabe=getch();
	cout << endl;
	switch (eingabe) {

	case 'j':
	case 'J':
	{
		cout << "das stimmt nicht!!!\n was ist mit Tischbeinen?\n";
	}
	break;
	case 'n':
	case 'N':
    {
		cout << " aha......\n also krabbelste?\n";
	}
	break;


	default:
	cout << "Falsche Eingabe! Programmende!";
	}
	  Sleep(5000);
   return 0;
}

Also ich habs mal nen bisschen umgeschrieben.
Das was du da mit zwei IF-Schleifen gebastelt hast, is Schwachsinn.(Sorry!)
Hab dir jetzt einfach ne CASE-Anweisung eingebaut, die löst - meiner Meinung nach - am besten das Problem.
H4ve Phun.