funkcja w funkcji :? Opcje

[Apo]

Witam
Napisałem sobie funkcje panelu który pojawia się obok wpisu do księgi gości po zalogowaniu jako admin. I w tej funkcji mam jeszcze jedną funkcją narazie odpowiedzialną za banowanie oto te funkcje :

[PHP] pobierz, plaintext 
<?php
///////////////////////////////////
//////////dodaj bana
///////////////////////////////////
function ban($ip) {
		if($_GET['action'] == 'ban' && $_GET['id']) {
		$ban_db = 'ip.txt';
		$hand = fopen($ban_db, 'w');
		flock($hand, LOCK_EX);
		fwrite($hand, $ip.'n');
		flock($hand, LOCK_UN);
		fclose($hand);
		echo 'Dodano bana';
	}
}
///////////////////////////////////
//////////panel po zalogwaniu
///////////////////////////////////
function paneladmina() {
global $PHP_SELF, $linki, $i;
if(logowanie() == TRUE) {
	echo '	<div id="title"><font class="decor">'.$linki[$i]['ip'].'</font>
			<font class="decor">#</font> <a href="'.$PHP_SELF.'&action=ban&id='.$i.'">Zbanuj</a>
			<font class="decor">#</font> <a href="'.$PHP_SELF.'&action=edit&id='.$i.'">Edytuj</a>  
			<font class="decor">#</font> <a href="'.$PHP_SELF.'&action=delete&id='.$i.'">Kasuj</a>
			<font class="decor">#</font> <a href="panel.php">Panel Admina</a></div>';
			ban($linki[$i]['ip']);
	}
}
?>
[PHP] pobierz, plaintext 

I teraz właśnie chyba wszystko powinno chodzić ale nic sie niedzieje gdy jest adres: action=edit&id='.$i.' :?:
co jest tego winą :?:

[nospor]

mowisz ze dajesz linka action=edit&id='.$i.' i nic sie nie dzieje. Natomiast w funkcji ban() wyswietlasz cokolwiek gdy action==ban a nie edit, wiec sie nie dziw ze funckja ban nic ci nie wyswietla.

pozatym do funkcji ban przekazujesz $ip, do ktorego wogole sie nie odwolujesz

[dr_bonzo]

A wywolujesz te funkcje?

[Apo]

No wywołuje tylko funkcje paneladmina. Dlatego dałem funkcje ban do funkcji paneladmina.

[dtb]

1. wydaje mi sie że nie można tworzyc funkcji wewnątrz funkcji, gdyż przy ponownym wywołaniu głównej funkcji wyskoczy błąd bo że nie można stwoprzyc drugiej funkcji o takiej nazwie. (kiedys mialem podobny problem)
2. czy nie chodzilło czy przypadkiem o action=ban&id='.$i.' ?
3. zmień $_GET['id'] na isset($_GET['id']) w ifie
4. fopen($ban_db, 'w'): w - otwiera plik tylko do zapisu. jeżeli plik istnieje wszystkie dane zostaną skasowane, jeżeli nie, php spróbuje go utworzyć. czy nie chodzilo ci o dopisywanie do pliku zamiast nadpisywanie (a zamiast w)?

Ten post edytował dtb 15.11.2005, 10:17:18

[nospor]

@dtb tutaj nie jest tworzona funkcja w funkcji, a jest wywoływana funkcja w funkcji. Lekka roznica

@Apo nie napisaleś czy moja skromna uwaga była słuszna co do linku i warunku

[Apo]

@nospor
c oto twojegi linku to z pośpiechu popełniłem błąd. A wszystkie te drobne błędy typu 'w' czy w if-ie isset(&_GET['id']) nie maja większego znaczenia, bo chce aby mi to ogółem chodziło a potem przechodze do szczegółów.
Aha nospor a jak nie przekazuje wartości $ip do funkcji jak wydaje mie sie że jest przekazywane tu : ban($linki[$i]['ip']); w tej drugiej funkcji na końcu w poscie 1.

Ten post edytował Apo 15.11.2005, 16:59:03

[nospor]

Ja nie napisalem ze nie przekazujesz, a ze sie nie odwolujesz (nie wykorzystujesz). To lekka roznica

Podaj jeszcze raz poprawiony kod, poprawione pytanie , bo przez te twoje bledy nie wiem oco ci chodzilo

[Apo]

heh narazie dałem sobie z tym spokój zrobie to potem ;] narazie proste rzeczy
ale jeszczy tylko 1 pytanie.
W pliku z kategoriami serializuje dane. Potem przy pomocy funkcji je odczytuje i tu mam kod:

[PHP] pobierz, plaintext 
<?php
   $linki = show('category_db.txt');
   $linki = unserialize($linki);
		  switch($_POST['cat']) {
		  for($i=0; $i < count($linki); $i++) {
		  case $i:
		  $category = $linki[$i]['kategoria'];
		  break;
			}
	 default:
	 $category = 'inna';
		}
?>
[PHP] pobierz, plaintext 

I odczytuje te kategorie i chce dać na switcha. Ake wyskakuje mi błąd:
Parse error: syntax error, unexpected T_FOR, expecting T_CASE or T_DEFAULT or '}' in g:\usr\apache\httpd\html\category\index.php on line 72

Linia 72:
for($i=0; $i < count($linki); $i++) {

No i moim zdaniem tu nie ma błedu ale coś robie źle

[nospor]

Skladani switch nie pozwala na wkladanie zadnych petli w srodek. musisz recznie robic te case a nie tak jak teraz dynamicznie

No, ewentualnie moglbyc stworzyc dynamiczne string reprezentujacy twoj kod php i potem go evalem potraktowac.

[Apo]

hmm poczytałem o tym eval ale to sie raczej nie nada.
Bo mam takie coś sobie
kategorie są zapisane w pliku i przez panel adminia można je dodawać i usuwać.
No wiec sie zmienia zawartość pliku. I przy dodanio strony do mojego katalogu stron www jest wybór kategorii. I tam właśnie sa generowane te wpisy z pliku w polu select. Po wyborze select 'switch' ustawiał kategorie z pola select do zmiennej $category i zapisywał do pliku:

[PHP] pobierz, plaintext 
<?php
switch($_POST['cat']) {
		case 0:
		$category = 'Webmaster';
		break;
		case 1:
		$category = 'Programowanie';
		break;
		case 2:
		$category = 'Grafika';
		break;
		}
 
		$linki[] = array('kategoria' => $category, 'autor' => $nickzform, 'www' => $wwwzform, 'tytul' => $titlezform, 'opis' => $opiszform, 'ip' => $numerip, 'id' => $numerid);
		$linki = serialize($linki);
		add('baza.txt', $linki);
?>
[PHP] pobierz, plaintext 

Coś takiego.
Może macie propozycje jak to rozwiązać

[Hacker]

A próbowałeś tak??

[PHP] pobierz, plaintext 
<?php
   $linki = show('category_db.txt');
   $linki = unserialize($linki);
   if ($_POST['cat'] > count($linki) || !$linki[$_POST['cat']])
   $kategoria = 'cośtam';
   else
   $kategoria = $linki[$_POST['cat']]['kategoria'];
 
?>
[PHP] pobierz, plaintext