Не работает выбор имени через switch

[Kukla19]

Есть такой вот конструктор, если указываю имя явно то все работает, если пытаюсь через рандом то вместо имени пустая строка. Помогите разобраться...

class mob : public character {

	public:
		string name;
		int hp = 75;
		int xp = 15;
		int lvl = 1;


			game::mob() {
				string setname;
				srand(time(0));
			int name = 1 + rand() % 5;
			switch (name) {
			case 1: setname = "Разбойник"; break;
			case 2: setname = "Волк"; break;
			case 3: setname = "Медведь"; break;
			case 4: setname = "Кабан"; break;
			case 5: setname = "Людоед"; break;
				this->name = setname;
				}
			}

			~mob() {
			
			}

[BDA]

Делайте "this->name = setname;" после switch, а не внутри одного из кейсов (тем более, что код после break недостижим).

[uberchel]

В консткукции switch нельзя писать код кроме как в case

class mob : public character {

	public:
		string name;
		int hp = 75;
		int xp = 15;
		int lvl = 1;
		
		game::mob() {
			string setname;
			srand(time(0));
			
			int name = 1 + rand() % 5;
			switch (name) {
				case 1: setname = "Разбойник"; break;
				case 2: setname = "Волк"; break;
				case 3: setname = "Медведь"; break;
				case 4: setname = "Кабан"; break;
				case 5: setname = "Людоед";
			}
			this->name = setname;
		}
		
		~mob() {
		
		}
}

[Kukla19]

BDA, спасибо, точно не увидел ошибку!
uberchel, спасибо!

[Алексей1153]

Сообщение от uberchel

В консткукции switch нельзя писать код кроме как в case

можно, но обычно это не имеет смысла или даже вредно
https://onlinegdb.com/fh95wjo7-

[Kukla19]

Еще спрошу в этой теме.

Почему-то не записывает в файл данные. Чтение файла сделано в конструкторе, запись в деструкторе подкласса character. После того как дописал метод battle перестало записывать данные в файл. А мне нужно чтобы здоровье и опыт записывались в файл.

Вот такой вот класс на 200 строк:

class game {

public:

	class item {

	public:
		string slot;
		int lvl;
		string name;
		string mark;
		string namestat1;
		string namestat2;
		string namestat3;
		int stat1;
		int stat2;
		int stat3;

		item() {

		}

		void iteminfo() {
			for (int i = 1; i > 10; i++) {
				if (this->mark == "выносливости" and lvl == i) {
					stat1 = i * 3;
					stat2 = 0;
					stat3 = 0;
					if (stat1 > 0)
						cout << this->name << " с печатью " << this->mark << endl;
				}
			}
		}
	};

	class character {

	public:
		string name;
		int hp;
		int lvl;
		int dmg;
		int xp;
		int crit;
		int block;
		string type = "knight";

		character() {
			ifstream fin("herodata");
			std::string attr;
			if (fin.is_open()) {
				fin >> attr >> this->name;
				fin >> attr >> this->hp;
				fin >> attr >> this->dmg;
				fin >> attr >> this->lvl;
				fin >> attr >> this->xp;
				fin >> attr >> this->crit;
				fin >> attr >> this->block;
				fin.close();
			}
		};

		~character() {
			std::ofstream out;   
			out.open("herodata");
			if (out.is_open())
			{
				out << "heroname" << " " << this->name << std::endl;
				out << "herohp" << " " << this->hp << std::endl;
				out << "herodmg" << " " << this->dmg << std::endl;
				out << "herolvl" << " " << this->lvl << std::endl;
				out << "heroxp" << " " << this->xp << std::endl;
				out << "herocrit" << " " << this->crit << std::endl;
				out << "heroblock" << " " << this->block << std::endl;
			};
		};

	};

	item shopitems[10];

	item genitem(int nitem) {
		item newitem;
		int noitem = 1 + rand() % (hero.lvl + (1 + rand() % 3));
		if (noitem == 1 and newitem.slot == "head")
		newitem.name = "простой шлeм";
		else if (noitem == 2 and newitem.slot == "head")
			newitem.name = "шлeм";
		else if (noitem == 3 and newitem.slot == "head")
			newitem.name = "простой шлeм";
	
		cout << newitem.name << endl;
		return newitem;
	};

	class mob : public character {

	public:
		string name;
		int hp = 75;
		int dmg = 15;
		int xp = 15;
		int lvl = 1;
		int crit = 10;
		int block = 15;
		string type;
	
			game::mob() {
				string setname;
				srand(time(0));
			int name = 1 + rand() % 5;
			switch (name) {
			case 1: setname = "Разбойник"; break;
			case 2: setname = "Волк"; break;
			case 3: setname = "Медведь"; break;
			case 4: setname = "Кабан"; break;
			case 5: setname = "Людоед"; break;
				}
			this->name = setname;
			}

			~mob() {
				deadbody body;
				body.type = this->type;
			}
	};

	character hero;
	mob mob;

	class deadbody {
	public:
		int lifetime = 30;
		string type;
	};

	void battle() {
		cout << "Вы сражаетесь с " << mob.name << " " << mob.lvl + (rand() % 3)<< " уровня" << endl;
		cout << endl;
		while (hero.hp > 0 and mob.hp > 0) {
			int heroatk = 1 + rand() % 100;
			int mobatk = 1 + rand() % 100;
			int heroblock = 1 + rand() % 100;
			int mobblock = 1 + rand() % 100;
			if (heroblock < hero.block) {
				cout << "Вы блокируете атаку." << endl;
			}
			else {
				if (mobatk < mob.crit) {
					cout << mob.name << " наносит вам критический удар " << (3 + rand() % mob.dmg) * 2 << "x2 урона" << endl;
					hero.hp -= (3 + rand() % mob.dmg)*2;
				}
				else {
					cout << mob.name << " наносит вам " << 3 + rand() % mob.dmg << " урона" << endl;
					hero.hp -= 3 + rand() % mob.dmg;
				}
			}
			if (mobblock < mob.block) {
				cout << mob.name << " блокирует атаку." << endl;
			cout << endl;
		}
			else {
				if (heroatk < hero.crit) {
					cout << "Вы наносите критический удар" << mob.name << " " << (5 + rand() % hero.dmg)*2 << "x2 урона" << endl;
					mob.hp -= (5 + rand() % hero.dmg)*2;
					cout << endl;
				}
				else {
					cout << "Вы наносите " << mob.name << " " << 5 + rand() % hero.dmg << " урона" << endl;
					mob.hp -= 5 + rand() % hero.dmg;
					cout << endl;
				}
				if (mob.hp <= 0) {
					cout << "Вы получили " << mob.xp << " опыта." << endl;
				
				}
			}
		}
		hero.xp = hero.xp + mob.xp;
	}
}

[maks1331]

Надо смотреть почему файловый поток не открывается. И да, писать любой значимый код, не считая очистки в деструкторе объетка считается плохой практикой

[Kukla19]

Сообщение от maks1331

Надо смотреть почему файловый поток не открывается. И да, писать любой значимый код, не считая очистки в деструкторе объетка считается плохой практикой

Спасибо!

[BDA]

А еще каждый вызов rand() возвращает новое число, так что в коде функции battle вы печатаете одни числа, а отнимаете другие, что может ввести в заблуждение. Сохраняйте результат выражения во временную переменную, выводите и отнимайте её.

[Kukla19]

После каждой команды battle накапливается опыт и изменяется здоровье, но не записывается в файл.

Возможно это из-за goto?
А когда я хардово (ну типа кнопкой крестиком) отключаю прогу деструктор срабатывает?