Forum PHP.pl

Pisze skrypt walki dwóch kont :

[PHP] pobierz, plaintext 
<?php
							$mypower = rand($pistolet[dmin],$pistolet[dmax])+$stat[hp]/10+$stat[strzelanie];
	$edefense = rand($pistolet2[dmin],$pistolet2[dmax])+$enemy[hp]/10+$enemy[refleks];
							$mydef = rand($pistolet[dmin],$pistolet[dmax])+$stat[hp]/10+$stat[refleks];
	$epower = rand($pistolet2[dmin],$pistolet2[dmax])+$enemy[hp]/10+$enemy[strzelanie];
 
	if ($stat[miasto] != $enemy[miasto] or $stat[page] != $enemy[page]){
 
print "Aby walczyć z przeciwnikiem musisz być w tym samym mieście oraz w tym samym pomieszczeniu co przeciwnik.";
echo '<br><br><h1><center><a href=' . $stat[miasto] . '.php>wróć do miasta</a><h1></center>';
 
 
}else{	
print "<center><b>$stat[user]   vs   $enemy[user]</b><br>";
	print "<center><br><img border=\"0\" src=\"$stat[avatar]\" width=\"80\" height=\"80\">";
	print "  <b>VS</b>  ";
	print "<img border=\"0\" src=\"$enemy[avatar]\" width=\"80\" height=\"80\"></center><br><br>";
 
$mojehp = ' . stat[hp] . ';
$ehp = ' . enemy[hp] . ';
 
while ($mojehp==0 or $ehp==0){
$los = rand(1,4);
If ($los = 1 or $los = 2){
$sila = 2;
mysql_query("update players set hp=hp-$sila where id=$enemy[id]");
print "<img src=\"images/good.jpg\">Strzelasz do <b>$enemy[user]</b>, zadejesz mu $sila obrażeń [pozostało $enemy[hp] życia]<br>";
}
}else{
$sila = 2;
mysql_query("update players set hp=hp-$sila where id=$stat[id]");
print "<img src=\"images/bad.jpg\"><b>$enemy[user]</b> strzela do Ciebie i zadej Ci $sila obrażeń [pozostało $stat[hp] życia]<br>";
}
}
}
?>
[PHP] pobierz, plaintext 

Ten skrypt powinien z kazda petla losowac kto uderzy i odejmowac 2 HP. Problem jest taki że atakuje zawsze tylko jeden gracz oraz nic nie odejmuje graczowi zycia.

Czyli user 1 atakuje 1000 razy usera 2 zadajac 2 pkt ograzen a ciagle przeciwnik ma 100 hp i walka sie nie konczy.

Ile razy można to przerabiać...
Sprawdź jak dokładnie wygląda zapytanie
echo "update players set hp=hp-$sila where id=$enemy[id]";
sprawdź czy się dobrze wykonuje - mysql_error().

O tym wszystkim masz napisane w temacie:
Temat: Jak poprawnie zada pytanie
który już ci dawałem do przestudiowania. Naprawdę podane tam informacje nie są naszym wybrykiem natury a przydatnymi informacjami w rozwiązywaniu banalnych problemów

edit:
while ($mojehp==0 or $ehp==0){
oznacza to, że pętla będzie się wykonywała dopóki twoje hp lub przeciwnika będzie równe zero - dziwny warunek.
Na dodatek, w pętli nigdzie nie zmieniasz tych wartości, więc jesli raz ci wejdzie w tę pętlę to z niej nie wyjdzie. Analizujesz czasami to co piszesz?

edit2:
$mojehp = ' . stat[hp] . ';
$ehp = ' . enemy[hp] . ';
A co to ma byc? przeciez przypisujesz tutaj tekst '. stat[hp] . ' a nie wartość $stat[hp]. Nic dziwnego ze masz spełniony warunek
while ($mojehp==0 or $ehp==0){
Tekst w porównaniu do 0 daje 0

Po pierwsze: A pętla while nie trwa do czasu aż warunek przestaje być prawdziwy? Jak więc while( $hp_moje == 0 || $hp_wroga == 0) ma być choć raz zrobione, skoro na dzień dobry ma FALSE? Jak dla mnie raczej while( $hp_moje > 0 && $hp_wroga > 0), bo wtedy przerwie w chwili gdy życie jednego spadnie do 0 lub niżej. To istotne zwłaszcza tym bardziej, że masz dynamiczne wartości ataku i obrony(!).

Po drugie: po co zarżynać bazę robiąc każdą akcję po kolei do niej? Prościej i o niebo wydajniej jest zapisywać wszystkie akcje/łupy/obrażenia w postaci tablicy, potem ją serializować i zapisać do bazy. Zwróć uwagę... Najpierw zapis wszystkiego do tablicy w PHP, a dopiero na koniec wrzucenie tego do bazy. Co lepsze? Zapis całej akcji w jednym rekordzie, czy kombinowanie z wieloma rekordami, których bez problemów może być kilkadziesiąt. Zwróć uwagę na wydajność. Obliczenia po stronie php są szybkie i całość ograniczy do nich+zapis do bazy 1 rekordu. Z drugiej strony kilkadziesiąt zapisów do bazy i brak w tej chwili zapisu stanu. Po prostu zamiast odejmować w PHP, robisz to w bazie. Gwoździe wbijasz za pomocą trzonka piły do drewna. Niby też można, tylko po co?

Po trzecie: wspomniane przez nospora zapomniane zmienianie wartości punktów hp obu graczy. To, że żaden z graczy nie walczy to efekt właśnie braku odejmowania i dodatkowo tego, że nawet nie wchodzi do pętli co opisałem w punkcie pierwszym.

@thek przeczytaj jeszcze raz moje edity z poprzedniego posta a zrozumiesz, że własnie mu wchodzi w pętle i nie chce wyjść

Jak może wejść w pętlę, skoro warunek w while jest nieprawidłowy? Ja rozumiem do-while i wykonanie się jednokrotne, bo wtedy OK. Ale powiedz jakim cudem przy jego warunku mogło do pętli wejść. Jedyne co to gdy chociaż jedna z nich była niezadeklarowana przed nią i php domyślnie ją utworzył oraz zainicjował zerem, bo wtedy spełniłby się warunek równości z 0. Innej możliwości nie widzę.

Aaaaa... Teraz widzę ten tekst zamiast zmiennych w Twoim edit2 Czyli niezainicjowane zmienne stają się zerami co ja opisałem jako podejrzenie. Swoją drogą odejmowanie życia graczom z użyciem bazy jest jak dla mnie poronione Gdyby to chociaż było w formie zapisu rekordu by potem raport wygenerować to jeszcze jakoś rozumiem, ale tak jak teraz to jest z deczka bezsensowne.

Ależ nie, zmienne są zainicjowane. Ino że tekstem:
$mojehp = ' . stat[hp] . ';
A sam wiesz jak wygląda przyrównanie tekstu do 0

edit: to też jest ciekawe:
If ($los = 1 or $los = 2){


@Lethys jesteś na forum tyle lat... mógłbyś już naprawdę znać podstawy języka php...

Obecnie mój skrypt wygląda tak :

[PHP] pobierz, plaintext 
$hp_moje = $stat[hp];
	print "moje życie $hp_moje<br>";
	$hp_wroga = $enemy[hp];
	print "życie przeciwnika $hp_wroga";
 
// sprawdzam czy życie nie jest wartością 0, jest 100 czyli wszystko ok
 
	while($hp_moje > 0 && $hp_wroga > 0){
 
// walka ma trwać do momentu kiedy jeden z graczy będzie miał 0 hp
 
$los = rand(1,4);
//losowanie kto pierwszy uderzy
If ($los = 1 or $los = 2){
 
// losowana siła uderzenia
$sila = rand(1,5);
 
// powinno odjąć graczowi z życia wartość wylosowanej siły, nie odejmuje oraz nie zwraca żadnego błędu
mysql_query("update players set hp=hp-$sila where id=$enemy[id]") or mysql_error();
print "<img src=\"images/good.jpg\">Strzelasz do <b>$enemy[user]</b>, zadejesz mu $sila obrażeń [pozostało $enemy[hp] życia]<br>";
 
// to samo tylko w przypadku kiedy przeciwnik pierwszy atakuje
}elseif ($los = 3 or $los = 4){
$sila = rand(1,5);
mysql_query("update players set hp=hp-$sila where id=$stat[id]") or mysql_error();
print "<img src=\"images/bad.jpg\"><b>$enemy[user]</b> strzela do Ciebie i zadej Ci $sila obrażeń [pozostało $stat[hp] życia]<br>";
}
}
[PHP] pobierz, plaintext 

dla mnie wszystko jest tutaj jasne czarne na białym a mimo to skrypt nie działa

W praktyce wygląda to tak :




Jest ktoś w stanie mi pomóc z tym skryptem ?

Z góry dziękuje

Dzięki nospor

zrobiłem to tak :

[PHP] pobierz, plaintext 
	while($hp_moje > 0 && $hp_wroga > 0){
$los = rand(1,4);
If ($los = 1 or $los = 2){
$sila = rand(1,5);
$hp_wroga = $hp_wroga - $sila; 
mysql_query("update players set hp=hp-$sila where id=$enemy[id]") or die(mysql_error());
print "<img src=\"images/good.jpg\">Strzelasz do <b>$enemy[user]</b>, zadejesz mu $sila obrażeń [pozostało $hp_wroga życia]<br>";
}elseif ($los = 3 or $los = 4){
$sila = rand(1,5);
$hp_moje = $hp_moje - $sila; 
mysql_query("update players set hp=hp-$sila where id=$stat[id]") or die(mysql_error());
print "<img src=\"images/bad.jpg\"><b>$enemy[user]</b> strzela do Ciebie i zadej Ci $sila obrażeń [pozostało $hp_moje życia]<br>";
}
}
}
[PHP] pobierz, plaintext 

I teoretycznie działa, tylko właśnie jest kolejny problem.

W pętli nie losuje kto będzie atakować tylko wyświetla.



przecież w kodzie mam zaraz na początku pętli zrobione losowanie.