Why doesn't this work?

If someone could fix this for me? WITHOUT USING ADVANCED METHODS, it would be great.

#include "stdafx.h"
#include <iostream>
#include <string>
using namespace std;

#define run cout

int main ()
{
run << "Welcome to Cytonia battle." << " " << "Please, young warrior, enter your name! \n";
string name;
cin >> name;

run << "Ah, so your name is" << " " << name << " " << "and what weapon did you bring with you? \n\n";
run << "Sword" << endl << "Staff" << endl << "Bow \n";

string choice;
cin >> choice;

if (choice == "Sword" || "sword")
{ run << "Aha, nothing better than the mighty blade!"; }
else
{

if (choice == "Staff" || "staff")
{ run << "Oh? I see you're a master of the dark arts! Great, we could use your services."; }

else
if (choice == "Bow" || "bow")
{ run << "Ah, fellow marksman are you?"; }


}
cin >> choice;





return 0;


}
Ah, nevermind, I took the long way out and rewrote it.

#include "stdafx.h"
#include <iostream>
#include <string>
using namespace std;

#define run cout

int main ()
{
run << "Welcome to Cytonia battle." << " " << "Please, young warrior, enter your name! \n";
string name;
cin >> name;

run << "Ah, so your name is" << " " << name << " " << "and what weapon did you bring with you? \n\n";
run << "Sword" << endl << "Staff" << endl << "Bow \n";

string sword = "Sword";
string staff = "Staff";
string Bow = "Bow";
string choice;
cin >> choice;

if (choice == sword)
{ run << "Aha, nothing better than the mighty blade!"; }


if (choice == staff)
{ run << "Oh? So you're a master of the dark arts, we could use your services!"; }


if (choice == Bow)
{
run << "Well well, you're a fellow marksman I see!";
}
cin >> choice;

Topic archived. No new replies allowed.