Inscris-toi au e-camp "Héberge ton jeu Facebook sur Azure" de Microsoft vendredi 25 mai à 13h30 !

> Le Site du Zéro - Informatique

> Les forums

> Communauté des zéros

> Discussions générales

> Les artistes du SdZ

> Lecture du sujet

Les artistes du SdZ

Pour accéder à cette section
Connectez-vous !

Auteur	Message
Page Précédente 1 2 3 ... 26 27 28 29 30 31 32 Suivante
5 visiteurs sur ce sujet (5 anonymes)
Page Précédente 1 2 3 ... 26 27 28 29 30 31 32 Suivante
Nelty	# Posté le 03/02/2008 à 19:13:28
Garfield in the US Groupe : Anciens Blog Twitter close Ville : Owatonna Pays : États-Unis	Reprise du dernier message de la page précédente : Voici ce qu'à fait N1K0L45 : http://www.nelty.fr/sportregistrer.zip
Publicité	# Posté le 03/02/2008 à 19:13:28

Angua	# Posté le 03/02/2008 à 23:52:41
	Citation @ aux candidats : vous pouvez mettre les liens vers vos codes dans le sujet, ça ne pose pas de soucis . Même si mon travail n'est pas un "code", je suis un "candidat"; je suppose que ça s'adresse aussi à moi, donc ? Mon travail est là (c'est peut-être un peu sombre, comme j'ai un écran plat je ne me rends pas toujours compte):
anonyme	# Posté le 04/02/2008 à 07:04:48
	Citation : Nelty Voici ce qu'à fait N1K0L45 : http://www.nelty.fr/sportregistrer.zip Merci de l'avoir fait à ma place :-)
Craw	# Posté le 04/02/2008 à 16:24:00
Rien n'est parfait !	Citation : Angua Citation @ aux candidats : vous pouvez mettre les liens vers vos codes dans le sujet, ça ne pose pas de soucis . Même si mon travail n'est pas un "code", je suis un "candidat"; je suppose que ça s'adresse aussi à moi, donc ? Mon travail est là (c'est peut-être un peu sombre, comme j'ai un écran plat je ne me rends pas toujours compte): Oui ça s'adresse aussi à toi.
Guillawme	# Posté le 04/02/2008 à 19:11:45
Blog Twitter close Études : Paris 6 - Université Pierre et Marie Curie (Jussieu)	Et oui, c'est un peu sombre... Dommage, pour le peu que j'arrive à distinguer c'est joli. Édité le 04/02/2008 à 19:13:14 par Guillawme
Angua	# Posté le 04/02/2008 à 22:30:50
	Une version plus claire: Édité le 05/02/2008 à 18:23:25 par Angua
bluestorm	# Posté le 05/02/2008 à 19:24:49
dont ask to ask Groupe : Anciens Blog Twitter close	Je pense que le plus pratique serait de toute mettre (temporairement) sur un site commun (par exemple celui du concours). J'ai rapidement survolé (ou pas) quelques codes, voici mes premières impressions; j'ai pas regardé le "contenu", mais juste la "tête" du code. Cyprien_ Si un jour tu trouves un hébergeur moins laid que MegaUpload, n'hésite pas (parce que là j'avoue qu'activer le flash, attendre, regarder la pub pendant le DL, je suis pas motivé). Ceci dit c'est moi qui suis curieux, donc théoriquement je pourrais faire un effort. savageman Des tests un peu bizarres dans les "if" : Code : PHP - Afficher / masquer les numéros de ligne if (1 == get_magic_quotes_gpc()) Code : PHP - Afficher / masquer les numéros de ligne if (false == $albums) Code : PHP - Afficher / masquer les numéros de ligne if (!empty($_POST['titre']) && !empty($_POST['description']) && !isset($_GET['id'])) { ... } if (!empty($_POST['titre']) && !empty($_POST['description']) && isset($_GET['id']) && array_key_exists($_GET['id'], $albums)) { ... } Ça manque d'une fonction auxiliaire message : : Code : PHP - Afficher / masquer les numéros de ligne function message($type, $message) { $_SESSION['message'] = array('type' => $type, 'texte' => $message); } Avec ça, tu pourrais simplifier plein de lignes redondantes (d'après grep, le code "array('type' =>" apparaît 17 fois dans ton fichier). Du code répétitif visuellement : Code : PHP - Afficher / masquer les numéros de ligne else if (UPLOAD_ERR_NO_TMP_DIR == $_FILES['fichier']['error']) { $_SESSION['message'] = array('type' => ERREUR, 'texte' => 'Erreur configuration serveur : pas de dossier temporaire.'); } else if (UPLOAD_ERR_CANT_WRITE == $_FILES['fichier']['error']) { $_SESSION['message'] = array('type' => ERREUR, 'texte' => 'Erreur serveur : écriture impossible dans le dossier temporaire.'); } ... Tu fais un tableau ERREUR => message, et pouf. N1K0L45 Il y a pas mal de code, j'ai juste regardé quelques fichiers au hasard. php.func.php est pas terrible : tu fais toujours comme ça : Code : PHP - Afficher / masquer les numéros de ligne $var = calcul; return $var; Alors que tu pourrais faire directement "return calcul;" : Code : PHP - Afficher / masquer les numéros de ligne function chrono($depart) { return 1000 * round(microtime(true) - $depart, 4); } Ensuite, ta fonction "place" est carrément tordue : pourquoi faire des replace dans une variable numérique ? Globalement il y a un truc très pénible dans ton code, c'est que tu commentes à mort : Code : PHP - Afficher / masquer les numéros de ligne // On crée un array contenant les différentes pages $array_pages = array('accueil', 'candidats', 'espace_perso', 'deconnexion'); // Si la page ne se trouve pas dans l'array, on définit la page comme l'accueil if(!in_array($page, $array_pages)) $page = 'accueil'; Qu'est-ce que les commentaires apportent ici ? Le code est assez clair tout seul. Avec tes commentaires, c'est comme si on était obligé de lire deux fois chaque ligne : d'abord on lit le commentaire, et après on lit la ligne, et les deux veulent dire exactement la même chose. Tu devrais essayer de réserver les commentaires aux passages importants, ou difficiles. Si tu trouves que tu as besoin de commentaires parce que ton code n'est "pas assez clair", il faut reprendre le code pour le clarifier. Et si tu mets juste une ligne de commentaire pour chaque ligne de code par habitude, je pense que tu devrais essayer d'arrêter Au passage, un bout de code très répétitif : Code : PHP - Afficher / masquer les numéros de ligne $connect = array(); $connect['pseudo'] = utf8_caracteres($data['connexion_pseudo']); $connect['pass'] = utf8_caracteres($data['connexion_pass']); $perso = array(); $perso['nom'] = utf8_caracteres($data['personnel_nom']); $perso['prenom'] = utf8_caracteres($data['personnel_prenom']); $perso['age'] = utf8_caracteres($data['personnel_age']); $perso['lieu'] = utf8_caracteres($data['personnel_lieu']); $perso['ville'] = utf8_caracteres($data['personnel_ville']); $perso['code_postal'] = utf8_caracteres($data['personnel_code_postal']); $sport = array(); $sport['carriere'] = utf8_caracteres(nl2br($data['sport_carriere'])); $sport['exploits'] = utf8_caracteres(nl2br($data['sport_exploits'])); $sport['club'] = utf8_caracteres($data['sport_club']); Solution proposée (pas testée) : Code : PHP - Afficher / masquer les numéros de ligne function extraire($prefixe, $valeurs) { $table = array(); foreach($valeurs as $val) $table[$val] = utf8_caracters($data[$prefixe . '_' . $val]); return $table; } $data['sport_carriere']= nl2br($data['sport_carriere']); $data['sport_exploits'] = nl2br($data['sport_exploits']); $connect = extraire('connexion', array('pseudo', 'pass')); $perso = extraire('personnel', array('nom', 'prenom', 'age', 'lieu', 'ville')); $sport = extraire('sport', array('carriere', 'exploit', 'club')); On trouve d'autre code de ce genre dans d'autres fichiers, par exemple "inscription.php". Tu pourrais gagner pas mal de place et de clarté en automatisant un peu tout ça. big tuto algorithmique \| tuto récursivité [PHP] \| tuto typage \| textes divers Ours & Hippy — le blog
jordan	# Posté le 05/02/2008 à 19:30:31
Développeur professionnel Blog Twitter close Ville : Couternon Pays : France métropolitaine Études : Université de Dijon	Et moi, tu n'as pas envie de jeter un coup d'oeil...
bluestorm	# Posté le 05/02/2008 à 19:50:17
dont ask to ask Groupe : Anciens Blog Twitter close	Si si, mais j'ai posté ce que j'avais regardé pour l'instant, voilà tout. Par ailleurs hum. Clairement, t'as déjà une méthodologie bien formée de développement en PHP, un langage que tu connais visiblement mieux que moi (étant donné que j'ai arrêté d'en faire depuis plus d'un an, c'est pas non plus über difficile, mais c'est la vie ). J'aime bien la méthode des templates .xsl. J'ai jamais vraiment étudié ces technos, mais à voir comme ça c'est plutôt sympa. Je trouve que le code reste plus lourd qu'un moteur de templates lightweight (genre... le mien ), mais ça reste correct. Sinon bah le code est correct, j'ai rien vu de frappant, à part deux petits détails : - la gestion des perms est un peu low-level par rapport au reste du script, ça jure. Au passage, une fonction "function check($n, $mask) { return ($mask == ($n & $mask); }" serait du plus bel effet. - une lourdeur dans admin/create.php big tuto algorithmique \| tuto récursivité [PHP] \| tuto typage \| textes divers Ours & Hippy — le blog
jordan	# Posté le 05/02/2008 à 19:54:52
Développeur professionnel Blog Twitter close Ville : Couternon Pays : France métropolitaine Études : Université de Dijon	Quand tu soulignes "plus lourd", tu parles de quoi (syntaxe, poids, accessibilité, performance,...) ? Je sais que c'est top secret, mais je serais content que les correcteurs de mon script se manifestent par MP. J'ai pleins de questions à poser.
Nelty	# Posté le 05/02/2008 à 19:58:38
Garfield in the US Groupe : Anciens Blog Twitter close Ville : Owatonna Pays : États-Unis	Citation : jordan Quand tu soulignes "plus lourd", tu parles de quoi (syntaxe, poids, accessibilité, performance,...) ? Je sais que c'est top secret, mais je serais content que les correcteurs de mon script se manifestent par MP. J'ai pleins de questions à poser. Ben si tu as des questions à nous poser, c'est à toi de nous envoyer le MP (en destinataire tu mets Tortue facile, vincent1870 et moi) .
bluestorm	# Posté le 05/02/2008 à 20:06:31
dont ask to ask Groupe : Anciens Blog Twitter close	Bah. Le principe d'un template c'est "t'as un tas de variable à assigner, et parfois t'as des variables qui contiennent d'autres variables, ou alors une liste de variables". La plupart des moteurs templates, et indirectement ton approche XSL, utilisent de la POO laide pour ça (la POO est un concept qui a tendance à devenir un peu laid un peu vite, et PHP est un langage qui devient très laid très vite, donc au final...). Le fait est qu'en PHP, on dispose _déjà_ d'une structure de donnée, intégrée au langage, et disposant d'une syntaxe de manipulation tout à fait correcte, pour faire exactement ça : les tableaux. Code : PHP - Afficher / masquer les numéros de ligne $child = $res->addChild("fichier"); $child->addAttribute("dirname", $pathParts["dirname"]."/"); $child->addAttribute("basename", $pathParts["basename"]); $child->addAttribute("extension", strtolower($pathParts["extension"])); $child->addAttribute("datetime", date("d/m/Y H:i:s", filemtime($filename))); $child->addAttribute("chmod", $this->perms($filename)); $child->addAttribute("filename", $pathParts["filename"]); $child->addAttribute("size", $size); Compare ça avec ce qu'on peut faire avec juste des tableaux (comme le permet par exemple mon moteur de templates génial ) Code : PHP - Afficher / masquer les numéros de ligne $tpl['fichier'][] = array( 'dirname' => $pathParts["dirname"]."/", 'basename' => $pathParts["basename"], 'extension' => strtolower($pathParts["extension"]), 'datetime' => date("d/m/Y H:i:s", filemtime($filename)), 'chmod' => $this->perms($filename), 'filename' => $pathParts["filename"], 'size' => $size ); Ne serait-ce que visuellement, je préfère la deuxième solution. Je pense qu'en plus elle est conceptuellement plus "propre". big tuto algorithmique \| tuto récursivité [PHP] \| tuto typage \| textes divers Ours & Hippy — le blog
Savageman	# Posté le 05/02/2008 à 20:09:25
Blog Twitter close Études : INSA Lyon	bluestorm, je suis OK avec tes remarques, sauf le dernier IF, je ne vois pas en quoi ça gêne (en regardant le fichier, je trouve que c'est plutôt joli visuellement et qu'on repère bien les parties, d'autant plus qu'il y a un joli commentaire ). Big-tuto : Créez votre site (presque) complet PHP : architecture MVC et bonnes pratiques
bluestorm	# Posté le 05/02/2008 à 20:12:43
dont ask to ask Groupe : Anciens Blog Twitter close	Si tu parles de "!empty($_POST['titre']) && !empty($_POST['description']", le problème c'est que tu testes deux fois la même chose. Tu pourrais mettre le deuxième if à l'intérieur du premier, comme ça tu n'aurais pas à re-tester le début. Si tu parles du truc tout à la fin, de gestion des erreurs, ben ça te fait 5 lignes par cas, alors qu'avec un tableau tu pourrais en faire une seule (et courte, et claire). big tuto algorithmique \| tuto récursivité [PHP] \| tuto typage \| textes divers Ours & Hippy — le blog
Savageman	# Posté le 05/02/2008 à 20:15:11
Blog Twitter close Études : INSA Lyon	Nan, je ne parlais pas des erreurs et du tableau pour lequel je suis d'accord (et que je ne vois pas pourquoi je n'y ai pas pensé, mais bon... passons :D). Je parlais bien du IF, et je ne vois pas en quoi ça gêne de retester les deux premières (à part "super, je peux gagner 0.001 milliseconde", mais je doute que tu veuilles parler de ça :-°) Big-tuto : Créez votre site (presque) complet PHP : architecture MVC et bonnes pratiques
bluestorm	# Posté le 05/02/2008 à 20:18:38
dont ask to ask Groupe : Anciens Blog Twitter close	répétition => laid. big tuto algorithmique \| tuto récursivité [PHP] \| tuto typage \| textes divers Ours & Hippy — le blog
Cyprien_	# Posté le 06/02/2008 à 07:50:08
	bluestorm >> Désolé pour MegaUpload, mais ça restera cet hébergeur pour cette première phase de concours. Je verrai si j'en trouve un autre pour la finale, car ne t'inquiète pas, je n'aime pas non plus cet hébergeur . En tout cas, si tu pouvais malgré tout surmonter ta réticence suprême( ) pour jeter un oeil sur mon code, je veux bien ! Merci d'avance, des conseils avisés font toujours plaisir .
Craw	# Posté le 06/02/2008 à 15:14:58
Rien n'est parfait !	On va bientôt passer au deuxième tour, le temps de voir si tous les résultats ont été publiés. Il manque le mapping je crois, je sais plus...
freecircus	# Posté le 06/02/2008 à 16:55:15
"Se coucher tard nuit"	Non il manque encore le C ...clap clap!
Craw	# Posté le 06/02/2008 à 21:39:38
Rien n'est parfait !	Ok.
nicoco_40	# Posté le 11/02/2008 à 18:02:36
Gavé bien mon gars § Ville : Villejuif Pays : France métropolitaine Études : EFREI	Encore longtemps avant le sujet redac ? comme on a encore deux tours je pensais que l'on aurait pu commencer avant ceux qui n'en ont qu'un.
Craw	# Posté le 11/02/2008 à 18:41:49
Rien n'est parfait !	Bah moi j'attends les résultats en C et en mapping, je ne sais pas... Je ne peux rien commencer avant, désolé.
Tortue facile	# Posté le 11/02/2008 à 19:31:33
DB2 + Cobol + CICS Groupe : Anciens Blog Twitter close Ville : Ivry-sur-seine Pays : France métropolitaine	Citation : Craw Bah moi j'attends les résultats en C et en mapping, je ne sais pas... Je ne peux rien commencer avant, désolé. Pour le mapping on a déjà le gagnant (le seul qui a rendu quelque chose). Grand fan de la tortue j'ai créé un site web avec notamment des infos sur l'hibernation de la tortue de terre. Je vous conseille aussi de lire cet article sur les pratiques contestables de Blogbang vis à vis de sa régie de pub.
Craw	# Posté le 11/02/2008 à 19:39:24
Rien n'est parfait !	Oui mais il faut le noter.
Craw	# Posté le 13/02/2008 à 17:14:49
Rien n'est parfait !	Je vais publier le deuxième tour de rédaction très prochainement, même si il n'y a pas les résultats des autres catégories, car là on traîne un peu trop.
Craw	# Posté le 14/02/2008 à 18:24:51
Rien n'est parfait !	Sujets de rédaction deuxième tour en ligne.
Saimoun	# Posté le 14/02/2008 à 19:08:56
Youhouuu !! Études : ENSIIE	Pour le sujet Rédac Groupe A : faudrait pas mettre une limite de mot pour le résumé ? (genre 10% de l'original ou un truc comme ça) Pour la question 4 : Vous interpréterez votre raisonnement. (au passage, y'a un accent sur interpréter), j'ai rien compris. Comment peut-on interpréter son propre raisonnement... ? En gros faut faire une dissert sur le texte qu'on aura proposé à la question 3 ? Saimoun. Programmeur JS/PHP/C/C++ à ses heures perdues Admin Sys à ses heures non perdues Servbwa, mon serveur en bois, hébergé chez moi
nicoco_40	# Posté le 14/02/2008 à 22:16:39
Gavé bien mon gars § Ville : Villejuif Pays : France métropolitaine Études : EFREI	ok je vois que je suis le seul con, ils sont ou les sujets, j'ai rien vu sur le site a Nelty...
freecircus	# Posté le 14/02/2008 à 22:20:00
"Se coucher tard nuit"	Pas dur pourtant, liste des sujet > tout en bas > Epreuve 2 ... Code : Autre - Afficher / masquer les numéros de ligne # [php] Épreuve 1, groupe B # [php] Épreuve 1, groupe A # [redaction] Épreuve 1, groupe A # [redaction] Épreuve 1, groupe B # [3d] Épreuve 1, groupe A # [3d] Épreuve 1, groupe B # [2d] Épreuve 1, groupe A # [2d] Épreuve 1, groupe B # [c] Épreuve 1, groupe A # [c] Épreuve 1, groupe B # [mapping] Épreuve 1, groupe A # [mapping] Épreuve 1, groupe B # [redaction] Épreuve 2, groupe A # [redaction] Épreuve 2, groupe B Retour à la liste des candidats Édité le 14/02/2008 à 22:21:02 par freecircus ...clap clap!
nicoco_40	# Posté le 14/02/2008 à 22:35:55
Gavé bien mon gars § Ville : Villejuif Pays : France métropolitaine Études : EFREI	Ouai je viens de le voir, mais j'ai juste pas acces au texte... J'ai une page d'erreur, surement du au parfeu de mon lycee... Je verrais cela plus tard.
Craw	# Posté le 14/02/2008 à 22:44:38
Rien n'est parfait !	Citation : Saimoun Pour le sujet Rédac Groupe A : faudrait pas mettre une limite de mot pour le résumé ? (genre 10% de l'original ou un truc comme ça) Pour la question 4 : Vous interpréterez votre raisonnement. (au passage, y'a un accent sur interpréter), j'ai rien compris. Comment peut-on interpréter son propre raisonnement... ? En gros faut faire une dissert sur le texte qu'on aura proposé à la question 3 ? Oui voilà tu as tout compris. Non je n'ai pas voulu imposer de limite.