Effet escompté obtenu mais mon code est-il bon ?

Kodi
Invité n'ayant pas de compte PHPfrance

14 juin 2007, 10:09

Bonjour, je suis mega newbie en dev PHP et je voulais savoir si ma façon de faire était bonne ou non pour ma première fonction. En quête de savoir je suis réceptif à toute critique et apport de connaissances :)

Sans chichi, la voici, la voila :
<? function convert($chaine) 
{
$chaine = str_replace("b","v",$chaine);
echo $chaine;
} 
$var = "b a ba b o bo b i bi b u bu";
convert($var);//renvoie donc v a va v o vo v i vi v u vu
?> 
Kodi

Avatar du membre
Administrateur PHPfrance
Administrateur PHPfrance | 13231 Messages

14 juin 2007, 10:22

A part le fait que je ne ferais pas le echo dans la fonction, c'est ce que j'aurais fait :-k

J'aurais écrit ça :
<?php
	function convert($chaine) 
	{
		return str_replace("b","v",$chaine);
	} 
	
	$var = "b a ba b o bo b i bi b u bu";
	echo convert($var); //renvoie donc v a va v o vo v i vi v u vu
?>
Connaître son ignorance est la meilleure part de la connaissance
Pour un code lisible : n'hésitez pas à sauter des lignes et indenter

twitter - site perso - Github - Zend Certified Engineer

Kodi
Invité n'ayant pas de compte PHPfrance

14 juin 2007, 10:40

"Zeus", ca en impose :D

Merci de ta réponse. Pourquoi le echo en dehors la fonction ? Quel est l'impact si ca reste dans la fonction ?

Kodi

Avatar du membre
Administrateur PHPfrance
Administrateur PHPfrance | 13231 Messages

14 juin 2007, 10:52

selon moi, cette fonction converti une chaine.
Tu désires afficher la chaine converti.

Imagine que je te demande non plus de convertir une seconde chaine et de la mémoriser dans un fichier, il te faudra créer une seconde fonction qui va faire le même travail que la 1ère mais qui va enregistrer le résultat dans un fichier.
Avec ma fonction, tu n'as qu'a ré-utiliser la fonction convert() mais, au lieu de faire un echo, tu mémorises le résultat dans un fichier ;)
Connaître son ignorance est la meilleure part de la connaissance
Pour un code lisible : n'hésitez pas à sauter des lignes et indenter

twitter - site perso - Github - Zend Certified Engineer

Kodi
Invité n'ayant pas de compte PHPfrance

14 juin 2007, 17:02

C'est pas faux, merci pour ce cas :D

Kodi