Discussion :

[laurentSc]

Bonjour,
désolé pour le titre non concis mais j'ai pas trouvé plus court.

Mon souci est que mon code parcourt les éléments d'un tableau avec un foreach mais je voudrais ajouter un traitement particulier sur une colonne. La seule chose à laquelle j'ai pensé est :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
 
foreach ($valTickets as $line)
{
    echo "<tr>";
    foreach ($line as $cellLine)
    {
        if ($cellLine=="xxx") {echo "<td>"; traitement particulier; echo "</td>";}
        else
            echo "<td>" . $cellLine . "</td>";
    }
    echo "</tr>";
}

c'est bon ou y a mieux ?

[rawsrc]

salut,

c'est bien mais la manière d'écrire ton code laisse vraiment à désirer : tu dois être hyper rigoureux et constant. Suis les règles PSR sans te poser de questions, elles ont été conçues par des développeurs aguerris et font consensus.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
foreach ($valTickets as $line) {
    echo '<tr>';
    foreach ($line as $cellLine) {
        echo '<td>';
        if ($cellLine=="xxx") {
            echo traitement particulier;
        } else {
            echo $cellLine;
        }
        echo '</td>';
    }
    echo '</tr>';
}

Pense quand même à bien échapper toutes les données.

[laurentSc]

OK,
je me suis d'abord demandé ce qu'étaient les règles PSR, puis en lisant ton code (le mien reformatté), j'ai compris et https://php.developpez.com/tutoriels...e/?page=normes.
En fait mon code initial était bien formatté mais c'est juste en écrivant ce post que c'est devenu mauvais.
J'ai repris mon code mal formatté et ai utilisé la commande "code->reformat code" de PHPStorm (que je vais acheter car je suis convaincu), mais le résultat est de travers

Code php :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
 
foreach ($valTickets as $line) {
    echo "<tr>";
    foreach ($line as $cellLine) {
        if ($cellLine == "xxx") {
            echo "<td>";
            traitement particulier; echo "</td>";} else
            echo "<td>" . $cellLine . "</td>";
    }
    echo "</tr>";
}

Cet outil de PHPStorm n'est pas valable ?

[rawsrc]

si c'est valable et c'est entièrement paramétrable :
File > Settings > Editor > Code Style > PHP
Et là c'est la fête au village.

[Trehinos]

Le if est la bonne façon de vérifier qu'on est dans la bonne colonne. Pensez à l'écriture foreach($row as $colName => $colValue) si $row est une ligne de résultat d'une requête BDD (ou un tableau associatif).

Si vous voulez modifier la valeur du tableau, on peut aussi travailler directement avec la référence de l'élément plutôt qu'une copie : foreach($row as &$colValue).

Pour PhpStorm : CTRL + ALT + L
Je crois que j'utilise ce raccourci encore plus souvent que CTRL + S ^^

[laurentSc]

Je peux certes améliorer les choses en modifiant les settings, mais y a un truc que j'ai pas trouvé :

Code php :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
 
foreach ($valTickets as $line)
{
    echo "<tr>";
    foreach ($line as $cellLine)
    {
        if ($cellLine == "xxx")
        {
            echo "<td>";
            traitement particulier; echo "</td>";}
        else
        {
            echo "<td>" . $cellLine . "</td>";
        }
    }
    echo "</tr>";
}
echo "</tr>";
}

Comment faire que la } de la ligne 10 soit renvoyée à la ligne suivante ?

[laurentSc]

Merci Trehinos du conseil sur l'écriture du foreach. Du coup, j'ai écrit

Code php :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
 
foreach ($valTickets as $line)
{
    echo "<tr>";
    foreach ($line as $colName=>$cellLine)
    {
        if ($colName == "id_Ticket")
        {
            echo "<td><a href=''>" . $cellLine . "</a></td>";
        }
        else
        {
            echo "<td>" . $cellLine . "</td>";
        }
    }
    echo "</tr>";
}

alors que j'étais parti sur un compteur de lignes (évidemment le href vide de la ligne 9, c'est du provisoire)