[php/mysql] logowanie a sesja i połączenie z mysql Opcje

[pijanyadmin]

Baza

[SQL] pobierz, plaintext 
CREATE TABLE IF NOT EXISTS `userdata` (
  `id` int(11) NOT NULL AUTO_INCREMENT,
  `login` varchar(15) NOT NULL,
  `haslo` varchar(32) NOT NULL,
  `email` varchar(40) NOT NULL,
  `data` varchar(19) NOT NULL,
  `adresip` varchar(25) NOT NULL,
  KEY `id` (`id`)
) ENGINE=MyISAM  DEFAULT CHARSET=utf8 AUTO_INCREMENT=12 ;
[SQL] pobierz, plaintext 

panel logowania

[PHP] pobierz, plaintext 
<?php
session_start();
require_once("functions.php");
require_once("connect.php");
if(logon()) {
	movetosite('panel.php');
	exit();
}
?>
 
<form action="panel.php" method="POST">
		<fieldset>
		<div>
		<label for="login">Login:</label>
		<input id="login" type="text" class="text"  name="user" maxlength="15" />
		</div>
		<br /><br />
		<div>
		<label for="haslo">Hasło:</label>
		<input id="haslo" type="password" class="text" name="pass" maxlength="15" />
		</div>
		<br /><br />
		<div>
		<input id="fSubmit" class="submit" type="submit" name="sprawdz" value='Zaloguj się' />
		</div>
	</fieldset>
</form>
[PHP] pobierz, plaintext 

sesja

[PHP] pobierz, plaintext 
<?php
require_once("connect.php");
 
 
 
function logon() {
	if(isset($_POST['user']) && isset($_POST['pass'])) {
		$_POST['user'] = md5($_POST['user']);
		$_POST['pass'] = md5($_POST['pass']);
 
			$q = mysql_query("SELECT * FROM userdata WHERE haslo = '".$_POST['user']."' AND haslo = '".$_POST['pass']."' LIMIT 1");
		if(!mysql_num_rows($q)) {
				echo 'Niepoprawne Login lub Hasło'; exit;
			return 0;
		}
 
	$_SESSION['logon'] = 1;
	$_SESSION['login'] = $_POST['user'];
	$_SESSION['password'] = $_POST['pass'];
	$_SESSION['ip'] = $_SERVER['REMOTE_ADDR'];
	$_SESSION['idu'] = md5(rand() + rand());
 
 
	return 1;
	}
 
 
 
		if(!isset($_SESSION['logon']) || !isset($_SESSION['login']) || !isset($_SESSION['password']) || !isset($_SESSION['ip']) || !isset($_SESSION['idu'])) {
					//	echo "3";
		return 0;
	}
	$login = $_SESSION['login'];
	$password = $_SESSION['password'];
	$idu = $_SESSION['idu'];
	$ip = $_SESSION['ip'];
 
	if($_SERVER['REMOTE_ADDR'] != $ip) {
				//	echo "4";
		return 0;
	}
 
 
if($_GET['idu'] != $_SESSION['idu']) {
				return 0;
} else {
	return 1;
}
 
}
 
 
 
function movetosite($strona) {
	echo "
	<script LANGUAGE=\"JavaScript\" TYPE=\"text/javascript\"> 
window.location.replace(\"$strona\"); 
</SCRIPT>";
}
 
function resetsession() {
$_SESSION['login'] = '';
$_SESSION['password'] = '';
$_SESSION['logon'] = '';
			 unset($_SESSION['login']);
			 unset($_SESSION['password']);
			 unset($_SESSION['logon']);
			 unset($_SESSION);
session_destroy();
}
 
?>
[PHP] pobierz, plaintext 

problem polega w zalogowaniu sie i otrzymaniu sesji:

[PHP] pobierz, plaintext 
<?php
$q = mysql_query("SELECT * FROM userdata WHERE haslo = '".$_POST['user']."' AND haslo = '".$_POST['pass']."' LIMIT 1");
		if(!mysql_num_rows($q)) {
				echo 'Niepoprawne Login lub Hasło'; exit;
			return 0;
		}
?>
[PHP] pobierz, plaintext 

tak mogte się zalogować wez problemu pod warunkiem że login jest taki sam jak haslo, niestety jak zmienie

[PHP] pobierz, plaintext 
<?php
haslo = '".$_POST['user']."'
?>
[PHP] pobierz, plaintext 

na

[PHP] pobierz, plaintext 
<?php
login = '".$_POST['user']."'
?>
[PHP] pobierz, plaintext 

to w ogóle nie moge się zalogować... co jest nie tak [pomijajnac fakt ze troche to chaotyczny kod]

Ten post edytował pijanyadmin 9.01.2008, 20:19:40

[xbigos]

@pijanyadmin. Ja czegoś tu nie rozumiem.

[PHP] pobierz, plaintext 
<?php
$zapytanie = "SELECT * FROM `userdata` WHERE `login`='haslo'";
?>
[PHP] pobierz, plaintext 

dziwne masz zapytanie. Porównujesz login do hasła?
potem dajesz

[PHP] pobierz, plaintext 
<?php
$idzapytania = mysql_query($zapytanie);
?>
[PHP] pobierz, plaintext 

Z tym sie zgodzę. Ale gdzie reszta?
Nie dziwie się, że się gubisz. Skoro twoje zmienne nie mają żadnego odzwierciedlenia w zawartości

[PHP] pobierz, plaintext 
<?php
session_start();
 
$haslo = $_SESSION['login'];
 
//pobiera dane z formularza
$email = $_POST['email'];
 
$zapytanie = "UPDATE  userdata SET email WHERE login = $haslo";
$idzapytania = mysql_query($zapytanie);
 
echo "dane zmieniione";
?>
[PHP] pobierz, plaintext 

poczytaj troche o zapytaniach sql

[PHP] pobierz, plaintext 
<?php
$zapytanie = "UPDATE  userdata SET email WHERE login = $haslo";
?>
[PHP] pobierz, plaintext 

Co chcesz umieścić w polu email? gdzie informujesz zapytanie, że chcesz dodać właśnie zawartość zmiennej $email.
zapytanie powinno wyglądać tak:

[PHP] pobierz, plaintext 
<?php
$zapytanie = "UPDATE  userdata SET email = `$email` WHERE login = $haslo";
?>
[PHP] pobierz, plaintext 

Według mnie cały kod powinien wyglądać tak:

[PHP] pobierz, plaintext 
<?php
session_start();
$login = 'haslo'; //uzyj w tym miejscu htmlspecialchars()
$zapytanie = "SELECT * FROM `userdata` WHERE `login`='$login`";
$idzapytania = mysql_query($zapytanie);
// Tutaj nie wiem co wpisać bo nie wiem co chciałeś tym osiągnąć
?>
<form action="edi.php" method="POST">
<?php
 
echo "
  <table border='0' width='100%' id='table1' height='321'>
		<tr>
				<td width='115'><font face='Tahoma'>login:</td>
				<td> <font face='Tahoma'>$login</font></td>
		</tr>
 
		<tr>
				<td width='115'><font face='Tahoma'>email:</font></td>
				<td> <input type='text' name='email'  value='$email'/></td>
		</tr>
		<tr>
				<td width='115' height='44'>&nbsp;</td>
				<td height='44'> <input type='submit' value='zmien dane'>
 
		</tr>
  </table>";
?>
[PHP] pobierz, plaintext 

edi.php (dziwna nazwa pliku:D)

[PHP] pobierz, plaintext 
<?php
session_start();
 
$zalogowany = $_SESSION['login'];
 
//pobiera dane z formularza
$email = htmlspecialchars($_POST['email']);
 
$zapytanie = "UPDATE  userdata SET email = `$email` WHERE login = `$zalogowany`";
$idzapytania = mysql_query($zapytanie);
if(!$idzapytania){
echo 'blad';
}
else {
echo 'rekord zaktualizowany';
}
?>
[PHP] pobierz, plaintext 

I staraj się nazywać zmienne na tej podstawie co przechowują.
no i przeczytaj jeszcze to Najczęściej popełniane błędy początkujących programistów. nie jest długie.

Według mnie ten skrypt jest napisany nieczytelnie, niedokładnie a co najgorsze. Nieprzemyślany.

Staraj się w php umieszczać jak najmniej znaczników html ponieważ zwalnia to działanie strony. Ponieważ PHP musi przetworzyć wszystko w echo jako string a jesli dasz normalnym sposobem to tym już się zajmie przeglądarka.

Ten post edytował xbigos 10.01.2008, 16:18:24