Import załącznika z e-maila do bazy danych, prośba o sprawdzenie kodu |
Import załącznika z e-maila do bazy danych, prośba o sprawdzenie kodu |
26.01.2022, 16:43:54
Post
#1
|
|
Grupa: Zarejestrowani Postów: 83 Pomógł: 0 Dołączył: 26.07.2019 Ostrzeżenie: (0%) |
Co dziennie otrzymuje na skrzynkę e-mail wiadomość z załącznikiem, który do tej pory pobierałem i ręcznie importowałem do programu. Zrobiłem skrypt, który pobiera załącznik ze skrzynki a następnie wpisuje dane zawarte w załączniku do bazy danych. Kod ogólnie działa jednak proszę o informacje czy wszystko jest zrobione tak jak trzeba, trochę nie daje mi spokoju czy dobrze zrobiłem że dwa razy występuje
|
|
|
26.01.2022, 16:52:38
Post
#2
|
|
Grupa: Moderatorzy Postów: 36 446 Pomógł: 6292 Dołączył: 27.12.2004 |
Ten csv ma tylko jeden wiersz?
-------------------- "Myśl, myśl, myśl..." - Kubuś Puchatek || "Manual, manual, manual..." - Kubuś Programista "Szukaj, szukaj, szukaj..." - Kubuś Odkrywca || "Debuguj, debuguj, debuguj..." - Kubuś Developer |
|
|
26.01.2022, 17:50:18
Post
#3
|
|
Grupa: Zarejestrowani Postów: 83 Pomógł: 0 Dołączył: 26.07.2019 Ostrzeżenie: (0%) |
|
|
|
26.01.2022, 17:51:43
Post
#4
|
|
Grupa: Moderatorzy Postów: 36 446 Pomógł: 6292 Dołączył: 27.12.2004 |
Nie wnikam czy to txt czy csv. Pytam sie czy ma tylko jeden wiersz?
-------------------- "Myśl, myśl, myśl..." - Kubuś Puchatek || "Manual, manual, manual..." - Kubuś Programista "Szukaj, szukaj, szukaj..." - Kubuś Odkrywca || "Debuguj, debuguj, debuguj..." - Kubuś Developer |
|
|
26.01.2022, 17:54:35
Post
#5
|
|
Grupa: Zarejestrowani Postów: 83 Pomógł: 0 Dołączył: 26.07.2019 Ostrzeżenie: (0%) |
Plik czasem ma nawet 30 lini
|
|
|
26.01.2022, 17:56:07
Post
#6
|
|
Grupa: Moderatorzy Postów: 36 446 Pomógł: 6292 Dołączył: 27.12.2004 |
To czemu o to:
unlink($filename); header('Location: employees.php'); robisz po kazdej linii? Jesli plik ma 30 linii to ty to odpalasz 30 razy. plik juz dawno nie istnieje a naglowek powinno sie wyslac tylko raz. Przeciez to ma byc zrobione raz po zakonczeniu petli a nie co linia -------------------- "Myśl, myśl, myśl..." - Kubuś Puchatek || "Manual, manual, manual..." - Kubuś Programista "Szukaj, szukaj, szukaj..." - Kubuś Odkrywca || "Debuguj, debuguj, debuguj..." - Kubuś Developer |
|
|
26.01.2022, 18:01:17
Post
#7
|
|
Grupa: Zarejestrowani Postów: 83 Pomógł: 0 Dołączył: 26.07.2019 Ostrzeżenie: (0%) |
Faktycznie, wyrzuciłem to po za pętle. Teraz powinno być chyba ok.
Ten post edytował ufo1990 26.01.2022, 18:02:23 |
|
|
26.01.2022, 18:02:51
Post
#8
|
|
Grupa: Moderatorzy Postów: 36 446 Pomógł: 6292 Dołączył: 27.12.2004 |
TEraz zajrzyj do manuala i zobacz co zwraca fwrite a nastepnie zastanow sie czy ten kod
$file = fwrite($filename_save, $attachment['attachment']); if(isset($file)) A konkrennie IF, ma jakikowliek sens -------------------- "Myśl, myśl, myśl..." - Kubuś Puchatek || "Manual, manual, manual..." - Kubuś Programista "Szukaj, szukaj, szukaj..." - Kubuś Odkrywca || "Debuguj, debuguj, debuguj..." - Kubuś Developer |
|
|
26.01.2022, 18:10:54
Post
#9
|
|
Grupa: Zarejestrowani Postów: 83 Pomógł: 0 Dołączył: 26.07.2019 Ostrzeżenie: (0%) |
ok, kolejna rzecz uporządkowana.
|
|
|
26.01.2022, 18:12:53
Post
#10
|
|
Grupa: Moderatorzy Postów: 36 446 Pomógł: 6292 Dołączył: 27.12.2004 |
Nie do konca uporzadkowana. Jedyne co zrobiles to wywaliles bezsensowny IF z isset ale tam powinien byc IF, ino ze bez isset. Przeciez tam moze byc FALSE..
$file = fwrite($filename_save, $attachment['attachment']); if($file) { ..... -------------------- "Myśl, myśl, myśl..." - Kubuś Puchatek || "Manual, manual, manual..." - Kubuś Programista "Szukaj, szukaj, szukaj..." - Kubuś Odkrywca || "Debuguj, debuguj, debuguj..." - Kubuś Developer |
|
|
26.01.2022, 18:14:36
Post
#11
|
|
Grupa: Zarejestrowani Postów: 83 Pomógł: 0 Dołączył: 26.07.2019 Ostrzeżenie: (0%) |
ok, poprawione.
|
|
|
26.01.2022, 18:19:12
Post
#12
|
|
Grupa: Moderatorzy Postów: 36 446 Pomógł: 6292 Dołączył: 27.12.2004 |
fclose($filename_save);
To powinno byc po zapisie a nie hen po -------------------- "Myśl, myśl, myśl..." - Kubuś Puchatek || "Manual, manual, manual..." - Kubuś Programista "Szukaj, szukaj, szukaj..." - Kubuś Odkrywca || "Debuguj, debuguj, debuguj..." - Kubuś Developer |
|
|
26.01.2022, 18:28:03
Post
#13
|
|
Grupa: Zarejestrowani Postów: 83 Pomógł: 0 Dołączył: 26.07.2019 Ostrzeżenie: (0%) |
|
|
|
Wersja Lo-Fi | Aktualny czas: 19.04.2024 - 00:39 |