Forum PHP.pl > [PHP][HTML]prosze o pomoc zmodyfikowac kod formularza

Pomoc - Szukaj - Użytkownicy - Kalendarz

Pełna wersja: [PHP][HTML]prosze o pomoc zmodyfikowac kod formularza

tomsson

25.03.2009, 11:52:27

prosze o pomoc zmodyfikowac kod formularza tak ,aby nie było pola "do" zeby ta wartość była juz zdefiniowana i nie ukazywała się w tabeli.
Zamiast pola "treść" zeby było "twoj email"

modyfikacja dotyczy poniższego kodu ,ktory znalazlem i nie za bardzo wiem jak przerobić

Cytat

<HTML>
<HEAD>
<TITLE>Wysyłanie poczty</TITLE>
</HEAD>
<BODY>
<FORM METHOD="post" ACTION="sendmail.php">
<TABLE border="1">
<TR>
<TD COLSPAN="2" ALIGN="center" BORDERCOLOR="#FFFFFF">
<H2>Wysyłanie poczty</H2>
</TD>
</TR>

<TR>
<TD BORDERCOLOR="#FFFFFF">Do</TD>
<TD BORDERCOLOR="#FFFFFF">
<INPUT TYPE="text" NAME="do" SIZE="20">
</TD>
</TR>

<TR>
<TD BORDERCOLOR="#FFFFFF">Temat</TD>
<TD BORDERCOLOR="#FFFFFF">
<INPUT TYPE="text" NAME="temat" SIZE="20">
</TD>
</TR>

<TR>
<TD BORDERCOLOR="#FFFFFF">Treść</TD>
<TD BORDERCOLOR="#FFFFFF">
<TEXTAREA NAME="contents" ROWS="5" COLS="40">
</TEXTAREA>
</TD>
</TR>

<TR>
<TD COLSPAN="2" ALIGN="CENTER" BORDERCOLOR="#FFFFFF">
<INPUT TYPE="submit" VALUE="Wyślij">
</TD>
</TR>
</TABLE>

</FORM>
</BODY>
</HTML>

w pliku sendmail.php mam

Cytat

<?PHP
function sendmail($do, $temat, $contents)
{
echo("$do $temat $contents");
if($do == "" || $temat == "" || $contents == ""){
print("Proszę wypełnić wszystkie pola formularza");
}
else if (mail($do, $temat, $contents)){
print("List został wysłany na adres $do");
}
else{
print("Nie udało się wysłać listu na adres $do");
}
}
if(isSet($_POST['do']) && isSet($_POST['temat']) && isSet($_POST['contents'])){
sendmail($_POST["do"], $_POST["temat"], $_POST["contents"]);
}
else{
echo("Niekompletne dane!");
}
?>

pomoze ktoś ?

rozny

25.03.2009, 14:24:19

Generalnie, to kiepski ten kod. Rozumiem, że w formularzu chcesz mieć pola "email gościa" i "temat". Ja bym to zrobił w ten sposób:

[HTML] pobierz, plaintext 
\\ formularz
 
<form action="sendmail.php" method="post">
Adres email: <input type="text" name="email">
Temat: <input type="text" name="temat">
<input type="hidden" name="check" value="ok">
<input type="submit" value="wyślij">
</form>
[HTML] pobierz, plaintext

A w pliku sendmail.php:

[PHP] pobierz, plaintext 
<?php
 
if($_SERVER['REQUEST_METHOD'] == 'POST') { //sprawdzasz, czy dane zostaly wyslane formularzem
 
if (isset($_POST['check'] && $_POST['check'] == ok) { //sprawdzasz, czy dane zostaly wyslane formularzem
 
$email = trim($_POST['email']); //usuwasz spacje na poczatku i na koncu wpisu (unikasz pustego wpisu)
$temat = trim($_POST['temat']); //usuwasz spacje na poczatku i na koncu wpisu (unikasz pustego wpisu)
$do = 'twój@email.pl'; //ustawiasz adres email, na ktory maja byc wysylane wiadomosci
 
function sendmail($do, $email, $temat) {
 
     if (empty($email) || empty($temat)) { //sprawdzasz, czy pola zostaly wypelnione
        echo 'Proszę wypełnić poprawnie formularz';
     }
 
     elseif (mail("$do", "$temat", "$email") { //wysylasz meila
        echo 'Wiadomość wysłana na adres'.$email;
     }
 
     else {
        echo 'Nie udało się wysłać wiadomości';
    }
}
}
}
else {
   die ('Dane nie pochodzą z formularza');
   exit();
}
?>
[PHP] pobierz, plaintext

Myślę, że to bezpieczniejsze i bardziej przejrzyste rozwiązanie.

To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.