[Conception] Une fonction édit?

Version imprimable

Voir 40 message(s) de cette discussion en une page

Bonjour !!!

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

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

31/03/2006, 16h07
onet

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 ;)
31/03/2006, 16h09
Vash_vador

oui,

Il n'y a rien qui s'affiche ....
31/03/2006, 16h27
hugo123

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.
31/03/2006, 16h38
Vash_vador

oki, merci de l'info

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

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

31/03/2006, 20h01
amika

essaye de remplacer cette portion:

Code:

$q = "UPDATE actualites SET type='".$type."' , titre='".$titre."' , texte='".$texte."' , pseudo='".$pseudo."' , image='".$image."' WHERE id='".$id."'";
31/03/2006, 20h42
Vash_vador

Hummmm

ça na rien changé ...

:cry:
31/03/2006, 21h27
ShinJava

Salut :)

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

Code:

$id = $_GET['id'];

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

Code:

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

<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
01/04/2006, 11h30
Vash_vador

Citation:

Envoyé par ShinJava

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

Code:

$id = $_GET['id'];

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

Jai tester ton bout de code, rien s'affiche :(
01/04/2006, 11h48
zulot

Citation:

Envoyé par Vash_vador

Citation:

Envoyé par ShinJava

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

Code:

$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.

ah?

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

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

01/04/2006, 23h50
ShinJava

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

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

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

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

Que faire :'(

07/04/2006, 21h55
ShinJava

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

Code:

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:

$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

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:

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

08/04/2006, 12h57
Vash_vador

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
08/04/2006, 14h26
ShinJava

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
08/04/2006, 17h29
Vash_vador
Oki
je vous poste le liens qui choisira la news a éditer :
le voici:
Code:

1 2 <a href="?p=actualitesmod&id=<? echo $t['id'] ?>"><? echo $t['titre'] ?>
en espérent que mes érreurs de viennent pas de là ^^

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

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

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

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

Voir 40 message(s) de cette discussion en une page