Comment imploser foreach

Comment imploser foreach() avec virgule?

foreach($names as $name) {
    //do something
    echo '<a href="' . $url . '" title="' . $title . '">' . $name .'</a>';
}

Souhaitez ajouter une virgule après chaque lien, sauf le dernier.

Mmh quel est l'intérêt d'avoir beaucoup de liens avec des noms différents mais la même URL?
Felix, êtes-vous un lecteur d'esprit qui peut voir ce qui est derrière "//faire quelque chose"? 🙂

OriginalL'auteur James | 2010-09-02

25

Raveren la solution est efficace et belle, mais voici une autre solution (qui peut être utile dans des scénarios similaires):
```
$elements = array();
foreach($names as $name) {
    //do something
    $elements[] = '<a href="' . $url . '" title="' . $title . '">' . $name .'</a>';
}
echo implode(',', $elements);
```
En général, celui-ci devrait être plus rapide que la concaténation de chaînes. Ce serait probablement encore plus rapidement si la chaîne a été construit en utilisant une chaîne de formatage ou de la substitution de variable: $elements[] = "<a href=\"$url\" title=...
Ceci est plus lent bien et n'est ni plus lisible ni fonctionnelle. (non pas que c'est pire, je suis juste à la défense de la micro-optimisation)
c'est juste une supposition, je ne peux pas trouver la source, mais concat et substr a été calibré pour être plus rapide dans tous les cas.
Vrai, c'est juste une supposition, mais en général, la mise en forme de chaîne est plus élégant aussi, je ne considère donc pas un mauvais changement, même si c'est plus lent. En outre, un tableau de chaînes qui est concaténé est, pour moi, plus lisible que de dépouiller le dernier caractère.
Bingo McBingo, a fait le tour pour moi, de la façon la plus simple d'obtenir ce damné virgule à une boucle foreach

OriginalL'auteur Emil Vikström
12

Vous avez besoin pour transformer votre tableau au lieu d'itération en utilisant foreach. Vous pouvez faire cela avec array_map.

PHP 5.3 syntaxe avec fermetures
```
echo implode(", ", array_map(function($name) use($url, $title)
{
    return '<a href="' . $url . '" title="' . $title . '">' . $name .'</a>';
}, $names));
```
Compatible syntaxe avant de PHP 5.3
```
function createLinkFromName($name)
{
    return '<a href="' . $url . '" title="' . $title . '">' . $name .'</a>';
}
echo implode(", ", array_map('createLinkFromName', $names));
```
PHP 5.3 syntaxe avec une meilleure fiabilité de
```
function a_map($array, $function)
{
    return array_map($function, $array);
}

echo implode(", ", a_map($names, function($name) use($url, $title)
{
    return '<a href="' . $url . '" title="' . $title . '">' . $name .'</a>';
}));
```
Franchement ce frontières illisible - surtout avec une tâche triviale. C'est aussi probablement le plus lent de la solution si vous êtes dans la performance.
Je voulais proposer la même chose, mais vous devez être prudent avec $url et $title. Ils ne seront pas ensemble (au moins à l'avant de PHP 5.3 exemple).
N'est-ce pas bien plus compliqué que nécessaire?
Il n'est pas si compliqué que ça, il y a vient de la array_map syntaxe qui met la fonction première qui n'est pas une sage décision quand on travaille avec des fermetures. Je suggère la définition d'une nouvelle fonction avec l'ordre inverse des arguments pour améliorer readbility.

OriginalL'auteur Vincent Robert
2
```
$s = '';
foreach ($names as $name) {
  if ($s) $s .= ', ';
  $s .= '<a href="' . $url . '" title="' . $title . '">' . $name . '</a>';
}
```
je pense que vous vouliez dire if(!$s), ou mieux if(strlen($s) == 0) $s = ', ';
knittl, vous pouvez enregistrer un appel de fonction en cochant la fonction isset($s[0]) au lieu de strlen \o/
Pas de. if ($s) est correct. Nous ne voulons pas commencer par une virgule. Nous voulons ajouter une qu'à partir du deuxième lien.
Mais vous avez réussi à me confondre suffisant pour modifier ma réponse, puis revenir.
ah ok, je ne savais pas que vous étiez à l'aide de $pour le séparateur et la chaîne elle-même. désolé pour la confusion

OriginalL'auteur TRiG
2
```
foreach($names as $name) {
    //do something
    $str .= '<a href="' . $url . '" title="' . $title . '">' . $name .'</a>,';
}
echo substr($str,0,-1);
```
EDIT:
comme les commentaires le soulignent, cette façon de faire les choses est un peu des erreurs si vous changer le séparateur (précisément sa longueur) et d'oublier les substr paramètre. Il faut donc utiliser la méthode foreach moins que l'exercice est absolument essentiel.

toute autre solution?
pourquoi, c'est plus rapide que d'imploser.
... jusqu'à ce que vous modifier le séparateur et oubliez pas de régler l' substr.
c'est très valable. Pas une raison pour me donner un moins bien :/
Désolé, je n'avais pas spécialement un signe moins; seulement cette approche. Je l'ai vu de trop nombreuses fois: la création d'une requête de cette façon et puis substring dans la syntaxe errordom. Je suis en train de décourager cette approche lorsqu'il y a une manière plus propre avec implode.

OriginalL'auteur raveren

$first = TRUE;
foreach($names as $name) {
    //do something
    if(!$first) { echo ', '; }
    $first = FALSE;
    echo '<a href="', $url, '" title="', $title, '">', $name, '</a>';
}

Micro-optimisation: avec de l'écho, de l'utilisation des virgules au lieu de concaténation: electrictoolbox.com/php-echo-commas-vs-concatenation

OriginalL'auteur knittl

Ici est un vilain solution à l'aide de l'écho:

 $total = (count($names) - 1 );

 foreach($names as $i => $name) 
 {
      if($i != $total) 
           echo '<a href="' . $url . '" title="' . $title . '">' . $name .'</a>, ';
      else
           echo '<a href="' . $url . '" title="' . $title . '">' . $name .'</a>';
 }

Je +1, sauf que le décompte du tableau à chaque itération est assez gaspillage (surtout si le tableau est importante)... peut-être pré-calculer ($numOfElements = count($names) - 1; foreach ...)...
Bon point - modifié

OriginalL'auteur Jake N

Vous devez vous connecter pour publier un commentaire.