Forum PHP.pl > [PHP]dodatkowy przecinek

Pomoc - Szukaj - Użytkownicy - Kalendarz

HTFhere

4.11.2013, 15:35:44

Cześć!
Mam 2 problemy z kodem, który powinien działać następująco-pobierać przez formularz imie i nazwisko, a następnie zapisywać je do odpowiedniego pliku. Z założenia imiona powinny być zapisywane w pliku tak : imie1,imie2,imie3 itd., analogicznie dla nazwisk.

[PHP] pobierz, plaintext 
<html>
	<head>
		<meta http-equiv="content-type" content="text/html; charset=utf-8">
	</head>
	<body>
		<font size="7" color="red"><b>Dodawanie osoby do bazy danych</b></font><br>
		<form>
			Podaj Imię:<input type="text" name="imie"><br>
			Podaj Nazwisko:<input type="text" name="nazwisko"><br>
			<input type="submit" value="Dodaj osobę"><br>
		</form>
		<?php
			@$imie = $_GET['imie'];
			@$nazwisko = $_GET['nazwisko'];
			$plik = 'imiona.txt';
			$zapis_imie = "$imie,";
			$uchwyt = fopen($plik, 'a');
			fputs($uchwyt, $zapis_imie);
			fclose($uchwyt);
			$plik2 = 'nazwiska.txt';
			$zapis_nazwisko = "$nazwisko,";
			$uchwyt2 = fopen($plik2, 'a');
			fputs($uchwyt2, $zapis_nazwisko);
			fclose($uchwyt2);
		?>
	<br><a href="menu.php">Powrót do menu</a>
 
 
 
 
	</body>
</html>
[PHP] pobierz, plaintext

Niestety z jakiegoś powodu, kod ten zawsze na poczatku pliku PRZED zapisaniem imienia/nazwiska daje przecinek, mam więc : ,imie1,imie2,imie3. Ktoś wie dlaczego dostaję ten dodatkowy przecinek na początku? I drugi problem, gdy odczytam tą bazę danych takim kodem:

[PHP] pobierz, plaintext 
<html>
	<head>
		<meta http-equiv="content-type" content="text/html; charset=utf-8">
	</head>
	<body>
	<?php
		$plik = 'imiona.txt';
		$uchwyt = fopen($plik, 'r');
		$zawartosc = fread($uchwyt, filesize($plik));
		fclose($uchwyt);
		$dane_imiona = explode(",", $zawartosc);
		//wczytalem wszystkie imiona
		$plik2 = 'nazwiska.txt';
		$uchwyt2 = fopen($plik2, 'r');
		$zawartosc2 = fread($uchwyt2, filesize($plik2));
		fclose($uchwyt2);
		$dane_nazwiska = explode(",", $zawartosc2);
		//wczytalem wszystkie nazwiska
		$ilosc_osob = count($dane_imiona);
		//sprawdzilem ile tych nazwisk jest
		for( $a=0; $a < ($ilosc_osob-1); $a++) {
		echo "$a. $dane_imiona[$a] $dane_nazwiska[$a]<br>";
		}
	?>
 
	<br><a href="menu.php">Powrót do menu</a>
	</body>
<html>
[PHP] pobierz, plaintext

A potem spróbuję dopisać nową osobę, dostaję "osobę ducha" w gratisie, tzn mam wtedy tak zapisane w plikach: ,imie1,imie2,imie3,,dopisaneimie, . Jak się pozbyć tych problemów? Ps. wszelkie uwagi dotyczące kodu, jego estetyczności albo wszystkiego innego co wam przyjdzie na myśl mile widziane

EDIT:Mała uwaga- zauważyłem, że ten przecinek dochodzi po prostu za każdym razem kiedy wchodzę albo odświeżam stronę z tym pierwszym kodem. może to jakoś pomoże ustalić przyczynę.

nospor

4.11.2013, 15:37:51

Odpalasz ten kod rowniez przed wyslaniem forma, wiec nic dziwnego ze pojawiaja ci sie puste przecinki.... ten kod ma byc odpalany dopiero po wyslaniu forma

HTFhere

4.11.2013, 15:41:50

To ma sens. Jak zapewne się domyślam, sprawienie, żeby kod odpalał się dopiero po wysłaniu formularza jest bardzo prostym zabiegiem? Móglbyś mnie nakierować na funkcję, której trzeba użyć? Jestem raczej początkujący

nospor

4.11.2013, 15:43:21

zrob sobie
print_r($_POST);
przed wyslaniem forma i po wyslaniu forma, a odpowiedz najdzie Cie sama.

mar1aczi

4.11.2013, 15:47:55

@nospor - Będzie miał zawsze pusto, korzysta z $_GETa

@autor: sprawdź tablicę $_GET

b4rt3kk

4.11.2013, 15:48:39

Albo sprawdź czy $_SERVER['REQUEST_METHOD'] jest równe POST.

nospor

4.11.2013, 15:49:04

Haha, racja, rob print_r($_GET);

Swoja drogą to na przyszlosc slij formy postem a nie getem

HTFhere

4.11.2013, 15:49:14

Nie wiem jak użyć tej funkcji, a:
"print_r() displays information about a variable in a way that's readable by humans."
niewiele mi mówi :/ wyświetla informacje o zmiennej sposobem który jest czytelny dla ludzi?

nospor

4.11.2013, 15:50:09

Nie miales patrzec co ona robi, tylko co zobaczysz na ekranie po jej uzyciu.... czytaj ze zrozumieniem, do tego nie trzeba miec wprawy z php.

I zrob
print_r($_GET);
i porownaj wyniki przed i po....

HTFhere

4.11.2013, 15:53:23

Ok, już rozumiem

mój pomysł jest taki- zrobię instrukcję warunkową, która dla pustego geta nie będzie nic robiła, co o tym sądzicie?

nospor

4.11.2013, 15:55:24

Od biedy moze byc

Lepiej sprawdzaj isset()em czy istnieje zadane pole z forma

mar1aczi

4.11.2013, 15:55:54

Cytat(HTFhere @ 4.11.2013, 16:53:23 )

Ok, już rozumiem

mój pomysł jest taki- zrobię instrukcję warunkową, która dla pustego geta nie będzie nic robiła, co o tym sądzicie?

Zrób, sprawdź, osądź sam ;]

Turson

4.11.2013, 15:55:57

Generalnie GET jest bezsensu jak nospor wpomniał.
Dla przykładu POST przycisk
<input type="submit" value="Dodaj osobę" name="wyslano">

potem w kodzie

[PHP] pobierz, plaintext 
if(isset($_POST["wyslano"]))
{
 //Twój kod
}
[PHP] pobierz, plaintext

//Edit: się ludzi do pomocy znalazło hehe

HTFhere

4.11.2013, 16:06:54

Dzięki wszystkim za pomoc! Poczytam o tym "POST" o którym piszecie

a jak narazie, żeby wszystko działało, zrobiłem tak:

[PHP] pobierz, plaintext 
<html>
	<head>
		<meta http-equiv="content-type" content="text/html; charset=utf-8">
	</head>
	<body>
		<font size="7" color="red"><b>Dodawanie osoby do bazy danych</b></font><br>
		<form>
			Podaj Imię:<input type="text" name="imie"><br>
			Podaj Nazwisko:<input type="text" name="nazwisko"><br>
			<input type="submit" value="Dodaj osobę"><br>
		</form>
		<?php
 
			@$imie = $_GET['imie'];
			@$nazwisko = $_GET['nazwisko'];
			if (isset($imie) == NULL) {
			echo "Dodaj jakąś osobę!";
			} else {
			$plik = 'imiona.txt';
			$zapis_imie = "$imie,";
			$uchwyt = fopen($plik, 'a');
			fputs($uchwyt, $zapis_imie);
			fclose($uchwyt);
			$plik2 = 'nazwiska.txt';
			$zapis_nazwisko = "$nazwisko,";
			$uchwyt2 = fopen($plik2, 'a');
			fputs($uchwyt2, $zapis_nazwisko);
			fclose($uchwyt2);
			}
		?>
	<br><a href="menu.php">Powrót do menu</a>
 
 
 
 
	</body>
</html>
[PHP] pobierz, plaintext

nospor

4.11.2013, 16:08:46

isset ma byc na $_GET a nie na zmiennej.
Do zmiennych masz wpisywac dopiero gdy są a nie bledy wygluszasz małpą @

HTFhere

4.11.2013, 16:10:42

w sensie wrzucić isset nad

[PHP] pobierz, plaintext 
@$imie = $_GET['imie'];
	@$nazwisko = $_GET['nazwisko'];
[PHP] pobierz, plaintext

i wpisać

[PHP] pobierz, plaintext 
if ( isset($_GET) == NULL)
[PHP] pobierz, plaintext

tak?

nospor

4.11.2013, 16:12:29

if ( isset($_GET['nazwisko']) )

A przypisanie zmiennych masz robic dopiero w tym IFie gdy jest spelniony

b4rt3kk

4.11.2013, 16:13:13

isset nie zwraca NULL tylko TRUE lub FALSE, także warunek nie ma szans by był spełniony.

nospor

4.11.2013, 16:14:21

Cytat

także warunek nie ma szans by był spełniony.

Oj ma, przeciez false == null jest prawdziwe. Co nie zmienia faktu, ze jego zapis jest bez sensu na dluższą mete dlatego podalem poprawny kod.

HTFhere

4.11.2013, 16:14:53

Heh jestem taki głupiutki w tym programowaniu, że aż mnie posądzają o nieczytanie

Dobra już zrobiłem jak poradziłeś. Dzięki za pomoc!

nospor

4.11.2013, 16:16:24

Jeszcze jedna rada: zawsze wyswietlaj wszystkie bledy, i nie uzywaj malp @, ktore te bledy wygluszają
http://nospor.pl/php-faq.html#faq-2

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.