Discussion :

[Konrad Florczak]

Bonjour je me demande comment je pourrais optimiser ce code très repetitif.
Je pensais mettre les valeurs debug, info, warn, error, fatal dans un table
et ensuite faire des boucles.

Mais je vois aps trop comment je pourrais faire ca ?

Si quelqu'un pouvait me filer un coup de main, ca serait bien sympatique.

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
<?php
if(isset($_GET['date'])&&isset($_GET['date2']))
{
    $date  = $_GET['date'];
    $date2 = $_GET['date2'];
    include("connect_db.php");
        if($date == $date2)
            {
            $sql_debug = mysql_query("select count(*) from log4j where time='$date' AND warning='error';")or die(mysql_error());
            $sql_info  = mysql_query("select count(*) from log4j where time='$date' AND warning='info'; ")or die(mysql_error());
            $sql_warn  = mysql_query("select count(*) from log4j where time='$date' AND warning='warn'; ")or die(mysql_error());
            $sql_error = mysql_query("select count(*) from log4j where time='$date' AND warning='fatal';")or die(mysql_error());
            $sql_fatal = mysql_query("select count(*) from log4j where time='$date' AND warning='error';")or die(mysql_error());
            }
        else
            {
            $sql_debug = mysql_query("select count(*) from log4j where (time BETWEEN'.$date' AND '$date2') AND warning='error';")or die(mysql_error());
            $sql_info  = mysql_query("select count(*) from log4j where (time BETWEEN'.$date' AND '$date2') AND warning='info' ;")or die(mysql_error());
            $sql_warn  = mysql_query("select count(*) from log4j where (time BETWEEN'.$date' AND '$date2') AND warning='warn' ;")or die(mysql_error());
            $sql_error = mysql_query("select count(*) from log4j where (time BETWEEN'.$date' AND '$date2') AND warning='error';")or die(mysql_error());
            $sql_fatal = mysql_query("select count(*) from log4j where (time BETWEEN'.$date' AND '$date2') AND warning='fatal';")or die(mysql_error());
            }
 
$debug = mysql_fetch_array($sql_debug);
$info  = mysql_fetch_array($sql_info) ;
$warn  = mysql_fetch_array($sql_warn) ;
$error = mysql_fetch_array($sql_error);
$fatal = mysql_fetch_array($sql_fatal);
 
    echo"<table><tr><td><b>Type</b></td><td><b>Number</b></td></tr>";
    echo"<tr><td>debug</td><td>".$debug."</td></tr>";
    echo"<tr><td>info </td><td>".$info."</td></tr>";
    echo"<tr><td>warn </td><td>".$warn."</td></tr>";
    echo"<tr><td>error</td><td>".$error."</td></tr>";
    echo"<tr><td>fatal</td><td>".$fatal."</td></tr>";
    echo"</table>";
}
else{
echo "Please enter select start date, end date and environment";
}
?>

[Xunil]

Sans doute avec un GROUP BY.

Code :

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

SELECT COUNT(*) FROM log4j WHERE time='$date' GROUP BY warning;

[Konrad Florczak]

non je voulais dire au niveau du PHP
mettre les variables dans un tableau php et mettre des boucles

je sais le faire en perl mais en php...

en perl ca sera quelque chose dans ce style :

Code :

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

1
2
3
4
5
6
my %Compteur = qw(WARN 0 INFO 0 ERROR 0 FATAL 0);
while(<IN>)
{
    ++$Compteur{$1} if (/(WARN|INFO|ERROR|FATAL)/);
}
close(IN);

[FCYPBA]

Tu peux utiliser un GROUP BY pour ne faire qu'une seule requete.

Ensuite, tu construit ton tableau de valeurs

Code :

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

1
2
3
4
5
6
 
$sql = "SELECT warning,COUNT(*) AS NB FROM log4j WHERE time='$date' GROUP BY warning;"
$res=mysql_query($sql);
while($tmp=mysql_fetch_array($res)){
$tab[$tmp['warning'] = $tmp['NB'];
}

Ensuite tu en fait ce que tu veux. Si tu ne veux faire que de l'affichage, tu peux faire des echos directs dans la boucle while

Code :

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

1
2
 
echo "<tr><td>" .$tmp['warning'] ."</td><td>" .$tmp['NB'] ."</td></tr>";

[Konrad Florczak]

Ok merci je comprend maintenant l'utilisation des mysql_fetch_array and co
bon je vais pouvoir faire des requetes plus denses et plus compliqué en mysql maintenant.

Merci beaucoup je vous tiens au courant

[Konrad Florczak]

mais alors si je comprend bien ton exemple le $res devient un tableau de type (5;2) avec comme valeurs (testées sur mon sql)


debug 2455
info 452
warn 11
error 1
fatal 0

ou c'est un tableau une dimension type :

debug 2455 info 452 warn 11 error 1 fatal 0

donc si je veux afficher la valeur 2455 je lui dis $res[2]

Désolé on y est presque mais j'ai regardé les docs et je trouve pas ca très clair, vu que mysql_fetch_array c'est plutot utilisé genre

while{
echo "$table_resultat['mavaleur_dun_select']";
}

[FCYPBA]

$res n'est pas un tableau, c'est un recordset.

Pour pouvoir utiliser les données contenues à l'intérieur ( sous forme de ligne ), il faut passer par les fonctions de fetch ( mysql_fetch_row,mysql_fetch_array,mysql_fetch_object)

Suivant la fonction que tu as choisi, tu obtiens un type de variable différent ( tableau, objet )

Dans le cas de mysql_fetch_array cela renvoie un tableau associatif ( nom de la colonne =>valeur + un index numérique ).

Chaque appel, à la fonction de fetch fait avancer le pointeur dans le recordset. Quand il n'y a plus de lignes, la fonction renvoie null et on sort de la boucle

Edit : Dans l'exemple que je t'ai donné, on parcours le jeu de résultat en plaçant les différentes variables au sein d'un tableau organisé comme ceci :
type de warning -> Nombre