[PHP] Funkcja i jej wywolanie Opcje

[kewes]

Witam,
Stworzyłem prosty formularz zawierający pola Checkbox i Opcion multiple:

[HTML] pobierz, plaintext 
<p>Jakiej przeglądarki WWW używasz najcześciej?</p>
<p><input type="checkbox" name="przeg[]" value="Firefox"/> Firefox</p>
<p><input type="checkbox" name="przeg[]" value="Safari"/> Safari</p>
<p><input type="checkbox" name="przeg[]" value="Opera"/> Opera</p>
<p><input type="checkbox" name="przeg[]" value="Chrome"/> Chrome</p>
<p><input type="checkbox" name="przeg[]" value="Internet Explorer"/> Internet Explorer</p>
<p><input type="checkbox" name="przeg[]" value="Netscape Navigator"/> Netscape Navigator</p>
<p><input type="checkbox" name="przeg[]" value="Ghost Navigator"/> Ghost Navigator</p>
<p><input type="checkbox" name="przeg[]" value="inna"/> inna</p>
<p style="text-align:center">Jakie popularne czasopisma komputerowe czytasz? </p><p style="text-align:center">eśli chcesz zaznaczyć wiecej niż jedną odpowiedĽ naciśnij klawisz CTRL</p>
<p><select name="gazeta[]" size="8" multiple="multiple">
<option value="PC World Komputer"> PC World Koputer </option> 
<option value="PC Format"> PC Format </option> 
<option value="Xploit">  Xploit </option> 
<option value="NEXT">  NEXT </option> 
<option value="Komputer Świat Ekspert">  Komputer Świat Ekspert </option> 
<option value="Komputer Świat">  Komputer Świat </option> 
<option value="Chip">  CHIP </option> 
<option value="Inne">  inne </option> 
</select>
[HTML] pobierz, plaintext 

Przetwarzam sobie go skryptem php, jako że pola wielokrotnego wyboru to zrobiłem sobie prostą funkcję do wypisywania tych pól:

[PHP] pobierz, plaintext 
<?php
function prze()
{
for ($i=0; $i<count($_POST['przeg']);$i++) {
echo " ".$_POST['przeg'][$i];
};
 
}
function gazeta()
{
for ($i=0; $i<count($_POST['gazeta']);$i++) {
echo " ".$_POST['gazeta'][$i];
};
 
}
echo "Korzystasz z nastepujacych przegladarek: "; prze();
 
echo ' .<br />Czytasz nastepujace gazety: '; gazeta(); echo ' .';
?>
[PHP] pobierz, plaintext 

Wszystko działa, tj wypisuje to co ma wypisywać, lecz problem pojawia się gdy chce wysłać te dane mailem,
Napisałem Sobie prosty skrypt do wysyłania mailem:

[PHP] pobierz, plaintext 
<?php
$uwagi =$_POST['uwagi'];
$imie=$_POST['podajimie'];
$nazwisko=$_POST['podajnazwisko'];
$wiek=$_POST['wiek'];
$plec=$_POST['plec'];
$adresat=$_POST['podajmail'];
$ip=$_SERVER['REMOTE_ADDR'];
$odbiorca="baranmic@gmail.com";
$tytul = "$dataa$imie$nazwisko";
$headers  = 'MIME-Version: 1.0' . "r\n";
$headers .= 'Content-type: text/html; charset=iso-8859-1' . "r\n";
$headers .= "From: $adresat";
$uwagi=$_POST['uwagi'];
$tresc =' ';
$tresc .='<table border="1"><tr><td>Imie</td><td>'.$imie.'</td></tr>';
$tresc .='<tr><td>Nazwisko</td><td>'.$nazwisko.'</td></tr>';
$tresc .='<tr><td>Adres IP</td><td>'.$ip.'</td></tr>';
$tresc .='<tr><td>Plec</td><td>'.$plec.'</td></tr>';
$tresc .='<tr><td>Wiek</td><td>'.$wiek.'</td></tr>';
$tresc .='<tr><td>Adres E-mail</td><td>'.$adresat.'</td></tr>';
$tresc .='<tr><td>Uwagi</td><td>'.prze().'</td></tr>';
$tresc .='<tr><td>Gazety:</td><td>'.gazeta().'</td></tr>';
@mail("$odbiorca","$tytul","$tresc","$headers");
echo '<br /><br /><br /><br /><center>wiadomosc wyslano</center>';
?>
[PHP] pobierz, plaintext 

Jednak pola gdzie teoretycznie chciałem otrzymać z wywołania funkcji nie wysyłają sie wraz z resztą danych na maila, tylko po prostu wyświetlają.
Co zrobić aby to zmienić?

[marcio]

Poniewaz w funkcjach gazeta() i prze() masz echo() ktore wyrzuca dane na ekran nigdzie ich nie zapisuje i jesli dane chcesz przeslac za pomoca mail() te 2 funkcje musza cos zwracac za pomoca return.

Czyli:

[PHP] pobierz, plaintext 
<?php
function prze() {
 
 for ($i=0; $i<count($_POST['przeg']);$i++) return $_POST['przeg'][$i];
 
}
 
 
function gazeta() {
 
 for ($i = 0; $i < count($_POST['gazeta']); $i++) return $_POST['gazeta'][$i];
 
}
?>
[PHP] pobierz, plaintext 

I potem ustawiasz sobie:

[PHP] pobierz, plaintext 
<?php
$agent = prze();
$gazeta = gazeta();
$tresc ='';
$tresc .='<table border="1"><tr><td>Imie</td><td>'.$imie.'</td></tr>';
$tresc .='<tr><td>Nazwisko</td><td>'.$nazwisko.'</td></tr>';
$tresc .='<tr><td>Adres IP</td><td>'.$ip.'</td></tr>';
$tresc .='<tr><td>Plec</td><td>'.$plec.'</td></tr>';
$tresc .='<tr><td>Wiek</td><td>'.$wiek.'</td></tr>';
$tresc .='<tr><td>Adres E-mail</td><td>'.$adresat.'</td></tr>';
$tresc .='<tr><td>Uwagi</td><td>'.$agent.'</td></tr>';
$tresc .='<tr><td>Gazety:</td><td>'.$gazeta.'</td></tr>';
?>
[PHP] pobierz, plaintext 

P.S powinno dzialac jak nie ustaw $agent i $prze jako tablice czyli $zmienna[] ale powinno dzialac

Ten post edytował marcio 27.05.2009, 15:18:46

[kewes]

Poniewaz w funkcjach gazeta() i prze() masz echo() ktore wyrzuca dane na ekran nigdzie ich nie zapisuje i jesli dane chcesz przeslac za pomoca mail() te 2 funkcje musza cos zwracac za pomoca return.

Czyli:

[PHP] pobierz, plaintext 
<?php
function prze() {
 
 for ($i=0; $i<count($_POST['przeg']);$i++) return $_POST['przeg'][$i];
 
}
function gazeta() {
 
 for ($i = 0; $i < count($_POST['gazeta']); $i++) return $_POST['gazeta'][$i];
 
}
?>
[PHP] pobierz, plaintext 

I potem ustawiasz sobie:

[PHP] pobierz, plaintext 
<?php
$agent = prze();
$gazeta = gazeta();
$tresc ='';
$tresc .='<table border="1"><tr><td>Imie</td><td>'.$imie.'</td></tr>';
$tresc .='<tr><td>Nazwisko</td><td>'.$nazwisko.'</td></tr>';
$tresc .='<tr><td>Adres IP</td><td>'.$ip.'</td></tr>';
$tresc .='<tr><td>Plec</td><td>'.$plec.'</td></tr>';
$tresc .='<tr><td>Wiek</td><td>'.$wiek.'</td></tr>';
$tresc .='<tr><td>Adres E-mail</td><td>'.$adresat.'</td></tr>';
$tresc .='<tr><td>Uwagi</td><td>'.$agent.'</td></tr>';
$tresc .='<tr><td>Gazety:</td><td>'.$gazeta.'</td></tr>';
?>
[PHP] pobierz, plaintext 

P.S powinno dzialac jak nie ustaw $agent i $prze jako tablice czyli $zmienna[] ale powinno dzialac

No niestety działa, ale w mailu dostaje tylko ostatnią ze zaznaczonych opcji.
Mam zmienić w takim wypadku
$agent = prze();
$gazeta = gazeta();
na tablice czy też wywołanie w $tresc ?

[erix]

A nie możesz po prostu wkleić tego kodu do pętli, tylko tworzysz niepotrzebną funkcję...? return" title="Zobacz w manualu PHP" target="_manual przerywa jej działanie, to trudno, aby zwracało więcej niż jeden rekord...