Discussion :

[pierrot10]

Bonjour à tous,

J'ai trouvé ce code et j'ai du mal a comprende ce que fais le dernier return

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
		if (!parent::install() OR		
			!$this->registerHook('rightColumn') OR
			!$this->registerHook('header'))
 
			return (false);
 
		$this->setDefaults();
 
		if (!$this->saveSettings()) return false;
 
		return true;

Je pense que c'est égal à:

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
 
		if (!parent::install() OR		
			!$this->registerHook('rightColumn') OR
			!$this->registerHook('header')){
 
			return (false);
}
 
		$this->setDefaults();
 
		if (!$this->saveSettings()){ return false;
}else{
 
		return true;
}

Je suis un peu confu car il se trouve à la même hauteur que le if alors qu'il devarit etre tabulé sur la droite, non?

[rawsrc]

Bonsoir,

tu as vu juste, ta reprise du code est exacte.
Voilà le problème lorsque l'auteur ne tient aucun compte des bonnes pratiques en matière de codage et de présentation.
Pour le relire c'est mariole. If faut toujours veiller à préserver la lisbilité du code source sinon bonjour le temps perdu. Avec la pratique, des bons réflexes s'installent et facilitent la vie.
Au début, j'ai commencé par relire mon propre code 2 ou 3 jours après l'avoir pondu, histoire de voir si c'était compréhensible et surtout à quelle vitesse je récupérais la logique. Le temps passé à me maudire m'a poussé à insérer des commentaires, à nommer mes variables de manière explicite et à présenter soigneusement mon code.

Voici le code présenté de manière lisible au premier coup d'oeil:

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
<?php
 
if ( ! (parent::install() && $this->registerHook('rightColumn') && $this->registerHook('header'))) {
   return false;
}
 
$this->setDefaults();
 
if ( ! $this->saveSettings()) {
   return false;  
}
 
return true;
 
?>

Un rarccourci quand tu as un test de ce genre

Code :

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

1
2
3
4
5
6
if (test) {
   return false;
}
else {
   return true;
}

Le else est optionnel vu que le résultat de la première partie du code te provoque un return.

[pierrot10]

Salut, merci pour ta réponse,

Mais qu'es-ce le dernier retun true?

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
16
 
<?php
 
if ( ! (parent::install() && $this->registerHook('rightColumn') && $this->registerHook('header'))) {
   return false;
}
 
$this->setDefaults();
 
if ( ! $this->saveSettings()) {
   return false;  
}
 
return true; 
 
?>

Il n'est pas perdu?

[rawsrc]

Mais qu'es-ce le dernier retun true?
Il n'est pas perdu?

Non du tout.
Si les étapes en ligne 4 se passent bien, le code passe à la ligne 8 puis en ligne 10 si la sauvegarde des paramètres se passe bien, on passe à la ligne 14 => on renvoie true.

[pierrot10]

Ben oui mais je ne comprends toujours pas,
si on passe à la ligne 14 par qu'a la ligne 10, tout ce passe bien, à quoi sert le return true
Ceci revient au même, non?

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
<?php
 
if ( ! (parent::install() && $this->registerHook('rightColumn') && $this->registerHook('header'))) {
 
}
 
$this->setDefaults();
 
if ( ! $this->saveSettings()) {
 
}
 
 
 
?>

[pierrot10]

Ce code me parrait farfelu

[rawsrc]

Pour supprimer le dernier return false/true il suffit de faire :

Code :

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

return $this->saveSettings();

J'ai pas voulu aller trop vite mais bon

[rawsrc]

Voilà comment je l'aurais présenté ce bout de code :

Code :

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

1
2
3
4
5
6
7
8
9
<?php
 
if (parent::install() && $this->registerHook('rightColumn') && $this->registerHook('header')) {
   $this->setDefaults();
   return $this->saveSettings();   
}
return false;
 
?>

Plus clair ?

[pierrot10]

Ben non, et ca me semble tellement basic.

Pourquoi pas faire ca alors.

Code :

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

1
2
3
4
5
6
7
8
 
<?php
 
if (parent::install() && $this->registerHook('rightColumn') && $this->registerHook('header')) {
   $this->setDefaults();
   $this->saveSettings();   
} 
?>

le return me pertube....

[rawsrc]

Parce que la fonction doit renvoyer un boolean.
Ta version ne renvoie rien c'est-à-dire null c'est-à-dire toujours false.
Du coup la fonction appelante ne sait pas si ce bout de code s'est bien déroulé ou pas... En ne retournant rien, elle considerera toujours que ce bout de code s'est mal déroulé.
N'oublies pas que la fonction appelante peut réorienter le traitement en fonction du résultat renvoyé par ce bout de code.

[pierrot10]

Ok super merci pour toutes tes explications

[pierrot10]

Re bonjour,
Je m'excuse pour mes questions de débutant, mais je me demande si je n'ai pas repris un code qui n'est pas propre. Et dans le doute de bien refaire ( et apprendre aussi) j'ai maimerais relancer quelques question différentes.

J'ai ceci

Code :

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

1
2
 
usort($this->tags, array($this, 'natorder'));

J'ai regardé la fonction usort et je devine que je peux aussi faire comme ceci
(en sachant que la fonction natorder existe)

Code :

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

usort($this->tags, 'natorder');

Mais pourquoi le gars mais un array($this,'natorder')?

Seonde question
J'ai ceci:

Code :

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

1
2
3
4
5
6
 
foreach ((array) $this->attributes as $key => $list){	
 
				uasort($this->attributes[$key]['values'], array($this, 'natorder'));
 
			}

Encore uen fois, pourquoi le gars met un array. Je pourrais faire comme ceci

Code :

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

1
2
3
4
5
6
 
foreach ($this->attributes as $key => $list){	
 
				uasort($this->attributes[$key]['values'], array($this, 'natorder'));
 
			}

Sachant que $this->attribut est déclaré comme ceci , dans ma class

Code :

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

public $attributes						= array();

Qu'es-ce que j'ai a gagner si je laissait mon freach avec le array()?

Merci pour vos éclaircissement!

[rawsrc]

Bonjour,

dans ce code : usort($this->tags, array($this, 'natorder')); la partie array($this, 'natorder'); correspond à un callback (fonction de rappel avec un contexte d'instance de classe): regardes ici

Ensuite pour ceci foreach ((array) $this->attributes as $key => $list) le (array) correspond à un cast explicit en type tableau : regardes ici.
Il est clair que si $attributes est typé en tableau, normalement cela est inutile. Mais dans son cas, la visibilité est public, donc il n'est pas aboslument certain que $attributes reste toujours un tableau. Pour s'en assurer, il faudrait réduire sa visibilité protected | private et lui rattacher un accesseur et/ou mutateur :

Code :

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

1
2
public function setAttribute($p) { ... }
public function getAttribute() { ... }