Система урона на php цикле, не всегда победа, почему?

Question

[Bicbai]

$mobxp = rand(2,5);//хп моба
					$mobyr = 1;//урон
					//------------------------
					$pitxp = 9;//хп питомца
					$pityr = 1;//урон
					//------------------------
					$vsego = $mobxp;
					//------------------------
					$sdelanudar = 0;
					$x=0;
					while ($x <= $vsego)
					{
						if($sdelanudar == 0) {
							if($mobxp == 0){//Если хп нет победил
								$sdelanudar = 1;
								//$x = $vsego;
								break;
							} else {
								$mobxp - $pityr;//Ударяет монстра
								$sdelanudar = 1;
							}
						} else if($sdelanudar == 1) {
							if($pitxp == 0){//Если хп нет проиграл
								$sdelanudar = 0;
								//$x = $vsego;
								break;
							} else {
								$pitxp - $mobyr;//Ударяет питомца
								$sdelanudar = 0;
							}
						}
						$x++; // Увеличение счетчика
						//echo $x;
					}
					if($sdelanudar == 0) {//не смог
						exit;
					} else if($sdelanudar == 1) {//победил
						exit;
					}

Что не так с моим циклам, почему когда у $pitxp цифра больше, получается поражение. Как сделать чтобы всё работало чётко. Ото это странно, когда у монстра 2 здоровья и ты проигрываешь со своими 9 здоровьем. Сейчас оно не всегда выдаёт победу, не могу понять почему (

Comments

[twobomb]

Вах щикарно. Игра - на php! Чего заслуживают только имена переменных.
И в итоге хоть ты смог хоть не смог результат всегда один, еще exit на die поменять и всё как в жизни будет.
А вот это интересный момент $pitxp - $mobyr;
Тут мне кажется должно быть или

$pitxp -= $mobyr;
//Или
$pitxp =$pitxp - $mobyr;

[Bicbai]

twobomb, разве? $pitxp - $mobyr; // это из 8 - 1 - вот что это.
каждый раз вычитает -1 число из основного
Ну я попробую потом. просто я не вижу в этом разницы) Оно же вычитается)

И в итоге хоть ты смог хоть не смог результат всегда один, еще exit на die поменять и всё как в жизни будет.

Это изменённый код, на практике там переменные аккаунта. С нужными цифрами питомца. Поэтому и результат будет разным. Просто сейчас это работает иногда странно. Потому что когда у тебя 9 здоровья. а у монстра 2 и ты проиграл. Это бред! При том что урон одинаковый. 1 и 1

[twobomb]

Bicbai, Хм, ну можно и так сказать. Вычитается но не сохраняется.
Можно и так написать

...
              } else {
                 8 - 1;
                2 * 5;
                "Hello world";
                $sdelanudar = 1;
              }
....

Это не ошибка, оно все вычитается всё работает
P.S. Для начала подучите язык

Answer 1

[Илья]

Вообще странно что у вас что-то работает если вы переменные даже не присваиваете. Вместо:
$mobxp - $pityr;//Ударяет монстра
Нужно:
$mobxp -= $pityr;//Ударяет монстра

Для питомца та же история. И вообще используйте нормальные названия и ООП. Так, совет на будущее который наверняка проигнорится

Comments

[Bicbai]

Не правильные поражения всё равно остались!

Можете дать рабочий вариант? Чтобы цикл работал как надо!

[Илья]

Bicbai, цикл работает как надо, в логике проблема. Вы сначала назовите переменные нормально, и просто текстом на русском распишите логику что и как должно делаться, а потом уже переносите это все на код PHP. У вас сейчас кол-во ударов, равно HP моба. Урон наносится через раз (т.е. 1 раз получает моб, 1 раз petyr, чтобы это не значило), дак и каким образом у вас вообще должна получиться победа? Вы если сами на уровне логики не понимаете что происходит, то это точно не будет работать на уровне кода

[Bicbai]

Илья, Сейчас я переделал цикл, работает как надо!
ps: я понимаю то что я там сделал. я не настолько нуб каким вы меня видите! Осуждаю.

[Илья]

Bicbai, ну замечательно что вы понимаете, но это не значит что у вас сделано хорошо. По факту у вас тут 1 цикл со странно-сложной логикой, если бы и в этом не разбирались, то было бы совсем грустно. Без обид. Но суть в том, что если бы вы сделали пару классов, то было бы понятнее и проще искать косяк.