Discussion :

[ceweb]

Bonjour,

Après de multiples essais avec asort je n'y vois plus clair !

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
/*--- Valeurs ---*/
$vendredi="vendredi";  $samedi="samedi";  $lundi="lundi";  $dimanche="dimanche";
 
/*--- Periode 1 ---*/
if(empty($vendredi)) { /*Pediode 1 vide*/    echo '';    }
else{
if ($vendredi===$samedi) {/*Pediode 1 va afficher seulement vendredi*/ echo'Le '.$vendredi.' ';    }
else{ /*Pediode 1 va afficher les 2 jours*/ echo 'Période1= '.$vendredi.' au '.$samedi.' '; }
}
/*--- periode 2 ---*/
if(empty($lundi)) { /*Periode 2 vide*/ echo '';    }
else{
if ($lundi===$dimanche) { /*Periode 2 va afficher seulement lundi*/    echo 'Le '.$lundi.' '; }
else{ /*Periode 2 va afficher  les 2 jours*/ echo 'Période2= '.$lundi.' au '.$dimanche.' ';    }
    }

Ce code ci dessus va afficher => Période1= vendredi au samedi et Période2= lundi au dimanche
Comment je peux faire pour trier mon résultat et faire => Période2= lundi au dimanche (période 2) et Période1= vendredi au samedi (periode 1)

[sabotage]

Avec un code aussi mal présenté et aussi illogique ($vendredi===$samedi) ce n'est pas étonnant que tu ne t'y retrouves pas.

Si tu veux la période 2 avant la période 1, change l'ordre de tes deux conditions.

[ceweb]

Pour ($vendredi===$samedi) ces 2 valeurs peuvent être parfois strictement identique en provenance de mon fichier xml (qui n'est pas ma source).

Désolé si cela est mal présenté, mais je dois avoir ces 3 conditions entre 2 valeurs. Si je remplace les jours de la semaine par ouverture1, ouverture2 etc.... cela ne sera pas forcement plus clair ;-)

Si c'est possible j'aimerai bien que tu me montre comment tu ferais un code plus clair :-)

[Spartacusply]

Moi je commencerai déjà par utiliser des noms de variables pertinents (du genre $dateDebutPeriode1) et indenter de manière propre le code.

[ceweb]

Bon j'espère que c'est plus clair

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
/*--- Valeurs Période 1 ---*/
$ouverture_1_debut="vendredi";  $ouverture_1_fin="samedi";  
/*--- Valeurs Période 2 ---*/
$ouverture_2_debut="lundi";  $ouverture_2_fin="dimanche";
 
/*--- Periode 1 ---*/
if(empty($ouverture_1_debut)) 
      { 
/*Pediode 1 vide*/
    echo ''
        }
else {
if ($ouverture_1_debut===$ouverture_1_fin) /*Dans mon fichier xml j'ai parfois début=vendredi et fin=vendredi lorsqu'il y a seulement un jour d'ouverture et que c'est fermé le samedi*/
     {
/*Pediode 1 va afficher seulement Début*/
    echo'Le '.$ouverture_1_debut.' ';    
        }
else {
/*Pediode 1 va afficher Début et fin*/
    echo 'Période1= '.$ouverture_1_debut.' au '.$ouverture_1_fin.' ';
        }
     }
 
/*--- periode 2 ---*/
if(empty($ouverture_2_debut))
      { 
/*Pediode 1 vide*/
    echo ''
        }
else {
if ($ouverture_2_debut===$ouverture_2_fin)
     {
/*Pediode 2 va afficher seulement Début*/
    echo'Le '.$ouverture_2_debut.' ';    
        }
else {
/*Pediode 2 va afficher Début et fin*/
    echo 'Période1= '.$ouverture_2_debut.' au '.$ouverture_2_fin.' ';
        }
     }

Toutes les valeurs sont aléatoires, je ne peux pas changer l'ordre du code

Comment je peux classer $ouverture_1_debut et $ouverture_2_debut par ordre logique des jours (lundi,mardi,mercredi,jeudi,vendredi,samedi,dimanche) ?

[sabotage]

Avec de l'imagination on peut tout faire

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
<?php
$ouverture_1_debut="vendredi";  $ouverture_1_fin="samedi";  
$ouverture_2_debut="lundi";  $ouverture_2_fin="dimanche";
$semaine = array('lundi'=>1, 'mardi'=>2, 'mercredi'=>3, 'jeudi'=>4, 'vendredi'=>5, 'samedi'=>6, 'dimanche'=>7);
function texte_periode($debut, $fin, &$resultat) {
   global $semaine;
 
   if (empty($debut)) {
      $texte = '';
   }
   elseif ($debut === $fin) {
       $texte = 'Le ' . $debut;    
   }
   else {
       $texte = 'Période du ' . $debut . ' au ' . $fin;
   }
 
    $resultat[$semaine[$debut]][] = $texte;
 
}
 
 
function show_periode($resultat) {
  sort($resultat);
  foreach ($resultat as $textes) {
    foreach ($textes as $texte) {
     echo '<p>' . $texte . '</p>';
    }
  }
}
 
$resultat = array();
texte_periode($ouverture_1_debut, $ouverture_1_fin, $resultat);
texte_periode($ouverture_2_debut, $ouverture_2_fin, $resultat);
show_periode($resultat);

[John Blobsmith]

On y voit plus clair.... mais sans la global ce serai quand même mieux.

[sabotage]

repeter la déclaration dans la fonction je trouve ça laid aussi.

[ceweb]

Merci beaucoup d'avoir pris du temps sur mon cas.

Mais Cela ne classe pas les jours dans le bon ordre

Ton code ci dessous donne

Code :

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

<p>Le jeudi</p><p>Le mercredi</p><p>Le samedi</p><p>Le vendredi</p><p>Période du lundi au mardi</p>

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
$ouverture_1_debut="vendredi";  $ouverture_1_fin="vendredi";
$ouverture_2_debut="samedi";  $ouverture_2_fin="samedi";
$ouverture_3_debut="lundi";  $ouverture_3_fin="mardi";
$ouverture_4_debut="mercredi";  $ouverture_4_fin="mercredi";
$ouverture_5_debut="jeudi";  $ouverture_5_fin="jeudi";
 
$semaine = array('lundi'=>1, 'mardi'=>2, 'mercredi'=>3, 'jeudi'=>4, 'vendredi'=>5, 'samedi'=>6, 'dimanche'=>7);
function texte_periode($debut, $fin, &$resultat) {
   global $semaine;
 
   if (empty($debut)) {
      $texte = '';
   }
   elseif ($debut === $fin) {
       $texte = 'Le ' . $debut;
   }
   else {
       $texte = 'Période du ' . $debut . ' au ' . $fin;
   }
 
    $resultat[$semaine[$debut]][] = $texte;
 
}
 
 
function show_periode($resultat) {
  sort($resultat);
  foreach ($resultat as $textes) {
    foreach ($textes as $texte) {
     echo '<p>' . $texte . '</p>';
    }
  }
}
 
$resultat = array();
texte_periode($ouverture_1_debut, $ouverture_1_fin, $resultat);
texte_periode($ouverture_2_debut, $ouverture_2_fin, $resultat);
texte_periode($ouverture_3_debut, $ouverture_3_fin, $resultat);
texte_periode($ouverture_4_debut, $ouverture_4_fin, $resultat);
texte_periode($ouverture_5_debut, $ouverture_5_fin, $resultat);
show_periode($resultat);

Bon au risque d'être encore confus je donne plus de détails

J'ai des fichiers xml qui me donne des résultats dans cet ordre :

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
<Ouverture>
 
<PlageJ fin="vendredi" début="vendredi">
<PlageH fin="12:00:00" début="08:00:00"/>
<PlageH fin="16:30:00" début="13:30:00"/>
</PlageJ>
 
<PlageJ fin="samedi" début="samedi">
<PlageH fin="12:00:00" début="09:00:00"/>
</PlageJ>
 
<PlageJ fin="mardi" début="lundi">
<PlageH fin="12:00:00" début="08:00:00"/>
<PlageH fin="17:30:00" début="13:30:00"/>
</PlageJ>
 
<PlageJ fin="mercredi" début="mercredi">
<PlageH fin="12:00:00" début="08:00:00"/>
</PlageJ>
 
<PlageJ fin="jeudi" début="jeudi">
<PlageH fin="12:00:00" début="08:00:00"/>
<PlageH fin="17:30:00" début="13:30:00"/>
</PlageJ>
 
</Ouverture>

Et voila comment sont les valeurs Php actuelles que j'utilise pour afficher les jours

Code :

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

1
2
3
4
5
$xml->Ouverture->PlageJ[0]['debut']     $xml->Ouverture->PlageJ[0]['fin']
$xml->Ouverture->PlageJ[1]['debut']     $xml->Ouverture->PlageJ[1]['fin']
$xml->Ouverture->PlageJ[2]['debut']     $xml->Ouverture->PlageJ[2]['fin']
$xml->Ouverture->PlageJ[3]['debut']     $xml->Ouverture->PlageJ[3]['fin']
$xml->Ouverture->PlageJ[4]['debut']     $xml->Ouverture->PlageJ[4]['fin']

[sabotage]

Pardon c'est ksort() pas sort()

[ceweb]

Oui avec ksort ca fonctionne, merci

Mais il reste un dernier truc que je ne comprends pas !

C'est comment sortir le echo de la fonction ?

Code :

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

1
2
3
4
5
6
7
8
9
10
function show_periode($resultat) {
  ksort($resultat);
  foreach ($resultat as $textes) {
    foreach ($textes as $texte) {
     //echo '<p>' . $texte . '</p>';
     return $texte;
    }
  }
}
  echo '<p>'.$texte.'</p>';

[sabotage]

Code :

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

1
2
3
4
5
6
7
8
9
10
function show_periode($resultat) {
  output = '';
  ksort($resultat);
  foreach ($resultat as $textes) {
    foreach ($textes as $texte) {
     $output.= '<p>' . $texte . '</p>';
    }
  }
  return $output;
}

[ceweb]

MERCI tout fonctionne bien.