Discussion :

[kanzarih]

Bonjour a tous,

Dans mon projet j'ai besoin de supprimer un enregistrement d'un tableau d'objet alors j'ai fait:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
 
public function deleteNode($nodename){
		if($this->initialized){
			for($i=0;$i<sizeof($this->myxmlnodes);$i++){
				if($this->myxmlnodes[$i]->isMe($nodename)){
					unset($this->myxmlnodes[$i]);
					$this->myxmlnodes = array_values($this->myxmlnodes);
					}
				}
			}
		}

Mais cette fonction ne marche pas ? alors y a t'il une erreur dans mon code?

Merci

[bricecol]

le unset suffit

si le code suivant ne marche pas, alors l'erreur vient de ta fonction isMe (que j'aimerais voir) :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
 
public function deleteNode($node)
{
    if (!$this->initialized)
        return;
 
    $nbNodes = count($this->myxmlnodes);
    for ($i = 0; $i < $nbNodes; $i++)
    {
        if ($this->myxmlnodes[$i]->isMe($node))
        {
            unset($this->myxmlnodes[$i]);
        }
    }
}

[kanzarih]

merci pour la réponse, après des tests j'ai remarquer que la taille du tableau a effectivement diminuer quand je supprime un nœud mais ce qui ne marche pas c'est le parcours du tableau après suppression en utilisant foreach :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
 
$xml = new MyXML('system/os/desktop/xml/deskmenu.xml');
$vect = $xml->getObjectsNodesVector();
$xml->deleteNode('Applications');
echo '<br>'. $xml->getObjectsNodesVectorsize();
foreach($vect as $objnode){
    echo $objnode->formatMe();
    echo $objnode->getIsActive();
    echo $objnode->getParentNode();
    echo $objnode->getIsSection();
}

ce code m'affiche, sachant que mon tableau contient 14 enregistrement au dépars :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
 
13
<Applications section ="true" ></Applications>1menu1
<Programing section ="true" ></Programing>1Applications1
<PHPEditor section ="false" ddd ="sdsd" ></PHPEditor>1Programing
<Graphic section ="true" ></Graphic>1Applications1
<MyPhoto section ="false" ></MyPhoto>1Graphic
<Multimedia section ="true" ></Multimedia>1Applications1
<MyPlayer section ="false" ></MyPlayer>1Multimedia
<Games section ="true" ></Games>1Applications1
<Tetris section ="false" ></Tetris>1Games
<System section ="true" ></System>1menu1
<ControllPanel section ="false" ></ControllPanel>1System
<Descktop section ="false" ></Descktop>1System
<Exit section ="true" ></Exit>1menu1

est ce que l'erreur vien du foreach?

IE : merci pour les modif mais isMe fonctionne correctement voila sont code :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
 
public function isMe($nodename){
	if($this->nodename == $nodename){
		return true;
	}else{
		return false;
	}
}

c'est une fonction de la classe a partir de la quelle je rempli le tableau

[kanzarih]

Ah oui, ca marche merci pour l'aide en fait c'est bête mais j'avais un echo qui m'affiche le formatme de l'enregistrement N°0 et comme par hasard l'élément que j'ai supprimer c'est l'élément N°0!!!!


Merci pour l'aide

[bricecol]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
public function isMe($nodename){
    if($this->nodename == $nodename){
        return true;
    }else{
        return false;
    }
}

malheureux ! jamais deux return dans une fonction :p
et quand tu peux préciser le type d'un paramètre, fais-le.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
public function isMe(string $nodename)
{
    return $this->nodename === $nodename;
}