Discussion :

[jeanfils]

Bonjour,


Je ne parviens pas à afficher les bonnes valeurs pour la méthode suivante:

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
//DEBUT CALCULER_MINRESTE
		public static double[] calculerMinReste(double...prix) {
			double min = prix[0];
			double reste=0.0;
			double minReste []= new double[2];
			for (int i=0;i<prix.length;i++) {
				reste = reste + prix[i];
				if (prix[i]< min){
					min = prix[i];
					}
				minReste[0]=min;
				minReste[1] = reste - min;		
			}
 
			return minReste;
		}
 
//FIN CALCULER_MINRESTE

Lorsque j'appelle la méthode avec le code suivant (extrait de code):

Code :

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

1
2
3
calculerMinReste(100.0,200.0,300.0);
			System.out.println("Min= " +  minReste[0] );
			System.out.println("Reste= " +  minReste[1] );

J'obtiens:

Min= 0.0
Reste= 0.0

Pourriez-vous me dire ce qui cloche?

[Noctis]

Tu retournes bien un tableau dans la methode calculerMinReste mais tu ne recuperes pas celui-ci lors de l'execution.

Code :

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

1
2
3
4
5
6
public static void main(String[] args) {
 
		double minReste[] = calculerMinReste(100.0, 200.0, 300.0);
		System.out.println("Min= " + minReste[0]);
		System.out.println("Reste= " + minReste[1]);
	}

[jeanfils]

Merci Noctis!

Je revenais pour dire que j'ai trouvé!
C'est la deuxième fois que je fais ce type d'erreur!
Je serai plus vigilant la prochaine fois!

Merci encore!

[emixam16]

Pour compléter la réponse de Noctis, faire des fonctions qui calculent plusieurs choses qui n'ont à priori aucun rapport est uen très mauvaise pratique et rend ton code non réutilisable (par exemple que faire si par la suite tu ne veux ne récupérer que le minimum d'un tableau ?) .

Je te conseille donc de séparer ton code en 2 fonctions:

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
 
private static double min(double... values) {
	double min = values[0];
	for(double value : values) if(value<min) min = value;
	return min;
}
private static double sum(double... values) {
	double sum = 0;
	for(double value : values) sum += value;
	return sum;
}
public static void main(String[] args)
{
	double[] values = {...} 
	double min = min(values);
	double reste = sum(values) - min;
}

Remarque au passage que ce code est plus simple à écrire et à comprendre (donc moins sujet au bugs)

[jeanfils]

OK c'est noté et pertinent!
Merci!

[adiGuba]

Salut,

Pour compléter la réponse de Noctis, faire des fonctions qui calculent plusieurs choses qui n'ont à priori aucun rapport est uen très mauvaise pratique et rend ton code non réutilisable (par exemple que faire si par la suite tu ne veux ne récupérer que le minimum d'un tableau ?) .

Dans ce cas précis on a quand même deux éléments très proche : on calcule des "infos" sur une tableau de valeur.
Ca peut être pratique d'avoir deux méthodes distinctes, mais cela oblige à parcourir deux fois les données...


Par contre le code peut être amélioré (for-each pour plus de lisibilité, création du tableau en fin de méthode, éviter de recalculer tout à chaque itération ce qui est inutile) :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
  public static double[] calculerMinReste(double...prix) {
	double min = prix[0];
	double sum = 0.0;
 
    for (double d : prix) {
     	sum += d;
      	if (d<min) {
        	min = d;
        }
    }
 
    return new double[] { min, sum-min };
  }

Au passage perso ce qui me gêne le plus c'est de retourner un double[].
Il serait nettement préférable d'utiliser un objet avec deux propriétés "min" et "reste" qui serait plus lisible et plus pratique.


Enfin avec Java 8 on peut faire cela très simplement avec les Streams :

Code :

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

1
2
3
  public static java.util.DoubleSummaryStatistics calculerMinReste(double...prix) {
	return Arrays.stream(prix).summaryStatistics();
  }

DoubleSummaryStatistics possède (entre autres) deux accesseurs getMin() et getSum()...


a++

[jeanfils]

Merci adiGuba

Etant tout juste débutant (moins d'une semaine de codage)
Est-il pertinent de "sauter les étapes" avec " les Streams".
Je vois bien que c'est plus simple mais j'essaye de suivre des cours en lignes.

[joel.drigo]

Salut,

Etant tout juste débutant (moins d'une semaine de codage)
Est-il pertinent de "sauter les étapes" avec " les Streams".

Je dirais que ça dépend de ce que tu veux apprendre. Il est utile (entendre indispensable ) d'apprendre les manipulations de tableaux, ainsi que le type de traitement que tu fais (calcul du min, somme des éléments d'un ensemble ou liste, etc), de manière générale. Cela fait partie du béaba, quelque soit le langage.

En revanche, connaître les Stream et leur manipulation fait partie de l'apprentissage de Java (ou des autres langages qui offrent le même type de structure). Même si ce n'est pas exactement du niveau débutant, une fois que tu as assimilé les parcours de tableau, passer au Stream est nécessaire pour ensuite écrire des du code plus concis et plus flexible dans un programme.

C'est comme pour les tris. Savoir coder les algorithmes de tri standard est un bon exercice d'apprentissage général. Mais au final, dans une application Java, il vaudra mieux savoir utiliser Comparator/Comparable et Arrays/Collections/sort et consort.