PéHáPé -- login (diskuse)

[midar]

Jee, já zapomněl na ten kod:
$connect = MySQL_Connect("mysql.webzdarma.cz", "...", "..."); if(!$connect)
{
echo "nelze se pripojit k databazi";
exit;
}

MySQL_Select_DB("usr");

$nick = $_POST["nick"];
$heslo = $_POST["heslo"];
$odesli = $_POST["odesli"];

if($odesli=="Odeslat")
{
$q = MySQL_Query("SELECT heslo FROM usr WHERE nick='$nick' AND heslo='$heslo'") or MySQL_Error();
$bflm = MySQL_Fetch_Row($q);
if($nick == "" or $heslo == "")
{
echo "Ou, neni heslo nebo nick.";
break;
}
if($bflm[3]==$heslo)
{
echo "Jsi přihlášenej.
";
}
else
{
echo "Špatné heslo.";
}
}

mysql_close();
?>

[pepiino]

Dobrý místo na dotaz :D

Proč jsi neposlal přesně tu hlášku? Neni to něco s tim, že parametr "is not valid mysql result?" Nebo tak nějak? Pokud jo, tak máš spíš problém v SQL dotazu. Zkus si ho hodit do proměnný a tu si před dotazem vypsat.

[midar]

Warning: mysql_fetch_row(): supplied argument is not a valid MySQL result --to tam ohlasi

[kavol]

Warning: mysql_fetch_row(): supplied argument is not a valid MySQL result --to tam ohlasi

hm, takže v databázi nenajde žádnou dvojici nick - heslo, která by odpovídala požadavku ...

[midar]

ale presto v databazi mam polozky nick a heslo a este tam mam id, ktery je autoincrement, ale to by se vubec nemelo brat v potaz.

[kavol]

no, to že to tam je ještě neznamená, že to tam musí najít ... jinak odpověď je jasná níže, nemůže nic najít, protože na to nemá práva ...

[midar]

jak tedy docilit zpristupneni, nebo nejak odstranit tyto prava?

[Pmx]

Warning: mysql_fetch_row(): supplied argument is not a valid MySQL result --to tam ohlasi

hm, takže v databázi nenajde žádnou dvojici nick - heslo, která by odpovídala požadavku ...

Chápu to správně tak, že podle vás se tato chybová hláška objeví vždy, když se nenajde žádný výsledek vyhovující podmínkám? Ve skutečnosti se tato hláška objeví při nějaké zásadní chybě, jako je špatně formulovaný SQL dotaz, nebo právě nepřipojení se k serveru... Pokud je prázdný výsledek (tj. ani jeden řádek), žádná chyba nevzniká, funkce mysql_fetch_row jen vrací false.

[kavol]

Pokud je prázdný výsledek (tj. ani jeden řádek), žádná chyba nevzniká, funkce mysql_fetch_row jen vrací false.

dík za opravu/upřesnění, to je tak, když si člověk něco nepamatuje přesně a je línej koukat do manuálu

[kavol]

hmm ... jak jsi přišel na to, že máš testovat if($bflm[3]==$heslo), když z databáze vybíráš pouze jeden sloupec (heslo), a tudíž to pole $bflm bude jednorozměrné (tedy hodnota hesla bude v položce $bflm[0])? - kdybys měl zapnuté warningy, nejspíš by ti to napsalo, že používáš neicializovanou proměnnou ...

btw, při daném dotazu je dostatečným testem už jen to, že vrátí nenulový počet řádků, a tedy do větve "Špatné heslo." by kód nikdy nemohl dojít, kdyby byl napsaný pořádně a otestoval, jestli vůbec nějaký výsledek dotazu dostal (konstrukce or MySQL_Error() jest velmi zvláštní, je tam na parádu či co? - řetězec, který vyplivne, se ani nezobrazuje, ani nikam neukládá ...)

jo a když už jsem u toho rozboru ... proměnná $connect, vzhledem k tomu, že se nevyužívá, je tam taky na nic, tu počáteční konstrukci by stačilo přepsat na:

if (!MySQL_Connect("mysql.webzdarma.cz", "...", "...")) {
 echo "nelze se pripojit k databazi";
 exit;
}

[midar]

hmh, upravil jsem to na bflm[0] a stejne to hlasi stejnou hlasku... To bude asi neco jinciho

[Honza Král]

1) http://stare.cz/otazky/

2) problem je v tom, ze ti to nevrati zadny radek, zkus: vypsat si SQL dotaz a pustit ho rucne otestovat pocet vracenych radku

3) nemyslim si, ze by blog na ABClinuxu mel fungovat jako poradna pro takove zacatecniky - nechci te odradit od programovani, ale nemyslis ze by bylo vhodnejsi presunout toto na nejake diskuzni forum, at uz specializovane na PHP, nebo na nejake ktere ma o PHP alespon klub?

Happy hackin'

[Honza Král]

1) http://stare.cz/otazky/

2) problem je v tom, ze ti to nevrati zadny radek, zkus: vypsat si SQL dotaz a pustit ho rucne otestovat pocet vracenych radku

3) nemyslim si, ze by blog na ABClinuxu mel fungovat jako poradna pro takove zacatecniky - nechci te odradit od programovani, ale nemyslis ze by bylo vhodnejsi presunout toto na nejake diskuzni forum, at uz specializovane na PHP, nebo na nejake ktere ma o PHP alespon klub?

Happy hackin'

[midar]

ano, je pravda, ze o tom blog neni, ale nemel jsem se kde zeptat, prvni co me napadlo byl abclinuxu, ale ja ani nezamyslel to, ze se v blogu budu ptat...Klidne me ukamenujte, ale nejdriv pls pomozte mi s timhle :-]

[Non_E]

Od podobných věcí je PHP fórum na wz.

[David Watzke]

Nebo diskuse.jakpsatweb.cz...

[kavol]

hmh, upravil jsem to na bflm[0] a stejne to hlasi stejnou hlasku... To bude asi neco jinciho

jistěže je to "něco jiného", jak už ostatně řešíš níže ... nicméně pointa měla být, i když se v mém příspěvku trochu schovala, že tak jak to bylo napsané to muselo napsat "špatné heslo" vždy, protože neprázdný řetězec $heslo (zaručeno předchozí podmínkou) jsi testoval proti prázdné (resp. neexistující) proměnné

[midar]

oka, oka

[Georgo10]

$q = MySQL_Query("SELECT heslo FROM usr WHERE nick='$nick' AND heslo='$heslo'") or MySQL_Error();

před mysql_error() musí být echo je možné, že třeba neexistuje tabulka usr, nebo sloupec nick či jiná chyba a vaším způsobem ji nezjistíte

@$q = MySQL_Query("SELECT heslo FROM usr WHERE nick='$nick' AND heslo='$heslo'"); if(!$q) echo MySQL_Error();

[Pmx]

Super, vzniká tu poradna pro začínající PHPčkaře. Na druhou stranu, je mi jasné, že každý nějak začínal...

$bflm[3] - co očekáváš, že dostaneš ve čtvrtém sloupci, když ti dotaz vrací jen jeden sloupec?

U or MySQL_Error() přidej print (MySQL_Error()), ať to aspoň trochu funguje. Nebo rovnou die().

Pokud je toto skutečně původní kód, upozornil bych na nutnost ošetření vstupních parametrů, kvůli script injection apod.. Na wz.cz sice asi budou mít zapnuté magic_quotes_gpc, ale 1) jinde to tak být nemusí, 2) největší sranda je, když to přes noc vypnou.

[midar]

hmh, ale dole porad mi to ukayuje ten neopravneny pristup.

[kavol]

Pokud je toto skutečně původní kód, upozornil bych na ...

no, především dlužno říci, že takto se projekty nepíšou ... jenže on má každej člověk jinej styl, takže bysme si mohli tak akorát zaflejmovat, jak to má "správně" vypadat

např. už jenom začátek ... pokud jde o pár stráneček, psal bych to ve stylu:

require_once("konfigurák") or die ("Fatální chyba: Nelze načíst kofiguraci");

if ($db_handle = MySQL_Connect($db_server, $db_user, $db_heslo)) {
  MySQL_Select_DB($db_mydb,$db_handle) or die ("Fatální chyba: Nelze vybrat databasi");
} else {
  die ("Fatální chyba: Nelze se pripojit k databazi");
}

ale pokud by šlo o něco většího ...

require_once("konstanty");
require_once("sys_funkce");
require_once("db_konfigurace");
require_once("db_funkce");

$db_handle = Pripoj_se_k_databazi($db_databaze_pro_tuto_stranku);
if (db_handle == false) {
  Vyhod_kritickou_chybu ($err_nepripojeno_k_databazi)
};

... výše uvedený kód prosím brát v tuto nekřesťanskou hodinu s rezervou

Na wz.cz sice asi budou mít zapnuté magic_quotes_gpc, ale 1) jinde to tak být nemusí, 2) největší sranda je, když to přes noc vypnou.

hm, magic_quotes_gpc je dle mého pěkná prasárna, ale když už se tu o tom mluví - existuje nějaký jednoduchý způsob, jak z PHPka komunikujícího s databází vymlátit na výstupu řetězec stejný jako na vstupu, tzn. bez přebývajícího/chybějícího oquotování? (tedy jinak než podívat se na konfigurační paramatry a podle toho používat od-quotovávácí funkce?)

... jinak totiž, pokud to někde "přes noc" vypnou, tak kód uzpůsobený na jedno nastavení, začne "přes noc" házet nesmysly

p.s. jak se rozumně česky řekne "oqutovat"?

[Sinuhet]

require_once("konfigurák") or die ("Fatální chyba: Nelze načíst kofiguraci");

K cemu tam je to die? Ja myslel, ze require_once samo vyhodi fatal error, takze za or se v pripade chyby program beztak nedostane.

[kavol]

říkám, že mi to po ránu nemyslí ... :-/

samozřejmě to mělo být s include

[skonciljsem]

Tedy já to obvykle dělám:

$spoj = mysql_connect(".....
$dotaz = "select.....
if (($spoj)&&(mysql_select_db("jmeno",$spoj}})
{
    $vydotaz = mysql_query($dotaz);
    if ($vydotaz)
    {
      $radek = mysql_fetch_array($vydotaz, MYSQL_ASSOC);
      // a $radek je asociativni pole, delam uz s tim
    };
}

Jo, a kde se ti tam vzala ta trojka? Nevraci to jen jedno pole (heslo)? {Já používám array, protože mi to vrací jméno a heslo, abych to jméno mohl zobrazit.)

[midar]

to bflm sem uz opravil, hodil sem tam MYSQL_ASSOC a objevila se mi jina hlaska :-] : Warning: Wrong parameter count for mysql_fetch_row(), obavam se ze tato hlaska jenom prekryla tu straou, ale mozna ne. Snad se mi to jednou povede.

[skonciljsem]

Ale ne, mysql_fetch_row() tenhle parametr nepouziva.

[midar]

co mam tedy pouzit pri overovani, bflm[0] nebo nechat to bez toho indexu?

[Pmx]

RTFM. Funkce mysql_fetch_... vrací pole a ne řetězec, tak s tím zacházej jako s polem.

[midar]

oka, ale to nevyresi to, ze mi to hlasi neopravneny pristup, viz dole.

[skonciljsem]

Podle mne se ti neprovedl MySQL_Select_DB, zkus pouzit konstrukci z helpu:
mysql_select_db('foo', $lnk) or die ('Can\'t use foo : ' . mysql_error());

[midar]

nemohl byste to nejak zkonkretizovat prosim?

[midar]

ne, jasne, chapu, hlasi mi to nepovoleny pristup: Access denied for user 'lopuch'@'%' to database 'usr'

[Pmx]

Databáze != tabulka. Tvoje databáze bude mít nejspíš název lopuch. To, s jakou tabulkou chceš pracovat, si zvolíš v konkrétním dotazu. Přes mysql_select_db volíš databázi.

[midar]

ahah , no nic, tak sem ze sebe udelal vola, no

[midar]

aha, nejde se mi pripojit k databazi...

[petr]

No to snad ani ne,.... vzdyt kouknu-vidim, pred podobnymi 'dotazy'doporucuji alespon se podivat na help wz, jmeno databaze je stejne jako login, takze porad nechapu kde beres nazev databaze 'usr' kdyz mas login lopuch.... mozna ze bych zacal teda jako konkretne timhle,.... ;)

[midar]

no ale stejne mi(i pres tu chybu v prihlasovani k databazi) to hlasi spatne heslo...

[midar]

Tak jsem to konecne vyresil. Udelal jsem novy script od zacatku a bez chyb. Dick vsem co se snazili mi pomoci. Opravdu diky.

[Luboš Doležel (Doli)]

Já dick nepotřebuju, já mám...už od narození Promiň, ale psát "dick" místo "dík" je hodně divný.

[Zdeněk Burda]

Možná to je tím, že netuší co to slovíčko znamená...

[midar]

ale vim, moc dobre to vim...

[David Watzke]

Tak to nerozdávej.