Mon code contient un bug : il double le nombre de cellules

Version imprimable

Salut tout le monde,

J'ai un code génial, mais malheureusement, il est équipé d'un petit bug :mouarf:

Le code permet de générer un tableau avec un certain nombre de lignes et de cellules (nombre de champ de la table).

Le hic, c'est qu'il me crée les cellules en double.

Donc si j'ai 4 champs, il me crée 8 cellules dont 4 vides.

Voici mon code, sauriez-vous me dire où je peux modifier par ce que ça prend le double du temps pour le chargement :

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
<?php
# script qui permet de créer des zones de texte à la volée et d'afficher le nom des champs de la table
# variable qui contient le nom de ma base de données.
$dbname = 'deliberations';
 
 
$link = mysql_connect('localhost', 'root', 'mdp');
	if(!$link)	  
	 {
   		echo 'Impossible de se connecter à MySQL';
   		exit;
	}
 
	// Rendre la base de données deliberation, la base courante
$db_selected = mysql_select_db($dbname, $link);
if (!$db_selected) 
	{
	   die ('Impossible de sélectionner la base de données : ' . mysql_error());
	}
 
# $_POST[select_Nom_Tables] récupère la valeur choisie par l'utilisateur, il faut ensuite cliquer sur le bouton pour qu'apparaisse les zones de texte	
$result = mysql_query('SELECT * FROM ' . $_POST['select_Nom_Tables']) or die (mysql_error());
 
 
echo "result vaut $result";
 
echo '<form action="" method="POST"><p>';
while ($row = mysql_fetch_assoc($result)) 
{
	echo "row vaut $row . ";
    // Ne montre que les noms des champs grâce à 'Field'
 
	echo '<table border="1">';
$query = mysql_query("SELECT * FROM .$_POST[select_Nom_Tables] LIMIT 20") or die (mysql_error());
echo '<tr>';
for($i=0;$i<=mysql_num_fields($query);$i++)
    {
    echo '<th>'.mysql_field_name($query,$i).'</th>';
    }
echo '</tr>';
while($row = mysql_fetch_array($query))
    {
    echo '<tr>';
    for($i=0;$i<count($row);$i++)
        {
        echo '<td> <textarea>'.$row[$i].'</textarea></td>';
        }
    echo '</tr>';
    }
echo '</table>';
 
 
 
}
echo '<input type="submit" name="Submit" value="Mettre la table à jour" />' ; 
echo '</p></form>';
 
?>

Un tout grand :merci: d'avance.

beegees

13/10/2008, 15h33
sabotage
Code:

for($i=0;$i<count($row);$i++)

Si $row à 2 elements et que tu vas de 0 à 2, tu fais 3 boucles.

Plus simplement pour parcourir le tableau
Code:

1 2 3 foreach ($row as $key=>$value) { echo '<td> <textarea>'.$value.'</textarea></td>'; }
13/10/2008, 15h46
ThomasR

Citation:

Envoyé par sabotage

Code:

for($i=0;$i<count($row);$i++)

Si $row à 2 elements et que tu vas de 0 à 2, tu fais 3 boucles.

Il y a bien écris, strictement inférieur au nombre d'occurence de $row. Si $row a deux éléments, il boucle 2 fois.

C'est à cet endroit que cela ne pas :

Code:

for($i=0;$i<mysql_num_fields($query);$i++)
13/10/2008, 15h54
beegees

Citation:

Envoyé par ThomasR

Il y a bien écris, strictement inférieur au nombre d'occurence de $row. Si $row a deux éléments, il boucle 2 fois.

C'est à cet endroit que cela ne pas :

Code:

for($i=0;$i<mysql_num_fields($query);$i++)

Bonjour Thomas,

Merci pour ta réponse.

Le fait de retirer le = ne change rien au problème :?

beegees
13/10/2008, 15h56
beegees
Citation:
Envoyé par sabotage

Code:

for($i=0;$i<count($row);$i++)

Si $row à 2 elements et que tu vas de 0 à 2, tu fais 3 boucles.

Plus simplement pour parcourir le tableau

Code:

1 2 3 foreach ($row as $key=>$value) { echo '<td> <textarea>'.$value.'</textarea></td>'; }
Salut Sabotage,

J'espère ne pas en demander de trop mais saurais-tu stp te baser sur mon code pour le code que tu m'as fourni ?

Je ne sais pas si il faut changer $row...

Voici ce que j'ai mis pour t'avancer :
Code:

1 2 3 foreach ($row as $key=>$value) { echo '<td> <textarea>'.mysql_field_name($query).'</textarea></td>'; }
Merci d'avance.

beegees
13/10/2008, 16h13
ThomasR

Etant donné que tu utilises mysql_field_name, ce serait quand meme mieux d'utiliser une boucle for()

Tu as écris deux fois ta requête ....

Voici le CODE COMPLET :

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
 
# script qui permet de créer des zones de texte à la volée et d'afficher le nom des champs de la table
# variable qui contient le nom de ma base de données.
$dbname = 'deliberations';
 
 
$link = mysql_connect('localhost', 'root', 'mdp');
	if(!$link)	  
	 {
   		echo 'Impossible de se connecter à MySQL';
   		exit;
	}
 
	// Rendre la base de données deliberation, la base courante
$db_selected = mysql_select_db($dbname, $link);
if (!$db_selected) 
	{
	   die ('Impossible de sélectionner la base de données : ' . mysql_error());
	}
 
echo '<table border="1">';
$query = mysql_query("SELECT * FROM .$_POST[select_Nom_Tables] LIMIT 20") or die (mysql_error());
echo '<tr>';
for($i=0;$i<mysql_num_fields($query);$i++)
    {
    echo '<th>'.mysql_field_name($query,$i).'</th>';
    }
echo '</tr>';
while($row = mysql_fetch_array($query))
    {
    echo '<tr>';
    for($i=0;$i<count($row);$i++)
        {
        echo '<td> <textarea>'.$row[$i].'</textarea></td>';
        }
    echo '</tr>';
    }
echo '</table>';

Ton probleme vient de mysql_fetch_array qui par défaut retourne un index numerique et un index portant le nom du champ :

Code:

1
2
3
4
5
6
7
8
9
while($row = mysql_fetch_array($query,MYSQL_NUM))
    {
    echo '<tr>';
    foreach($row as $value)
        {
        echo '<td> <textarea>'.$value.'</textarea></td>';
        }
    echo '</tr>';
    }

13/10/2008, 16h25
ThomasR

Citation:

Envoyé par sabotage

Ton probleme vient de mysql_fetch_array qui par défaut retourne un index numerique et un index portant le nom du champ

Pas la peine de chercher midi à 14h :)

Citation:

Envoyé par ThomasR

Tu as écris deux fois ta requête ....

Voici le CODE COMPLET :

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
 
# script qui permet de créer des zones de texte à la volée et d'afficher le nom des champs de la table
# variable qui contient le nom de ma base de données.
$dbname = 'deliberations';
 
 
$link = mysql_connect('localhost', 'root', 'mdp');
	if(!$link)	  
	 {
   		echo 'Impossible de se connecter à MySQL';
   		exit;
	}
 
	// Rendre la base de données deliberation, la base courante
$db_selected = mysql_select_db($dbname, $link);
if (!$db_selected) 
	{
	   die ('Impossible de sélectionner la base de données : ' . mysql_error());
	}
 
echo '<table border="1">';
$query = mysql_query("SELECT * FROM .$_POST[select_Nom_Tables] LIMIT 20") or die (mysql_error());
echo '<tr>';
for($i=0;$i<mysql_num_fields($query);$i++)
    {
    echo '<th>'.mysql_field_name($query,$i).'</th>';
    }
echo '</tr>';
while($row = mysql_fetch_array($query))
    {
    echo '<tr>';
    for($i=0;$i<count($row);$i++)
        {
        echo '<td> <textarea>'.$row[$i].'</textarea></td>';
        }
    echo '</tr>';
    }
echo '</table>';

Un tout grand merci à toi Thomas.

Malheureusement ça ne marche pas, j'obtiens toujours les colonnes en double.

Je te remercie encore pour le temps que tu consacres à nous aider sur ce forum, c'est vraiment très sympa de ta part :king:

beegees

Citation:

Envoyé par sabotage

Ton probleme vient de mysql_fetch_array qui par défaut retourne un index numerique et un index portant le nom du champ :

Code:

1
2
3
4
5
6
7
8
9
while($row = mysql_fetch_array($query,MYSQL_NUM))
    {
    echo '<tr>';
    foreach($row as $value)
        {
        echo '<td> <textarea>'.$value.'</textarea></td>';
        }
    echo '</tr>';
    }

Merci pour ta réponse.

J'ai fait un copy and paste de ton code, ça fonctionne super bien, merci beaucoup.

beegees

13/10/2008, 16h35
beegees

Citation:

Envoyé par ThomasR

Pas la peine de chercher midi à 14h :)

En effet, le problème venait de là, encore un grand :merci: à vous deux.

beegees