Bug : boucle qui spam

Version imprimable

Bonjour,

j'ai un tableau qui affiche les données d'une table
le problème est que chaque résultat est affiché 16 fois ce qui fait un spam intensif, une page longue de x Millions de pixels et un temps de génération de la page beaucoup trop important

Voici un extrait mon code, ou est le problème ? Merci d'avance

Code:

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
 
$GetIss = 'SELECT issuetask.idissue, issuetask.noissue, issuetask.close, issuetask.priority, issuetask.title, issuetask.datetimestart, user.name 
               FROM project, issuetask join user on issuetask.iduser = user.iduser 
               WHERE noproject LIKE "%'.$_GET["Keyword"].'%" 
               ORDER BY idissue';
 
$ResultGetIss = mysql_query($GetIss);
 
 
 
 
<table class="sorTableView">
    <thead>
        <tr>
            <th data-sort="string" class="userTdTitle" scope="col" width="15%">#Issue</th>
            <th data-sort="string" class="userTdTitle" scope="col" width="35%">Title</th>
            <th data-sort="string" class="userTdTitle" scope="col" width="25%">Client</th>
            <th data-sort="string" class="userTdTitle" scope="col" width="15%">Responsible</th>
            <th data-sort="int" class="userTdTitle" scope="col" width="10%">Priority</th>
        </tr>
     </thead>
 
     <tbody>
        <?PHP
            if ($_GET["Keyword"]=='')
            {
 
             }
             else while($ResultIss = mysql_fetch_array($ResultGetIss))
             {
               echo "<tr>";
 
               echo "<td class='userTd1'>"; ?>
               <a href="edittask.php?IDIssue=<?PHP echo $ResultIss["idissue"]; ?>"><?PHP
               echo $ResultPro["noproject"]."-".$ResultIss["noissue"]."</a>";
 
                echo "<td align='center' class='userTd1'>".$ResultIss["title"]."</td>";
 
                echo "<td align='center' class='userTd1'>";?>
                <a href="admin_client_info.php?IDClient=<?PHP echo $ResultPro["idclient"]; ?>">
                <?PHP echo $ResultPro["clientname"]."</a>";
                 echo "</td>";
 
                 echo "<td align='right'  class='userTd1'>".$ResultIss["title"]."</td>";
 
                 echo "<td align='right'  class='userTd1'>".$ResultPro["priority"]."</td>";
 
                 echo "</tr>";
                }
            ?>
    </tbody>
</table>

PS : aucune erreur est affichée il y a juste une generation a l'infini
Merci :zoubi:

[EDIT] J,ai maintenant l'erreur Fatal error: Maximum execution time of 30 seconds exceeded qui s'affiche (sans avoir touché au code)

23/02/2015, 23h35
sabotage
- On ne met pas des données $_GET directement dans une requête
- Il manque une jointure avec la table project
Je suis dubitatif également sur ton else while, moi je mettrais des {
Code:

1 2 3 else { while($ResultIss = mysql_fetch_array($ResultGetIss)) {

Je dirais mêm plus, mon cher DuponT :

Code:

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
<table class="sorTableView">
    <thead>
        <tr>
            <th data-sort="string" class="userTdTitle" scope="col" width="15%">#Issue</th>
            <th data-sort="string" class="userTdTitle" scope="col" width="35%">Title</th>
            <th data-sort="string" class="userTdTitle" scope="col" width="25%">Client</th>
            <th data-sort="string" class="userTdTitle" scope="col" width="15%">Responsible</th>
            <th data-sort="int" class="userTdTitle" scope="col" width="10%">Priority</th>
        </tr>
     </thead>
 
     <tbody>
        <?PHP
            if ($_GET["Keyword"]=='')
            {
				// .......... ?
             } else {
		$GetIss = 'SELECT issuetask.idissue, issuetask.noissue, issuetask.close, issuetask.priority, issuetask.title, issuetask.datetimestart, user.name 
				FROM project, issuetask join user on issuetask.iduser = user.iduser 
				WHERE noproject LIKE "%'.mysql_real_escape_string($_GET["Keyword"]).'%" 
				ORDER BY idissue';
 
		$ResultGetIss = mysql_query($GetIss);
		while($ResultIss = mysql_fetch_array($ResultGetIss))
		{
               echo "<tr>";
 
               echo "<td class='userTd1'>"; ?>
               <a href="edittask.php?IDIssue=<?PHP echo $ResultIss["idissue"]; ?>"><?PHP
               echo $ResultPro["noproject"]."-".$ResultIss["noissue"]."</a>";
 
                echo "<td align='center' class='userTd1'>".$ResultIss["title"]."</td>";
 
                echo "<td align='center' class='userTd1'>";?>
                <a href="admin_client_info.php?IDClient=<?PHP echo $ResultPro["idclient"]; ?>">
                <?PHP echo $ResultPro["clientname"]."</a>";
                 echo "</td>";
 
                 echo "<td align='right'  class='userTd1'>".$ResultIss["title"]."</td>";
 
                 echo "<td align='right'  class='userTd1'>".$ResultPro["priority"]."</td>";
 
                 echo "</tr>";
                }
               }
            ?>
    </tbody>
</table>

Par contre :

Code:

1
2
3
4
5
<?PHP
                echo "<td align='center' class='userTd1'>";?>
                <a href="admin_client_info.php?IDClient=<?PHP echo $ResultPro["idclient"]; ?>">
                <?PHP echo $ResultPro["clientname"]."</a>";
                 echo "</td>";

mettre le code HTML tantôt dans le PHP, tantôt en dehors m'échappe...

24/02/2015, 15h17
EkrazNeophite

Merci pour votre aide

Sabotage, j'ai corrigé le GET
Pour le while, je vais de suite le mettre dans le else car c'est depuis que j'ai créé cette boucle while que la page met beaucoup de temps a charger

jreaux62, le premier if est vide car ce que je souhaite c'est que quand la variable Keyword est vide, les tableaux soient vides, et lorsque l'utilisateur arrive sur page c'est le cas.
Pour les alternances avec le PHP/HTML je vais corriger cela car c'est vrai que je me perd das mon propre code.
24/02/2015, 15h38
-Rpass-

je suis également perplexe quant à ton else et while côte à côte ce n'est pas courant. Ne serait ce que pour la lisibilité je te conseillerais de mettre ton while à l'interieur de la boucle else.

De plus nous ne connaissons pas la provenance de la variable $ResultPro
24/02/2015, 15h43
EkrazNeophite

D'accord je corrige tout ça et je vous redonne ma nouvelle version :)

Alors voila j'ai corrigé comme vous me l'avez dit : la page se lance automatiquement ! Cependant, je ne comprend pas pourquoi elle mettait autant de temps auparavant sachant que je n'ai pas modifié grand chose :

Code:

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
    <tr>
      <td>
        <?PHP 
          include("connect.php");     
 
          $GetIss = 'SELECT client.idclient, client.clientname, issuetask.noissue, issuetask.title, issuetask.priority, project.noproject, issuetask.iduser, project.priority
                     FROM CLIENT, ISSUETASK, PROJECT, USER
                     WHERE client.idclient = project.idclient
                     AND issuetask.iduser = user.iduser
                     AND project.noproject LIKE "%'.mysql_real_escape_string($_GET["Keyword"]).'%"
                     ORDER BY noproject ASC';
 
                    $ResultGetIss = mysql_query($GetIss);
        ?>
 
                    <table class="sorTableView">
                        <thead>
                          <tr>
                            <th data-sort="string" class="userTdTitle" scope="col" width="15%">#Issue</th>
                            <th data-sort="string" class="userTdTitle" scope="col" width="35%">Title</th>
                            <th data-sort="string" class="userTdTitle" scope="col" width="25%">Client</th>
                            <th data-sort="string" class="userTdTitle" scope="col" width="15%">Responsible</th>
                            <th data-sort="int" class="userTdTitle" scope="col" width="10%">Priority</th>
                          </tr>
                        </thead>
 
                       <tbody>
              <?PHP
 
                          $KeyW = $_GET["Keyword"];
 
                          if ($KeyW=='')
                          {
                            // Le tableau sera vide lorsque l'utilisateur arrivera sur la page
                          }
                          else
                          {
                            while($ResultIss = mysql_fetch_array($ResultGetIss))
                            {
              ?>
                              <tr>
                                <td class='userTd1'>
                                  <a href="edittask.php?IDIssue=<?PHP echo $ResultIss["idissue"]; ?>"><?PHP echo $ResultIss["noproject"]."-".$ResultIss["noissue"]; ?></a>
                                </td>
                                <td align='center' class='userTd1'><?PHP echo $ResultIss["title"]; ?></td>
                                <td align='center' class='userTd1'>
                                  <a href="admin_client_info.php?IDClient=<?PHP echo $ResultPro["idclient"]; ?>"><?PHP echo $ResultPro["clientname"]; ?></a>
                                </td>
                                <td align='center' class='userTd1'><?PHP echo $ResultIss["responsible"]; ?></td>
                                <td align='right' class='userTd1'><?PHP echo $ResultIss["priority"]; ?></td>
                            </tr>
              <?PHP
                            }
                          }
              ?>
                      </tbody>
                  </table>
        </td>
    </tr>

Est-ce les balises PHP ouvertes et refermées de partout ? Le if==GET ? la boucle else while ? la requête SQL ?

Bonjour,
restons logique et concis jusqu'au bout :

Code:

1
2
3
4
5
6
                         $KeyW = (isset($_GET["Keyword"])? $_GET["Keyword"] : '';
 
                          if ($KeyW != '')
                          {
                            while($ResultIss = mysql_fetch_array($ResultGetIss))
                            {

Et de préférence, on écrit <?php en minuscules.

24/02/2015, 20h12
EkrazNeophite

Ok merci :)
(Je ne vois aucune différence ^^)