Aff, To Ficando Doido, Erro Ao Editar..

[taudujas]

Pessoal o que pode estar errado aqui:

<?

include "config.php";

foreach ($_POST as $campo => $valor)

$atualiza = "update dados set nome,email, telefone, icq, MSN, marca, modelo, valor, anuncio where id = " . $_POST['id'];

mysql_query($atualiza);

header("Location: index.php\n");

?>

[taudujas]

ninguém viu o erro ainda??

[ppgsalomao]

o erro tah na query !!!

tipo:

$atualiza = "update dados set nome = " . $_POST['nome'] . ", email = " . $_POST['email] . ", telefone = " . $_POST['telefone'] . ", icq = " . $_POST['icq] . ", MSN = " . $_POST['MSN'] . " , marca = " $_POST['marca] . ", modelo = " . $_POST['modelo'] . ", valor = " . $_POST['valor] . " , anuncio = " . $_POST['anuncio'] . " where id = " . $_POST['id'];

[taudujas]

Obrigado Pedro, mas pensava que so o foreach resolvia. valeu.

[ppgsalomao]

Não resolve !!

Na verdade aih no caso ele só vai executar um monte de vezes desnescessariamente !!

[taudujas]

é, deu erro olha ai:

Parse error: parse error, unexpected T_VARIABLE in C:\root\classficados\admin\atualizar.php on line 6

<?

include "config.php";

foreach ($_POST as $campo => $valor)

$atualiza = "update dados set nome = " . $_POST['nome'] . ", email = " . $_POST['email'] . ", telefone = " . $_POST['telefone'] . ", icq = " . $_POST['icq'] . ", MSN = " . $_POST['MSN'] . " , marca = " $_POST['marca'] . ", modelo = " . $_POST['modelo'] . ", valor = " . $_POST['valor'] . " , anuncio = " . $_POST['anuncio'] . " where id = " . $_POST['id'];

mysql_query($atualiza);

header("Location: index.php\n");

?>

[ppgsalomao]

cara,

você presto atenção se todos esses campos existem ??

O campo nome, email, telefone, icq, MSN, marca, modelo, valor e anuncio ???

[taudujas]

Existe sim, estao todos certo.

[taudujas]

A tabela ai:

CREATE TABLE `dados` (

  `id` int(10) NOT NULL auto_increment,

  `nome` varchar(70) NOT NULL default '',

  `email` varchar(70) NOT NULL default '',

  `telefone` varchar(70) NOT NULL default '',

  `icq` varchar(40) NOT NULL default '',

  `MSN` varchar(70) NOT NULL default '',

  `marca` varchar(70) NOT NULL default '',

  `modelo` varchar(70) NOT NULL default '',

  `ano` varchar(10) NOT NULL default '',

  `valor` varchar(70) NOT NULL default '',

  `uf` varchar(20) NOT NULL default '',

  `anuncio` varchar(250) NOT NULL default '',

  PRIMARY KEY  (`id`)

) TYPE=MyISAM AUTO_INCREMENT=10 ;

[Illidan]

Experimenta colocar aspas simples assim:

<?

include "config.php";

foreach ($_POST as $campo => $valor)

$atualiza = "update dados set nome = '" . $_POST['nome'] . "', email = '" . $_POST['email] . "', telefone = '" . $_POST['telefone'] . "', icq = '" . $_POST['icq] . "', MSN = '" . $_POST['MSN'] . "' , marca = '" $_POST['marca] . "', modelo = '" . $_POST['modelo'] . "', valor = '" . $_POST['valor] . "' , anuncio = '" . $_POST['anuncio'] . "' where id = '" . $_POST['id] . "' ";

mysql_query($atualiza);

header("Location: index.php\n");

?>

Falou!

[taudujas]

Mesmo erro:

Parse error: parse error, unexpected T_VARIABLE in C:\root\classficados\admin\atualizar.php on line 6

[Illidan]

Pow, assim era pra funcionar... :/

TODAS essas variáveis estão definidas??? Já experimentou fazer um teste colocando um "@" antes de cada uma delas? Só não coloque antes da que carrega o ID, pois essa cláusula UPDATE (assim como DELETE) é uma bomba no seu bd... nunca se esqueça de usar a cláusula WHERE com elas...

Falow!

[ppgsalomao]

Faltava um ponto !!!

<?

include "config.php";

foreach ($_POST as $campo => $valor)

$atualiza = "update dados set nome = '" . $_POST['nome'] . "', email = '" . $_POST['email'] . "', telefone = '" . $_POST['telefone'] . "', icq = '" . $_POST['icq'] . "', MSN = '" . $_POST['MSN'] . "' , marca = '" . $_POST['marca] . "', modelo = '" . $_POST['modelo'] . "', valor = '" . $_POST['valor'] . "' , anuncio = '" . $_POST['anuncio'] . "' where id = '" . $_POST['id'] . "' ";

mysql_query($atualiza);

header("Location: index.php\n");

?>

[taudujas]

Beleza ,vou testar.

Obrigado.

[ikkinet]

na verdade o erro não esta na linha 6 e sim na linha 4...

depois do foreach ($_POST as $campo => $valor) ficou faltando o { };

este e um comando q precisa das chaves e principalmente do ponto e virgula no final....

[Fabyo]

Assim :

Vai pegar todos os campos e criar as variaveis com o mesmo nome e ainda tirar as tags html

foreach ($_POST as $campo => $valor) { $$campo = strip_tags($valor); }

e não esqueça do :

$atualiza = "update dados set nome = '$nome'  , email = '$email' , telefone = '$telefone', icq = '$icq', MSN = '$MSN' , marca = '$marca', modelo = '$modelo', valor = '$valor' , anuncio = '$anuncio' where id = '$id' ";

OBS: Use foreach se você tiver muitos campos no formulario dai agiliza seu trabalho , mas se tiver poucos campos use o $_POST["campo"]

OBS2: Se esse id estiver vindo por url não esqueça de pegar ele com o $_GET

$id = $_GET["id"];

falou

ate mais

[taudujas]

Beleza fabyo já arrumei agora já esta certinho..

obrigado.

[taudujas]

podem fechar o tópico.

[Sperto_Rox]

é o seuinte ... esse foreach cria todas as variaveis .. mas sendo q aqui em casa se eu fizer assim : $campo .. sem declarar a variavel funciona corretamente ...

será q eu sou privilegiado? ahuauhuha

[ikkinet]

Sperto Rox

não e privilegiado não....

no meu server e aki em casa tb acontece isso.... isso é devido ao nosso php.ini estar com o Globals ON....

isso faz com q todas as variaveis q vierem do formulario já estejam definidas.....

[Fabyo]

Que no caso não é bom

use o register em off

http://php.planetmirror.com/manual/pt_BR/s...sterglobals.php

[Nêutron]

agora o register_globals vem OFF por padrão...

e isso é bom, porque se não, acaba sujando o escopo de globais... (palavras do manual... sei lá o que é ESCOPO hehehehehehe)