[PHP] Skrypt do zbierania maili nie działa prawidłowo., Skrypt z kursu php, ale chyba źle napisany Opcje

[kamjot]

Witam. Uczę się PHP od jakichś dwóch tygodni i utknąłem w miejscu.Moje środowisko do XAMPP i PHP 5.6.31

Korzystam z kursu na kursphp.com, jestem teraz na końcu jednego z rozdziałów ( https://kursphp.com/rozdzial-5/podsumowanie...tego-rozdzialu/ ) i utknąłem.

Skrypt ma "czytać" daną stronę lub blik i zbierać adres email dopasowany według wyrażenia regularnego, po czym zapisać zebrane maile do pliku a następnie wysłać je mailem.

Oryginalny skrypt wygląda tak:

[PHP] pobierz, plaintext 
<?php
 
function zdobadz_email($strona) //tutaj podajemy adres strony lub ścieżke pliku, wywołanie funkcji i podanie argumentu odbywa sie za pomocą require_once () z osobnego pliku
{
   // formuła prawidłowego adresu e-mail 
   $sprawdz = '/^[a-zA-Z0-9.\-_]+@[a-zA-Z0-9\-.]+\.[a-zA-Z]
   {2,4}$/';
 
   $plik = fopen($strona,'r'); // otwarcie pliku strony
   // utworzenie naszego pliku
   $moj_plik = fopen('tymczasowy_index.txt','a');
   flock($moj_plik, 2); // blokada pliku
 
   // przeszukujemy plik dopóki nie znajdziemy sie na końcu
   while(!feof($plik))
   {
      $linia = fgets($plik); // pobieramy jedną linię
 
      // sprawdzamy, czy znajduje się tam adres e-mail
      // jeśli tak, zapisujemy do naszego pliku
      if (ereg($sprawdz, $linia, $wynik)) 
         fputs($moj_plik, $wynik);
   }
   fclose($plik); // zamykamy plik strony
 
   // po zapisaniu danych, wskaźnik znajduje się na końcu pliku
   // musimy go przewinąć znów na początek funkcją rewind()
   rewind($moj_plik);
 
   // zawartość zapisanego pliku wczytujemy do tablicy adresów
   // $adresy, z wykorzystaniem funkcji file
   $adresy = file($moj_plik);
 
   // procedura wysyłania maila
   $adres = "przyklad@uzycia.pl";
   $tytul = "Adresy e-mail";
   $wiadomosc = "Znalezione adresy e-mail to: $adresy";
   mail($adres, $tytul, $wiadomosc);
 
   // odblokowanie pliku
   flock($moj_plik, 3);
 
   // zamknięcie
   fclose($moj_plik);
 
   // usunięcie, po wysłaniu e-mailem
   unlink($moj_plik);
}
 
?>
[PHP] pobierz, plaintext 

Skrypt jest dziwny chociażby przez użycie ereg w 21 linijce (przez co przy wywołaniu funkcji wyrzuca Warning: ereg(): REG_ERANGE) natomiast składnia wyrażenia z 6 linijki wygląda na preg_match i taką też zmianę zastosowałem.

Następne warningi jakie dostaje to:

Warning: file() expects parameter 1 to be a valid path, resource given in C:\xampp\htdocs\kursphp\roz-4-6\5\funkcja_szukaj_maila.php on line 72 //(w poście linijka 32)
Warning: unlink() expects parameter 1 to be a valid path, resource given in C:\xampp\htdocs\kursphp\roz-4-6\5\funkcja_szukaj_maila.php on line 87 //(w poście linijka 47)

Domyśliłem się, że lokalizacja pobrana ze zmiennej $moj_plik jest nieprawidłowa ze względu na obecność w niej funkcji fopen więc podałem samą ścieżkę i co prawda warningi zniknęły, ale teraz funkcja zwraca taki notice:
Notice: Array to string conversion in C:\xampp\htdocs\kursphp\roz-4-6\5\funkcja_szukaj_maila.php on line 77 (w poście linijka 37).

Co prawda skrypt wykonuje się do końca (kasuje plik za pomocą unlink() na samym końcu, ale po moich modyfikacjach) i wysyła maila ale w treści nie ma maili, które powinny być pobranie z podanego pliku.

Czy będzie ktoś miły i pomoże mi rozwikłać dlaczego ten skrypt nie działa poprawnie albo chociaż naprowadzić? Korzystałem z manuala php i czytałem na temat użytych w skrypcie funkcji i ich parametrów i w sumie nie mam się do czego doczepić, wszystkie podane parametry wyglądają w porządku.

Dziękuję serdecznie, pozdrawiam.

Ten post edytował kamjot 5.09.2017, 14:43:14

[nospor]

Polecam zmienic kurs bo pisal go jakis poczatkujacy... takie bledy... koles nawet nie testowal tego co publikuje :/

Tobie zas polecam manual i zagladniecie do funkcji, ktore rzucaja bledem.

[kamjot]

Polecam zmienic kurs bo pisal go jakis poczatkujacy... takie bledy... koles nawet nie testowal tego co publikuje :/

Tobie zas polecam manual i zagladniecie do funkcji, ktore rzucaja bledem.

Z tym kursem to się zgadzam, już nieraz musiałem poprawiać kod aby zaczął działać Myślałem, że tutaj też dam radę, ale poległem, straciłem pół dnia na próby i dalej jestem w tym samym miejscu.

Co do błędów ereg, to zmieniłem na preg_match i jak dla mnie jest dobrze. Jednak już zaczyna mi się rozjaśniać - ja dobrze rozumiem 3 parametr preg_match() zapisuje znaleziony wzór w formie tablicy do pliku $moj_plik, więc jak dobrze rozumiem w pliku "tymczasowy_index.txt" poiwnienem się spodziewać czegoś w rodzaju $wynik[0] = "mail@mail.com" itp. natomiast plik ten jest pusty (wyłączyłem unlink, żeby sprawdzić zawartość pliku).

Co do ostatniego błędu Array to string conversion to domyślam się, że powinienem przekonwertować tablicę do stringa np za pomocą explode albo wywołać każdy indeks tablicy ręcznie?

[nospor]

Warning: file() expects parameter 1 to be a valid path, resource given in

Mowie poraz kolejny: zajrzyj do manuala co oczekuje funkcja file() jako pierwszy parametr a co ty dajesz.

[kamjot]

Mowie poraz kolejny: zajrzyj do manuala co oczekuje funkcja file() jako pierwszy parametr a co ty dajesz.

Ok, gdy używam parametrów z oryginalnego kodu czyli

[PHP] pobierz, plaintext 
$adresy = file($moj_plik);
[PHP] pobierz, plaintext 

to dostaję warning file() expects parameter 1 to be a valid path

ale

gdy użyję według mnie (na podstawie manuala wynika, że wystarczy tylko ścieżka do pliku, flagi mi tutaj raczej nie będą potrzebne) poprawnie skonfigurowanej funkcji file czyli:

[PHP] pobierz, plaintext 
$adresy = file('tymczasowy_index.txt');
[PHP] pobierz, plaintext 

(plik 'tymczasowy_index.txt' znajduje się w tym samym katalogu co plik .php)

to warny znikają.

Ale problem jest jeszcze przed funkcją file() gdyż pętla while() powinna zapełnić plik 'tymczasowy_index.txt' a mimo to plik nie uzupełnia się danymi a wykonywanie funkcji kończy się komunikatem: Notice: Array to string conversion in C:\xampp\htdocs\kursphp\roz-4-6\5\funkcja_szukaj_maila.php on line 77

[viking]

Zacznij poprawiać błędy od początku a nie końca. Masz gdzieś tablicę wcześniej.