Salut,
Ton code pourrait etre largement simplifié grace aux commandes natives à php.

Exemple:

$url = "http://maps.google.com/maps/api/[...]&format=png";
$filename = "./files/mamap.png";

file_put_contents($filename, file_get_contents($url));

Evidement il manque la gestion d'erreur, mais tu vois l'idée.

Hello,

De l'objet pour un code comme ça n'aurait pas grand intérêt d'autant plus qu'il n'y a qu'une fonction et qu'instancier un objet est coûteux...

Par contre oui séparer le html du code php, la fonction devrait retourner un code d'erreur, thrower des exceptions ou encore retourner true/false mais pas afficher de html.

L'autre reproche qu'on pourrait faire c'est que récemment (enfin il y a quelques mois) une source similaire a été postée:
http://www.phpcs.com/codes/ALLUMER-MACHINE-DANS-RESEAU-WAKE-ON-LAN_45455.aspx


a+

@dededede4:

Hum je vois vraiment pas pourquoi un switch serait plus compliqué pour un débutant qu'un if/then/else.
Honnêtement, dès que tu pige ce que fait le switch tu peux pas vraiment ne pas comprendre...

De toute façon, les points à retenir de mon code n'est pas switch vs if, mais code indenté et propre vs pas indenté, laid, plein de mauvaises habitudes et bourré de fautes d'orthographe.

=== vs == fond la même chose dans ton cas, oui et alors ? En programmation le typage est la chose la plus basique et la plus essentielle.

Je vois que tu es débutant, tu ne l'as peut être pas encore appris, mais si des personnes plus expérimentées que toi tel que Codefalse, Teclis01 ou encore moi même te dise que quelques chose est mal fait ou mauvais, faudrait peut être remettre en doute tes connaissances aux lieu de dire qu'on a tord...


@Teclis01:
C'est vrai pour la coloraison, j'avais pas pensé ;-). Toutefois pour la vitesse, moi je dis que c'est à bencher :P

@Codefalse:
Ben voyons, ça ne va pas aujourd'hui ? T'as vraiment cru que Teclis en avait contre toi :P

Salut,

Si, mais on peut précharger l'image avec du javascript avant l'affichage, enfin sur un de mes sites le javascript préload avec un imgPreload = new Image(); imgPreload.src = "toto.png"; et tu l'affiche qu'une fois chargée. Ça évite les clignotements pour les mouseover et pour les défilements d'images dans mon cas.

Peut être que je me trompe, mais je suis persuadé que c'est possible parce que google me retourne 88 900 résultats pour preload image javascript :P

Sinon mes suggestions sont à prendre avec des pincettes, je n'ai pas du tout réfléchie à la quantité de travail qu'elles représentent :o

1. Je sais à quoi sert exit, c'est que lui avait mis le echo $msg à la fin, au moment d'écrire le exit je n'avais pas encore déplacé le echo.

2. Ouais on devrait remplacer case '' par un default.
3. Pourquoi ne pas afficher le form via un echo ? D'après toi c'est plus rapide d'ouvrir/fermer les balises php que de passer par un echo ?

De plus l'utilisation d'un default revient à ce que je fais, si le $_GET est pas setté, on le set pour pas que PHP affiche une notice.

En fait dans un vrai projet la conception de l'auteur de mettre le captcha et le formulaire dans le même script est simplement non viable, donc un tel débat/dilemme n'a pas lieu d'être :P.

Moi aussi je vois le zip, faudrait penser à se sortir la tête du c** et chercher un peu par soit même ! Et je suggère à mehdikobra de lire les règlements, pas de langage SMS :-/

J'aime bien ta source, j'ai eu beaucoup de plaisir à dessiner des champs avec des signes extra terrestres à l'aide de la fonction mettre en relief :)

Quelques petites suggestions toutefois:
- Précharger les images avant de les afficher: Ça éviterait un clignotement désagréable à chaque modification.

- Puisque tu gardes tous les fichiers temporaires des modifications, pourquoi ne pas ajouter une fonction "undo" ou "annuler un changement" ?

- Je sais pas si je m'y suis mal pris, mais chez moi la rotation ne fonctionne pas.

- Peut être stocker les fichiers temporaires un peu plus proprement. Du genre:
/tmp/session_id/image 1,2,3,4,5,6
Comme ça un dossier par session dans un sous dossier tmp/, ça fait moins "bordel"

- Peut être utiliser des "slider" ou "scrollbar" pour ajuster la luminosité/contraste plutôt que de demander une valeur avec des input

Je n'ai pas regardé le code du tout, seulement le résultat alors je vais m'arrêter là :P

Bonne continuation dans ton projet :-)

Afin de te donner un coup de main, j'ai réorganisé ton code pour le rendre plus clair:

<?php
session_start();
isset($_GET['pg']) or $_GET['pg'] = '';

switch($_GET['pg']) {
    case '':
        echo '<img src="index.php?pg=image" border="0" /><br />Tape le code :
              <form method="post" action="index.php?pg=verifie">
              <input type="text" name="code" /><input type="submit" value="OK" />
              </form>';
      break;
      
    case 'verifie':
        if ( isset($_POST['code']) && $_SESSION['chaine'] === $_POST['code'] ) {
          $msg = 'Bon';
        } else {
          $msg = 'Pas bon';
        }
        echo $msg;
        unset($_SESSION['chaine']);
      break;
      
    case 'image':
        header ('Content-type: image/png');
        $chaine = 'ABCDEFGHIJKLMNOPQRSTUVWXYZ123456789';
        $chaine = str_shuffle($chaine);
        $_SESSION['chaine'] = substr($chaine, 0, 5);
        $image = imagecreate(42,15);
        $noir = imagecolorallocate($image, 0, 0, 0);
        $blanc = imagecolorallocate($image, 255, 255, 255);
        imagestring($image, 4, 1, 0, $chaine, $blanc);
        imagepng($image);
        exit;
      break;
}
?>



À mon avis, le code que je viens de faire est beaucoup plus clair, simple et utile pour un débutant que ton code. Si je me trompe n'hésitez pas à me contredire ;)

Salut,

Aussi après avoir vérifié si l'image était valide ou pas tu dois détruire la variable de session, sinon ton captcha sert à rien, une fois qu'on a le bon code stocké en session on peut très bien flooder puisque le code ne changera pas tant qu'on ne réaffiche pas l'image...

Et teclis a bien raison, ton code manque cruellement d'indentation. On s'en fiche qu'il soit simple, si il n'est pas facile à lire ça ne sert à rien.

Dernière chose ton captcha se casse en quelques secondes avec un OCR, il faudrait ajouté quelques difficultés !

Bonne continuation
a++

On ne peut pas compiler du php en exe. Le programme en question "bamcompile" fait juste écrire le script php à la fin d'un launcher, qui n'est ni plus ni moins que le moteur php avec toutes ses librairies compilé statiquement.

Ça donne un fichier énorme, a vu de nez je dirais qu'un simple hello world doit faire pas loin d'un megaoctet alors qu'un hello world en C ça fait quelques octets a tout casser... Plus tu as besoin d'options dans php, plus le launcher sera gros.

Bref c'est pas mal n'importe quoi.


Bon maintenant une question pour l'auteur: Pourquoi google, qui est le site le plus commun et le plus simple qu'on puisse trouver,n'est pas pris en charge ?

Hello,

Une toute petite question, pourquoi tu ouvre un pointeur vers le stdin alors que php en initialise un directement, à savoir STDIN ?

Je me demande surtout si il y a une raison spécifique à ce choix, si STDIN a volontairement été oublié :P


Sinon excellente source, comme toujours, bien que je sois trop paresseux pour la tester, j'ai quand même pris la peine de lire le code ^^ :)