coucou747
Messages postés12303Date d'inscriptionmardi 10 février 2004StatutMembreDernière intervention30 juillet 201244 28 oct. 2005 à 20:07
idem...
l'idée est pas mal, mais postes le code ici, on pourra surement t'aider...
monoceros01
Messages postés420Date d'inscriptionvendredi 28 novembre 2003StatutMembreDernière intervention20 mars 2006 28 oct. 2005 à 13:20
C'est bizard ludo, j'ai beau cliquer il ne se passe rien o_o
Pourtant je suis sur firefox, je choisi l'item et bon y a pas de bouton valider donc je pense que c'est ça qui manque o.o
cs_Anthomicro
Messages postés9433Date d'inscriptionmardi 9 octobre 2001StatutMembreDernière intervention13 avril 20078 28 oct. 2005 à 12:09
bah je pense que c'est affaire de goût, je dirais plutôt "une fenêtre popup" et "un popup"
je sais pas pourquoi, chacun dit comme il veut je pense ;-)
ImmortalPC
Messages postés954Date d'inscriptionmardi 11 mai 2004StatutMembreDernière intervention11 novembre 20082 28 oct. 2005 à 11:55
lol
on dit pas "une" popup ?
cs_Anthomicro
Messages postés9433Date d'inscriptionmardi 9 octobre 2001StatutMembreDernière intervention13 avril 20078 28 oct. 2005 à 11:40
Bouh un popup ^^
ImmortalPC
Messages postés954Date d'inscriptionmardi 11 mai 2004StatutMembreDernière intervention11 novembre 20082 28 oct. 2005 à 11:31
Salut,
Si tu veux un bon code revoie le de a-z
Voici ce que moi j'ai fait récément :
http://abctest.net/ @+
cs_Anthomicro
Messages postés9433Date d'inscriptionmardi 9 octobre 2001StatutMembreDernière intervention13 avril 20078 28 oct. 2005 à 10:22
y'a toujours autant de requêtes mal faites et inutiles, on ne peut pas personnaliser les réponses... etc...
ludovicanceaux
Messages postés440Date d'inscriptionmardi 21 octobre 2003StatutMembreDernière intervention 6 mai 2010 28 oct. 2005 à 01:45
lol, en fait je l'ai modifié entre temps regarde et dis moi si tu trouve autant de faille k'avant!!!!!!!!!!!!
cs_Anthomicro
Messages postés9433Date d'inscriptionmardi 9 octobre 2001StatutMembreDernière intervention13 avril 20078 28 oct. 2005 à 00:30
ce qui n'améliore pas le script et le rend d'autant plus inutile :-(
ludovicanceaux
Messages postés440Date d'inscriptionmardi 21 octobre 2003StatutMembreDernière intervention 6 mai 2010 27 oct. 2005 à 23:41
nan, mais c'etai juste pour te dire k'il y a possibilité de mettre 2 sondages sur la même page, puis ke tu pe mettre la kestion ke tu ve (ce n'est ke du texte) etc.....
coucou747
Messages postés12303Date d'inscriptionmardi 10 février 2004StatutMembreDernière intervention30 juillet 201244 27 oct. 2005 à 22:31
je ne regardes que les sources ...
si je vais sur ton site, je n'ai pas les sources php...
<html>
<head>
<!-- DEBUT DU SCRIPT -->
<SCRIPT LANGUAGE="JavaScript">
bon, déja, dans les premières lignes, t'es pas régulier : des majuscules et des minuscules dans le code html... ensuite, type="text/javascript" est préférable à language.
<!--BALISES META-->
<TITLE>bing[...]
bon, deux choses, on sait ce qu'est une balise meta... alors ne met pas de commentaire pour la signaler... de plus, ici, c'est la balise de titre, pas meta...
<table width="752" border="0" align="center" cellpadding="0" cellspacing="0" class="texteclaire">
dès la première ligne après le body, tu aurais pu utiliser des CSS, et tu ne l'as pas fait...
BORDER=0
il manque des guillemets...
ludovicanceaux
Messages postés440Date d'inscriptionmardi 21 octobre 2003StatutMembreDernière intervention 6 mai 2010 27 oct. 2005 à 21:51
coucou747
Messages postés12303Date d'inscriptionmardi 10 février 2004StatutMembreDernière intervention30 juillet 201244 27 oct. 2005 à 10:50
cf mon premier commentaire, mais j'ajoutes :
if ($Nul <> 0)
équivaut à if ($Nul != 0), mais c'est plus corect..
$Nul = $Nul + 1; doit être remplacé par $Nul++;
<td height="168"> sinon, les CSS seraient un +...
Aparement, on ne peut pas choisir les réponses, et on ne peut pas non plus mettre plusieurs sondages, bref, je n'ai pas vu les fruits de ton amélioration... t'as pas tennu compte de mes remarques....
monoceros01
Messages postés420Date d'inscriptionvendredi 28 novembre 2003StatutMembreDernière intervention20 mars 2006 27 oct. 2005 à 07:47
ludovicanceaux
Messages postés440Date d'inscriptionmardi 21 octobre 2003StatutMembreDernière intervention 6 mai 2010 26 oct. 2005 à 21:25
J'ai fai la MAJ
oooook
Messages postés1Date d'inscriptionmercredi 26 octobre 2005StatutMembreDernière intervention26 octobre 2005 26 oct. 2005 à 01:14
je capte pas je dois un base de donnée et je met quoi dedans? rien n'est précisé je comprend rien
ludovicanceaux
Messages postés440Date d'inscriptionmardi 21 octobre 2003StatutMembreDernière intervention 6 mai 2010 3 oct. 2005 à 20:59
bah ok je t'remercie ,c vrai ke c + beau je ne pensai pô faire ce du tout!!!!!
FhX
Messages postés2350Date d'inscriptionmercredi 13 octobre 2004StatutMembreDernière intervention18 avril 20153 3 oct. 2005 à 18:16
Jte refait ton code un peu plus jolie :)
<?php
$query = mysql_query("SELECT MAX(tres_bien)as tres_bien,
MAX(bien) as bien,
MAX(moyen) as moyen,
MAX(nul) as nul FROM sondage1");
$result = mysql_fetch_array($query);
// Donc, $result['tres_bien'] contient la valeur de Très bien etc...
//La c'est + beau ;)
?>
cs_Anthomicro
Messages postés9433Date d'inscriptionmardi 9 octobre 2001StatutMembreDernière intervention13 avril 20078 30 sept. 2005 à 20:12
bah un sondage ne reste pas fixe éternellement, tu poses pas toujours la même question, sinon je vois pas l'intérêt...
Ensuite c'est vrai que c'est harchi crade mais bon...
ludovicanceaux
Messages postés440Date d'inscriptionmardi 21 octobre 2003StatutMembreDernière intervention 6 mai 2010 30 sept. 2005 à 20:05
de toute facon, je ne vois pô l'interet de vouloir faire plusieurs inscriptions, ou de se déconnecter plusieurs fois ou alors encore de tromper le code rien que pour voter plusieurs fois, ce n'est qu'un sondage!!!!!!!
Et pour la propreté du code je vous l'accorde, mais ca fait seulement 1 mois ke je fais du php et seulement 1 semaine ke je m'interresse o mySQL, dc bah voilà!!!!!!
cs_Anthomicro
Messages postés9433Date d'inscriptionmardi 9 octobre 2001StatutMembreDernière intervention13 avril 20078 30 sept. 2005 à 18:47
"Ba de toutes facons, le mieux pour un sondage, c'est de ne proposer les votes qu'à des membres inscrits d'une manière ou d'une autre. Car il existe toujours des solutions pour tromper le script et voter plusieurs fois."
Après c'est pareil je suis d'accord avec toi, mais tu peux t'inscrire plusieurs fois :-)
FhX
Messages postés2350Date d'inscriptionmercredi 13 octobre 2004StatutMembreDernière intervention18 avril 20153 30 sept. 2005 à 18:34
Moi ce qui me chagrine le plus, c'est pas forcément l'utilisation du code, mais sa propreté. Car quand on met 4 requetes pour avoir 4 MAX() alors qu'une seule requète aurait pu suffir, je trouve ca un peu limite !
Ensuite, des echo() sans ; à la fin des phrases, je trouve ca très moche.
De plus, dans ton switch(), tu fais 4 versions d'un INSERT dans ta DB. Pourquoi ne pas faire une variable commune et l'utiliser ensuite pour faire ton query ?
Genre :
switch($x) {
case y:
$sql = "INSERT INTO ...";
break;
case x:
$sql = "INSERT INTO ...";
break;
}
mysql_query($sql);
Avant ton switch, initialise tes variables :
$tres_bien = 0;
$bien = 0;
etc...
Sinon, tu te retrouves avec des erreurs de notices... beurk quoi :)
Bref, encore un peu de boulot niveau propreté =)
massacr
Messages postés233Date d'inscriptionvendredi 2 juillet 2004StatutMembreDernière intervention 4 janvier 2007 30 sept. 2005 à 18:10
Ba de toutes facons, le mieux pour un sondage, c'est de ne proposer les votes qu'à des membres inscrits d'une manière ou d'une autre. Car il existe toujours des solutions pour tromper le script et voter plusieurs fois.
ImmortalPC
Messages postés954Date d'inscriptionmardi 11 mai 2004StatutMembreDernière intervention11 novembre 20082 30 sept. 2005 à 16:53
Merci :-)
cs_Anthomicro
Messages postés9433Date d'inscriptionmardi 9 octobre 2001StatutMembreDernière intervention13 avril 20078 30 sept. 2005 à 15:41
" Bah si tu met le texte ke tu ve!!!!!!!!!!, et ce n'est qu'un sondage tu ve mettre koi toi ds le tien???????"
Ce que je veux dire par là c'est qu'il faut agir au niveau du code php pour modifier l'intitulé de ton sondage, il aurait été judicieux de faire une interface d'admin des sondages existants.
"concernant le fait qu'on ne puisse pas voter 2 fois à cause de l'ip... lol... il suffit de redémarrer la connexion... et hop, une nouvelle ip."
ouais, vas redémarrer 36 fois ta connexion, tu vas vite arrêter...
"non, le mieux, c'est d'avoir un cookie"
et là tu fais "refuser les cookies" dans ton navigateur et hop ton code fonctionne plus et il est possible de voter encore plus rapidement (sauf si tu empêches le vote si le navigateur refuse les cookies)
"Si non pour éviter de voter 2 fois moi j'utilise les cookie, l'ip, les session et le host."
ça c'est propre ;-)
ImmortalPC
Messages postés954Date d'inscriptionmardi 11 mai 2004StatutMembreDernière intervention11 novembre 20082 30 sept. 2005 à 10:38
Salut,
l'@ c'est pour éviter d'afficher l'erreur ( si il y en a une ).
Et die c'est si il y a une erreur tu peut afficher un message et ça arrete le script.
Si non pour éviter de voter 2 fois moi j'utilise les cookie, l'ip, les session et le host.
@+
sebmafate
Messages postés4936Date d'inscriptionlundi 17 février 2003StatutMembreDernière intervention14 février 201437 30 sept. 2005 à 08:53
concernant le fait qu'on ne puisse pas voter 2 fois à cause de l'ip... lol... il suffit de redémarrer la connexion... et hop, une nouvelle ip.
non, le mieux, c'est d'avoir un cookie
ludovicanceaux
Messages postés440Date d'inscriptionmardi 21 octobre 2003StatutMembreDernière intervention 6 mai 2010 30 sept. 2005 à 01:12
le fait de mettre @ ou or die etc.... ca change koi exactement????
ludovicanceaux
Messages postés440Date d'inscriptionmardi 21 octobre 2003StatutMembreDernière intervention 6 mai 2010 30 sept. 2005 à 01:11
Bah si tu met le texte ke tu ve!!!!!!!!!!, et ce n'est qu'un sondage tu ve mettre koi toi ds le tien???????
cs_Anthomicro
Messages postés9433Date d'inscriptionmardi 9 octobre 2001StatutMembreDernière intervention13 avril 20078 29 sept. 2005 à 21:54
Salut,
heu apparemment à part "bien, très bien" etc... tu peux pas mettre autre chose sans grosse modif ?
coucou747
Messages postés12303Date d'inscriptionmardi 10 février 2004StatutMembreDernière intervention30 juillet 201244 29 sept. 2005 à 21:24
selon moi, tu devrais utiliser des or die au lieu de mettre des @, et tu devrais aussi mettre des <?php à la place de <?
$Adresse_IP=$_SERVER['REMOTE_ADDR'];
une ip = 4 octets, ça tient dans un int sous la forme suivante :
$ip=explode('.',$_SERVER['REMOTE_ADDR']);
$ip=$ip[0]*256*256*256+$ip[1]*256*256+$ip[2]*256+$ip[3];
28 oct. 2005 à 20:07
l'idée est pas mal, mais postes le code ici, on pourra surement t'aider...
28 oct. 2005 à 13:20
Pourtant je suis sur firefox, je choisi l'item et bon y a pas de bouton valider donc je pense que c'est ça qui manque o.o
28 oct. 2005 à 12:09
je sais pas pourquoi, chacun dit comme il veut je pense ;-)
28 oct. 2005 à 11:55
on dit pas "une" popup ?
28 oct. 2005 à 11:40
28 oct. 2005 à 11:31
Si tu veux un bon code revoie le de a-z
Voici ce que moi j'ai fait récément :
http://abctest.net/
@+
28 oct. 2005 à 10:22
28 oct. 2005 à 01:45
28 oct. 2005 à 00:30
27 oct. 2005 à 23:41
27 oct. 2005 à 22:31
si je vais sur ton site, je n'ai pas les sources php...
<html>
<head>
<!-- DEBUT DU SCRIPT -->
<SCRIPT LANGUAGE="JavaScript">
bon, déja, dans les premières lignes, t'es pas régulier : des majuscules et des minuscules dans le code html... ensuite, type="text/javascript" est préférable à language.
<!--BALISES META-->
<TITLE>bing[...]
bon, deux choses, on sait ce qu'est une balise meta... alors ne met pas de commentaire pour la signaler... de plus, ici, c'est la balise de titre, pas meta...
<table width="752" border="0" align="center" cellpadding="0" cellspacing="0" class="texteclaire">
dès la première ligne après le body, tu aurais pu utiliser des CSS, et tu ne l'as pas fait...
BORDER=0
il manque des guillemets...
27 oct. 2005 à 21:51
27 oct. 2005 à 10:50
if ($Nul <> 0)
équivaut à if ($Nul != 0), mais c'est plus corect..
$Nul = $Nul + 1; doit être remplacé par $Nul++;
<td height="168"> sinon, les CSS seraient un +...
Aparement, on ne peut pas choisir les réponses, et on ne peut pas non plus mettre plusieurs sondages, bref, je n'ai pas vu les fruits de ton amélioration... t'as pas tennu compte de mes remarques....
27 oct. 2005 à 07:47
http://monoceros01.free.fr/code.php?script=suggestions_sondage
26 oct. 2005 à 21:25
26 oct. 2005 à 01:14
3 oct. 2005 à 20:59
3 oct. 2005 à 18:16
<?php
$query = mysql_query("SELECT MAX(tres_bien)as tres_bien,
MAX(bien) as bien,
MAX(moyen) as moyen,
MAX(nul) as nul FROM sondage1");
$result = mysql_fetch_array($query);
// Donc, $result['tres_bien'] contient la valeur de Très bien etc...
//La c'est + beau ;)
?>
30 sept. 2005 à 20:12
Ensuite c'est vrai que c'est harchi crade mais bon...
30 sept. 2005 à 20:05
Et pour la propreté du code je vous l'accorde, mais ca fait seulement 1 mois ke je fais du php et seulement 1 semaine ke je m'interresse o mySQL, dc bah voilà!!!!!!
30 sept. 2005 à 18:47
Après c'est pareil je suis d'accord avec toi, mais tu peux t'inscrire plusieurs fois :-)
30 sept. 2005 à 18:34
Ensuite, des echo() sans ; à la fin des phrases, je trouve ca très moche.
De plus, dans ton switch(), tu fais 4 versions d'un INSERT dans ta DB. Pourquoi ne pas faire une variable commune et l'utiliser ensuite pour faire ton query ?
Genre :
switch($x) {
case y:
$sql = "INSERT INTO ...";
break;
case x:
$sql = "INSERT INTO ...";
break;
}
mysql_query($sql);
Avant ton switch, initialise tes variables :
$tres_bien = 0;
$bien = 0;
etc...
Sinon, tu te retrouves avec des erreurs de notices... beurk quoi :)
Bref, encore un peu de boulot niveau propreté =)
30 sept. 2005 à 18:10
30 sept. 2005 à 16:53
30 sept. 2005 à 15:41
Ce que je veux dire par là c'est qu'il faut agir au niveau du code php pour modifier l'intitulé de ton sondage, il aurait été judicieux de faire une interface d'admin des sondages existants.
"concernant le fait qu'on ne puisse pas voter 2 fois à cause de l'ip... lol... il suffit de redémarrer la connexion... et hop, une nouvelle ip."
ouais, vas redémarrer 36 fois ta connexion, tu vas vite arrêter...
"non, le mieux, c'est d'avoir un cookie"
et là tu fais "refuser les cookies" dans ton navigateur et hop ton code fonctionne plus et il est possible de voter encore plus rapidement (sauf si tu empêches le vote si le navigateur refuse les cookies)
"Si non pour éviter de voter 2 fois moi j'utilise les cookie, l'ip, les session et le host."
ça c'est propre ;-)
30 sept. 2005 à 10:38
l'@ c'est pour éviter d'afficher l'erreur ( si il y en a une ).
Et die c'est si il y a une erreur tu peut afficher un message et ça arrete le script.
Si non pour éviter de voter 2 fois moi j'utilise les cookie, l'ip, les session et le host.
@+
30 sept. 2005 à 08:53
non, le mieux, c'est d'avoir un cookie
30 sept. 2005 à 01:12
30 sept. 2005 à 01:11
29 sept. 2005 à 21:54
heu apparemment à part "bien, très bien" etc... tu peux pas mettre autre chose sans grosse modif ?
29 sept. 2005 à 21:24
$Adresse_IP=$_SERVER['REMOTE_ADDR'];
une ip = 4 octets, ça tient dans un int sous la forme suivante :
$ip=explode('.',$_SERVER['REMOTE_ADDR']);
$ip=$ip[0]*256*256*256+$ip[1]*256*256+$ip[2]*256+$ip[3];