Forum PHP.pl

wielkie dzięki za sugestię
nie bardzo wiem o czym piszesz ale wiem o czym doczytac

niżej mój I dziąłający skrypt coś na wzór subskrybcji (nie licząc uploadu plików jpg i wyświetlania ich w 4 kolumnach co działa w 1/2)

wszystko w nim działa ale wiem że można to napisac lepiej jeżeli ktoś ma chwilke aby pomoc mi w dążeniu do doskonałości to będę wdzięczny.
jeżeli poprawki wymagają jakiejś "zaawansowanej wiedzy" to bardzo proszę o szczegóły bo jeszcze wiele mi zostało do nauki

kod "checkmail.php" znalazłem gdzieś w necie mam zamiar napisać coś w tym stylu samemu ( wydaje sie dość proste) jak juz uporam sie z ewentualnymi poprawkami w "getmail.php"

baza -> id (autoincrement) | mail (unique) | status (1 lub 0)

form.htm

[PHP] pobierz, plaintext 
<?php
 
......
<form action=&#092;"getmail.php\" method=\"POST\">
		mail: <input type=&#092;"text\" name=\"mail\" maxlength=\"50\"><br><br>
		<input type=&#092;"submit\" name=\"Submit\" value=\"Wyślij\">
</form>
.........
 
?>
[PHP] pobierz, plaintext 

getmail.php

[PHP] pobierz, plaintext 
<?php
include_once(&#092;"./checkemail.php\");
 
 if(!$_POST['mail'])
 { echo &#092;"podaj mail\"  ;
 
 }
  else
  {
	  if (!ismail($_POST['mail']))
	  {
 
		  echo &#092;"to nie jest poprawny mail\";
	  }
 else
 {
 
mysql_connect( &#092;"localhost\", \"root\", \"\" );
mysql_select_db( &#092;"mydb\");
$res = mysql_query( 'SELECT * FROM maillista' );
$mail = $_POST['mail'];
$val = mysql_fetch_array($res);
 
 
mysql_query (&#092;"INSERT INTO maillista (mail) VALUES ('$mail')\");
 
if (mysql_errno()==0)
	{
		echo &#092;"$mail <br><br>mail został dodany do bazy\";
	}
	 else
	 {
		$status=$val['status'];
		if($status==1)
		{
			mysql_query (&#092;"UPDATE maillista SET status='0' WHERE mail='$mail'\");
			echo &#092;"mail usunięty z listy mailingowej\";
		}
 
		else
		  {
			mysql_query (&#092;"UPDATE maillista SET status='1' WHERE mail='$mail'\");
			echo &#092;"mail dodany do listy mailingowej\";
		  }
	 }
 mysql_close();
 }
  }
 
?>
[PHP] pobierz, plaintext 

checkmail.php

[PHP] pobierz, plaintext 
<?php
 
function ismail($email)
{
 $pos = strpos($email, '@');
 if ($pos === false){
  return false;
 }
 else
 {
  $pos = strpos($email, '.', $pos);
  if ($pos === false){
   return false;
  }
  else
  {
	 return true;
   }
		}
 }
 
?>
[PHP] pobierz, plaintext