Criticas para programar mejor

HiToGoRoShi · #1 (**permalink**) 02/03/2010, 16:08

Que tal amigos, estaba aburrido y quise poner a prueba mi conocimiento tratando de hacer esas peleas de los juegos que hay en php, debe haber visto tipo BITEFIGHT, MAFIAS entre otros...

Bueno me gustaria que me den consejos y tips para programar mejor, que cosas no debo hacer, que debo hacer que le falat para que la programacion sea impecable y mas perfecta...

Código PHP:

Ver original<?php
session_start();
$tipo = "submit";
 
$vidaA = 100;
$vidaB = 100;
 
$hpA = 500;
$hpB = 500;
 
$hpcolorA = "#66FF00";
$hpcolorB = "#66FF00";
 
$heroA = "Lord of Avernus";
$heroB = "Warchief";
 
//Skils NOMBRES y ATAQUE
$skillA = "Death Coil";
$skillB = "Return";
 
$skillAdmg = rand(65,75);
$skillBdmg = 30;
 
//Atake
$atkA = rand(45,60);
$atkB = rand(55,65);
 
if($_POST['action'] == "true") {
 
//ATAQUE MAGICO
$da&#241;o = $atkB-($atkB*0.10);
    if ($_POST['atacar'] == "Skill") {
        //Atake Abaddon
        $msj= "+<b><font color=green>".$heroA."</font></b> uso <font color=brown><i>".$skillA."</i></font> en <b><font color=blue>".$heroB."</font></b> causando <b><font color=red>".$skillAdmg. "</font></b> daño magico<br>";
        //Atake Centauro
        $rpta = "-<b><font color=blue>".$heroB."</font></b> respondio al ataque de <b><font color=green>".$heroA."</font></b> causando <b><font color=red>".$da&#241;o. "</font></b> daño fisico<br>";
        //Calcular daño causado
        $hpBtmp = $skillAdmg;
        $hpAtmp = $da&#241;o;
    }
//Ataque FISICO
    else {
        $da&#241;o = $atkA-($atkA*0.08);
        $da&#241;o2 = $atkB-($atkB*0.10);
         
        $skill = rand(0,100);
        if ($skill <= 30) {
            $da&#241;oRE = $atkA-($atkA*0.20);
            $return= "--<b><font color=blue>".$heroB."</font></b> uso <font color=brown><i>".$skillB."</i></font> contra <b><font color=green>".$heroA."</font></b> devolviendo <b><font color=red>".$skillBdmg. "</font></b> de daño fisico y reduciendo el ataque en <font color=brown><b>".$da&#241;oRE."</b></font><br>";
            $hpBtmp = $da&#241;oRE;
            $hpAtmp = ($da&#241;o2+$skillBdmg);
        }else {
            $hpBtmp = $da&#241;o;
            $hpAtmp = $da&#241;o2;
        }
        //Ataque Abaddon
        $msj= "+<b><font color=green>".$heroA."</font></b> ataco a <b><font color=blue>".$heroB."</font></b> causando <b><font color=red>".$da&#241;o. "</font></b> daño fisico<br>".$return;
        //Ataque Centauro
        $rpta = "-<b><font color=blue>".$heroB."</font></b> respondio al ataque de <b><font color=green>".$heroA."</font></b> causando <b><font color=red>".$da&#241;o2. "</font></b> daño fisico<br>";
 
    }
    
//Calculando el % de la vida y la cantidad
 
    $hpA = (int)(($hpA - $hpAtmp));
    $_SESSION['hpAresult'] = $_SESSION['hpAresult']+(500-$hpA);
        
    $hpB = (int)(($hpB - $hpBtmp));
    $_SESSION['hpBresult'] = $_SESSION['hpBresult']+(500-$hpB);
    
    $vidaA = (int)($vidaA-($vidaA-(100*($hpA/500))));
    $_SESSION['vidaAresult'] = $_SESSION['vidaAresult']+(100-$vidaA);
    
    $vidaB = (int)($vidaB-($vidaB-(100*($hpB/500))));
    $_SESSION['vidaBresult'] = $_SESSION['vidaBresult']+(100-$vidaB);
    
//Colorear VIDAS
    if ($_SESSION['vidaAresult'] >= 50) {
        $hpcolorA = "#FF3300";
    }
    if ($_SESSION['vidaBresult'] >= 50) {
        $hpcolorB = "#FF3300";
    }
    
//Destruir la Session carajo
if ($_SESSION['hpAresult'] > 500 or $_SESSION['hpBresult'] > 500 ){
    session_destroy();
    $tipo = "hidden";
    $combate = "true";
}
 
if ($_SESSION['hpAresult'] < $_SESSION['hpBresult'] ){
    $ganador = $heroA;
}else{
$ganador = $heroB;  
}
 
print $_SESSION['hpAresult']." ".$_SESSION['hpBresult'];
}
?>
<form id="form1" name="form1" method="post" action="dota.php">
    <table width="100%">
        <tr>
            <td width="50%"><fieldset>
                    <legend style="font-weight: bold; font-family: Verdana, Geneva, sans-serif; font-size: 24px;">Abaddon</legend>
                    <table width="100%">
<tr>
                            <td width="12%"><span style="font-size: 36px; font-weight: bold;"><img src="abaddon.gif" alt="" width="64" height="64"></span></td>
                            <td width="76%" align="center" bgcolor="#333333" style="font-size: 36px; color: #FFF;"><?php print $heroA; ?></td>
                            <td width="12%" align="center" bgcolor="#333333"><img src="skill1.gif" alt="" width="64" height="64"></td>
                        </tr>
                        <tr>
                            <td colspan="3"><table width="100%">
                                    <tr>
                                        <td><span style="font-size: 14px; font-weight: bold; font-family: Verdana, Geneva, sans-serif;">Damage:</span></td>
                                        <td colspan="3"><span style="font-size: 14px; font-family: Verdana, Geneva, sans-serif;"> 45 - 60 </span></td>
                                    </tr>
                                    <tr>
                                        <td width="13%" style="font-weight: bold; font-family: Verdana, Geneva, sans-serif; font-size: 14px;">Armor:</td>
                                        <td colspan="3">10</td>
                                    </tr>
                                    <tr>
                                        <td><span style="font-size: 12px; font-weight: bold; font-family: Verdana, Geneva, sans-serif;">VIDA:</span></td>
                                      <td width="67%" bgcolor="#333333"><table width="%<?php print 100-$_SESSION['vidaAresult']; ?>">
                                                <tr>
                                                    <td bgcolor="<?php print $hpcolorA ;?>">&nbsp;</td>
                                                </tr>
                                            </table></td>
                                        <td width="20%" colspan="2" align="center"><span style="font-size: 18px; font-family: Verdana, Geneva, sans-serif;"> [<span style="font-weight: bold">500 </span>- <?php print 500-$_SESSION['hpAresult'];?>]</span></td>
                                    </tr>
                                </table></td>
                        </tr>
                        <tr>
                            <td></td>
                        </tr>
                    </table>
                </fieldset></td>
            <td width="50%"><fieldset>
                    <legend style="font-family: Verdana, Geneva, sans-serif; font-weight: bold; font-size: 24px;">Centauro</legend>
                    <table width="100%">
                      <tr>
                            <td width="12%"><span style="font-size: 36px; font-weight: bold;"><img src="centauro.gif" alt="" width="64" height="64"></span></td>
                            <td width="76%" align="center" bgcolor="#333333" style="font-size: 36px; color: #FFF;"><?php print $heroB; ?></td>
                            <td width="12%" align="center" bgcolor="#333333"><img src="skill1a.gif" alt="" width="64" height="64"></td>
                        </tr>
                        <tr>
                            <td colspan="3"><table width="100%">
                                    <tr>
                                        <td><span style="font-size: 14px; font-weight: bold; font-family: Verdana, Geneva, sans-serif;">Damage:</span></td>
                                        <td colspan="3"><span style="font-size: 14px; font-family: Verdana, Geneva, sans-serif;"> 55 - 65 </span></td>
                                    </tr>
                                    <tr>
                                        <td width="13%" style="font-weight: bold; font-family: Verdana, Geneva, sans-serif; font-size: 14px;">Armor:</td>
                                        <td colspan="3">8</td>
                                    </tr>
                                    <tr>
                                        <td><span style="font-size: 12px; font-weight: bold; font-family: Verdana, Geneva, sans-serif;">VIDA:</span></td>
                                        <td width="67%" bgcolor="#333333"><table width="%<?php print 100-$_SESSION['vidaBresult']; ?>>">
                                                <tr>
                                                    <td bgcolor="<?php print $hpcolorB ;?>">&nbsp;</td>
                                                </tr>
                                            </table></td>
                                        <td width="20%" colspan="2" align="center"><span style="font-size: 18px; font-family: Verdana, Geneva, sans-serif;"> [<span style="font-weight: bold">500 </span>- <?php print 500-$_SESSION['hpBresult'];?>]</span></td>
                                    </tr>
                                </table></td>
                        </tr>
                        <tr>
                            <td></td>
                        </tr>
                    </table>
                </fieldset></td>
        </tr>
        <tr>
            <td colspan="2"><input type="<?php print $tipo; ?>" name="atacar" id="atacar" value="Atacar">
                <input type="<?php print $tipo; ?>" name="atacar" id="atacar" value="Skill" />
                <input name="action" type="hidden" id="action" value="true"></td></tr>
    </table>
</form>
<?php
if ($_POST['action'] == "true") {
    ?>
<fieldset><legend>Resultado</legend>
    <table width="100%" bgcolor="#E8E8E8"><tr><td>
    <?php print $msj.$rpta; ?>
    </td></tr></table>
</fieldset>
 
    <?php
}
 
?>
 
<?php
if ($combate == true){
?>
<fieldset><legend>Resumen:</legend>
 
    <table width="100%"><tr><td align="center" bgcolor="#333333"><span style="font-size: 36px; color: #FFF;"><?php print $ganador; ?> ha resultado ser el ganador</span></td></tr>
 
    </table>
</fieldset>
<?php
}
?>

El ejemplo en linea: http://peruguitar.net/DOTA/dota.php

EXTRAS:

El centauro tiene un 20% de chance de utilizar su habilidad return el cual devuelve 30 de daño
Las armaduras defienden a los hereos del ataque fisico (no magico) , es decir ARMOR 10, un 10% de proteccion del ataque TOTAL causado

santris · #2 (**permalink**) 02/03/2010, 16:16

me ganó el Warchief con un ataque no se que!!

Pues yo tu código lo veo muy limpio, bien anidado con comentarios. Muy bien

Saludos

jackson666 · #3 (**permalink**) 02/03/2010, 20:15

Un error garrafal es usar caracteres no convencionales en los nombres de las variables, como por ejemplo

Código Desastre:

Ver original$daño;
$canción;
$áéíóú;

Y cosas por el estilo....

HiToGoRoShi · #4 (**permalink**) 02/03/2010, 22:35

te refieres a la ñ? pero no vota errores... igual tratare de no usarlso

memoadian · #5 (**permalink**) 02/03/2010, 23:08

me salio error en la linea 32???
T_IF mmm o soy solo yo?.

HiToGoRoShi · #6 (**permalink**) 02/03/2010, 23:51

Cita:

Iniciado por memoadian

me salio error en la linea 32???
T_IF mmm o soy solo yo?.

creo que solo tu, lo acabo de probar en FF e Internet Explorer y todo bien. D:

memoadian · #7 (**permalink**) 03/03/2010, 03:13

de verdad que me sale ese error, en fin yo tampoco le veo, pero hasta haria una captura de pantalla, si no me diera flojera.

CHuLoSoY · #8 (**permalink**) 03/03/2010, 05:18

Para empezar, utilizar eñes en los nombres de variables es una locura, puede que te funcione, pero en ese servidor exclusivamente.
Luego está el tema de las tablas. Yo las utilizo así que no debería hablar, pero se recomienda la distribución del contenido con cajas (<div>) y controladas con CSS.
El juego podrías meterle un poco de Javascript y lo harías muchísimo más dinámico.
Yo he empezado un piloto de juego(aguardando por mapas y personajes, además de historias xD yo me encargo únicamente de la programación) en www.bouzas2.es . Ahí verás que puedes pelear contra un macaco mal hecho (de prueba) y fíjate qué bien está la batalla (se juega con el teclado, puedes elegir arma, disparar y moverte).

Saludos!

memoadian · #9 (**permalink**) 03/03/2010, 08:43

Es que el chiste del juego es ese, que no tenga javascript si no puro php, en lo personal a mi me encantan esos juegos y estoy aprendiendo php para lograr hacer el mio. aunque falta un laaaaargo camino, quizas este año.

HiToGoRoShi · #10 (**permalink**) 03/03/2010, 09:06

bueno en realidad si vas a tener que usar javascript, para adornar mejor o el uso de AJAX