Witam mam problem z formularzem kontaktowym w php

problem polega na tym że np. mimo iż nie wszystkie pola są wypełnione, lub email jest niepoprawny wiadomość jest wysyłana

Wiem że w kodzie jest namieszane, ale w php jestem zielony


kontakt.php:

[PHP] pobierz, plaintext 
//sprawdzenie czy załącznik pochodzi z formularza
if ($_POST["wyslij"]==1)
{
srand((double)microtime()*1000000);
$znacznik = md5(uniqid(rand()));
 
// dane o odbiorcy, nadawcy
$odbiorca="adres odbiorcy mailów";
$tytul= $_POST["temat"];
$nadawca_mail= $_POST["email"];
 
// treść listu
$tresclistu = $_POST["tresc"];
 
// definicja nagłówków
  $naglowki  = "From: $nadawca_mail do <$odbiorca>\n";
  $naglowki .= "Content-Type: multipart/mixed;\n";
  $naglowki .= "\tboundary=\"___$znacznik==\"";
 
// nagłówki listu
  $tresc="--___$znacznik==\n";
  $tresc .="Content-Type: text/plain; charset=\"UTF-8\"\n";
  $tresc .="\n$tresclistu\n";
 
// nagłówki i obsługa załącznika
  $plik_nazwa_tmp = $_FILES["plik1"]["tmp_name"];
  $plik_nazwa_oryginalna = $_FILES["plik1"]["name"];
  $plik_wielkosc = $_FILES["plik1"]["size"];
 
  $plik_nazwa_tmp2 = $_FILES["plik2"]["tmp_name"];
  $plik_nazwa_oryginalna2 = $_FILES["plik2"]["name"];
  $plik_wielkosc2 = $_FILES["plik2"]["size"];
 
  $plik_nazwa_tmp3 = $_FILES["plik3"]["tmp_name"];
  $plik_nazwa_oryginalna3 = $_FILES["plik3"]["name"];
  $plik_wielkosc3 = $_FILES["plik3"]["size"];
 
  $plik_nazwa_tmp4 = $_FILES["plik4"]["tmp_name"];
  $plik_nazwa_oryginalna4 = $_FILES["plik4"]["name"];
  $plik_wielkosc4 = $_FILES["plik4"]["size"];
 
if (is_uploaded_file($plik_nazwa_tmp))
  {
    $tresc .="--___$znacznik==\n";
    $tresc .="Content-Type: image/gif\n";
    $tresc .="Content-Disposition: attachment;\n";
    $tresc .=" filename=\"$plik_nazwa_oryginalna\"\n";
    $tresc .="Content-Transfer-Encoding: base64\n\n";
    $f = fopen($plik_nazwa_tmp,"rb");
    $dane = fread($f,filesize($plik_nazwa_tmp));
    fclose($f);
    $tresc .= chunk_split(base64_encode($dane));
  }
 
if (is_uploaded_file($plik_nazwa_tmp))
  {
    $tresc .="--___$znacznik==\n";
    $tresc .="Content-Type: image/gif\n";
    $tresc .="Content-Disposition: attachment;\n";
    $tresc .=" filename=\"$plik_nazwa_oryginalna2\"\n";
    $tresc .="Content-Transfer-Encoding: base64\n\n";
    $f = fopen($plik_nazwa_tmp2,"rb");
    $dane = fread($f,filesize($plik_nazwa_tmp2));
    fclose($f);
    $tresc .= chunk_split(base64_encode($dane));
  }
 
if (is_uploaded_file($plik_nazwa_tmp))
  {
    $tresc .="--___$znacznik==\n";
    $tresc .="Content-Type: image/gif\n";
    $tresc .="Content-Disposition: attachment;\n";
    $tresc .=" filename=\"$plik_nazwa_oryginalna3\"\n";
    $tresc .="Content-Transfer-Encoding: base64\n\n";
    $f = fopen($plik_nazwa_tmp3,"rb");
    $dane = fread($f,filesize($plik_nazwa_tmp3));
    fclose($f);
    $tresc .= chunk_split(base64_encode($dane));
  }
 
 if (is_uploaded_file($plik_nazwa_tmp))
  {
    $tresc .="--___$znacznik==\n";
    $tresc .="Content-Type: image/gif\n";
    $tresc .="Content-Disposition: attachment;\n";
    $tresc .=" filename=\"$plik_nazwa_oryginalna4\"\n";
    $tresc .="Content-Transfer-Encoding: base64\n\n";
    $f = fopen($plik_nazwa_tmp4,"rb");
    $dane = fread($f,filesize($plik_nazwa_tmp4));
    fclose($f);
    $tresc .= chunk_split(base64_encode($dane));
  }
 
// wysłanie listu
     if (mail($odbiorca,$tytul,$tresc,$naglowki))
   {
   echo "";
   }
   else
   {
   print "";
   }
}
if(!empty($tytul) && !empty($nadawca_mail) && !empty($tresclistu)) {
 
//--- początek funkcji weryfikującej adres e-mail ---
function checkMail($checkmail) {
  if(filter_var($checkmail, FILTER_VALIDATE_EMAIL)) {
    if(checkdnsrr(array_pop(explode("@",$checkmail)),"MX")){
        return true;
      }else{
        return false;
      }
  } else {
    return false;
  }
}
//--- koniec funkcji ---
 
if(checkMail($nadawca_mail)) {
  echo "<script type='text/javascript'>window.alert('Wiadomość została wysłana');</script>";
} else {
  echo "<script type='text/javascript'>window.alert('Podany Email jest nieprawidłowy!');</script>";
}
 
} else {
  //komunikat w przypadku nie powodzenia
  echo "<script type='text/javascript'>window.alert('Wypełnij wszystkie pola! (załączniki opcjonalne)');</script>";
}
[PHP] pobierz, plaintext 

kontakt.html

[HTML] pobierz, plaintext 
<FORM METHOD="post" ACTION="kontakt.php" ENCTYPE="multipart/form-data">
<input type="hidden" name="wyslij" value="1" />
<input type="hidden" name="MAX_FILE_SIZE" value="2000000" />
<input name="email" id="tematmail" placeholder="twój email">
<input name="temat" id="tematmail" placeholder="temat wiadomości"><br>
<br><textarea name="tresc" placeholder="treść wiadomości"></textarea><br>
<br><input type="file" name="plik1" id="file"><br>
<input type="file" name="plik2" id="file"><br>
<input type="file" name="plik3" id="file"><br>
<input type="file" name="plik4" id="file"><br>
<input type="submit" value="wyślij!" id="button"><input style="margin-left:3px;" type="reset" value="resetuj" id="button">
</form>
[HTML] pobierz, plaintext 

z góry dziękuje za pomoc

[PHP] pobierz, plaintext 
if(!empty($tytul) && !empty($nadawca_mail) && !empty($tresclistu)) {
 
//--- początek funkcji weryfikującej adres e-mail ---
function checkMail($checkmail) {
  if(filter_var($checkmail, FILTER_VALIDATE_EMAIL)) {
    if(checkdnsrr(array_pop(explode("@",$checkmail)),"MX")){
        return true;
      }else{
        return false;
      }
  } else {
    return false;
  }
}
//--- koniec funkcji ---
 
if(checkMail($nadawca_mail)) {
  echo "<script type='text/javascript'>window.alert('Wiadomość została wysłana');</script>";
} else {
  echo "<script type='text/javascript'>window.alert('Podany Email jest nieprawidłowy!');</script>";
}
 
} else {
  //komunikat w przypadku nie powodzenia
  echo "<script type='text/javascript'>window.alert('Wypełnij wszystkie pola! (załączniki opcjonalne)');</script>";
}
[PHP] pobierz, plaintext 

Tutaj masz ładny fragment sprawdzający, szkoda że sprawdza po fakcie wysyłki wiadomości.
Po co tyle razy sprawdzasz is_uploaded_file?

Jak już pisałem, w php jestem zielony. Ten kod raczej sklejałem z kilku różnych, a nie pisałem samodzielnie

Sprawdzam is_uploaded_file kilka razy bo jest kilka załączników. Z tym też mam wsumie problem...bo jeżeli załącze np 1 plik, to na poczcie pojawia się ten jeden plik i 3 kolejne o nazwie "noname"

A więc mogłbym prosić o przekształcenie tego kodu aby to działało?? I przy okazji podać jakiś sposób na te załączniki

(żeby np po załączeniu jednego pliku na poczcie pojawiał się 1 plik bez tych trzech innych plików z nazwą "noname", potem tak samo z dwoma plikami itd.)

Nie ma nic za darmo
Tutaj raczej uczymy robić samemu niż kopiować bezmyślnie.
Masz kilka załączników, ale w warunku sprawdzającym zapomniałeś sprawdzać kilku załączników, a zawsze sprawdzasz jeden i ten sam.
Kod wysyłający i uploadujący powinieneś wrzucić przed komunikatem Wiadomość została wysłana.

Dzięki wielkie!

wszystko pozmieniałem jak mówiłeś/aś i narazie działa

i rozumiem że tam gdzie są

[PHP] pobierz, plaintext 
if (is_uploaded_file($plik_nazwa_tmp))
[PHP] pobierz, plaintext 

trzeba pozmieniać na:

[PHP] pobierz, plaintext 
if (is_uploaded_file($plik_nazwa_tmp1))
if (is_uploaded_file($plik_nazwa_tmp2))
[PHP] pobierz, plaintext 

itd??

Dokładnie tylko bez 1 bo nie masz takiej zmiennej