Chce to zoptymalizowac - tylko jaka opcja rozsadniej i jak?, wiec jak to z tym :) ? Opcje

[ApC]

witam,
mam cos takeigo ze w jednym pliku wykorzystuje z tabeli `users` tylko `login` i `mail`.
i wyglada zapytanie tak:

$user=mysql_fetch_assoc(mysql_query("SELECT * FROM `users` WHERE `login`='".$_SESSION['login']."' AND `haslo`='".$_SESSION['haslo']."'"));

i jak cos biore to tak:
$user[login] - $user[mail]

I pobieram tylko ten login i mail, a w tej tabeli `users` jest tego pelno, login, mail, plec, adres, miasto, kod, ip itp... chce to zoptymalizowac.

A wiem ze idzie jakos tak

$user=mysql_fetch_assoc(mysql_query("SELECT `login,mail` FROM `users` WHERE `login`='".$_SESSION['login']."' AND `haslo`='".$_SESSION['haslo']."'"));

Tylko jak biore tak:
$user[login] - $user[mail]
to nie dziala. pomoze ktos? z gory dzieki

[l0ud]

Nie bierz jednocześnie login i mail pomiędzy ``,a oddzielnie:

Kod

SELECT `login`,`mail`

W odniesieniu tablicy dodawaj apostrofy

[PHP] pobierz, plaintext 
<?php
$user['login']
?>
[PHP] pobierz, plaintext 

[ApC]

dzieki bardzo za odpowiedz.
wiec udalo sie ;]

a czemu uzywac apostrofow? to co daje? bo nigdy nie stosuje i jest ok? i mysle, ze tak jak teraz zmieniam jest korzystniej nie pobierajac wszystkiego nie?

[l0ud]

Jak coś działa w PHP to nie znaczy że jest poprawnie zrobione. Jeżeli nie chce Ci się poprawiać tych apostrofów, chociaż za każdą następną okazją już je stosuj. W ten sposób wyrabiasz sobie złe nawyki i innym, którzy czytają Twój kod. Jak rozwiązałbyś problem odniesienia się do indeksu w tablicy zbudowanego z zawartości pewnej zmiennej i jakiegoś stringu?

[PHP] pobierz, plaintext 
<?php
$array[$zmienna.'asd']
?>
[PHP] pobierz, plaintext 

Zrób to bez cudzysłowów ani apostrofów

[edit]

Na pewno jest korzystniej pobierać tylko potrzebne dane - zmniejszasz ruch pomiędzy serwerem mysql a http, poza tym mysql nie musi wyszukiwać niepotrzebnych danych...

Ten post edytował l0ud 12.03.2008, 20:50:16