newbie having trouble with php and forms

demiancalixto · 09-25-2008, 06:57 AM

hello guys!
i'm newbie to php and now trying to make some nice stuff.
i'm developing a project to manage some data related to public health using php and xml.
well, but going to the point, i have a page with forms like this:

Code:

<html>
<head>
<title>Incluir Estabelecimento</title>
</head>
<body>
<form action="salva.php" method="post">
<p><b>Nome*: </b> <input type="text" name="nome" /></p>
<p><b>Respons&aacute;vel*: </b> <input type="text" name="resp" /></p>
<p><b>Endere&ccedil;o*: </b> <input type="text" name="ende" /></p>
<p><b>Telefone: </b> <input type="text" name="tele" /></p>
<p><b>Ramo de atividade: </b> <input type="text" name="ramo" /></p>
<p><b>Emiss&atilde;o do &uacute;ltimo alvar&aacute;: </b> <input type="text" name="emis" /></p>
<p><b>Vencimento do alvar&aacute;: </b> <input type="text" name="venc" /></p>
<input type="hidden" value="OK" name="esco" />
<p><input type="submit" value="Incluir dados" name="incl" /></p>
</body>
</html>

and this one submits data from the form to salva.php:

Code:

<?php
$tabela = new DOMDocument('1.0', 'utf8');
	$tabela->formatOutput = true;
	
$nomes = trim($_POST['nome']);
$resps = trim($_POST['resp']);
$endes = trim($_POST['ende']);
$teles = trim($_POST['tele']);
$ramos = trim($_POST['ramo']);
$emiss = trim($_POST['emis']);
$vencs = trim($_POST['venc']);

$spn = strlen($nomes);
$spr = strlen($resps);
$spe = strlen($endes);

// ---------------------------------

function escreveNovo() {
if (!isset($nomes) || !isset($resps) || !isset($endes)) {
	echo("Preencha os campos obrigatórios! <a href=\"http://localhost/visa/inclui.php\">Voltar </a>");
	exit;
}
else {	
	header("Content-type: text/xml");
	$indi = strpos($resps, " ");
	$espa = $indi - 1;
	$titu = substr($resps, 0, $espa);

	$pri = $tabela->createElement("estabelecimento");
	$tabela->appendChild($pri);
	$sec = $tabela->createAttr("dono");
	$pri->appendChild($sec);
	$ter = $tabela->createElement("nome");
	$pri->appendChild($ter);
	$qua = $tabela->createElement("responsavel");
	$pri->appendChild($qua);
	$qui = $tabela->createElement("endereco");
	$pri->appendChild($qui);
	$sex = $tabela->createElement("telefone");
	$pri->appendChild($sex);
	$set = $tabela -> createElement("ramo");
	$pri->appendChild($set);
	$oit = $tabela -> createElement("emissao");
	$pri->appendChild($oit);
	$non = $tabela -> createElement("vencimento");
	$pri->appendChild($non);

	$utit = $tabela->createTextNode($titu);
	$sec->appendChild($utit);
	$enom = $tabela->createTextNode($nomes);
	$ter->appendChild($enom);
	$pser = $tabela->createTextNode($resps);
	$qua->appendChild($pser);
	$edne = $tabela->createTextNode($endes);
	$qui->appendChild($edne);
	$elet = $tabela->createTextNode($teles);
	$sex->appendChild($elet);
	$omar = $tabela->createTextNode($ramos);
	$set->appendChild($omar);
	$sime = $tabela->reateTextNode($emiss);
	$oit->appendChild($sime);
	$cnev = $tabela->createTextNode($vencs);
	$non->appendChild($cnev);
}
}

// -----------------------------------------

escreveNovo();
$tabela->save('estab.xml');
?>

i'm in trouble, becuase even when all text fields are filled, the script ends up to the first "if" clause in function escreveNovo(), and returns the message (which should appear only if one of the first three fields were empty).

i'm really stuck with that. Another important thing is that i've already tried to echo the strings from the forms, and they are actually there, working perfectly!

thanks in advance for the support!

**dm01** · 09-25-2008, 03:39 PM

Try this:

Replace

PHP Code:


			
if (!isset($nomes) || !isset($resps) || !isset($endes)) {

    echo("Preencha os campos obrigatórios! <a href=\"http://localhost/visa/inclui.php\">Voltar </a>");

    exit;

}

with

PHP Code:


			
if (!isset($nomes) {

    echo("Preencha os campo Nomes! <a href=\"http://localhost/visa/inclui.php\">Voltar </a>");

    exit;

}

elseif (!isset($resps) {

    echo("Preencha os campo Respons&aacute;vel! <a href=\"http://localhost/visa/inclui.php\">Voltar </a>");

    exit;

} 

elseif (!isset($endes) {

    echo("Preencha os campo Endere&ccedil;o <a href=\"http://localhost/visa/inclui.php\">Voltar </a>");

    exit;

}

The elseif statement is used if there is more than one condition that needs to be checked. In this case, !isset(nomes), !isset(resps), and !isset(endes) all need to be FALSE in order for the program to continue nomally. The elseif statement works just like the if statement, checking the conditions for truth. If any of the statments are TRUE, then the message "Preencha o campo <field>!" is returned. Please note I am guessing at the Portuguese grammar with some help with an on-line translator.

09-25-2008, 03:39 PM	#2 (permalink)
dm01 Moderator: Design Join Date: Oct 2006 Location: Richmond, B.C.; Canada Posts: 1,446 OS: Windows XP [Version 5.1.2600] SP3 \| Ubuntu Jaunty Jackalope \| Windows 7 BETA My System	Re: newbie having trouble with php and forms Try this: Replace PHP Code: `if (!isset($nomes) \|\| !isset($resps) \|\| !isset($endes)) { echo("Preencha os campos obrigatórios! <a href=\"http://localhost/visa/inclui.php\">Voltar </a>"); exit; }` with PHP Code: `if (!isset($nomes) { echo("Preencha os campo Nomes! <a href=\"http://localhost/visa/inclui.php\">Voltar </a>"); exit; } elseif (!isset($resps) { echo("Preencha os campo Responsável! <a href=\"http://localhost/visa/inclui.php\">Voltar </a>"); exit; } elseif (!isset($endes) { echo("Preencha os campo Endereço <a href=\"http://localhost/visa/inclui.php\">Voltar </a>"); exit; }` The elseif statement is used if there is more than one condition that needs to be checked. In this case, !isset(nomes), !isset(resps), and !isset(endes) all need to be FALSE in order for the program to continue nomally. The elseif statement works just like the if statement, checking the conditions for truth. If any of the statments are TRUE, then the message "Preencha o campo <field>!" is returned. Please note I am guessing at the Portuguese grammar with some help with an on-line translator. __________________ Validate your Markup Validate your CSS Notepad++ Please use [html], [php], and [code] when posting code or markup. I do not help by Private Message or e-mail. If for some reason I have over-looked a reply to a thread that I have previously replied to, then send me a message.