Discussion :

[ChPr]

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 :

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
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

[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 ?

[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).

[ChPr]

... 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

"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

[CosmoKnacki]

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

Code :

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
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 :

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

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.

[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