[PHP]Undefined index Opcje

[Dukov]

Hej, dzisiaj mam problem z czymś takim

[PHP] pobierz, plaintext 
if ($_POST['name'] != null)
[PHP] pobierz, plaintext 

wywala błąd undefined index
A to pole formularza

[HTML] pobierz, plaintext 
<input type="text" name="name">
[HTML] pobierz, plaintext 

Jest ktoś w stanie to wytłumaczyć? Chyba się nie pomyliłem i do formularza odwołuje się poprzez index o nazwie atrybutu name, a nie id?

[trueblue]

A jaką metodą wysyłasz formularz?

[Dukov]

Teraz mam kolejny problem też z indexami

[PHP] pobierz, plaintext 
if($_FILES['filename']['error']!=4){
			$avatar = fileReceive();
		}
[PHP] pobierz, plaintext 

Funkcja

[PHP] pobierz, plaintext 
function fileReceive(){
		$max_size = 1024*1024;
		if (is_uploaded_file($_FILES['filename']['tmp_name'])) {
    		if ($_FILES['filename']['size'] > $max_size) {
        		echo 'Błąd! Plik jest za duży!';
    		} 
    		else {
        		echo 'Odebrano plik. Początkowa nazwa: '.$_FILES['filename']['name'];
        		echo '<br/>';
 
        		if (isset($_FILES['filename']['type'])) {
            		echo 'Typ: '.$_FILES['filename']['type'].'<br/>';
        		}
 
       	 		$strExt = substr( $_FILES['filename']['name'], strrpos( $_FILES['filename']['name'], '.' ) + 1 );
       	 		echo "Rozszerzenie: ". $strExt;
       	 		$len = 20;
       	 		$randstring =  substr(sha1(rand(1,10000)),0,$len).".jpg";
       	 		echo $randstring;
       	 		echo $_FILES['filename']['name'];
 
        		if ($strExt == 'jpg' || 'JPG' || 'jpeg' || 'JPEG'){
        			move_uploaded_file($_FILES['filename']['tmp_name'],
                	$_SERVER['DOCUMENT_ROOT'].'../avatars/'.$_FILES['filename']['name']);
 
        		}	
    		else{ 
    		echo "niy!!";}
    		}
		} 
		else {
   		echo 'Błąd przy przesyłaniu danych!';
		}
		rename('avatars/'.$_FILES['filename']['name'], 'avatars/'.$randstring);
		return $randstring;
	}
[PHP] pobierz, plaintext 

[HTML] pobierz, plaintext 
<form name="regform" action="scripts/registry.php" method="post" enctype="multipart/form-data>
		<input type="text" name="login">
		<input type="password" name="pass">
		<input type="text" name="mail">
		<input type="text" name='name'>
		<input type="text" name="city">
		<textarea name="desc"></textarea>
		<input type="hidden" name="MAX_FILE_SIZE" value="512000" /><input type="file" name="filename">
		<input type="submit">
	</form>
[HTML] pobierz, plaintext 

Co ciekawe poszedłem tutaj po najmniejszej lini oporu, przerobiłem pod swoje potrzeby skrypt z neta, stosowałem go też na innych stronach i działał - tutaj nie chce działać. Te echa są tylko dopóki zacznie działać, żeby nie było, że tak zostanie, bo docelowo będzie Location: header i komunikat.

Ten post edytował Dukov 25.07.2017, 11:28:52

[nospor]

No ale jesli ten kod odpalasz rowniez przed wyslaniem formularza to rowniez bedziesz mial ten blad. Moze pokaz caly kod, wlacznie z formularzem

[Pyton_000]

nawet jak wyślesz puste pole to nie będzie null tylko pusty string.

Sprawdza się przez:

[PHP] pobierz, plaintext 
if(!empty($_POST['name'])) {...}
[PHP] pobierz, plaintext 

[Dukov]

@Pyton_100

POSTy działają, w skrypcie logowania też mam null, najpierw było jako isset, jak mnie pamięć nie myli i wywaliło błąd i w kodzie błędy zaproponowąło null i działa, ten skrypt też działa, problem mam z plikiem, a kodu się wstydzę jeszcze, dopóki go nie dopracuje Ale formularz wysyłam DO tego skryptu metodą POST, więc formularz jest wysłany do skryptu i skrypt zaczyna działać, więc kod i tak się odpala PO wysłaniu formularza.

Ale proszę

[PHP] pobierz, plaintext 
<?php
	$avatar;
	if ($_POST['name'] != null){
		$userName = $_POST['name'];	
	}
	if ($_POST['city'] != null ) {
		$userCity = $_POST['city'];
	}
	if ($_POST['desc'] != null){
		$userDesc = $_POST['desc'];
	}
	$host = 'localhost';
	$baza = 'test';
	$uzytkownik = 'adam';
	$haslo = 'aaaaa5';
	if ($_POST['login'] != null || $_POST['pass'] != null || $_POST['mail'] != null) {
		$userLogin = $_POST['login'];
		$userPass = $_POST['pass'];	
		$userMail = $_POST['mail'];
		$len = 20;
		$userID = substr(sha1(rand(1,10000)),0,$len);
		echo $userID;
		if($_FILES['filename']['error']!=4){
			$avatar = fileReceive();
		}
		reg();
 
	}
	else
	{
		header("Location: ../reg.php?reg=emptyf");
	}
	function reg(){
		global $userID, $userMail, $userPass, $userLogin, $userDesc, $userCity, $userName, $avatar;
		try {
 
 
 
    		$conn = new PDO("mysql:host=$host;dbname=$baza", $uzytkownik, $haslo);
			$conn->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
			$stmt = $conn->prepare("SELECT Nazwa FROM Users WHERE Nazwa = '$userLogin'"); 
			$stmt->execute();
			$row = $stmt->fetch();
			if($row[0] <> 1){
			$stmt = $conn->prepare("INSERT INTO 'Users' ('ID', 'Nazwa','Haslo', 'mail','name','city', 'image', 'desc', 'activate', 'admin') VALUES ('$userID','$userLogin', '$userPass', '$userMail', '$userName', '$userCity, $avatar', '$userDesc','false','false'"); 
			$stmt->execute();
 
			sendMail();
 
			}
 
			else{
				header("Location: ../reg.php?reg=userexists");
			}
   		}
   		catch(PDOException $e)
    	{
   	 		return 1; #brak polaczenia
    	}
	}
 
 
 
 
	function fileReceive(){
		$max_size = 1024*1024;
		if (is_uploaded_file($_FILES['filename']['tmp_name'])) {
    		if ($_FILES['filename']['size'] > $max_size) {
        		echo 'Błąd! Plik jest za duży!';
    		} 
    		else {
        		echo 'Odebrano plik. Początkowa nazwa: '.$_FILES['filename']['name'];
        		echo '<br/>';
 
        		if (isset($_FILES['filename']['type'])) {
            		echo 'Typ: '.$_FILES['filename']['type'].'<br/>';
        		}
 
       	 		$strExt = substr( $_FILES['filename']['name'], strrpos( $_FILES['filename']['name'], '.' ) + 1 );
       	 		echo "Rozszerzenie: ". $strExt;
       	 		$len = 20;
       	 		$randstring =  substr(sha1(rand(1,10000)),0,$len).".jpg";
       	 		echo $randstring;
       	 		echo $_FILES['filename']['name'];
 
        		if ($strExt == 'jpg' || 'JPG' || 'jpeg' || 'JPEG'){
        			move_uploaded_file($_FILES['filename']['tmp_name'],
                	$_SERVER['DOCUMENT_ROOT'].'../avatars/'.$_FILES['filename']['name']);
 
        		}	
    		else{ 
    		echo "niy!!";}
    		}
		} 
		else {
   		echo 'Błąd przy przesyłaniu danych!';
		}
		rename('avatars/'.$_FILES['filename']['name'], 'avatars/'.$randstring);
		return $randstring;
	}
 
function sendMail(){
	ob_start();
	$addressee = 'ajjambor912@gmail.com'; 	
	@$email = $_POST['email'];
 
	@$content = 'Name '.$_POST['username'].'  Phone: '.$_POST['userphone'].' Content: '.$_POST['message'];
 
	$header = 	"From: ".$_POST['email']." \nContent-Type:".
			' text/plain;charset="iso-8859-2"'.
			"\nContent-Transfer-Encoding: 8bit";
	if (mail($addressee, 'Message from NAhandyman: ', $content , $header))
 
			header('Location:index.php?send=true');
	else 
			header('Location:index.php?send=false');
}
 
?>	
[PHP] pobierz, plaintext 

Z góry zaznacze, że funkcja za wysyłanie maila, jest niegotowa, tylko skopiowana z innego skryptu, a wrzuciłem ją tam, żebym miał pogląd jak to mniej więcej było ostatecznie zrobione, tak jak nie są hasowane hasła itd, jak zacznie to działać, to będe dopracowywał powoli ten skrypt. Oczywiście większość nie działa, to szkielet, pisałem to wczoraj po nocy i chciałem sobie naszkicować, teraz go sukcesywnie poprawiam.

Ten post edytował Dukov 25.07.2017, 11:59:29

[nospor]

enctype="multipart/form-data>

Kazde rozpoczete " trzeba tez zamknac "

[Dukov]

Zamknąłem, a efekt ten sam, póki co wykomentuje tę funkcje i poprawiam dalej, zajrzę co jakiś czas, jak ktoś znalazł rozwiązanie, najwyżej jak będę miał wszystko inne "pochytane" to wyrzucę ten warunek i funkcje i napiszę inaczej od nowa.

[nospor]

To moze napisz co ci nie dziala, bo jak poki co nie widze. Widzialem ze nie dzialal ci pierwszy warunek, ktory rzekomo poprawiles. Potem napisales ze nie dziala ci co innego ale co to juz nie napisales, tylko ogolnikowo cos o indeksie. Konkrety prosze

[Dukov]

Notice: Undefined index: filename in D:\wamp64\www\scripts\registry.php on line 23

[PHP] pobierz, plaintext 
if($_FILES['filename']['error']!=4){
			$avatar = fileReceive();
		}
[PHP] pobierz, plaintext 

Linia 23 to ten warunek.

[HTML] pobierz, plaintext 
	<input type="hidden" name="MAX_FILE_SIZE" value="512000" /><input type="file" name="filename">
[HTML] pobierz, plaintext 

A ta linia odpowiada za odbieranie plików, pełen kod jest wyżej

[nospor]

Pokaz jeszcze raz poprawiony formularz, co by miec pewnosc ze poprawiles to poprawnie

[Pyton_000]

Nie sprawdzasz czy !empty($_FILES) a powinieneś. błąd mówi jasno że nie ma indeksu filename ergo nie wysyłasz pliku.

[Dukov]

Proszę uprzejmie

[HTML] pobierz, plaintext 
<form name="regform" action="scripts/registry.php" method="post" enctype="multipart/form-data">
		<input type="text" name="login">
		<input type="password" name="pass">
		<input type="text" name="mail">
		<input type="text" name='name'>
		<input type="text" name="city">
		<textarea name="desc"></textarea>
		<input type="hidden" name="MAX_FILE_SIZE" value="512000" /><input type="file" name="filename">
		<input type="submit">
	</form>
[HTML] pobierz, plaintext 

[nospor]

@Pyton jakby nie wyslal pliku to nadal by byl w $_FILES ale oznaczony jako pusty. No chyba, ze juz skleroze mam na starosc

[Dukov]

A no właśnie, ten warunek odpowiada za sprawdzenie pliku, 4 to plik niewysłany 3 albo 2 to niepełny plik bądź uszkodzony, a 1 albo 0 to plik wysłany poprawnie, zresztą docelowo będzie na to 0 albo 1 wywołana funkcja, a pozostałe będzie header location powrót do rejestracji i komunikat zależny od zmiennej z GET albo return jeszcze pomyślę

EDIT Pyton ma racje i poleciał 'pomógł' , zmieniłem ten warunek na if(!empty($_FILES)){ i działa dokładnie tak jak chcę teraz walczę z resztą błędów

Ten post edytował Dukov 25.07.2017, 12:48:17

[Pyton_000]

UPLOAD_ERR_NO_FILE mówi o tym że plik został załączony do formularza ale z jakiegoś powodu nie dotarł do PHP a dokładniej jeśli tmp_name jest pusty czyli np. plik nie móg się zapisać na dysk do tmp.

Ten post edytował Pyton_000 25.07.2017, 13:22:13

[Dukov]

Powiem tak, zmieniłem na echo, już zacząłem głupieć od tego, bo znowu były problemy, ogólnie warunek nie działa, nieistotne czy plik jest załączony, czy nie warunek spełniony możesz mi powiedzieć pokrótce jak mam napisać ten warunek, żeby sprawdził, czy jest załączony plik, czy nie?

Teraz sobie musze jeszcze opis na forum zmienić.

[Pyton_000]

- najpierw sprawdzasz czy $_FILES nie jest pusty
- potem sprawdzasz czy $_FILES['filename']['error'] nie ma błedów
- dalej lecisz normalnie

[Dukov]

Tak, ale za każdym razem (nawet jak jest pusty) to zwraca, że nie jest pusty

[PHP] pobierz, plaintext 
if(!empty($_FILES['filename'])){
			echo "pusty";
		}
[PHP] pobierz, plaintext 

Obojętnie czy jest czy nie, warunek jest spełniony, tutaj chcę sprawdzić TYLKO czy jest pusty, bo czy są błędy mam w funkcji, którą chce wywołać zamiast echo "pusty";

EDIT bez indeksu efekt ten sam, sprawdzałem.

Ten post edytował Dukov 25.07.2017, 13:54:49

[Pyton_000]

Zrób sobie `var_dump($_FILES);` i będziesz wiedział co tam jest a czego nie ma przy wysłaniu pustego formularza i z plikiem/