Forum PHP.pl > [PHP]Cannot redeclare

Pomoc - Szukaj - Użytkownicy - Kalendarz

djgarsi

9.03.2011, 14:57:41

Witam. Mam pewien problem. Wywołuję funkcję w pliku php, jednak dostaję taki oto komunikat.

Kod

Cannot redeclare logadm() (previously declared in /home/domena/domainsdomena.pl/public_html/login.php:59) in /home/domena/domains/domena.pl/public_html/login.php

Czym może być to spowodowane?

nospor

9.03.2011, 15:02:25

Ty nie wywolujesz a ponownie ją deklarujesz. Nie mozna kilkukrotnie deklarować funkcji o tej samej nazwie.

Pewnie includujesz kilka razy ten sam plik z funkcją
Albo w petli deklarujesz tę samą funkcje non stop

djgarsi

9.03.2011, 15:06:13

Oto i mój plik:

[PHP] pobierz, plaintext 
<?php
include('db_connect.php');
 
if(isset($_POST['logadm'])) {
	$login = htmlspecialchars($_POST['loginadm']);
	$passwordzik = htmlspecialchars(md5($_POST['passadm']));
 
	if(strstr($login, "'") == true || strstr($login, "table") == true || strstr($login, "database") == true)
	{ 
		header('Location: '.$_SERVER['HTTP_REFERER']); 
	}
	else {
		$_POST['loginadm'] = str_replace( "'", "", $_POST['loginadm']);
		$_POST['loginadm'] = str_replace( "DELETE", "", $_POST['loginadm']);
		$_POST['loginadm'] = str_replace( "DROP", "", $_POST['loginadm']);
		$_POST['loginadm'] = str_replace( "UPDATE", "", $_POST['loginadm']);
		$_POST['loginadm'] = str_replace( "SELECT", "", $_POST['loginadm']);
		$login = htmlspecialchars($_POST['loginadm']);
		$passwordzik = htmlspecialchars(md5($_POST['passadm']));
 
		$dane = @mysql_query('SELECT login, password FROM tabela WHERE login = "'.$login.'" AND password= "'.$passwordzik.'"') or die(mysql_error());
	}
	if(mysql_num_rows($dane) == 1) {
		$_SESSION['logadm'] = 'ok';
		$_SESSION['nick2'] = $login;
		$Query='SELECT * FROM users WHERE login="'.$_SESSION['nick2'].'"'; 
		$result = mysql_query($Query);
		while ($row = mysql_fetch_array($result)) 
		{
			$_SESSION['nick'] = $row["login"];
			$_SESSION['dostep'] = $row["dostep"];
		}
	}
	else {
		$_SESSION['logadm'] = '<P class=blad>'.$l_bllu.'';
	}
		unset($_POST['logadm']);
	}
	if(isset($_POST['unlogadm'])) {
		unset($_SESSION['nick']);
		unset($_SESSION['dostep']);
		unset($_SESSION['logadm']);
		unset($_POST['unloaddm']);
	}
 
	function logadm() {
		if($_SESSION['logadm'] == 'ok') {
			echo'<form action="'.getenv(REQUEST_URI).'" method="post">';
			echo'<input type="submit" name="unlogadm" value="Wyloguj '.$_SESSION['nick'].'" class="button" />';
			echo'</form> </center>';
		}
		else 
		{
		$string = '<form action="'.getenv(REQUEST_URI).'" method="post">';
		if(isset($_SESSION['logadm']))
			$string .= '<center>'.$_SESSION['logadm'].'';
			$string .= 'Login: <input type="text" autocomplete="off" name="loginadm" style="width: 100px" class="textbox"/>';
			$string .= ' Haslo: <input type="password" autocomplete="off" name="passadm" style="width: 100px" class="textbox"/>';
			$string .='  <input type="submit" name="logadm" value="Login" class="button"/><br>';
			$string .= '<a href=/register.html>Zarejestruj się</a>    <a href=/lostpassword.php>Przypomnij hasło</a></center>';
			$string .= '</form>';
		}
	}  //w tej linii pokazuje błąd
	return $string;
?>
[PHP] pobierz, plaintext

konole

9.03.2011, 15:50:13

Czekaj, czekaj.

Na początku pytasz się o to, czy istnieje $_POST['logadm'], a potem w tym samym bloku dołączasz funkcję z tabelką logowania. A co, jeśli nie istnieje $_POST['logadm']? W jaki sposób chcesz dołączyć funkcję? Jeszcze raz wstawiasz gdzieś jej kod (else)?

I to

Kod

$_POST['loginadm'] = str_replace( "'", "", $_POST['loginadm']);
        $_POST['loginadm'] = str_replace( "DELETE", "", $_POST['loginadm']);
        $_POST['loginadm'] = str_replace( "DROP", "", $_POST['loginadm']);
        $_POST['loginadm'] = str_replace( "UPDATE", "", $_POST['loginadm']);
        $_POST['loginadm'] = str_replace( "SELECT", "", $_POST['loginadm']);
        $login = htmlspecialchars($_POST['loginadm']);
        $passwordzik = htmlspecialchars(md5($_POST['passadm']));

Możesz równie zastąpić

Kod

$login = mysql_real_escape_string($_POST['loginadm']);
$passwordzik = mysql_real_escape_string($_POST['passadm']);

greycoffey

9.03.2011, 16:09:47

Masz podany bład jak na tacy, przetłumacz sobie (jak nie znasz angielskiego to masz translatroy) i rozwiąż problem.

djgarsi

9.03.2011, 16:18:30

Cytat(greycoffey @ 9.03.2011, 16:09:47 )

Masz podany bład jak na tacy, przetłumacz sobie (jak nie znasz angielskiego to masz translatroy) i rozwiąż problem.

No fajnie, tylko że do tej pory działało. Przestało gdy dopisałem linijki ze str_replace.

greycoffey

9.03.2011, 16:34:23

Redeklarujesz funkcje logadm(). Daj cały kod, a nie tylko ten plik, jest coś po tym, lub przed tym jeszcze? Btw. czy domainsdomena.pl nie powinno być domains/domena.pl - to twój błąd, czy taka jest prawdziwa treść zwracana przez praser?

Do tego return $string; powinno być w funkcji, nie poza nią.

djgarsi

9.03.2011, 16:38:56

domains/domena.pl - mój błąd

podałem cały plik login.php

Fifi209

9.03.2011, 16:43:25

Cytat(greycoffey @ 9.03.2011, 16:34:23 )

Do tego return $string; powinno być w funkcji, nie poza nią.

Nie zawsze, swego czasu (póki nie używano Registry) w config wyglądał tak:

[PHP] pobierz, plaintext 
$config[jakisklucz] = 'jakas_wartosc';
 
return $config;
[PHP] pobierz, plaintext

Co dawało możliwość globalnego dostępu do $config

O ile mnie pamięć nie myli.

djgarsi

9.03.2011, 17:47:12

Niestety, przestawienie return nic nie daje.
Jakieś inne propozycje?

greycoffey

9.03.2011, 19:41:03

Gdzie wykorzystujesz ten plik? Może go inkludujesz potem, bo on sam jest niezbyt przyjazny?

djgarsi

9.03.2011, 23:18:54

greycoffey, masz rację. includowałem plik w innym miejscu. i chyba to było to bo inaczej go wywołałem i działa:)
Pozdrawiam.

To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.