[Conception] Une fonction édit?

**Vash_vador** · 31/03/2006, 16h06

Bonjour !!!

Voilà, cela fait plusieurs heures que je narrive pas a trouver mon erreur pour cette fonction edit ....

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
 
<?php
$id = $_GET['id']; 
 
$q = "UPDATE `actualites` SET type='type' , titre='titre' , texte='texte' , pseudo='pseudo' , image='$image' WHERE id='$id'";
$r = mysql_query($q,C_MYSQL) or die($q.' : '.mysql_error());
unset($_GET);
 ?>
 
 
 
<form method="post" action="" enctype="multipart/form-data">
<table width="500" border="0">
<tr>
<td>Pseudo:</td>
<td><input name="pseudo" type="text" class="post" value="<? echo $_GET['pseudo'] ?>"></td>
</tr>
<tr>
<td>Type:</td>
<td><select name="type">
<option value="generale"<?php if($_GET['type'] == 'site'){?> selected<?php }?>>site</option>
<option value="scenenews"<?php if($_GET['type'] == 'japom'){?> selected<?php }?>>japon</option>
<option value="generale"<?php if($_GET['type'] == 'manga'){?> selected<?php }?>>manga</option>
</select>
</td>
</tr>
<tr>
<td>Titre</td>
<td><input name="titre" type="text" class="post" value="<? echo $_GET['titre']?>"></td>
</tr>
<tr>
<td>Texte:</td>
<td><textarea name="texte" class="post" cols="50" rows="10"><? echo $_GET['texte'] ?></textarea>
</td>
</tr>
<td>Image:</td>
<td><input name="image" class="post" value="<? echo $_GET['image']?>"></td>
<tr>
<td><input class="post" name="action" type="submit" value="Envoyer"></td>
</tr>
</table>
</form>

Voilà, si quelqun pourai m'aider ...

merci .

**onet** · 31/03/2006, 16h07

Peux-tu nous dire quelle erreur tu as ?

Onet
[edit] en fait, y a 1-2 choses qui me semblent bizzaredans ton code. J'attends donc ton erreur pour regarder plus loin

**Vash_vador** · 31/03/2006, 16h09

oui,

Il n'y a rien qui s'affiche ....

**hugo123** · 31/03/2006, 16h27

Je vois de multiples choses qui n'ont pas l'air bonnes :
Dans ton update, tu fais des SET type='type' etc... T'es sur que ca devrait pas être type='$type' avec $type = $_POST['type']; ?
(idem pour les autres champs)

Ensuite, tu devrais préciser ton action plutot que de la laisser vide.
Tu dis que ton formulaire est en POST mais tu utilises partout $_GET
Le unset $_GET ne sert à rien.
A aucun moment je ne vois une connexion au SGBD de faite.

edit : ah oui j'oubliais, lors de la première arrivée sur la page, tu dois vérifier tes valeurs pour ne pas faire des updates sans arrêt.

**Vash_vador** · 31/03/2006, 16h38

oki, merci de l'info

**Vash_vador** · 31/03/2006, 19h37

Salut
j'ai modfier pas ça mais ca naffiche toujour rien :'(

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
 
<?php
$id = $_GET['id']; 
 
// On commence par récupérer les champs
if(isset($_GET['type']))      $type=$_GET['type'];
else      $type="";
 
if(isset($_GET['titre']))      $titre=$_GET['titre'];
else      $titre="";
 
if(isset($_GET['texte']))      $texte=$_GET['texte'];
else      $texte="";
 
if(isset($_GET['pseudo']))      $pseudo=$_GET['pseudo'];
else      $pseudo="";
 
if(isset($_GET['image']))      $image=$_GET['image'];
else      $image="";
 
$q = "UPDATE `actualites` SET `type`='$type' , `titre`='$titre' , `texte`='$texte' , `pseudo`='$pseudo' , `image`='$image' WHERE `id`='$id'";
$r = mysql_query($q,C_MYSQL) or die($q.' : '.mysql_error());
 
 ?>
 
<form method="GET" action="" enctype="multipart/form-data">
<table width="500" border="0">
<tr>
<td>Pseudo:</td>
<td><input name="pseudo" type="text" class="post" value=""></td>
</tr>
<tr>
<td>Type:</td>
<td>
</td>
</tr>
<tr>
<td>Titre</td>
<td><input name="titre" type="text" class="post" value="<? echo $_GET['titre']?>"></td>
</tr>
<tr>
<td>Texte:</td>
<td><textarea name="texte" class="post" cols="50" rows="10"><? echo $_GET['texte'] ?></textarea>
</td>
</tr>
<td>Image:</td>
<td><input name="image" class="post" value="<? echo $_GET['image']?>"></td>
<tr>
<td><input class="post" name="action" type="submit" value="Envoyer"></td>
</tr>
</table>
</form>

**amika** · 31/03/2006, 20h01

essaye de remplacer cette portion:

Code :

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

$q = "UPDATE actualites SET type='".$type."' , titre='".$titre."' , texte='".$texte."' , pseudo='".$pseudo."' , image='".$image."' WHERE id='".$id."'";

**Vash_vador** · 31/03/2006, 20h42

Hummmm

ça na rien changé ...

**ShinJava** · 31/03/2006, 21h27

Salut

1/ Il y a un problème pour le ID. A quoi fait référence $_GET['id'] dans :

Code :

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

$id = $_GET['id'];

2/ Ensuite, je ne suis pas sur que le code suivant fonctionne :

Code :

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

<form method="GET" action="" enctype="multipart/form-data">

J'ai jamais essayé... ca se trouve ca marche, mais sinon je fais cela de la manière classique qui est utilisé par beaucoup de monde :

Code :

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

<form method="GET" action="<?php echo ($_SERVER['PHP_SELF']); ?>" enctype="multipart/form-data">

Voila voila, repond à la 1ère question surtout, c'est de là que vient ton problème.

++
ShinJava

**Vash_vador** · 01/04/2006, 11h30

Envoyé par ShinJava

1/ Il y a un problème pour le ID. A quoi fait référence $_GET['id'] dans :

Code :

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

$id = $_GET['id'];

Il fait réference pour le WHERE `id`='$id'

Jai tester ton bout de code, rien s'affiche

**zulot** · 01/04/2006, 11h48

Envoyé par Vash_vador

Envoyé par ShinJava

1/ Il y a un problème pour le ID. A quoi fait référence $_GET['id'] dans :

Code :

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

$id = $_GET['id'];

Il fait réference pour le WHERE `id`='$id'

Jai tester ton bout de code, rien s'affiche

Ce qu'il voulait dire pour le $_GET['id'] c'est que tu ne transmet jamais la variable id dans l'url donc il fait un where de rien puisqu'il n'a pas de valeur.

**Vash_vador** · 01/04/2006, 12h10

ah?

donc j'enlève le Get id en gros?

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
 
<?php
 
 
// On commence par récupérer les champs
if(isset($_GET['type']))      $type=$_GET['type'];
else      $type="";
 
if(isset($_GET['titre']))      $titre=$_GET['titre'];
else      $titre="";
 
if(isset($_GET['texte']))      $texte=$_GET['texte'];
else      $texte="";
 
if(isset($_GET['pseudo']))      $pseudo=$_GET['pseudo'];
else      $pseudo="";
 
if(isset($_GET['image']))      $image=$_GET['image'];
else      $image="";
 
$q = "UPDATE actualites SET type='".$type."' , titre='".$titre."' , texte='".$texte."' , pseudo='".$pseudo."' , image='".$image."' WHERE id='".id."'";
$r = mysql_query($q,C_MYSQL) or die($q.' : '.mysql_error());
 
 ?>
 
<form method="GET" action="<?php echo ($_SERVER['PHP_SELF']); ?>" enctype="multipart/form-data"><table width="500" border="0">
<tr>
<td>Pseudo:</td>
<td><input name="pseudo" type="text" class="post" value=""></td>
</tr>
<tr>
<td>&nbsp;</td>
<td>
</td>
</tr>
<tr>
<td>Titre</td>
<td><input name="titre" type="text" class="post" value="<? echo $_GET['titre']?>"></td>
</tr>
<tr>
<td>Texte:</td>
<td><textarea name="texte" class="post" cols="50" rows="10"><? echo $_GET['texte'] ?></textarea>
</td>
</tr>
<td>Image:</td>
<td><input name="image" class="post" value="<? echo $_GET['image']?>"></td>
<tr>
<td><input class="post" name="action" type="submit" value="Envoyer"></td>
</tr>
</table>
</form>

**ShinJava** · 01/04/2006, 23h50

Oui mais il y a un gros problème de conception là.

Ton ID faisait référence à une ligne que tu devais mettre à jour dans ta base de donnée n'est-ce pas ?
En gros il faut pas l'enlever, mais il faut lui attribuer l'ID de la ligne que tu veux updater. Maintenant à savoir comment faire ? Cela depend de comment tu as codé tout ça, ou comment tu vas coder tout ça.

A toi de jouer

++
ShinJava

**Vash_vador** · 07/04/2006, 19h51

Salut
donc jai toujour le meme problème
j'ai changer quelque truc mais bon

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
 
<?php
print_r($_GET);
 
// On récupére les champs
if(isset($_GET['type']))      $type=$_GET['type'];
else      $type="";
 
if(isset($_GET['titre']))      $titre=$_GET['titre'];
else      $titre="";
 
if(isset($_GET['texte']))      $texte=$_GET['texte'];
else      $texte="";
 
if(isset($_GET['pseudo']))      $pseudo=$_GET['pseudo'];
else      $pseudo="";
 
if(isset($_GET['image']))      $image=$_GET['image'];
else      $image="";
 
$q = "UPDATE news SET type='".$type."' , titre='".$titre."' , texte='".$texte."' , pseudo='".$pseudo."' , image='".$image."' WHERE id='".id."'";
$r = mysql_query($q,C_MYSQL) or die($q.' : '.mysql_error());
 
 ?>
 
<form method="GET" action="<?php echo ($_SERVER['PHP_SELF']); ?>" enctype="multipart/form-data"><table width="500" border="0">
<tr>
<td>Pseudo:</td>
<td><input name="pseudo" type="text" class="post" value=""></td>
</tr>
<tr>
<td>&nbsp;</td>
<td>
</td>
</tr>
<tr>
<td>Titre</td>
<td><input name="titre" type="text" class="post" value="<? echo $_GET['titre']?>"></td>
</tr>
<tr>
<td>Texte:</td>
<td><textarea name="texte" class="post" cols="50" rows="10"><? echo $_GET['texte'] ?></textarea>
</td>
</tr>
<td>Image:</td>
<td><input name="image" class="post" value="<? echo $_GET['image']?>"></td>
<tr>
<td><input class="post" name="action" type="submit" value="Envoyer"></td>
</tr>
</table>
</form>

J'ai un ligne qui saffiche en haut du genre :

Code :

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

Array ( [p] => actualitesmod [id] => 25 )

Que faire :'(

**ShinJava** · 07/04/2006, 21h55

Salut Vash_Vador

,
Alors la ligne qui s'affiche en haut c'est surement à cause de :

Code :

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

print_r($_GET);

Cela permet d'afficher les informations de ta variable $_GET.

Je pense qu'il y'a un problème dans requete :

Code :

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

$q = "UPDATE news SET type='".$type."' , titre='".$titre."' , texte='".$texte."' , pseudo='".$pseudo."' , image='".$image."' WHERE id='".id."'";

C'est plutot $id ? Si c'est le cas, a quoi fait-il reference ?
L'id est tres important pour ta requete, si tu en as pas, tu ne pourra pas mettre à jour ta ligne.
Mais avant d'aller plus loin, et pour être sur qu'on va dans le bon sens, peux-tu nous expliquer ce que tu aimerais faire avec ce code (mettre a jour une information? ajouter, effacer, ???)

++
ShinJava

**rfufu** · 08/04/2006, 00h03

c'est pas gagné

bon, alors je te propose pas de solution, mais une amélioration étant donné que ton code ne marchera effectivement que lorsque tu fera venir ta variable $id de quelque part.

Donc voilà :

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
 
<?php
// si tu met pas ton if ici, tu ferras un update a chaque foi
// que ta page sera chargée.
if(isset($_GET)){
	// extraction des variables contenu dans le tableau GET
	extract($_GET);
	$q = "
		UPDATE	news
		SET	
			type='".$type."',
			titre='".$titre."',
			texte='".$texte."',
			pseudo='".$pseudo."',
			image='".$image."'
		WHERE id='".$id."'";
		//effectivement ton $id, il faut le récupérer quelque part.
		//mais comme un print_r t'as renvoyé id => 25, j'imagine que
		//tu sais d'ou tu le fais venir.
 
	$r = mysql_query($q,C_MYSQL) or die($q.' : '.mysql_error());
 
	//enfin, vide tes variables pour éviter
	//qu'une de ces valeurs ne servent à updater un autre champ
	//dont la valeur ne serait pas précisé.
	$type = null;
	$titre = null;
	$texte = null;
	$pseudo = null;
	$image = null;
	$id = null;
}
 ?>

mais décris nous un peu mieux ton but...

**Vash_vador** · 08/04/2006, 12h57

Salut !

Bé en fait le but de ce code et de faire une fonctrion 'éditer' pour modifier les news.

et que dans le tableau, il y a le texte qui saffiche pour pouvroir directement le modifier

**ShinJava** · 08/04/2006, 14h26

Ok c'est ce qu'il me semblait.
On va continuer par étape maintenant.
A quel moment choisis-tu la news à modifier (j'imagine qu'il y'en aura pas qu'une seule) ?

Ce que je veux dire, c'est que tu dois d'abord definir la news à modifier parmis tant d'autres. Et quand tu l'auras choisi, il te faudra passer son ID dans ta nouvelle page (dans ton code que tu proposes), afin que la requete MySQL puisse trouver l'ID de la news à modifier.

Donc repond à ma question du début de ce message, voir comment procéder =)

++
ShinJava

**Vash_vador** · 08/04/2006, 17h29

Oki
je vous poste le liens qui choisira la news a éditer :
le voici:

Code :

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

1
2
 
<a href="?p=actualitesmod&amp;id=<? echo $t['id'] ?>"><? echo $t['titre'] ?>

en espérent que mes érreurs de viennent pas de là ^^

**ShinJava** · 08/04/2006, 18h06

Ah le voila le fameux ID ^^ !
Donc voici comment doit être ton code :

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
59
60
61
62
63
64
65
<?php
 
// Si le formulaire a etait validé, alors on execute la condition if.
// Le nom "action" fait reference à celui que tu as donné dans ton bouton envoyer.
if ( $_GET['action'] )						
{
 
	if(isset($_GET['id']))      
	$id=$_GET['id'];
	else      $id="";
 
 
	if(isset($_GET['type']))      $type=$_GET['type'];
	else      $type="";
 
	if(isset($_GET['titre']))      $titre=$_GET['titre'];
	else      $titre="";
 
	if(isset($_GET['texte']))      $texte=$_GET['texte'];
	else      $texte="";
 
	if(isset($_GET['pseudo']))      $pseudo=$_GET['pseudo'];
	else      $pseudo="";
 
	if(isset($_GET['image']))      
	$image=$_GET['image'];
	else      $image="";
 
 
 
 
 
$q = "UPDATE news SET type='".$type."' , titre='".$titre."' , texte='".$texte."' , pseudo='".$pseudo."' , image='".$image."' WHERE id='".$id."'";
$r = mysql_query($q,C_MYSQL) or die($q.' : '.mysql_error());
}
 ?>
 
<form method="GET" action="<?php echo ($_SERVER['PHP_SELF']); ?>" enctype="multipart/form-data"><table width="500" border="0">
<input type="hidden" name="id" value="<?php echo $_GET['id']; ?>">
<tr>
<td>Pseudo:</td>
<td><input name="pseudo" type="text" class="post" value=""></td>
 
</tr>
<tr>
<td>&nbsp;</td>
<td>
</td>
</tr>
<tr>
<td>Titre</td>
<td><input name="titre" type="text" class="post" value="<? echo $_GET['titre']?>"></td>
</tr>
<tr>
<td>Texte:</td>
<td><textarea name="texte" class="post" cols="50" rows="10"><? echo $_GET['texte'] ?></textarea>
</td>
</tr>
<td>Image:</td>
<td><input name="image" class="post" value="<? echo $_GET['image']?>"></td>
<tr>
<td><input class="post" name="action" type="submit" value="Envoyer"></td>
</tr>
</table>
</form>

Voila voila, je t'explique juste cette ligne :

Code :

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

<input type="hidden" name="id" value="<?php echo $_GET['id']; ?>">

Il s'agit d'un champs caché où tu donne à la variable id, la valeur "id" qui a etait passé dans l'adresse que tu m'a donné c'est à dire :

Code :

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

id=<? echo $t['id'] ?>

Si tu ce n'est pas clair ou si tu as des questions, n'hesite pas.
En fait malgré quelques petites erreurs de syntaxe, tu vas oublié de faire passé l'ID entre ta page où il y avait le choix de news et la page où on pouvait editer les news. C'est cette ID qui permet de dire à la table de ta base de donné quel ligne il faut modifier.

Voila voila,
bon courage

++
ShinJava

[Conception] Une fonction édit?

PHP & Base de données

Vue hybride

Discussions similaires

Partager

Partager