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

**steve.ma** · 04/11/2013, 07h12

là ça me saoule. ça fait plus de 4h non-stop que je suis dessus et j'y arrive pas.
j'ai pu aussi faire fonctionner le code de keaton en faisant comme ceci:

Code :

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

1
2
3
4
5
6
<?php
function convertTime($hour = 0, $minutes = 0, $seconds = 0) {
    return 3600 * $hour + 60 * $minutes + $seconds;	}
 
// on convertie l'heure d'arrivée en secondes
$convertedTime_a = convertTime($_POST['arrive_h'],$_POST['arrive_m'],$_POST['arrive_s']);

ça s'affiche bien pour l'heure d'arrivée, mais dès que je dois faire la même chose avec des trajets variables j'y arrive plus: MODE PAUSE.

Invité · 04/11/2013, 08h42

Bonjour,
1/ il te faut comprendre comment fonctionnent les array !
=> Relis mon message (corrigé et commenté)

2/ Attention de ne pas faire de confusion entre les DIFFÉRENTS $i !
Le $i de la boucle for(...) N'EST PAS celui de la boucle foreach(...) !
Pour éviter les confusions, tu peux mettre $i, $j ou $k,...

Par conséquent, le traitement de "trajet" se fait de la même manière que celui de "ville" :

Code php :

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

$trajet_array = (!empty($_POST['trajet']))? $_POST['trajet'] : ''; // array !

**steve.ma** · 13/11/2013, 07h34

Après des heures et des heures de recherches, impossible pour mes compétences de récupérer les valeurs de chaque clés pour les utiliser pour faire mes calculs, j'ai donc abandonné l'idée de réduction de code: il est certes à rallonge mais il a l'avantage d'être fonctionnel.

Je repasserai si je suis à nouveau bloqué dans la suite de ce que je veux faire de ce code dans un nouveau sujet.

Merci de votre aide, ça m'aura au moins permis d'approfondir pas mal de choses et d'en découvrir d'autres grâce aux recherches effectuées depuis vos réponses.

Problème non résolu mais sujet clos.

**ascito** · 17/11/2013, 05h20

ouep , j'ai un truc vraiment pas esthétique mais qui fonctionne bien pour comprendre un peu le truc des dates

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
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
<?php
Class jcalandar{
     public function enDate($date){
 
          $dateJour = $date ;
          $dateTab = explode('-',$dateJour );
          if (strlen($dateTab[0])==4){$ln='en';}
          if (strlen($dateTab[2])==4){$ln='fr';}
          $date=   $ln=='fr' ? $dateTab[2].'-'.$dateTab[1].'-'.$dateTab[0] : $date;      
          return $date;          
 
     }  
  	public function mktimeDate($date){
          $dateJour = $date ;
          $dateTab = explode('-',$dateJour );
          if (strlen($dateTab[0])==4){$ln='en';}
          if (strlen($dateTab[2])==4){$ln='fr';}
          $m =  $ln=='fr' ? (int)$dateTab[1] : (int)$dateTab[1];
          $Y =  $ln=='fr' ? (int)$dateTab[2] : (int)$dateTab[0];
          $d=   $ln=='fr' ? (int)$dateTab[0] : (int)$dateTab[2];
          $lnOut =  $ln =='fr' ? "d-m-Y" : "Y-m-d";
          if (is_int($d) && is_int($m) && is_int($Y) ){
          $mktime  =  mktime(0, 0, 0, $m, $d , $Y ) ;		
          }
          else{
          $mktime   ="";	
          }
	return $mktime;
	}
 
   	public function parseDate($date){
          $date=$this->enDate($date); 
          $this->m =   date ( 'm' ,$this->mktimeDate($date) ) ;
          $this->y =   date ( 'Y' ,$this->mktimeDate($date) ) ;
          $this->d =   date ( 'd' ,$this->mktimeDate($date) ) ;
  	}
 
     public function dayInYear(){
 
          $date=$this->enDate($date);
          return date ( 'w' ,$this->mktimeDate($date) );
 
     }
 
     public function prev($date){
 
          $dateJour = $date ;
          $dateTab = explode('-',$dateJour );
 
          if (strlen($dateTab[0])==4){$ln='en';}
          if (strlen($dateTab[2])==4){$ln='fr';}
          $date=   $ln=='fr' ? $dateTab[2].'-'.$dateTab[1].'-'.$dateTab[0] : $date;
          $m =   date ( 'n' ,$this->mktimeDate($date) ) ;
          $y =   date ( 'Y' ,$this->mktimeDate($date) ) ;
          $d=    date ( 'j' ,$this->mktimeDate($date) ) ;
          if( $d-1 == 0 ){
          $m--;
          $d=$this->numerDayInMonth($y.'-'.$m.'-'.$d);
               if($m==0){
               $y--;
               $m=12;
               $d=$this->numerDayInMonth($y.'-'.$m.'-'.$d);         
               }
          }else{
          $d--;     
          }
          $date = (string) $d.'-'.$m.'-'.$y;
          $date = (string) date ( 'd' ,$this->mktimeDate($date) ).'-'.date ( 'm' ,$this->mktimeDate($date) ).'-'.$y;
 
          return $date  ;
     }
     public function next($date){
 
          $dateJour = $date ;
          $dateTab = explode('-',$dateJour );
 
          if (strlen($dateTab[0])==4){$ln='en';}
          if (strlen($dateTab[2])==4){$ln='fr';}
          $date=   $ln=='fr' ? $dateTab[2].'-'.$dateTab[1].'-'.$dateTab[0] : $date;
          $m =   date ( 'n' ,$this->mktimeDate($date) ) ;
          $y =   date ( 'Y' ,$this->mktimeDate($date) ) ;
          $d=    date ( 'j' ,$this->mktimeDate($date) ) ;
 
          $dm=$this->numerDayInMonth($date);
 
          if( $d+1 > $dm ){
          $m++;
          $d=1;    
               if($m>12){
               $y++;
               $m=1;
               $d=1;         
               }
          }else{
          $d++;     
          }
 
          $date = (string) $d.'-'.$m.'-'.$y;
          $date = (string) date ( 'd' ,$this->mktimeDate($date) ).'-'.date ( 'm' ,$this->mktimeDate($date) ).'-'.$y;
 
          return $date  ;
     }
      public function nextMonth($date){
           $dateJour = $date ;
          $dateTab = explode('-',$dateJour );
           if (strlen($dateTab[0])==4){$ln='en';}
          if (strlen($dateTab[2])==4){$ln='fr';}
          $date=   $ln=='fr' ? $dateTab[2].'-'.$dateTab[1].'-'.$dateTab[0] : $date;
          $m =   date ( 'n' ,$this->mktimeDate($date) ) ;
          $y =   date ( 'Y' ,$this->mktimeDate($date) ) ;
          $d=    date ( 'j' ,$this->mktimeDate($date) ) ;
 
          $dm=$this->numerDayInMonth($date);
               $m++;
               if($m>12){
               $y++;
               $m=1;
               } 
           $date = (string) $d.'-'.$m.'-'.$y;
          $date = (string) date ( 'd' ,$this->mktimeDate($date) ).'-'.date ( 'm' ,$this->mktimeDate($date) ).'-'.$y;
 
          return $date  ;
     }
       public function prevMonth($date){
           $dateJour = $date ;
          $dateTab = explode('-',$dateJour );
           if (strlen($dateTab[0])==4){$ln='en';}
          if (strlen($dateTab[2])==4){$ln='fr';}
          $date=   $ln=='fr' ? $dateTab[2].'-'.$dateTab[1].'-'.$dateTab[0] : $date;
          $m =   date ( 'n' ,$this->mktimeDate($date) ) ;
          $y =   date ( 'Y' ,$this->mktimeDate($date) ) ;
          $d=    date ( 'j' ,$this->mktimeDate($date) ) ;
 
          $dm=$this->numerDayInMonth($date);
               $m--;
               if($m==0){
               $y--;
               $m=12;
               } 
           $date = (string) $d.'-'.$m.'-'.$y;
          $date = (string) date ( 'd' ,$this->mktimeDate($date) ).'-'.date ( 'm' ,$this->mktimeDate($date) ).'-'.$y;
 
          return $date  ;
     }    
     public function dayInMonth($date ,$format = 'N' ){
          $date=$this->enDate($date);
          return date ( $format ,$this->mktimeDate($date) );
          }
 	function monthNumToName($mois) {
		$tableau = Array("", "Janvier", "Février", "Mars", "Avril", "Mai", "Juin", "Juillet", "Aôut", "Septembre", "Octobre", "Novembre", "Décembre");
		return (intval($mois) > 0 && intval($mois) < 13) ? $tableau[intval($mois)] : "Indéfini";
	}
 
 
     public function numerDayInMonth($date ,$format = 't' ){
          $date=$this->enDate($date);  
          return date ( $format ,$this->mktimeDate($date) );
      }
 
     public function dayInWeek(){
 
          return date ( 'w' ,$this->mktimeDate($date) );
 
     }
 
}

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

Discussions similaires

Partager

Partager