Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

 
Reply to this topicStart new topic
> [PHP] Czy można poprawić bardziej ten kod.
Giluś
post
Post #1





Grupa: Zarejestrowani
Postów: 377
Pomógł: 9
Dołączył: 2.11.2010

Ostrzeżenie: (0%)
-----


Cześć chciałbym się zapytać czy ten kod który zaraz pokażę można jeszcze jakoś zoptymalizować aby było mniej linijek lub szybszy.

  1. $wynik = str_replace("{imie}", $_POST['imie'], $wiadomoscmysql);
  2. $wynik1 = str_replace("{nazwisko}", $_POST['imie'], $wynik);
  3. $wynik2 = str_replace("{login}", $_POST['login'], $wynik1);
  4. $wynik3 = str_replace("{email}", $_POST['email'], $wynik2);
  5. $wynik4 = str_replace("{telefon}", $_POST['telefon'], $wynik3);
  6. $wynik5 = str_replace("{miejscowosc}", $_POST['miejscowosc'], $wynik4);
  7. $wynik6 = str_replace("{ulica}", $_POST['ulica'], $wynik5);
  8. $wynik7 = str_replace("{dom}", $_POST['dom'], $wynik6);
  9. $wynik8 = str_replace("{wojewodztwo}", $_POST['wojewodztwo'], $wynik7);
  10. $wynik9 = str_replace("{kodpocztowy}", $_POST['kodpocztowy'], $wynik8);
  11. $wynik10 = str_replace("{gg}", $_POST['gg'], $wynik9);
  12. $wynik11 = str_replace("{pesel}", $_POST['pesel'], $wynik10);
  13. $wynik12 = str_replace("{nip}", $_POST['nip'], $wynik11);


Całość działa na takiej zasadzie:
W Bazie MySQL zapisuję treść wysyłanego email (tutaj mam tylko kod do zmiany tekstu) i gdy dałem w MySQL np.: $_POST['imie'] to w email wysyłało: $_POST['imie'], wiec zrobiłem takie coś, że w mysql zapisuję np.: jako {imie} i później mi to zmienia na $_POST['imie'].
Czy można to zmodyfikować jeszcze bardziej ?
Go to the top of the page
+Quote Post
Fifi209
post
Post #2





Grupa: Zarejestrowani
Postów: 4 655
Pomógł: 556
Dołączył: 17.03.2009
Skąd: Katowice

Ostrzeżenie: (0%)
-----


Użyć funkcji tylko raz, a jako argument przekaż tablice. Więcej w manualu str_replace
Go to the top of the page
+Quote Post
Swirek
post
Post #3





Grupa: Zarejestrowani
Postów: 221
Pomógł: 20
Dołączył: 4.05.2006
Skąd: Polska

Ostrzeżenie: (0%)
-----


i tak troszkę nie w temacie, ale rzucił mi się w oczy błąd. Przypisujesz do nazwiska również zmienną imię.

pozdrawiam (IMG:style_emoticons/default/smile.gif)
Go to the top of the page
+Quote Post
Giluś
post
Post #4





Grupa: Zarejestrowani
Postów: 377
Pomógł: 9
Dołączył: 2.11.2010

Ostrzeżenie: (0%)
-----


Cytat(Fifi209 @ 6.08.2011, 11:30:11 ) *
Użyć funkcji tylko raz, a jako argument przekaż tablice. Więcej w manualu str_replace


Dzięki za podpowiedz coś takiego ma być:
  1. $search = array ("{imie}" , "{nazwisko}" , "{login}" , "{email}" , "{telefon}" , "{miejscowosc}" , "{ulica}" , "{dom}" , "{wojewodztwo}" , "{kodpocztowy}" , "{gg}" , "{pesel}" , "{nip}");
  2. $replace = array ($_POST['imie'] , $_POST['nazwisko'] , $_POST['login'] , $_POST['email'] , $_POST['telefon'] , $_POST['miejscowosc'] , $_POST['ulica'] , $_POST['dom'] , $_POST['wojewodztwo'] , $_POST['kodpocztowy'] , $_POST['gg'] , $_POST['pesel'] , $_POST['nip']);
  3. $wynik = str_replace($search, $replace, $wiadomoscmysql);



Cytat(Swirek @ 6.08.2011, 12:17:21 ) *
i tak troszkę nie w temacie, ale rzucił mi się w oczy błąd. Przypisujesz do nazwiska również zmienną imię.

pozdrawiam (IMG:style_emoticons/default/smile.gif)



Dzięki (IMG:style_emoticons/default/smile.gif) bardzo już poprawiłem.



Ps. Jak wyjaśni się temat to każdemu kto pomógł dam Plusa

Ten post edytował Giluś 6.08.2011, 11:37:01
Go to the top of the page
+Quote Post

Reply to this topicStart new topic
2 Użytkowników czyta ten temat (2 Gości i 0 Anonimowych użytkowników)
0 Zarejestrowanych:

 



RSS Aktualny czas: 24.08.2025 - 07:50