[php] zmienne i fwrite Opcje

[kshyh]

Witam Was,
borykam się już dluższą chwilę nad zapisaniem zmiennych do pliku. Chodzi o to, ze w formularzu definiuję liczbę osób które chcemy zaprosić. Następnie pętlą "for" generuję tyle "input type=text" imie i nazwisko, ile wpisaliśmy we wcześniejszym formularzu. Wszystko gra, zmienne mają po kolei imie1=...&nazwisko1=...&imie2=...&nazwisko2=...& itd itd.
W skrypcie zapisu zrobiłem rownież pętle ale coś mi nie działa. Kombinowalem na wszelkie sposoby i nie moge.

Poniżej skrypt zapisu.

[PHP] pobierz, plaintext 
<?php
$file=fopen("baza.txt", "w");
		for( $m = 1; $m < $_SESSION['ilosc']+1; $m++)
		{
				$imie[$m] = $_GET['imie'][$m];
				fwrite($file, $imie[$m] );
				fwrite($file, ";" );
				$nazwisko[$m] = $_GET['nazwisko'][$m];
				fwrite($file, $nazwisko[$m] );
				fwrite($file, ";" );
				fwrite($file, "#" );
		 };
?>
[PHP] pobierz, plaintext 

Co robię źle?

[strife]

A co chcesz osiągnać?

Nie lepiej przesyłać takie dane metodą $_POST. A dodawać kolejne wpisy poprzez użycie tablicy, formularz wyglądał by w ten sposób ( oczywiście jak tam chcesz, generowany dynamicznie ).

[HTML] pobierz, plaintext 
<form method="post" action="index.php">
    <input type="text" name="imie[]" value="Romek" />
    <input type="text" name="imie[]" value="Stefan" />
    <input type="text" name="imie[]" value="Zbyszek" />
    <input type="text" name="nazwisko[]" value="Kowalski" />
    <input type="text" name="nazwisko[]" value="Nowak" />
    <input type="text" name="nazwisko[]" value="Uszatek" />
    <input type="submit" value="Dodaj" />
</form>
[HTML] pobierz, plaintext 

Od strony php, dodajesz te zmienne poprzez użycie pętli:

[PHP] pobierz, plaintext 
<?php
if ( isset( $_POST['imie'] ) ) {
	$string = NULL;
	foreach( $_POST['imie'] as $key => $value ) {
		$string .= $value . ';' . $_POST['nazwisko'][$key] . '#';
	}
	file_put_contents('baza.txt', $string );
}
?>
[PHP] pobierz, plaintext 

Pisane z palca. Kilka uwag:

• Sprawdzaj czy dane zostały wysłane
• Lepiej zapisywać tylko raz na końcu skryptu ( patrz przykład )
• Jeśli chcemy wysłać jakieś większe dane używamy metody $_POST

Pozdrawiam.

[Grzyw]

A jaki efekt przynosi Ci w działaniu ten skrypt?

*************************************

Zdecydowane tak do 2. uwagi końcowej Strife'a.
Wielokrotne wywołanie funkcji fwrite() zmusza do wielokrotnego zapisu do pliku. W przypadku, gdy wyślesz wszystko raz przez jedną zmienną, plik zostanie nadpisany tylko raz. To zwiększy prędkość wykonania skryptu i zmniejszy prawdopodobieństwo błędu (to przy jednym zapisie w pliku jest mniejszy, niż np. przy kilkuset zapisach).

Ten post edytował Grzyw 7.05.2007, 23:46:06

[kshyh]

Robię GET'em, bo chcę widzieć zmienne, później przerobię to na POST.

O to chodzi, że nie wiem jak duża ma być tablica. Definiowanie na sztywno ilości input'ów w tablicy nie jest w tym przypadku odpowiednim rozwiązaniem. Za każdym razem będzie to różna ilość.


W $_SESSION['ilosc'] zapisana jest jakaś liczba np. 4, która to we wcześniejszym formularzu "ilegosci.html" jest podawana.

[PHP] pobierz, plaintext 
<?
	printf('<form action=zapisz.php method=get>');
 
	for( $j = 1; $j < $_SESSION['ilosc']+1; $j++ )
	{
		echo "<table>";
		echo "<tr>";
		echo "<td width=100%>";
		printf("<p>   ");
		printf("</p>");
		echo "Imie: <input type=text name='imie$j' size='40' maxlength='40'> ";
		printf("</p>");
		echo "Nazwisko: <input type=text name='nazwisko$j' size='40' maxlength='40'> ";
		printf("</p>");
		echo "</td>";
		echo "</tr>";
		echo "</table>";
		printf("<p>   ");
		printf("</p>");
	};
	printf('<p align="right"><input name="dalej" type="submit" value="Dalej"></p>');
	printf('</form>');
?>
[PHP] pobierz, plaintext 

Skrypt ten wygeneruje 4 pola imie i 4 pola nazwisko. Powstaną zmienne $imie1, $imie2, ..., $imie4 oraz $nazwisko1, ..., $nazwisko4. Zmienne są przesyłane bo widzę je w pasku adresu.

Skrypt zapisz.php ma te zmienne zapisać do pliku, i tutaj tkwi problem. Stąd wzięła się pętla którą chciałem odpowiednie zmienne przechwycić i zapisać.

*Grzyw
efekt zapisu jest następujący

;;#;;#;;#;;#

[strife]

Ja bym to rozwiązał inaczej, pierwsza strona na której wybierasz ilość gości, może być trzymana w sesji. Następnie przechodząc do następnej strony, na podstawie wartości zapisanej w sesji, generowane są formularze, i tutaj nie rozumiem, czemu nie chcesz użyć tablic, po to one zostały przecież stworzone. Przykład poglądowy:

[PHP] pobierz, plaintext 
<?php
$intQuests = 12; // moze byc wartosc sesji
 
$form = NULL;
for ( $i = 0; $i < $intQuests; $i++ ) {
	$form .= '<tr><td>';
	$form .= '<input type="text" name="imie[]" />';
	$form .= '</td>';
	$form .= '<td>';
	$form .= '<input type="text" name="nazwisko[]" />';
	$form .= '</td></tr>';	
}
 
// wyswietlanie dynamicznie generowanego formularza
 
echo '<form method="post">';
echo '<table><tr><th>Imie</th><th>Nazwisko</th></tr>' . $form . '</table>';
echo '<input type="submit" />';
echo '</form>';
echo '<pre>';
var_export( $_POST );
echo '</pre>';
?>
[PHP] pobierz, plaintext 

PS. Spróbuj zamiast

[PHP] pobierz, plaintext 
<?php
$_GET['imie'][$m]
?>
[PHP] pobierz, plaintext 

dać:

[PHP] pobierz, plaintext 
<?php
$_GET['imie' . $m]
?>
[PHP] pobierz, plaintext 

[kshyh]

dzięki strife,

[PHP] pobierz, plaintext 
<?php
$_GET['imie' . $m]
?>
[PHP] pobierz, plaintext 

zadziałał.

ps.To nie jest tak, że nie chcę używać tablic. Tylko wpadłem na takie rozwiązanie i dla mnie, dla beginner'a jest ono łatwiejsze. Zaraz przetestuje rozwiązanie z tablicami. Dzieki jeszcze raz.