[PHP] Regex // Preg_Replace Opcje

[JakubBab]

Hej,

Staram sie odszukac w tablicy , uzywajac Regex, slow ktore zawieraja wiecej niz 7 znakow a nastepnie zmodyfikowac je (maja byc z malej litery) i podmienic ze starymi uzywajac funkcji preg_replace.

Znajduje wyrazy ktore mnie interesuja ale po uruchomieniu funkcji preg_replace ,zmiana jest ale nie nie ta oczekiwana. Kod i opis dalszy ponizej :

[PHP] pobierz, plaintext 
$wyrazenie_zlozone = array(" Serwer", " Router", " Klawiatura"," Mysz"," Karta sieciowa"," Monitor");
[PHP] pobierz, plaintext 

Tutaj kod w ktorym znajduje tego czego szukam( zamiescilem odrazu wywowalnie dla sprwadzenia)

[PHP] pobierz, plaintext 
$pattern = '@([A-Za-z]{7,})@'; 							// szukam liter od [A-za- z] i z min. 7 znakami
 
 
 
$doZmiennej = array();
$doZmiennej[]= $pattern;
 
 
$pasujace = preg_grep($pattern,$wyrazenie_zlozone);		        // znajdz pattern w tablicy $wyrazenie_zlozone, przechowaj w $pasujace
$pasujace1 = array_map('strtolower', $pasujace);			// to co w pasujace1 zmniejsz 'lowercase'
print_r ($pasujace1);
echo "<br/>";
[PHP] pobierz, plaintext 

Funkcja wywoluje: Array ( [2] => klawiatura [4] => karta sieciowa [5] => monitor )

-------------------------------------------------------------------------------------------------------------------------------------------------
I teraz magia, po uruchomieniu funkcji preg_replace

[PHP] pobierz, plaintext 
$zamiana = preg_replace($doZmiennej,$pasujace1,$wyrazenie_zlozone);
 
 
print_r ($zamiana);
[PHP] pobierz, plaintext 

Wychodzi wynik taki :

Array ( [0] => Serwer [1] => Router [2] => klawiatura [3] => Mysz [4] => Karta klawiatura [5] => klawiatura )


ZAMIAST

Array ( [0] => Serwer [1] => Router [2] => klawiatura [3] => Mysz [4] => karta sieciowa [5] =>monitor )

Bede wdzieczny za sugestie.

Pozdr

Jakub.

[KsaR]

[PHP] pobierz, plaintext 
$tablica = array(" Serwer", " Router", " Klawiatura"," Mysz"," Karta sieciowa"," Monitor");
$tablica2=array();
foreach ($tablica as $v)
{
 if (isset($v[7]) && ctype_alnum(str_replace(' ','',$v))) # wiecej niz 7 i jest alfa numeryczne.
 {
  $tablica2[]=lcfirst($v); # więc dodajemy do $tablica2 oraz z malej 1szy znak.
 }
}
print_r($tablica2);
[PHP] pobierz, plaintext 

O cos takiego chodzilo?
--
Poprawilem, nie lapalo bo spacje byly.

Ten post edytował KsaR 28.09.2015, 22:50:29

[JakubBab]

Super, dziala. Zminilem tylko :

[PHP] pobierz, plaintext 
$tablica2[]=strtolower($v); # więc dodajemy do $tablica2 oraz z malej wsztstko.
[PHP] pobierz, plaintext 

Dzieki za pomoc!

Ten post edytował JakubBab 28.09.2015, 22:54:00

[com]

po pierwsze twoje wyrażenie nie uwzględnia spacji, wiec pewnie wpisuje tam ostatni pasujący bo reszta pewnie nie ma 7 znaków

Ten post edytował com 28.09.2015, 22:49:58

[KsaR]

po pierwsze twoje wyrażenie nie uwzględnia spacji, wiec pewnie wpisuje tam ostatni pasujący bo reszta pewnie nie ma 7 znaków

w moim to samo, chwile temu mi sie przypomnialo to poprawilem tzn. str_replace dodalem.

[JakubBab]

Super, dziala. Zminilem tylko :

[PHP] pobierz, plaintext 
   $tablica2[]=strtolower($v); # więc dodajemy do $tablica2 oraz z malej wsztstko. 
[PHP] pobierz, plaintext 

Dzieki za pomoc!


Ps. Respect za predkosc :] !

PS2. Niechesz pobawic sie w mentora?

Ten post edytował JakubBab 28.09.2015, 22:57:13

[com]

do kogo to było

Ksar teraz trochę przekombinowałeś i ctype nie jest chyba domyślny na większości serwerów

Ten post edytował com 28.09.2015, 23:05:16

[JakubBab]

do Ksar :]

Com - rowniez dzieki za sugestie.

[KsaR]

do Ksar :]

Com - rowniez dzieki za sugestie.

Gorszego mentora byś nie znalazł
( Temat: PHP Praca praktyki )
acz kolwiek moge pomoc od czasu do czasu, o ile ogarne.
Jak co to priw

[Pyton_000]

[PHP] pobierz, plaintext 
<?php
 
$wyrazenie_zlozone = array(" Serwer", " Router", " Klawiatura"," Mysz"," Karta sieciowa"," Monitor");
 
array_walk($wyrazenie_zlozone, function(&$value) {
    if(mb_strlen($value) > 7) {
        $value = strtolower($value);
    }
});
 
var_dump($wyrazenie_zlozone);
[PHP] pobierz, plaintext 

[com]

albo twoim sposobem, z moja sugestią
http://ideone.com/ONGuJf

no i niepotrzebnie robiłeś tam preg_replace

ale rozwiązanie Pyton_000 wydaje się tu być najlepsze

[JakubBab]

com - dziala i to jeszcze z REGEX - czapka z glowy . Prosba o rozbicie teoretyczne ponizszego:

[PHP] pobierz, plaintext 
$pattern = '@([A-Za-z]{7,}+[ ]{0,})@ 
[PHP] pobierz, plaintext 

Pyton - " fanks brah" -



...i wyrywam sobie wlosy zadajac pod nosem pytanie - kiedy, no kiedy ja sie naucze i wyskocze jak z gotowa odpowiedzia na podobny problem.

[com]

to trochę przekombinowałem teraz widzę sam że jedynym błędem był twój preg_replace patern był dobry

[KsaR]

albo twoim sposobem, z moja sugestią
http://ideone.com/ONGuJf

no i niepotrzebnie robiłeś tam preg_replace

ale rozwiązanie Pyton_000 wydaje się tu być najlepsze

[PHP] pobierz, plaintext 
<?php
$start=microtime(1);
for ($x=0;$x<10000;++$x)
{
$tablica = array(" Serwer", " Router", " Klawiatura"," Mysz"," Karta sieciowa"," Monitor");
$tablica2=array();
foreach ($tablica as $v)
{
 if (isset($v[7]) && ctype_alnum(str_replace(' ','',$v))) # wiecej niz 7 i jest alfa numeryczne.
 {
  $tablica2[]=strtolower($v); # więc dodajemy do $tablica2 oraz z malej znaki.
 }
 else $tablica2[]=$v;
}
}
echo 'Ksar: ',bcsub(microtime(1),$start,5),'<br/>';
print_r($tablica2);
echo '<br/><br/>';
unset($v,$tablica,$tablica2,$start,$x);
 
$start=microtime(1);
for ($x=0;$x<10000;++$x)
{
$wyrazenie_zlozone = array(" Serwer", " Router", " Klawiatura"," Mysz"," Karta sieciowa"," Monitor");
array_walk($wyrazenie_zlozone, function(&$value)
{
 if(mb_strlen($value) > 7)
 {
  $value = strtolower($value);
 }
});
}
echo '<br/>Python_000: ',bcsub(microtime(1),$start,5),'<br/>';
print_r($wyrazenie_zlozone);
[PHP] pobierz, plaintext 

Ksar: 0.05350
Array ( [0] => Serwer [1] => Router [2] => klawiatura [3] => Mysz [4] => karta sieciowa [5] => monitor )


Python_000: 0.05680
Array ( [0] => Serwer [1] => Router [2] => klawiatura [3] => Mysz [4] => karta sieciowa [5] => monitor )

No niby mam szybsze o ile dobrze przetestowalem. (ps. Poprawilem z radami PW zeby dodawalo kazdy).
Dodatkowo Pyton_000 nie sprawdza czy alfa-numeryczne, ale jesli to zbedne? To u mnie tez mozna wywalic i znow jeszcze szybsze

--
A tu live wklejka gdzie skasowalem to sprawdzanie alfa-num:
http://ideone.com/o45lic

Ten post edytował KsaR 29.09.2015, 00:03:40

[JakubBab]

Tak jak ja sobie to tlumacze to:

[PHP] pobierz, plaintext 
    $pattern = '@([A-Za-z]{7,}+[ ]{0,})@ 
[PHP] pobierz, plaintext 

Szukaj wyrazenia ktore ma w sobie litery A-Za-z z minimalna liczba 7 znakow (+) i [] spacji {0,} <- a to to juz nie wiek jak zinterpretowac.

W kazdym razie, z mojego oryginalnego kodu zmienilem tylko:

[PHP] pobierz, plaintext 
$zamiana = preg_replace($doZmiennej,$pasujace1,$wyrazenie_zlozone);
[PHP] pobierz, plaintext 

Na to:

[PHP] pobierz, plaintext 
$zamiana = array_replace($wyrazenie_zlozone,$pasujace1);
[PHP] pobierz, plaintext 

I tez smiga.

Zachodze w glowe dlaczego preg_replace nie zadzialalo w tym przypadku. Anyone?

[com]

tak, napisałem to, że przekombinowałem i nie zweryfikowałem potem a to nie robiło nic zapomniałem że on pomija spacje
http://www.php.net/preg_replace dlatego że, to na co miał zamienić miało być definicja schematu zamiany a nie było

Dla tak małej próbki to nie ma znaczenia, owszem rozwiązanie jest ok, ale jak wspomniałem jak na serwerze masz ctype, a pewnie nie masz. i wystarczy dać pare białych znaków i już warunek legnie, ale tego również rozwiania inne nie uwzględniają

Ten post edytował com 29.09.2015, 00:19:52

[Pyton_000]

...i wyrywam sobie wlosy zadajac pod nosem pytanie - kiedy, no kiedy ja sie naucze i wyskocze jak z gotowa odpowiedzia na podobny problem.

Jak będziesz miał 666 pomógł na profilu i 2000 like na FB ;D

Moje rozwiązanie nie zakładało tylko alfa-num bo nie było określonego konkretnego warunku, ale lekko modyfikując dodając warunki da się zrobić.

@KsaR może i szybsze ale przekombinowane Poza tym ma komentarze, a jak ma komentarze to trzeba się zastanowić czy kod napisany nie powinien być zmieniony tak żeby się komentarzy pozbyć. (Temat komentarzy był na forum więc tam możemy podyskutować)


Suma sumarum Każde rozwiązanie jest dobre bardziej lub mniej

[com]

Pyton_000 dobrze to ujął, w mojej wypowiedzi lepsze chodziło o to, że od razu widać co tu się dzieję, w twoim przykładzie trzeba się zastanawiać co miałeś na myśli wiec jak wspomniał Pyton_000 trzeba pomyśleć czy nie dało by sie tego napisać inaczej