[php][mysql] Sprawdzanie w bazie Opcje

[terreska]

Mam poniższy skrypcik.
Nie ważne co wpisze zawsze pojawia się ten sam komuniakat "TAK".
Wiem, że błąd tkwi w zapytaniu lub w if, albo tu i tu. Kombinowałam na wszystkie sposoby i nic.
Proszę o pomoc.

[PHP] pobierz, plaintext 
<?php
case "domena":
			$query="select * from members where domena='%$by%'";
			if ($query='%$by%') 
			{
			echo '<font color="red"><center>TAK</center></font>';
			}	else{
			echo '<font color="green"><center>NIE</center></font>';
			} 
			break;
?>
[PHP] pobierz, plaintext 

Ten post edytował terreska 5.11.2006, 09:27:56

[milus]

Masz zapytanie ale nie wysyłasz go.
mysql_query

[terreska]

Niestety nie rozumiem. Co i gdzie muszę wpisać?

Ten post edytował terreska 5.11.2006, 09:51:53

[TomASS]

po pierwsze masz zapytanie ($query), ale nigdzie go nie wykonujesz (np. mysql_query), także $query jest "zwykłym" stringiem.

po drugie znaczek "=" jest operatorem przypisania, w Twoim przypadku wyrażenie:

[PHP] pobierz, plaintext 
<?php
($query='%$by%')
?>
[PHP] pobierz, plaintext 

przyjmie wartość '%$by%', które w if'ie jest zawsze prawdą.

Operatorem porównania jest np. '=='

Może napisz co chciałaś osiągnąć?

Ten post edytował TomASS 5.11.2006, 09:57:12

[terreska]

Teraz mam tak i znów jest to samo. Co teraz jest źle?

[PHP] pobierz, plaintext 
<?php
case "domena":
			$query="select * from members where domena='%$by%'";
			$q=mysql_query ($query);
			if ($q=='%$by%') 
			{
			echo '<font color="red"><center>TAK</center></font>';
			}	else{
			echo '<font color="green"><center>NIE</center></font>';
			} 
			break;
?>
[PHP] pobierz, plaintext 

Chcę sprawdzić czy określony wpis znajduje się w bazie czy nie.

[TomASS]

W zasadzie to zobacz co zwraca mysql_query.

Powiedz co chcesz osiągnąć?

Dlaczego masz takie zapytanie:

[PHP] pobierz, plaintext 
<?php
$query="select * from members where domena='%$by%'";
?>
[PHP] pobierz, plaintext 

?
Może powinno być:

[PHP] pobierz, plaintext 
<?php
$query="SELECT * FROM members WHERE domena LIKE '%$by%'";
?>
[PHP] pobierz, plaintext 

[skowron-line]

mysql_num_rows sprawdz ile rekordow zwroci zapytanie i wtedy jak bedzie = 0 to daj ze nie ma takiego wpisu

[terreska]

Ok, dziękuję teraz działa, ale nie do końca dobrze...

[PHP] pobierz, plaintext 
<?php
				case "domena":
						$query="select * from members where domena like '%$by%'";
						$q=mysql_query ($query);
						$num_rows = mysql_num_rows($q); 
						if ($num_rows==0) 
						{
						$content=error_page("Nie ma wpisu.");
						}	else{
						$content=error_page("Jest wpis.");
				  break;
?>
[PHP] pobierz, plaintext 

W bazie znajduje się przykładowo słowo "telewizor".
Gdy wpiszę telewizor, tele, albo nawet samą litere występującą w tym wyrazie to wyświetli mi, że jest wpis.
Jak zrobić, aby tylko szukał pełnych nazw w bazie?

Ten post edytował terreska 6.11.2006, 10:59:43

[Amorph]

[PHP] pobierz, plaintext 
<?php
$query="select * from members where domena like '$by'";
?>
[PHP] pobierz, plaintext 

[terreska]

Że też sama na to nie wpadam... (IMG:http://forum.php.pl/style_emoticons/default/laugh.gif)
Dziękuje.

[TomASS]

A nie lepiej

[PHP] pobierz, plaintext 
<?php
$query="SELECT * FROM members WHERE domena= '$by'";
?>
[PHP] pobierz, plaintext 

?
Dodatkowo, możesz określić zamiast * tylko niezbędne pola - powinno to przyspieszyć zapytanie.
Spróbuj też zabezpieczyć swój skrypt przed atakami SQL Injection np. przy pomocy mysql_escape_string, ale to już osobny temat.

[Kangu]

z tego co wiem wielkosc liter w zapytaniu sql nie ma znaczenia, chociaz sam uzywam wielkich dla wyeksponowania polecen, no a nazwy tabel pisze w `ciapkach`. co do * to sie zgodze, lepiej jej nie stosowac. powinno sie raczej wymieniac interesujace elementy. to ma wplyw na ' jakosc ' zapytania. mysql_real_escape_string tez jest przydatne - zgadzam sie. polecam. pozdro.

Ten post edytował Kangu 7.11.2006, 11:30:16

[mls]

z tego co wiem wielkosc liter w zapytaniu sql nie ma znaczenia, chociaz sam uzywam wielkich dla wyeksponowania polecen, no a nazwy tabel pisze w `ciapkach`. co do * to sie zgodze, lepiej jej nie stosowac. powinno sie raczej wymieniac interesujace elementy. to ma wplyw na ' jakosc ' zapytania. mysql_real_escape_string tez jest przydatne - zgadzam sie. polecam. pozdro.

Za bardzo nie przyzwyczajaj się do tego, że "wielkość liter w zapytaniu sql nie ma znaczenia" oraz "nazwy tabel pisze w `ciapkach`" - gdy będziesz musiał skorzystać z PostgreSQL, zrozumiesz dlaczego (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

[TomASS]

@Kangu :
Oczywiście nie chodzi o wielkość liter, lecz o samo wykonanie zapytania:

[SQL] pobierz, plaintext 
WHERE domena= '$by'";
[SQL] pobierz, plaintext 

vs

[SQL] pobierz, plaintext 
WHERE domena LIKE '$by'";
[SQL] pobierz, plaintext 

to pierwsze jest zdecydowanie szybszym rozwiązaniem.

Powiem nawet więcej - do wyszukiwania najlepiej chyba nadaje się użycie poleceń match() i against():

[SQL] pobierz, plaintext 
SELECT cos1, cos2 
FROM TABLE WHERE match(cos1) against('szukane_wyrazenie');
[SQL] pobierz, plaintext 

Co do wielkich liter oraz do brania nazw tabel/pól w "ciapki" to ja zdecydowanie preferuje style jaki panuje na mysql.com. Wielkie literki + brak ciapek.

co do * to sie zgodze, lepiej jej nie stosowac.

Słyszałem, że nawet wyliczenie wszystkich pól jest szybsze niż stosowanie *, aczkolwiek na pewno mniej wygodne.

Za bardzo nie przyzwyczajaj się do tego, że "wielkość liter w zapytaniu sql nie ma znaczenia" oraz "nazwy tabel pisze w `ciapkach`" - gdy będziesz musiał skorzystać z PostgreSQL, zrozumiesz dlaczego

Nie znam się za bardzo na PostgreSQL, ale przeglądając to forum można natrafić np. na takie zapytanie gdzie nie ma żadnego ujmowania w "ciapki" (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif)


Pozdrawiam

Ten post edytował TomASS 7.11.2006, 23:12:27