les espaces en trop ne servent qu'a rajouter des octets inutile dans le code...
if(isset($pwet))
est aussi lisible que...
if( isset ($pwet) )
Ca n'a rien à voir avec l'indentation du code... après à chacun sa façon de coder mais dans ce cas je ne vois pas l'intérêt d'enlever des points à une note parce qu'il manque des espaces à TON gout... c'est comme enlever des points parce que j'aurais préféré que la variable $pwet s'appelle $plop... bref... discussion inutile et sans intérêt à mon sens... merci pour ton com, bonne continuation... .. . :o)
@ tchaOo°
ps: au boulot je bosse actuellement sur une source d'à peu près 45000 lignes et personnellement je trouve tout aussi clair et même plus clair un bloc de condition simple sans accolade... mais comme tu l'as dis c'est une question d'habitude... les gouts et les couleurs... .. . ;o)
Comme je l'ai dit ça dépend que de nous, de la façon dont on code.
Personnellement pour de longs codes (pour des scripts, sites, etc), je préfère faire des blocs de condition même s'il y a qu'une seule exéctuin.
Les espaces, c'est ma façon à moi d'écrire du code, après certes tu fais comme tu veux.
Il y a une différence entre "mon" code et le tien, moi j'aime bien espacer : )
Je vois toujours pas pourquoi tu dis ça... hormis les accolades il n'y a pas de différence entre ton code et mon code et je ne vois pas l'interêt de faire un bloc de condition pour une seule instruction... .. .
Qui plus est je ne vois pas le rapport avec les espaces... si tu parle de l'indentation de mon code c'est surement parce que tu as utilisé firefox lorsque tu as récupéré le code... fais un copié collé à partir d'IE et tu retrouvera l'indentation comme sur la présentation du code... .. . ;o)
Je devais recréer la fonction pour un site que je développais en PHP 5 mais l'hébergeur tournait en PHP 4 et comme j'avais la flem de réécrire cette fonction, bref thx.
Je n'avais pas connaissance de cette fonction dans PHP5, en plus de m'avoir appris que je n'etais pas fou d'avoir écris une fonction qui faisait ca (il y a longtemps) j'ai appris l'existence de cette fonction native dans php5 !
Oui je connais cette version, qui n'est d'ailleur pas celle de php.net mais d'un posteur tout comme moi ici... seulement je ne l'aime pas pour plusieurs raisons... tout d'abord si le séparateur n'est pas fournis tu remarquera que ce dernier est instancié à la fin de la fonction ce qui ne sert à rien du tout donc la chaine est faussée... je voulais une fonction similaire à l'originale d'une part au niveau de la déclaration donc pas de récursivité possible (nombre d'arguments) et je ne voulais pas passer par un builder externe comme dans celle de PEAR (package PHP_Compat) d'autre part celle ci dessus ne formate pas la chaine exactement comme l'originale encore que cela est facilement modifiable... .. .
après j'aurais pu passer par un tableau plutôt qu'une chaine mais c'est un choix les deux solutions se valent en terme de démarche et de performances... le plus chiant c'est le builder en interne en fait... je trouve pas ça des plus propre mais je vois pas d'autre solution si on exclu un builder externe ou la récursion... j'y réfléchirais quand j'aurais le temps... .. .
Salut amezghal... pour répondre à ton commentaire... .. .
- oui la coloration de phpcs ne supporte pas les chaines de caractères sur plusieurs lignes... :oS
- détrompe toi cette fonction peut être très utile et le fait qu'elle n'apparaisse qu'a partir de php5 ne veut rien dire quand à son utilité... str(r)ipos() et str_ireplace() par exemple sont très utile et n'apparaisse qu'avec PHP5... de même pour le model objet (digne de ce nom) de PHP... .. .
- kek elle a ma façon de coder... .. . ;o)
Merci pour ton commentaire et pour la note... .. .
-le script du phpcs n'a pas bien mis en forme votre script($builderCore):
-la fonction a mon avis rarement utilisable(la preuve c'est qu'elle n'apprait qu'avec la version 5 du php :D ).
-par contre j'aime bien ta façon de coder xd je note 8
if(isset($pwet))
est aussi lisible que...
if( isset ($pwet) )
Ca n'a rien à voir avec l'indentation du code... après à chacun sa façon de coder mais dans ce cas je ne vois pas l'intérêt d'enlever des points à une note parce qu'il manque des espaces à TON gout... c'est comme enlever des points parce que j'aurais préféré que la variable $pwet s'appelle $plop... bref... discussion inutile et sans intérêt à mon sens... merci pour ton com, bonne continuation... .. . :o)
@ tchaOo°
ps: au boulot je bosse actuellement sur une source d'à peu près 45000 lignes et personnellement je trouve tout aussi clair et même plus clair un bloc de condition simple sans accolade... mais comme tu l'as dis c'est une question d'habitude... les gouts et les couleurs... .. . ;o)
Personnellement pour de longs codes (pour des scripts, sites, etc), je préfère faire des blocs de condition même s'il y a qu'une seule exéctuin.
Les espaces, c'est ma façon à moi d'écrire du code, après certes tu fais comme tu veux.
Il y a une différence entre "mon" code et le tien, moi j'aime bien espacer : )
Je te remercie encore pour ton code.
Qui plus est je ne vois pas le rapport avec les espaces... si tu parle de l'indentation de mon code c'est surement parce que tu as utilisé firefox lorsque tu as récupéré le code... fais un copié collé à partir d'IE et tu retrouvera l'indentation comme sur la présentation du code... .. . ;o)
@ tchaOo°
if(!is_array($formdata) && !is_object($formdata))
return false;
elseif(empty($formdata))
return '';
J'aurais écrit cela :
if ( ! is_array ( $formdata ) && ! is_object ( $formdata ) )
{
return FALSE;
}
else if ( empty ( $formdata ) )
{
return '';
}
Bon bien sur, je peux pas te reprocher ça, étant donné que cela dépend de la manière de coder des gens ; )