PHP zapis emaila Opcje

[jakub333]

więc tak mam takie coś

[PHP] pobierz, plaintext 
<?php
print '<table border="1" width="100%" align="center">';
print '<tr>';
print '<td bgcolor="teal">';
print '<p align="center">PODAJ SWOJ E-MAIL</p>';
print '</td>';
print '<td bgcolor="gold" align="center">';
function  walidacja_email($email)
 
 {
  if (!eregi("^[_a-z0-9-]+(\.[_a-z0-9-]+)*@[a-z0-9-]+(\.[a-z0-9-]+)*(\.[a-z]{2,4})$", $email)) 
  { 
   echo '<font color="red"> wprowadzony adres email jest niepoprawny</font>';
	}
	else echo '<font color="green">wprowadziłeś poprawny adres email</font>';
}
 
 
$email = $_POST ["email"];
 echo <<<KOD
 
<form name="formularz" action="index.php" method="post">
  <input type"text" name="email" value="$email" />
  <input type="submit" value="Zapisz" />
</form>
KOD;
if ($email) walidacja_email($email);
 
{
$plik = fopen ("email.txt","a");
fputs ($plik, "$email  ").'';
fclose ($plik);
}
print '</td>';
print '</tr>';
print '</table>';
print '</form>';
?>
[PHP] pobierz, plaintext 

i tu moja pośba skrypt ma zapisywać tylko kiedy email jest poprawny a zapisuje nawet jak nie jest co mam z tym zrobić ? z góry dziękuje

Ten post edytował jakub333 22.09.2011, 18:45:18

[Ziem]

[PHP] pobierz, plaintext 
if ($email) walidacja_email($email);
[PHP] pobierz, plaintext 

Zwracaj z funkcji walidacja_email true (w przypadku poprawnego maila) i false (w przeciwnym wypadku) i wtedy robisz tak:

[PHP] pobierz, plaintext 
if(walidacja_email($email))
...
[PHP] pobierz, plaintext 

[jakub333]

nie wiem czy robię coś źle bo dalej nie działa tak jak powinno

[Ziem]

[PHP] pobierz, plaintext 
<?php
print '<table border="1" width="100%" align="center">';
print '<tr>';
print '<td bgcolor="teal">';
print '<p align="center">PODAJ SWOJ E-MAIL</p>';
print '</td>';
print '<td bgcolor="gold" align="center">';
function  walidacja_email($email)
 {
  if (!eregi("^[_a-z0-9-]+(\.[_a-z0-9-]+)*@[a-z0-9-]+(\.[a-z0-9-]+)*(\.[a-z]{2,4})$", $email)) 
  { 
      return false;
 
	}
	else return true;
}
 
 
$email = $_POST ["email"];
 echo <<<KOD
 
<form name="formularz" action="index.php" method="post">
  <input type"text" name="email" value="$email" />
  <input type="submit" value="Zapisz" />
</form>
KOD;
if (walidacja_email($email))
{
echo '<font color="green">wprowadziłeś poprawny adres email</font>';
$plik = fopen ("email.txt","a");
fputs ($plik, "$email  ").'';
fclose ($plik);
}
else
{
   echo '<font color="red"> wprowadzony adres email jest niepoprawny</font>';
}
print '</td>';
print '</tr>';
print '</table>';
print '</form>';
?>
[PHP] pobierz, plaintext 

Powinno działać.

[jakub333]

niestety nie "Parse error: syntax error, unexpected $end in D:\httpd\index.php on line 44"

[Ziem]

[PHP] pobierz, plaintext 
    <?php
    print '<table border="1" width="100%" align="center">';
    print '<tr>';
    print '<td bgcolor="teal">';
    print '<p align="center">PODAJ SWOJ E-MAIL</p>';
    print '</td>';
    print '<td bgcolor="gold" align="center">';
    function walidacja_email($email)
    {
    if (!eregi("^[_a-z0-9-]+(\.[_a-z0-9-]+)*@[a-z0-9-]+(\.[a-z0-9-]+)*(\.[a-z]{2,4})$", $email))
    {
    return false;
 
    }
    else return true;
    }
 
 
    $email = $_POST ["email"];
    echo '<form name="formularz" action="test2.php" method="post">
      <input type"text" name="email" value="$email" />
      <input type="submit" value="Zapisz" />
    </form>';
    if (walidacja_email($email))
    {
    echo '<font color="green">wprowadziłeś poprawny adres email</font>';
    $plik = fopen ("email.txt","a");
    fputs ($plik, "$email ").'';
    fclose ($plik);
    }
    else
    {
    echo '<font color="red"> wprowadzony adres email jest niepoprawny</font>';
    }
    print '</td>';
    print '</tr>';
    print '</table>';
    print '</form>';
    ?>
[PHP] pobierz, plaintext 

[klocu]

No to dziwne bo u mnie działa.

Ale do rzeczy, zawsze jak widzę taki sajgon to strasznie się złoszczę.
Uporządkuj to, mimo że to kod strukturalny. Wszystkie funkcje w jedno miejsce, reszta w drugie.

Co do kodu:
Sprawdzaj czy otrzymujesz jakiekolwiek dane od użytkownika - bo przepisujesz na ślepo dane z POST'a do zmiennej, potem te dane walidujesz. Ale po co jak nic w takiej zmiennej może nie być.
Po drugie czytamy dokumentację - eregi jest funkcją zdeprecjonowaną - używamy do tego preg_match. (zakładając, że mówimy o testach na najnowszej wersji parsera)

I w HTMLu znaczniki zamykamy od najpóźniej występującego, a nie jak nam się podoba.