Réduire un code php à répétition

**steve.ma** · 30/10/2013, 06h35

Bonjour,

J'ai un bout de code, ou plutôt un pavé lol, et je suis certain qu'il y a moyen de réduire ce code en quelques lignes tant il y a de répétitions. ^^

Donc question: Comment réduire le code suivant pour chaque cas svp?
NB: le code est coupé, il n'y a pas 10fois mais 20 fois la même chose à chaque fois.

cas 1

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
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
/////////////////////////////////////////////////// 0
if(isset($_POST['heure0']))
$my_strh0=$_POST['heure0'];
else
$_POST['heure0'] = "00-00-00";
/////////////////////////////////////////////////// 1
if(isset($_POST['heure1']))
$my_strh1=$_POST['heure1'];
else
$_POST['heure1'] = "00-00-00";
 
if(!isset($a1))
$a1 = 0;
/////////////////////////////////////////////////// 2
if(isset($_POST['heure2']))
$my_strh2=$_POST['heure2'];
else
$_POST['heure2'] = "00-00-00";
 
if(!isset($a2))
$a2 = 0;
/////////////////////////////////////////////////// 3
if(isset($_POST['heure3']))
$my_strh3=$_POST['heure3'];
else
$_POST['heure3'] = "00-00-00";
 
if(!isset($a3))
$a3 = 0;
/////////////////////////////////////////////////// 4
if(isset($_POST['heure4']))
$my_strh4=$_POST['heure4'];
else
$_POST['heure4'] = "00-00-00";
 
if(!isset($a4))
$a4 = 0;
/////////////////////////////////////////////////// 5
if(isset($_POST['heure5']))
$my_strh5=$_POST['heure5'];
else
$_POST['heure5'] = "00-00-00";
 
if(!isset($a5))
$a5 = 0;
/////////////////////////////////////////////////// 6
if(isset($_POST['heure6']))
$my_strh6=$_POST['heure6'];
else
$_POST['heure6'] = "00-00-00";
 
if(!isset($a6))
$a6 = 0;
/////////////////////////////////////////////////// 7
if(isset($_POST['heure7']))
$my_strh7=$_POST['heure7'];
else
$_POST['heure7'] = "00-00-00";
 
if(!isset($a7))
$a7 = 0;
/////////////////////////////////////////////////// 8
if(isset($_POST['heure8']))
$my_strh8=$_POST['heure8'];
else
$_POST['heure8'] = "00-00-00";
 
if(!isset($a8))
$a8 = 0;
/////////////////////////////////////////////////// 9
if(isset($_POST['heure9']))
$my_strh9=$_POST['heure9'];
else
$_POST['heure9'] = "00-00-00";
 
if(!isset($a9))
$a9 = 0;
/////////////////////////////////////////////////// 10
if(isset($_POST['heure10']))
$my_strh10=$_POST['heure10'];
else
$_POST['heure10'] = "00-00-00";
 
if(!isset($a10))
$a10 = 0;
 ////////////////////////////////////////////////////

cas 2

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
/* VARIABLES */
$my_strh0=$_POST['heure0'];
$my_strh1=$_POST['heure1'];
$my_strh2=$_POST['heure2'];
$my_strh3=$_POST['heure3'];
$my_strh4=$_POST['heure4'];
$my_strh5=$_POST['heure5'];
$my_strh6=$_POST['heure6'];
$my_strh7=$_POST['heure7'];
$my_strh8=$_POST['heure8'];
$my_strh9=$_POST['heure9'];
$my_strh10=$_POST['heure10'];

cas 3

Code :

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

1
2
3
4
5
6
7
8
9
10
11
$h0=strtotime($my_strh0);
$h1=strtotime($my_strh1);
$h2=strtotime($my_strh2);
$h3=strtotime($my_strh3);
$h4=strtotime($my_strh4);
$h5=strtotime($my_strh5);
$h6=strtotime($my_strh6);
$h7=strtotime($my_strh7);
$h8=strtotime($my_strh8);
$h9=strtotime($my_strh9);
$h10=strtotime($my_strh10);

cas 4

Code :

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

1
2
3
4
5
6
7
8
9
10
if($h0 > $h1) {$a1=($h0-$h1);	}
if($h0 > $h2) {$a2=($h0-$h2); 	}
if($h0 > $h3) {$a3=($h0-$h3); 	}
if($h0 > $h4) {$a4=($h0-$h4); 	}
if($h0 > $h5) {$a5=($h0-$h5); 	}
if($h0 > $h6) {$a6=($h0-$h6); 	}
if($h0 > $h7) {$a7=($h0-$h7); 	}
if($h0 > $h8) {$a8=($h0-$h8); 	}
if($h0 > $h9) {$a9=($h0-$h9); 	}
if($h0 > $h10) {$a10=($h0-$h10);}

cas 5

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
if($h1 > $h0) {$a1=86400-($h1-$h0);  }
echo "Trajet".$my_strh1.""; 
@$date1 = getdate($h1);
 
if($h2 > $h0) {$a2=86400-($h2-$h0);  }
echo "Trajet".$my_strh2.""; 
@$date2 = getdate($h2);                
 
if($h3 > $h0) {$a3=86400-($h3-$h0);  }
echo "Trajet".$my_strh3.""; 
@$date3 = getdate($h3);
 
if($h4 > $h0) {$a4=86400-($h4-$h0);  }
echo "Trajet".$my_strh4.""; 
@$date4 = getdate($h4);
 
if($h5 > $h0) {$a5=86400-($h5-$h0);  }
echo "Trajet".$my_strh5.""; 
@$date5 = getdate($h5);
 
if($h6 > $h0) {$a6=86400-($h6-$h0);  }
echo "Trajet".$my_strh6.""; 
@$date6 = getdate($h6);
 
if($h7 > $h0) {$a7=86400-($h7-$h0);  }
echo "Trajet".$my_strh7.""; 
@$date7 = getdate($h7);
 
if($h8 > $h0) {$a8=86400-($h8-$h0);  }
echo "Trajet".$my_strh8.""; 
@$date8 = getdate($h8);
 
if($h9 > $h0) {$a9=86400-($h9-$h0);  }
echo "Trajet".$my_strh9.""; 
@$date9 = getdate($h9);
 
if($h10 > $h0) {$a10=86400-($h10-$h0);  }
echo "Trajet".$my_strh10.""; 
@$date10 = getdate($h10);

cas 6

Code :

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

1
2
3
4
5
6
7
8
9
10
$a1=$a1-3600;
$a2=$a2-3600;
$a3=$a3-3600;
$a4=$a4-3600;
$a5=$a5-3600;
$a6=$a6-3600;
$a7=$a7-3600;
$a8=$a8-3600;
$a9=$a9-3600;
$a10=$a10-3600;

cas 7

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
echo "Depart:";
//reconversion du timestamp en heure pr?format?e (ici H M)
echo strftime("%H:%M:%S",$a1);
echo strftime("%H:%M:%S",$a2);
echo strftime("%H:%M:%S",$a3);
echo strftime("%H:%M:%S",$a4);
echo strftime("%H:%M:%S",$a5);
echo strftime("%H:%M:%S",$a6);
echo strftime("%H:%M:%S",$a7);
echo strftime("%H:%M:%S",$a8);
echo strftime("%H:%M:%S",$a9);
echo strftime("%H:%M:%S",$a10);

D'avance merci.

**grunk** · 30/10/2013, 08h40

Au lieu d'appeler tes champs name="heure0" , utilise un tableau : name="heure[]"Et du coup tu peux tout traiter dans une boucle. Dans le premie cas ça donnerais quelque chose comme :

Code :

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

1
2
3
$total = count($_POST['heure']);
for($i = 0; $i < $total; $i++)
	$_POST['heure'][$i] = (!empty($_POST['heure'][$i])) ? $_POST['heure'][$i] : "00-00-00";

**keaton7** · 30/10/2013, 14h12

Oui, la solutuion de grunk est bonne.

Sinon pourquoi reparcourir a chaque fois tes éléments ? Tu ne peux pas mettre tout dans une boucle for, et gérer avec des variables dynamiques ?
(CF : http://www.php.net/manual/fr/languag...s.variable.php)

PS : Fait gaffe à la lisibilité de tes noms de variables

**Bebel** · 30/10/2013, 14h14

ou tu peux utiliser le $$
ca donnerait quelque chose 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
 
$i = 0;
 
if(isset($_POST['heure' . $i]))
 
$variable = 'my_strh' . $i;
 
$$variable = $_POST['heure' . $i];
else
$_POST['heure' . $i] = "00-00-00";
 
------
$varh = 'h' . $i;
$variable = 'my_strh' . $i ;
 
$$varh = strtotime($$variable);

et ainsi de suite.

**steve.ma** · 01/11/2013, 08h41

Tout d'abord merci beaucoup pour vos réponses qui, en tant que débutant, m'ont permises d'apprendre encore quelques petites choses intéressantes dont les "variables des variables $$" qui sont très pratiques.

J'ai donc utilisé l'idée de bebel que j'ai pu comprendre grâce au tuto donné par keaton ce qui m'a donné pour la 1ère partie:

Code :

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

1
2
3
4
5
6
7
8
9
<?
$i = 0;
 
if (isset($_POST['heure' . $i])) {
$variable = 'my_strh' . $i;
$$variable = $_POST['heure' . $i]; }
else {
$_POST['heure' . $i] = "erreur calcul"; }
?>

(j'ai du ajouter les accolades pour le "if")

Je ne pense pas m'être trompé sur la 2è partie où j'ai remplacé:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
<?
$my_strh1=$_POST['heure1'];
$my_strz0=$_POST['zarrive'];
//Conversion des heures en TimeStamp
$h1=strtotime($my_strh1);
$z0=strtotime($my_strz0);
//Si la deuxieme heure est bien plus tard que la premiere
if($z0 > $h1) {$a1=($z0-$h1);       }
//Si c'est la premiere heure qui est plus tard que la deuxieme
if($h1 > $z0) {$a1=86400-($h1-$z0);   }	
?>

par ceci en adaptant avec votre aide:

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
<?
$varh = 'h' . $i;
$variable = 'my_strh' . $i ;
 
$$variable = $_POST['heure' . $i];
$my_strz0 = $_POST['zarrive'];
//Conversion des heures en TimeStamp
$$varh = strtotime($$variable);
$z0=strtotime($my_strz0); 
 
//Si la deuxieme heure est bien plus tard que la premiere
$vara = 'a' . $i ;
if($z0 > $varh) {$vara=($z0-$varh); }
//Si c'est la premiere heure qui est plus tard que la deuxieme
if($varh > $z0) {$vara=86400-($varh-$z0);   }
?>

NB: Il s'agit d'une soustraction de H:M:S où
- $z0 est l'heure d'arrivée
- $varh est le temps qu'on soustrait (mes anciens h1, h2, h3, h4,... h20)

Le problème actuel est que suite à l'insertion de ce code, les soustractions ne se font plus, sans doute à cause du tableau qui les appelle mais là je sèche.
J'utilisais à la base ces input:

Code html :

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

1
2
3
4
<INPUT type="text" name="heure1" value="HH:MM:SS">
<INPUT type="text" name="heure2" value="HH:MM:SS">
<INPUT type="text" name="heure3" value="HH:MM:SS">
... que je répète jusqu'à  name="heure20"

J'ai donc tenté ceci mais ça ne fonctionne pas :/

Code html :

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

1
2
3
4
<INPUT type="text" name="heure[]" value="HH:MM:SS">
<INPUT type="text" name="heure[]" value="HH:MM:SS">
<INPUT type="text" name="heure[]" value="HH:MM:SS">
... et ceci 20 fois car besoin d'afficher jusqu'à 20 soustractions.

Si ça peut aider, dans mon code de base trouvé sur le net et que j'essai d'adapter, j'affiche l'heure $varh comme ceci:

Code :

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

1
2
3
4
<?
echo "".$$variable."";
@$date1 = getdate($varh);
?>

Et depuis la modif du code, le message d'erreur "erreur calcul" s'affiche à la place de mon $varh en H:M:S ("erreur calcul" est le texte d'erreur mis dans le 1er code de ce message ci)

Bref, d'où vient mon erreur:
- De mon code de départ qui envoi mal l'information? (page1.php)
- Ai-je oublié de faire un tableau qui appelle mes "Heure"? (page2.php)
- Comment alors faire ce tableau ?
- Et à quoi correspond ceci svp, le problème pourrait-il venir de là:
@$date1 = getdate($varh);

(désolé du long message mais j'ai voulu être le plus explicite possible)

**sabotage** · 01/11/2013, 09h00

Code html :

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

1
2
3
4
<INPUT type="text" name="heure[]" value="HH:MM:SS">
<INPUT type="text" name="heure[]" value="HH:MM:SS">
<INPUT type="text" name="heure[]" value="HH:MM:SS">
... et ceci 20 fois car besoin d'afficher jusqu'à 20 soustractions.

Effectivement tu as du boulot en simplification :

Code :

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

1
2
3
for($i = 0; $i < 20; $i++) {
    echo '<input type="text" name="heure[]" value="HH:MM:SS" />';
}

D'ailleurs j'ai peur que le reste de ton code, d'après les bouts que tu nous as montré, ne contienne plein d'étapes inutiles : par exemple le cas 5 et grosso modo le cas 6 à l'envers.
Explique nous exactement ce que tu veux obtenir et on pourra t'aider. Tel quel ton code est tellement alambiqué que c'est difficile de s'y retrouver.

Si on reprend juste la différence entre la première valeur et les autres (je passe sur le fait que convertir des heures en date avec strtotime est une erreur) :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
foreach($_POST['heure'] as $key=>$value) {
    if ($key == 0) {
        $premiere_heure = strtotime($value);
    }
    else {
         $heure = strtotime($value);
         if ($heure > $premiere_heure) {
              $diff = 86400 - ($heure - $premiere_heure);
         }
         else {
              $diff $premiere_heure - $heure;
        }
    }
}

**steve.ma** · 01/11/2013, 17h33

Pour répondre à ta question, à la base il s'agit d'un code trouvé sur le net que j'ai voulu adapter à mes besoin. Malheureusement mes connaissance sont limitées.
J'ai donc commencer à faire ce que je pouvais en venant demander de l'aide au besoin, parce que je me voyais mal arriver et demander à ce qu'on me fasse le boulot complet, puis de toutes manières je me serai fait remballer. lol.
D'un autre côté c'est intéressant d'apprendre aussi, de comprendre ce qu'il se passe, seulement les tutos sur le net donnent des exemples sur des cas simples et dès qu'on veut pousser la chose c'est la galère.

C'est pour ça que j'ai super apprécié d'avoir à la fois un exemple et un lien vers le tuto pour comprendre. Encore merci.

Bref, je fais une autre réponse qui va parler du code, je crois que tout le monde s'en fiche de ma vie. mdr

**steve.ma** · 17/11/2013, 12h37

Mon problème était surtout dans la réduction du code, chose que je n'ai pas réussi à faire. Mon projet était plutôt simpliste à la base donc ce code à rallonge m'embêtait, mais en avançant j'ai souhaité ajouté un paquet d'options et ce code à rallonge finalement, je l'ai totalement modifié et je le trouve bien pratique car il m'aide beaucoup pour la compréhension et mise en oeuvre de ce que je souhaite faire de toutes ces variable: c'est certes pas très joli à voir mais c'est tellement plus facile de retrouver et utiliser une variable quand elle est clairement visible!

De plus, j'ai passé 30 minutes à faire mon code à rallonge finalement, contre plus de 15 jours non-stop sans pouvoir réduire mon code: faites le calcul. ^^

Il est vrai par contre que cette discussion m'a permis d'apprendre un paquet de choses sur les dates, utilisant finalement une astuce qu'on m'a proposé: tout traduire en secondes pour ensuite faire mes calculs: le + pratique et + clair pour ce que je souhaite faire de mes dates.

Je regrette encore moins dans le sens ou je suis parti à la base d'un code pris sur le net que je souhaitais modifier, et je suis arrivé au final avec un code que j'ai fait moi-même, et même si c'est pas grand chose finalement, j'en suis assez fier.

Au final et avec du recul, si j'ai abandonné le but originel de cette discussion par dépits, elle m'aura été très bénéfique et je souhaite une fois de plus remercier tous ceux qui m'ont apporté leur aide: j'ai testé tout ce qui m'a été proposé en faisant des recherches autour, ce qui m'a permis d'apprendre un paquet de choses.

A l'heure actuelle j'essaie de jouer avec mes variables en les faisant tourner en bourrique à l'aide du javascript, histoire de me venger. mdr.

Je précise donc une nouvelle fois que le problème n'est pas résolu mais que finalement je ne souhaite pas le résoudre lol, car mon code à rallonge m'arrange bien.

NB: L'objet de ce message étant devenu caduque, je ne sais pas si je dois ou non marquer ce sujet comme résolu. Dans le doute je ne touche à rien.

Réduire un code php à répétition

Langage PHP

Vue hybride

Discussions similaires

Partager

Partager