[PHP][MYSQL]Skrypt rejestracji Opcje

[slawo123]

Witam, od niedawna bawię się php, i postanowiłem napisać własnego CMS'a (IMG:http://forum.php.pl/style_emoticons/default/blinksmiley.gif)
Napisałem między innymi skrypt rejestracji, ale nie wiem czy nie popełniłem w nim jakiś rażących błędów, oprócz tego że nie miesza się html'a z php.
Nie wiem również czy jest on zoptymalizowany i czy jest bezpieczny...

Proszę o szczere opinie, i wytykanie mi błędów. Nie śmiejcie się z nazewnictwa zmiennych i funkcji (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif)

Skrypt można przetestować pod tym adresem: http://e50.com.pl/test/register.php

A kod to: http://e50.com.pl/test/kod.txt

Za długi był by go wkleić tutaj (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif)

[peter13135]

niechce mi sie wszystkiego czytać, ale już widze pare błedów optymalizacji, a konkretnie na początku pobierasz z jednej tabeli nr 1 wartośc nr 1 a potem z tej samej tabeli wartosć nr 2, niemożesz wykonać raz zapytanie pobierając obie wartości?

[slawo123]

Czyli w jaki sposób mam pobrać kilka rekordów z tabeli i zapisać je osobno w zmiennych?

Proszę o dalsze opinie.

[rupek]

kurde, tego sie nie dało normalnie przeczytać, popraw formatowanie i wrzuć ponownie, a wtedy na pewno ludzie będą chętniej pomagać, zauważy się więcej błedów a i Tobie będzie się ten kod łatwiej poprawiać

[slawo123]

Racja. http://www.wklejto.pl/8614

Czekam na sugestie...

Ok, to może inaczej...

Jak za pomocą jednego zapytania do mysql uzyskać osobno wartości z kilku rekordów? Za pewne przez pętlę, ale nie wiem jak potem te dane wyciągnąć...

Mam zapytania:

[PHP] pobierz, plaintext 
<?php
//odczyt nazwy domeny
$zapytanie3 = @mysql_query("SELECT wartosc FROM `config` WHERE nazwa = 'domena'");
$domena = @mysql_fetch_array($zapytanie3);
$domena = $domena['wartosc'];
 
//odczyt ścieżki path
$zapytanie2 = @mysql_query("SELECT wartosc FROM `config` WHERE nazwa = 'script_path'");
$path = @mysql_fetch_array($zapytanie2);
$path = $path['wartosc'];
 
//odczyt zawartości konfiguracji email
$zapytanie1 = @mysql_query("SELECT wartosc FROM `config` WHERE nazwa = 'email'");
$obecny_email = @mysql_fetch_array($zapytanie1);
$obecny_email = $obecny_email['wartosc'];
?>
[PHP] pobierz, plaintext 

[ferrero2]

Pętla while :

Jeśli np "nazwa" to u Ciebie 2-ga kolumna licząc od lewej (pierwsza kolumna ma index 0) to robisz tak

Kod

$zapytanie = "SELECT * FROM config";
$result = mysql_query($zapytanie);
while($row = mysql_fetch_array($result))
{
echo $row['1'] . '<br />';
}

Niewiem czy konkretnie o to chodzi.

[slawo123]

Ale ja nie chcę wyświetlać wyników tylko je przypisywać do różnych zmiennych.

Mam tabelę config a w niej 2 kolumny: nazwa i wartosc.

Rekordów mam 3.

I chcę w jednym zapytaniu pobrać wszystkie dane z tabeli i każdy rekord przypisać do innej zmiennej, tak jak poradził peter13135

[golaod]

Wszystkie dane z tabeli SELECT * FROM tabela
Funkcja http://pl.php.net/manual/pl/function.eval.php
while($wynik = mysql_fetch_array($zapytanie)
eval("\$$wynik['nazwa']=$wynik['wartosc'];");

Dzieki temu stworza Ci sie 3 zmienne o nazwach jak w tabeli (chyba ze beda spacje) i wartosciach jak w tabeli.

Ten post edytował golaod 23.08.2008, 12:01:20

[ferrero2]

To dodaj :

Kod

$zapytanie = "SELECT nazwa, wartosc FROM config";
  $result = mysql_query($zapytanie);
  $licznik = 1;
  while($row = mysql_fetch_array($result))
  {
$wartosc[$licznik] =$row['wartosc'];
  $nazwa[$licznik] = $row['nazwa'];
  licznik++;
  }

Ten post edytował ferrero2 23.08.2008, 12:08:35

[golaod]

@ferrero2 nie ucz go zlych nawykow
PHP jest sprytne i potrafi sam indeksowac tablice zatem wystarczy tylko $nazwa[]=$row['nazwa'],$wartosc[]=$row['wartosc']. A w ogole to po co mu dwie tablice do tego ?

Ten post edytował golaod 23.08.2008, 12:37:46

[peter13135]

<?
$sql=@mysql_query("SELECT wartosc FROM `config` );
$row=mysql_fetch_array($sql);
$domena=$row['domena'];
$path = $row['script_path'];
$email=$row['email];
?>

zdaje sie że powinno być tak, jeśli źle to niech mnie ktoś poprawi
ale myśle że dobrze myśle więc napisałem co myślałem (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

edit, coś mi sie pomyliło, myślałem że masz inną strukture tabeli
w twoim przypadku powinno sie zastosować petle

$sql=@mysql_query("SELECT wartosc FROM `config` );
for($x=0,$x>3;$x++)
{
$row=mysql_fetch_array($sql);
$nazwa[$i]=row[wartosc];
}
no i $nazwa[0] to będzie domena
$nazwa[1] to patch
$nazwa[2] to email
(tak mi sie wydaje że tak być powinno, jak źle to poprawcie)

Ten post edytował peter13135 25.08.2008, 13:16:48