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

**beegees** · 13/10/2008, 16h16

Salut tout le monde,

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

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 :

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

d'avance.

beegees

**sabotage** · 13/10/2008, 16h33

Code :

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

 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 :

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

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

**ThomasR** · 13/10/2008, 16h46

Envoyé par sabotage

Code :

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

 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 :

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

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

**beegees** · 13/10/2008, 16h54

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 :

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

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

**beegees** · 13/10/2008, 16h56

Envoyé par sabotage

Code :

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

 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 :

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

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 :

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

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

Merci d'avance.

beegees

**ThomasR** · 13/10/2008, 17h13

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

**ThomasR** · 13/10/2008, 17h15

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

Voici le CODE COMPLET :

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
 
# 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>';

**sabotage** · 13/10/2008, 17h19

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

Code :

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

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>';
    }

**ThomasR** · 13/10/2008, 17h25

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

**beegees** · 13/10/2008, 17h33

Envoyé par ThomasR

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

Voici le CODE COMPLET :

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

beegees

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 :

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

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>';
    }