Podwójne iterowanie po tablicy each(), problem z resetowaniem Opcje

[Zick4]

Mam menu rozwijane posiadające elementy oraz podelementy. U mnie $menu to obiekt klasy menu gdzie jest tablica elementów, a każdy element ma swoją tablicę podelementów. Obie tablice (elementów i podelementów) są indeksowane numerycznie. Jeżeli usuwam jakiś element bądz podelement unsetuje go. Przez to tworzy się tablica z dziurami. Poniższy przykład jest tylko oglądowy co chcę pokazać, bo tak naprawdę to są obiekty bardziej skomplikowane, które mają swoje właściwości i metody.

[PHP] pobierz, plaintext 
<?php
array(
	0=>array(0=>'a', 1=>'b', 2=>'c'),
	1=>array(0=>'a', 1=>'b', 2=>'c'),
	2=>array(0=>'a', 1=>'b', 2=>'c')
);
?>
[PHP] pobierz, plaintext 

Jeżeli teraz usunę element o indeksie 1 to pozostanie mi

[PHP] pobierz, plaintext 
<?php
array(
	0=>array(0=>'a', 1=>'b', 2=>'c'),
	2=>array(0=>'a', 1=>'b', 2=>'c')
  );
?>
[PHP] pobierz, plaintext 

i teraz jeszcze usunę podelement pierwszej tablicy o indeksie 1 to

[PHP] pobierz, plaintext 
<?php
array(
	  0=>array(0=>'a', 2=>'c'),
	  2=>array(0=>'a', 1=>'b', 2=>'c')
	);
?>
[PHP] pobierz, plaintext 

co mi bardzo nie odpowiada, bo zostają dziury więc napisałem funkcję porządkującą, która nie działa, bo według mnie resetowanie nie jest dobrze zrobione. Np. jeżeli zmodyfukiję trochę poniższą funkcję i zrobię tylko jedną pętle np. dla samym elementów, bądź dla samych podelementów to działa, ale jeżeli chcę zrobić obie i przejść przez wszystko to dochodzi tylko do pierwszego elementu zagłębia się do podelementów poprawia je ale już nie przechodzi do następnego elementu.

[PHP] pobierz, plaintext 
<?php
function uporzadkujMenu($menu)
{
	reset ($menu->elementy);
	$tab = array();
	for ($index = 0; list ($klucz, $wartosc) = each($menu->elementy); $index++)
	{
		$tab[$index] = $wartosc;
	  reset($tab[$index]->podelementy);
 
		$tab2 = array();
		for ($i = 0; list ($klucz2, $wartosc2) = each($tab[$index]->podelementy); $i++)
		{
		   $tab2[$i] = $wartosc2;
		}
		$tab[$index] = $tab2;
	}
 
	$menu->elementy = $tab;
	return $menu;
}
?>
[PHP] pobierz, plaintext 

Jakieś pomysły?

[phpion]

Użyj foreach zamiast for lub w for'ze sprawdzaj czy dany index istnieje (isset()).

[krowal]

A tak właściwie to czemu Ci te dziury przeszkadzają, przecież tak czy owak kolejność jest nadal zachowana. Podejrzewam, że wyświetlanie tego zrobiłeś w jakiś dziwny sposób, zamiast użyć normalnie foreach dla każdego z elementów i podelementów ? No chyba że sztuka dla sztuki (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

[Zick4]

Zrobiłem tak ponieważ mam funkcję

[PHP] pobierz, plaintext 
<?php
function znajdzElement($nazwa_elementu) //dostaję stringa i jeżeli znalazłem to zwracam index
	{
		$index = 0;
		foreach ($this->podelementy as $a)
		{
			if ($a->nazwa == $nazwa_elementu)
			{
				return $index;
			}
			$index++;
		}
		return false;
	}
?>
[PHP] pobierz, plaintext 

Bez uporządkowania zwraca zły index.To był pierwszy powód, a drugi to mam funkcję dodajElement()

[PHP] pobierz, plaintext 
<?php
function dodajPodElement($nazwa)
	{
		$nowy_podelement = new PodElementMenu($nazwa);
		$this->podelementy[] = $nowy_podelement;
	}
?>
[PHP] pobierz, plaintext 

Bez uporządkowania dodaje elementy w róznym miejscu raz za tym elementem a raz za innym. A ja bym chciał na końcu. (hmm można by chyba dać array.pusha() i to chyba dodał by na końcu). Oto moja klasa menu

[PHP] pobierz, plaintext 
<?php
class Menu
{
	public $nazwa;
	public $elementy = array();
	function __construct($nazwa_menu)
	{
		$this->nazwa = $nazwa_menu;
	}
	function dodajElement($nazwa)
	{
		$nowy_element = new ElementMenu($nazwa);
		$this->elementy[] = $nowy_element;
 
	}
 
 
	function znajdzElement($element)
	{
		$index = 0;
		foreach ($this->elementy as $a)
		{
			if ($a->nazwa == $element)
			{
				return $index;
			}
			$index++;
		}
		return false;
	}
}
 
class ElementMenu extends PodElementMenu
{
	public $nazwa;
	public $sciezka;
	public $podelementy = array();
	function __construct($nazwa_elementu, $sciezka_do_elementu=null)
	{
		$this->nazwa = $nazwa_elementu;
		$this->sciezka = $sciezka_do_elementu;
	}
	function edytuj($nazwa_elementu, $sciezka_do_elementu=null)
	{
		$this->nazwa = $nazwa_elementu;
		if ($sciezka_do_elementu != null)
			 $this->sciezka = $sciezka_do_elementu;
	}
	function dodajPodElement($nazwa)
	{
		$nowy_podelement = new PodElementMenu($nazwa);
		$this->podelementy[] = $nowy_podelement;
	}
 
	//funkcja dostaje stringa i szuka podelementu o takiej nazwie  
	function znajdzPodElement($nazwa_elementu)
	{
		$index = 0;
		foreach ($this->podelementy as $a)
		{
			if ($a->nazwa == $nazwa_elementu)
			{
				return $index;
			}
			$index++;
		}
		return false;
	}
}
?>
[PHP] pobierz, plaintext 

[jang]

[PHP] pobierz, plaintext 
<?php
$menu = array(
	0=>array(0=>'a', 1=>'b', 2=>'c'),
	2=>array(0=>'a', 2=>'c'),
	5=>array(1=>'a', 3=>'b', 7=>'c'),
  );
var_export ($menu);
echo '<br /><br />';
 
function uporzadkujMenu($menu)
{
	$tab = array();
	for ($index = 0; list ($klucz, $wartosc) = each($menu); $index++)
	{
		$tab[$index] = $wartosc;
 
		$tab2 = array();
		for ($i = 0; list ($klucz2, $wartosc2) = each($tab[$index]); $i++)
		{
		   $tab2[$i] = $wartosc2;
		}
		$tab[$index] = $tab2;
	}
 
	$menu = $tab;
	return $menu;
}
 
$menu = uporzadkujMenu($menu);
var_export ($menu);
?>
[PHP] pobierz, plaintext 

Wynik:

[HTML] pobierz, plaintext 
array ( 0 => array ( 0 => 'a', 1 => 'b', 2 => 'c', ), 2 => array ( 0 => 'a', 2 => 'c', ), 5 => array ( 1 => 'a', 3 => 'b', 7 => 'c', ), )
 
array ( 0 => array ( 0 => 'a', 1 => 'b', 2 => 'c', ), 1 => array ( 0 => 'a', 1 => 'c', ), 2 => array ( 0 => 'a', 1 => 'b', 2 => 'c', ), )
[HTML] pobierz, plaintext 

[Zick4]

Bez uporządkowania dodaje elementy w róznym miejscu raz za tym elementem a raz za innym. A ja bym chciał na końcu. (hmm można by chyba dać array.pusha() i to chyba dodał by na końcu). Oto moja klasa menu

Sprawdziłem i jednak bzdury gadam bo i pushem i $tab[] dodaje na końcu (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Bez uporządkowania zwraca zły index.To był pierwszy powód, a drugi to mam funkcję dodajElement()

[PHP] pobierz, plaintext 
<?php
function znajdzElement($element)
	{
		$index = 0;
		foreach ($this->elementy as $a)
		{
			if ($a->nazwa == $element)
			{
				return $index;
			}
			$index++;
		}
		return false;
	}
?>
[PHP] pobierz, plaintext 

Zmieniłem na

[PHP] pobierz, plaintext 
<?php
function znajdzElement($element)
{
 
	while (list ($klucz, $wartosc) = each($this->elementy))
		if ($wartosc->nazwa == $element)
			return $klucz;
	return false;
}
?>
[PHP] pobierz, plaintext 

i teraz działa zwracanie dobrego indeksu. Teraz tak tylko z ciekawości się zastanawiam, jak zrobić aby iterować po tablicy wielowymiarowej z użyciej each

Jang świetnie, że Ci chodzi. Mi niestety nie. Narazie mam usuwanie bez uporządkowania , ale warto coś z tym zrobić więc jeszcze podrążę temat. Moja klasa:

[PHP] pobierz, plaintext 
<?php
class Menu
{
	public $elementy = array(); //w środku obiekty klasy ElementMenu
	...
}
class ElementMenu extends ElementPodMenu
  {
	  public $podelementy = array();//w środku obiekty klasy PodElementMenu
	  ...
  }
class ElementPodMenu
	{
		...
	}
?>
[PHP] pobierz, plaintext 

I dałem do nich taką funkcję, która nie działa

[PHP] pobierz, plaintext 
<?php
function uporzadkujMenu(&$menu)
{
	$elementy = array();
	for ($index = 0; list ($klucz, $wartosc) = each($menu->elementy); $index++)
	{
		$elementy[$index] = $wartosc;
		$podelementy = array();
		for ($i = 0; list ($klucz2, $wartosc2) = each($elementy[$index]->podelementy); $i++)
		{
		   $podelementy[$i] = $wartosc2;
		}
		$elementy[$index]->podelementy = $podelementy;
	}
	$menu->elementy = $elementy;
}
uporzadkujMenu($menu);
?>
[PHP] pobierz, plaintext