CODE ATTAQUE CRITIQUE

CODE ATTAQUE CRITIQUE

Zamentur
Hors ligne

Manitou

Messages : 814
Sujets : 44
Inscription : Jan 2007

#16

24-10-2007, 03:24 PM (Modification du message : 24-10-2007, 03:29 PM par Zamentur.)

Code PHP :
<?php 

$reponse = mysql_query("SELECT * FROM unites where login='test'");

$donnees = mysql_fetch_array($reponse);

$rangersatt=$donnees['rangers'];

$marinesatt=$donnees['marines'];

$snipersatt=$donnees['snipers'];

$Troupes_attaquantes=($rangersatt+$marinesatt+$snipersatt);

$reponse = mysql_query("SELECT * FROM unites where login='jldbaro'");//installer le def

$donnees = mysql_fetch_array($reponse);

$rangersdef=$donnees['rangers'];

$marinesdef=$donnees['marines'];

$snipersdef=$donnees['snipers'];

$Troupes_défendantes=($rangersdef+$marinesdef+$snipersdef);

echo"total: $Troupes_défendantes<br /> <br />  ";

$rangersvie=1000;

$marinesvie=1300;

$snipersvie=700;

$rangersdegats1=15;

$marinesdegats1=20;

$snipersdegats1=40;

$rangersdegats2=25;

$marinesdegats2=32;

$snipersdegats2=67;

$rangersvietotaldef=($rangersvie*$rangersdef);

$marinesvietotaldef=($marinesvie*$marinesdef);

$snipersvietotaldef=($snipersvie*$snipersdef);

$rangersvietotalatt=($rangersvie*$rangersatt);

$marinesvietotalatt=($marinesvie*$marinesatt);

$snipersvietotalatt=($snipersvie*$snipersatt);

$snipers_def_temp=true;

$snipers_att_temp=true;

$marines_def_temp=true;

$marines_att_temp=true;

Ben perso là dans ce cas précis j'aurais creer un array, je trouve que tes variables se repetent beaucoup et presentent des similitudes
Avec un array il est possible de faire des traitement avec foreach alors que sinon il faut utilisé $$text_mavar (avec $text_mavar='marinesvietotalatt' par exemple)

donc j'aurrais fais ceci(enfin plus comme çà):

Code PHP :
<?php 

$reponse = mysql_query("SELECT marines,rangers,snipers FROM unites where login='test' LIMIT 1");

$att = mysql_fetch_array($reponse);

$reponse = mysql_query("SELECT marines,rangers,snipers FROM unites where login='jldbaro' LIMIT 1");

$def = mysql_fetch_array($reponse);

//On calcul le nombre d'unités dans chaque camps

$troupes_attaquantes=array_sum($att);

$troupes_défendantes=array_sum($def);

echo"total: $troupes_défendantes<br /> <br />  ";

//On charge les variables de config

/*

 dans l'absolue je regroupe ce type de variable dans un autre fichier que j'inclue.

 La raison est simple si tu as besoin de ces varaible dans un autre script,

 il faut eviter de faire un copier coller car quand tu voudras faire un reglage,

 il faudra changer sur les 2 script (si c'est pas 20 scripts :) )

Alors que l'include de ton fichier de config te permet de modifier çà une fois

Bref attention à la centralisation des données

*/

$vie=array(

   'rangers'=>1000,

   'rangers'=>1300,

   'snipers'=>700

);

$degats=array(

   'rangers'=>array(1=>15,25),

   'rangers'=>array(1=>20,32),

   'snipers'=>array(1=>40,67)

);

//Calcul de la vie total des 2 camps

/*Bon je trouve que la foreach raccourcie le code 

et là encore pour ajouter une unité les donnée sont plus centraliser 

(mais pas encore assez à mon sens...

Le & sert à transmettre la varaiable par reference et non en la clonnant.

Je le met juste pour le fun :)

*/

foreach($vie as &$unite=>&$vie_unite) {

   $vietotaldef[$unite]=($vie_unite*$def[$unite]);

   $vietotalatt[$unite]=($vie_unite*$att[$unite]);

   $att_temp[$unite]=true;

   $def_temp[$unite]=true;

}

/*Là tu remarques qu'avec ce systeme,

 il te sera beaucoup plus simple de passer tes variables en parametre

 et non en varaible global (et meme avec les variables global,

 çà raccourcie enormement le script visuelement)*/

Sinon j'ai remarqué une faute grave en passant:

Code PHP :
<?php 

header ('Location: index.php'); 

Tu vas me dire que çà marche trés bien, mais moi je repond que çà marche sur ta config serveur/navigateur et pas forcement sur les autres config!
Il m'ai arrivé une fois il ya longtemps de mettre une url relative dans le header comme çà et la redirection ne se faisait pas sur Firefox (ou IE je sais plus) . J'ai cherché durant 5h...

de3.php.net a écrit :Note: HTTP/1.1 demande une URI absolue comme argument de » Location:,
y compris le protocole, hôte et chemin absolu.
Mais certains navigateurs acceptent les URI relatives. Vous pouvez généralement utiliser les variables globales $_SERVER['HTTP_HOST'], $_SERVER['PHP_SELF'] et
dirname() pour construire vous-même une URI absolue :

Exemple 1740. Redirection à l'aide de header()
Code PHP :
<?php

header("Location: http://" . $_SERVER['HTTP_HOST']

. rtrim(dirname($_SERVER['PHP_SELF']), '/\\')

. "/" . $relative_url);

?>

Pour les fonction je pense pareil que les autres, il faut utiliser les parametre et les return !

Et aussi pour les echo qui se produisent avant le <html>
Il est possible de pallier à ce probleme soit avec les valeur de retour des fonction, soit avec http://de3.php.net/manual/fr/function.ob-start.php
et http://de3.php.net/manual/fr/function.ob-get-flush.php

D'ailleurs tant qu'on parle des textes, je te conseil de centraliser tes textes dans un fichier de config!
avec un array par exemple

Citation :Pourquoi?

Pour pouvoir corriger/modifier les textes sans se casser la tete à parcourir tout un code
Et en plus un jour tu pourras traduire ton site de cette façon... (en incluant un fichier de config different selon la langue)

Citation :Et si je veux mettre une varaible dans mon texte?

Je te conseil de regarder là...
http://de3.php.net/manual/fr/function.sprintf.php

et ton body il est ou?

Code PHP :
<?php 

<html>

<head>

<title>Espace Jeu WAR FACTORY</title>

</head>

<p align="center">

Sinon pour le calcul du temps de generation je te conseil de te creer une fonction dans un fichier de fonction à part que tu inclueras dans chacune de tes pages... Je supposes que tu n'affiches pas le temps generé que pour l'attaque...
Ainsi il te suffira de faire un lancement de ta fonction au debut de ton script
et un arret à la fin
Pour ce cas précis l'utilisation des classes peut etre utiles pour que les 2 fonction ai accés à la valeur temps Smile

donc à la fin il faudrait arriver à un trucs

Code PHP :
<?php 

require_once('time_generated.class.php');

$temps=new time_generated();

$temps->lancer_compteur();

//Ton script ...

$temps->arreter_compteur();

echo $temps->compteur;

Ce systeme te permetrait en plus de pouvoir optimiser tes scripts au niveau de leur temps de generatuion en regardant ce qui prend le plus de temps Smile

(et oui rien ne t'empeche de deplacer $temps->lancer_compteur(); et $temps->arreter_compteur(); ou meme d'en creer de nouveau)

En tout cas bravo pour la tentative de separation de la vue, des données et de l'action du script. (en dehors des echo des fonctions c'est en bonne voie)
Et l'utilisation des fonctions c'est bien sauf que dans ce cas précis et avec toutes tes variables global tes fonctions sont inutile... ( dsl )
L'interet des fonction c'est quand meme de les utiliser plusieurs fois et dans plusieurs endroit, là avec toutes ces variables global c'est pas possible!
Et c'est pour çà que les fonctions doivent etre le plus independante possible d'un script (en dehors des parametres). Et c'est aussi pour çà qu'il vaut mieux les mettre dans un fichier de fonction

Bon voilà, bon faut dire que j'ai pas regarder le code meme, j'ai pas eu le courage, j'ai donc fait des remarques uniquement sur la structure et pas sur l'action reel du code...

Si vous voulez voir ce que je fais, visitez mon Book!
Al-Gol
Date de l'idée:25/01/2005
Date de lancement officiel de Ragol la v1: 11/03/2006
Date de fermeture officiel de Ragol: 23/09/2008

« Sujet précédent | Sujet suivant »

Utilisateur(s) parcourant ce sujet : 2 visiteur(s)

Messages dans ce sujet

CODE ATTAQUE CRITIQUE - par jldbaro - 23-10-2007, 11:15 AM

Sujets apparemment similaires…
Sujet		Auteur	Réponses	Affichages	Dernier message
	Critique de mon Premier Vrai Script de POO	Eluox	21	6 744	30-05-2008, 06:38 PM Dernier message: Cartman34
	Algorithm/fonction D'un Code D'attaque	covin85	16	6 654	22-08-2007, 10:54 PM Dernier message: elazard
	code d'attaque bis	simonhibou89	1	1 821	27-01-2007, 09:43 AM Dernier message: tashimitsu
	[résolu] code d'attaque	simonhibou89	16	5 350	21-12-2006, 12:16 PM Dernier message: simonhibou89

JeuWeb Créez votre jeu par navigateur !