[PHP]Przenosiny z ereg na preg Opcje

[DestiX]

Witajcie, mam do napisania pewien skrypt i potrzebne jest mi tutaj wyszukiwanie słów w pliku. Udało mi się to zrobić przy pomocy jak się okazało przestarzałego "ereg". Próbowałem samemu przerobić to na "preg_match" no ale niestety nie potrafię. Zamieszczam poniżej mój kod i liczę na jak najszybszą pomoc.

[PHP] pobierz, plaintext 
<?php
if (isset($_POST['loguj'])){
	if ($_POST['login'] == '' || $_POST['pass'] == ''){
  echo ('Musisz uzupełnić pola.');
    } else {
		$login = $_POST['login'];
		$haslo = $_POST['pass'];
 
 
 
		//wczytuje plik
		$tekst=file_get_contents('db/konta.(IMG:style_emoticons/default/haha.gif) ');
 
		//czy zmienne są puste
 
		if($login!='' || $haslo!='')
		{
 
   		//szukanie konta
   			if(ereg($login,$tekst)|| ereg($haslo,$tekst))
			{
			echo 'Zalogowałeś się na konto:'.$login;
   			}
   			else
   			{
   			echo 'Konto: '.$login.'nie istnieje';
			}
			}
	}
}
?>
[PHP] pobierz, plaintext 

[Turson]

Nie masz żadnego wzorca więc samo strpos wystarczy.

[nospor]

@Turson wystarczyc moze i wystarczy, co nie zmienia faktu, ze bedzie do kitu....

przyklad:
koles wpisujesz w pole LOGIN 'bam'
a w "bazie" ma 'bambus'
i ups, zalogouje sie

[Daimos]

nospor, ale chyba o to autorowi tematu chodzi (IMG:style_emoticons/default/wink.gif) bo z tego co widzę, to chce szukać loginu w jakimś tekście. Po co i dlaczego, to już nie nam oceniać, ale jeśli to system logowania, to faktycznie...

Ten post edytował Daimos 17.11.2014, 21:26:51

[nospor]

@Daimos to że autor robi logicze bledy nieznaczy, że mamy mu ich niewypomnieć (IMG:style_emoticons/default/wink.gif)

[DestiX]

Robię prosty skrypt sklepu do szkoły, nie musi to być nie wiadomo co dlatego też proszę nie wnikać w to jak to jest napisane albo co w czym szukam bo jest to nie istotne. Kod, który wstawiłem wyżej działa w sposób dla mnie odpowiedni. Jedyne co jest nie tak to fakt, że na stronie mam info o tym, że funkcja ereg jest przestarzała dlatego też proszę was o zamienienie tej funkcji na ponoć nowszą "preg". To wszystko (IMG:style_emoticons/default/smile.gif)

[nospor]

NO to dostales odpowiedz: strpos... Ile razy mozna pisac o tym?

[DestiX]

No ale wyczytałem też jak sam podałeś, że można podać "bam" a zaloguje się na konto "bambus". W ereg tego nie było, jeżeli nie znalazło dokładnie całego "bambus" to nie logowało.

[nospor]

A, zrozumialem, ze w ereg miales podobny blad (IMG:style_emoticons/default/smile.gif)

edit: no ale chwila... w ereg miales wpisac BAM a nie BAMBUS. Jak wpisales BAM to ci nie znalazl mimo ze byl BAMBUS? cos krescisz (IMG:style_emoticons/default/wink.gif)

edit:
if (ereg('bam','blabla bambus blala')) echo 'ok'; else echo 'nieok';
nom, czyli jest jak mowilem, wiec rownie dobrze mozesz stosowac STRPOS

[DestiX]

Rzeczywiście, wpisuje "bam" i loguje na "bambusa". Widocznie mi się tylko wydawało. No dobra to niech już tak to zostanie.

[PHP] pobierz, plaintext 
if(strpos($login, $tekst)|| strpos($haslo, $tekst))
[PHP] pobierz, plaintext 

Tak to ma/może wyglądać?

[Turson]

Zajrzyj do manuala. Zobaczysz, że strpos jest pehapowym wyjątkiem od reguły i kolejność argumentów jest inna

[DestiX]

[PHP] pobierz, plaintext 
<?php
if (isset($_POST['loguj'])){
	if ($_POST['login'] == '' || $_POST['pass'] == ''){
  echo ('Musisz uzupełnić pola.');
    } else {
		$login = $_POST['login'];
		$haslo = $_POST['pass'];
 
 
 
		//odczyt pliku - wczytanie jego treści do stringa
		$tekst=file_get_contents('db/konta.(IMG:style_emoticons/default/haha.gif) ');
 
		//Warunek: Czy w zmiennej $wyraz coś istnieje?
 
		if($login!='' || $haslo!='')
		{
   		$log = strpos($tekst, $login);
		$pas = strpos($tekst, $haslo);
 
   			if($log !== FALSE|| $pas !== FALSE)
			{
			echo 'Zalogowałeś się na konto:'.$login;
			$plik = "db/logged.(IMG:style_emoticons/default/haha.gif) ";
        	$fp = fopen($plik, "a+");
        	fwrite($fp, $login);
        	fclose($fp);
   			}
   			else
   			{
   			echo 'Konto: '.$login.'nie istnieje';
			}
			}
	}
}
?>
[PHP] pobierz, plaintext 

No teraz to nie wiem ocb. Za każdym razem dostaje info, że zalogowano na konto. Zmieniałem "FALSE" na "TRUE", usuwałem i dodawałem"!" i jedyna różnica to taka, że albo za każdym razem loguje na konto (nawet jak nie istnieje) albo nie loguje (nawet jak istnieje).

[nospor]

No ale logika sie klania...
nie: if($log !== FALSE|| $pas !== FALSE)
a: if($log !== FALSE && $pas !== FALSE)

To jak miales
if($log !== FALSE|| $pas !== FALSE)
to wystarczy ze albo zgadza sie login albo haslo i juz loguje

To samo tutaj:
if($login!='' || $haslo!='')

tak ma byc:
if($login!='' && $haslo!='')
I login i haslo mają byc wypelnione

[DestiX]

A no fakt, myślenie o tej godzinie nie działa u mnie najlepiej. Jak na razie wszystko działa tak, że dzięki za pomoc. Gdy jeszcze coś zepsuje to napiszę (IMG:style_emoticons/default/smile.gif) .

[Daimos]

@Daimos to że autor robi logicze bledy nieznaczy, że mamy mu ich niewypomnieć (IMG:style_emoticons/default/wink.gif)

co prawda to prawda (IMG:style_emoticons/default/smile.gif)

Robię prosty skrypt sklepu do szkoły, nie musi to być nie wiadomo co dlatego też proszę nie wnikać w to jak to jest napisane albo co w czym szukam bo jest to nie istotne. Kod, który wstawiłem wyżej działa w sposób dla mnie odpowiedni...

Tak samo podejrzewam, że twierdziła słynna studentka z Łodzi, z jej prostą aplikacją za 500tys zł do liczenia głosów (IMG:style_emoticons/default/wink.gif)
Lepiej się uczyć od razu dobrych nawyków, bo to jak jest teraz napisane, odbije się na Twoich aplikacjach w przyszłości, każdy to potwierdzi. Sam mam nawyki, za które bym się zabił, a jednak czasami z przyzwyczajenia je stosuje, żałuje, ale tak się nauczyłem i czasami ponosi (IMG:style_emoticons/default/smile.gif)