Utilisation de break

Version imprimable

Bonjour à toutes et à tous,

En refondant des fichiers php utilisant mySQL vers mySQLi, je tombe sur d'autres problèmes :

Dans la fonction suivante, j'utilise plein de break pour diminuer le temps d'exécution.

Apparemment, cela ne fonctionne plus avec php 7 :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
function covno($L)
{
  if(ord($L) >= 76 and ord($L) < 81)
  {
    return chr(ord($L)-28);
    break;
  }
  if(ord($L) >= 112 and ord($L) < 117)
  {
    return chr(ord($L)-63+2*(116-ord($L)));
    break;
  }
  if(ord($L) >= 81 and ord($L) < 91)
  {
    return chr(ord($L)+7+2*(90-ord($L)));
    break;
  }
  if(ord($L) >= 117 and ord($L) < 123)
  {
    return chr(ord($L)-10);
    break;
  }
  if(ord($L) >= 65 and ord($L) < 75)
  {
    return chr(ord($L)+39-2*(ord($L)-74));
    break;
  }
  if ($L == 'K')
  {
    return 'A';
    break;
  }
  if(ord($L) >= 107 and ord($L) < 112)
  {
    return chr(ord($L)-45-2*(ord($L)-111));
    break;
  }
  if(ord($L) >= 97 and ord($L) < 107)
  {
    return chr(ord($L)-26);
    break;
  }
  if(ord($L) >= 48 and ord($L) < 58)
  {
    return chr(ord($L)+33);
    break;
  }
}

Si je supprime tous les break, cela fonctionne !

je me fourvoie ou quoi ?

Cordialement.

Pierre

01/11/2019, 11h12
mathieu

return provoque la fin de la fonction avec le retour de la valeur.
donc le code après, que ça soit break ou autre chose n'est jamais exécuté.

peut-être que ces tests étaient utilisés dans une boucle avant la refonte de votre code ?
01/11/2019, 11h44
Invité

Bonjour,

1- même constat.
Tous les break; sont à supprimer.

2- Pour les opérateurs logiques, il est préférable d'utiliser && et || plutôt que and et or (qui on un fonctionnement particulier).
01/11/2019, 11h56
ChPr

Citation:

Envoyé par mathieu

... peut-être que ces tests étaient utilisés dans une boucle avant la refonte de votre code ? ...

Le code n'a pas été modifié, et cela fonctionnait avec php 5

Citation:

Envoyé par mathieu

"return" provoque la fin de la fonction avec le retour de la valeur.
donc le code après, que ça soit "break" ou autre chose n'est jamais exécuté. ...

Trop imbibé de Pascal où il n'y a pas de return, mais un result qui lui, ne termine pas la fonction. D'où mes break.

En php 7, ces break, inutiles, créent une erreur.

Cordialement.

Pierre

Moi cette fonction, je la réécrirai comme ça:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
function covno($L) {
    $ord = ord($L);
 
    switch($ord) {
        case $ord < 48: break;
        case $ord < 58: return chr($ord + 33);
        case $ord < 65: break;
        case $ord < 75: return chr($ord + 39 - 2*($ord - 74));
        case 75: return 'A';
        case $ord < 81: return chr($ord - 28);
        case $ord < 91: return chr($ord + 7 + 2*(90 - $ord));
        case $ord < 97: break;
        case $ord < 107: return chr($ord - 26);
        case $ord < 112: return chr($ord - 45 - 2*($ord - 111));
        case $ord < 117: return chr($ord - 63 + 2*(116 - $ord));
        case $ord < 123: return chr($ord - 10);
    }
}

en mettant les plages dans l'ordre pour réduire le nombre de tests.

Dans l'exemple précédent break; et return; sont pour ainsi dire interchangeables, car break sort du switch puis on arrive à la fin de la fonction, et return; sort immédiatement de la fonction. Ce n'est pas tout à fait pareil mais le résultat est le même.

Mais si on fait la même chose avec des if:

Code:

1
2
3
4
5
6
7
8
9
10
11
function covno($L) {
    $ord = ord($L);
 
    if ( $ord < 48 )
        return;
 
    if ( $ord < 58 )
        return chr($ord + 33);
 
   //...
}

il n'est plus possible d'utiliser break.

02/11/2019, 09h53
ChPr

Merci "Cosmoknacki" pour cette remise en forme de ma fonction "covno". Je vais l'utiliser car elle est beaucoup plus claire écrite de cette manière.

Cordialement.

Pierre