Discussion :

[§Gama§]

Bonjour,
mon pb du jour, qui n'est pas bloquant, pcq j'ai trouvé une solution ...mais qui ne me semble pas propre (perte de temps de calcul inutile).
est le suivant... je dois rajouter

Code :

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

 $result['players'][$key] =$array;

pour conserver les modifications que j'ai faites dans array, alors que je pensais qu'elles se faisaient également dans le niveau du dessus... j'ai tenté de rajouté un passage par référence avec un & devant le array, mais ca n'a pas l'air d'être cela...
y a t'il moyen d'éviter cette dernière ligne dans la déclaration du foreach? je rajoute la dimension 'board' qui n'existait pas.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
foreach( $result['players'] as $key =>$array) { 	
		$this->arrayColors[$array['colorid']]=$key;
		$array['board']['nbsat'] = 0;	
		$array['board']['eqp'] = array('NONE','NONE','NONE');	
  		$array['board'][2] = 0;
  		$array['board'][3] = 0;
		$array['board'][5] = 0;
		$array['board']['vobj'] = array();	
  		$array['board']['vsen'] = array();	
		$array['board']['vspy'] = array();	
		$this->log($array);
		$result['players'][$key] =$array; 	
  	}

[mathieu]

si le tableau "board" est le même pour tous les joueurs, tu pourrais par exemple l'extraire de la boucle

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
17
18
19
 
$arrayBoard = array();
 
$arrayBoard['nbsat'] = 0;
$arrayBoard['eqp'] = array('NONE','NONE','NONE');
$arrayBoard[2] = 0;
$arrayBoard[3] = 0;
$arrayBoard[5] = 0;
$arrayBoard['vobj'] = array();
$arrayBoard['vsen'] = array();
$arrayBoard['vspy'] = array();
 
 
foreach($result['players'] as $key => $array) {
	$this->arrayColors[$array['colorid']] = $key;
 
	$result['players'][$key]["board"] = $arrayBoard;
	$this->log($result['players'][$key]);
}

mais sinon je ne vois pas d'autre optimisation possible juste pour ce code.

[§Gama§]

Ce n'est pas le cas
mais je trouve cela curieux que ca ne soit pas possible d'enlever

Code :

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

1
2
 
$result['players'][$key]["board"] = $arrayBoard;

en changeant la symbolique d'entrée pour que les opérations soient directement effectuées... un peu comme ci on utilisait le "with" d'autres langages.

[Tsilefy]

Code :

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

foreach( $result['players'] as $key => &$array) {

te permets de faire ce que tu veux. Teste sur un tableau plus simple et tu verras.

[§Gama§]

malheureusement pour ce cas là, ce n'est pas le cas:

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
17
18
 
$sql = "SELECT player_id id, player_name name, player_color color,player_colorid colorid, player_score score FROM player ";
        $result['players'] = self::getCollectionFromDb( $sql );
	//completed with extra datas deducted form cards position (here initiate defaults)
	foreach( $result['players'] as $key =>$array) { 	
		$this->arrayColors[$array['colorid']]=$key;
		$array['board']['nbsat'] = 0;	
		$array['board']['eqp'] = array('NONE','NONE','NONE');	
  		$array['board'][2] = 0;
  		$array['board'][3] = 0;
		$array['board'][5] = 0;
		$array['board']['vobj'] = array();	// victory objectives	
  		$array['board']['vsen'] = array();	// victory sentinels
		$array['board']['vspy'] = array();	// victory opponents spy
		$this->log('ICI',$result['players']);
		$result['players'][$key] =$array; 
		$this->log('LA',$result['players']);
  	}

Les deux logs sont différents, le premier ne contient pas les modifs de Array.. le second les contient.

[Tsilefy]

Tu as oublié l'esperluette devant $array.

Et si tu l'as mis dans tes tests, bien évidemment tu ne vas pas logger cette variable en cours de boucle, ça te donnera une variable incomplète (et des redondances à chaque boucle). Vérifie la valeur de la variable après la boucle.

[§Gama§]

tu veux dire faire comme cela?:

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
 
foreach( $result['players'] as $key =>&$array) { 	
		$this->arrayColors[$array['colorid']]=$key;
		$array['board']['nbsat'] = 0;	
		$array['board']['eqp'] = array('NONE','NONE','NONE');	
  		$array['board'][2] = 0;
  		$array['board'][3] = 0;
		$array['board'][5] = 0;
		$array['board']['vobj'] = array();	// victory objectives	
  		$array['board']['vsen'] = array();	// victory sentinels
		$array['board']['vspy'] = array();	// victory opponents spy
		//$this->log('ICI',$result['players']);
		//$result['players'][$key] =$array; 
		//$this->log('LA',$result['players']);
  	}
	$this->log('OULA',$result['players']);

Si c'est le cas... par exemple le champs 'nbsat' n'existe pas.... la phrase $result['players'][$key] =$array; est nécessaire.
(merci pour ton aide déjà )

[sabotage]

Quelle est la structure de $result['players'] ?
En testant un retour classique de base de donnée, j'ai bien le résultat attendu.

[§Gama§]

J'ai tort.... "le &array" a bien fait son effet, j'ai pu supprimer le result=""
Le fichier n'avait pas encore eu le temps d'être mis à jour quand j'ai retesté!

merci!!!