for étendu: Warning d'inutilisation [Résolu]

[rtg57]

Bonjour,

J'ai un tableau à 2 dimensions que je souhaite initialiser à '0'.
Ce tableau est transmis en paramètre d'une méthode dont voici la partie du code qui nous intéresse:

Code :

1
2
3
4
5
6
7
8
9
10
public void chargeDonnees( short[][] tablo_pointage )
{
  /.../
  // Init du tableau:
  for( short ligne[] : tablo_pointage )
    for( short element : ligne )
      element = 0;
 
  /.../
}

Le souci est que le compilateur m'envoie un Warning en indiquant que 'element' n'est pas utilisé. Avez-vous une explication à cela ?

Bien sûr, je pourrais utiliser la propriété length du tableau est faire 2 boucles for imbriquées:

Code :

1
2
3
for( int ligne = tablo_pointage.length; ligne >= 0; ligne-- )
  for( int element = tablo_pointage[ ligne ].length; element >= 0; element-- )
    tablo_pointage[ ligne ][ element ] = 0;

Je trouve quand même que la 1ère version plus élégante !

[adiGuba]

Salut,


Tu ne peux pas modifier les éléments d'un tableau avec le for étendus.
Les valeurs sont simplement stocké dans une variable locale, que tu n'utilise pas (d'où le warning).

Mais bon le plus simple serait d'utiliser Arrays.fill() :

Code :

1
2
for( short ligne[] : tablo_pointage )
	Arrays.fill(ligne, (short)0);

a++

[Lady]

Envoyé par rtg57

Bonjour,

J'ai un tableau à 2 dimensions que je souhaite initialiser à '0'.
Ce tableau est transmis en paramètre d'une méthode dont voici la partie du code qui nous intéresse:

Code :

Le souci est que le compilateur m'envoie un Warning en indiquant que 'element' n'est pas utilisé. Avez-vous une explication à cela ?

Je pense que ce bout de code ne va pas faire ce que tu attends ...
La ligne élément = 0; ne met pas une des case de ton tableau à 0 mais fait juste pointé ton paramètre élément sur l'espace en mémoire qui vaut 0 pour un short ....

Sachant que de base le tableau va être initialisé à 0 quoiqu'il arrive (valeur par défaut des type primitifs numérique)

[rtg57]

Bonjour et merci de m'avoir répondu rapidement,

Envoyé par adiGuba

Tu ne peux pas modifier les éléments d'un tableau avec le for étendus.
Les valeurs sont simplement stocké dans une variable locale, que tu n'utilise pas (d'où le warning).

A mon grand étonnement, cela fonctionne: je modifie bien le tableau fourni par le code appelant.
D'après ce que j'ai cru comprendre, seuls les éléments primitifs sont copiés localement sur la pile. Ce qui fait que la modification de leurs valeurs n'est vue que localement.
Par contre, lorsque ce sont des objets (et les tableaux en font partie), c'est l'adresse de l'objet qui est passée. Du coup, la fonction appelée travail réellement sur l'objet situé à cette adresse.

Votre réponse m'a peut être mis sur la piste : peut être que le compilateur ne saisit pas cette astuce. Les objets ligne[] et element sont créés juste pour la boucle d'initialisation. De ce point de vue, ils sont mis à '0' et jamais utilisés par la suite puisque leur portée est simplement cette boucle. D'où l'avertissement.

En réalité, ces zones mémoires sont ensuite remplies par des valeurs, mais elles sont adressées explicitement, genre

Code :

tablo_pointage[ index1 ][ index2 ] = 36;

En tout cas, merci aussi pour

Code :

Arrays.fill(ligne, (short)0);

Je ne connaissais pas. Et cela ne génère pas de Warning.
En ce qui concerne ce petit désagrément, j'ai aussi trouvé ceci:

Code :

1
2
3
for( short ligne[] : tablo_pointage )
  for( @SuppressWarnings( "unused" ) short element : ligne )
    element = 0;

[adiGuba]

Envoyé par rtg57

A mon grand étonnement, cela fonctionne: je modifie bien le tableau fourni par le code appelant.

Non cela ne fonctionne pas. Je viens de vérifier avec ce code :

Code :

1
2
3
4
5
6
	short[] ligne = { 1, 2, 3, 4, 5, 6 };
 
	for( @SuppressWarnings( "unused" ) short element : ligne )
		element = 0;
 
	System.out.println(Arrays.toString(ligne));

Comme indiqué par Lady, cela vient surement du fait que les tableaux sont initialisé à zéro par défaut...


a++