Débutant recherche de l'aide

Version imprimable

Bonjour à tous

Je débute et je suis confronté a un problème surement de base !
voici le code

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
package com.moi.package1;
 
import java.lang.Double;
 
public class Stat {
	static double[] tb;
 
	  double somme() {
    	double r = 0;
    	for (int i = 0; i <= tb.length; i++){
    		r = r + tb[i];
    	};
    	return r;
    }
	  double maximum() {
	        double r = tb[0];
	        for (int i = 1; i <= tb.length; i++){
	        	if (tb[i] > r) {r = tb[i];}
	        }
	        return r;
	    }
 
	 double minimum() {
	    	double r = tb[0];
	        for (int i = 1; i <= tb.length; i++){
	        	if (tb[i]< r){r = tb[i];}
	        }
	        return r;
	     } 	
 
 
	public static void main(String[] args) {
		// TODO Auto-generated method stub
		for (int i = 0;  i <= (args.length); i++){
		  tb[i]= Double.parseDouble(args[i]);
		}
		System.out.println(somme()+" "+maximum()+" "+minimum());

à la compilation j'ai une erreur, dans le main l'évaluation de somme(), maximun(), et minimum() ne s'effectue pas !

quelqu'un pourrais t'il me dire pourquoi ???

MERCI

20/10/2010, 18h39
hugo123

Les méthodes somme, maximum et minimum ne sont pas statiques mais tu les appelles dans ton main qui est statique.

Tu ne peux pas appeler des méthodes non statiques dans une méthode statique, tu dois instancier ton objet Stat auparavant ou bien passer tes méthodes en statiques.

je te conseille d'ailleurs d'utiliser un IDE (genre eclipse) qui te détecte tout de suite ce type d'erreur.
20/10/2010, 18h41
sinok

il te faut déclarer tes méthodes somme, maximum et minimum en tant que publiques si tu veux pouvoir les appeler en dehors de ta classe

Comme dit précédemment, Soit tu mets tous en statique, soit tu mets tous en public et tu utilise une instance de Stat :

Code:

1
2
3
4
5
6
7
8
9
10
11
public double somme() {//...}
public double maximum(){//...}
//...
 
public static void main(String[] args) {
		// TODO Auto-generated method stub
		for (int i = 0;  i <= (args.length); i++){
		  tb[i]= Double.parseDouble(args[i]);
		}
		Stat stat = new Stat();
		System.out.println(stat.somme()+" "+stat.maximum()+" "+stat.minimum());

Merci

Bonjour,

Tout d'abord merci pour vos réponses....
Bon j'ai suivi vos conseils....Mais voilà !!! ça ne fonctionne toujours pas !
J'ai exécuté cette classe dans eclipse mais les erreurs ne me guident guère dans l'orientation vers laquelle me diriger....
Voici mon nouveau code :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 package Exemples; import java.lang.Double; public class Stat { public double[] tb; public double somme() { double r = 0; for (int i = 0; i <= tb.length; i++){ r = r + tb[i]; }; return r; } public double maximum() { double r = tb[0]; for (int i = 1; i <= tb.length; i++){ if (tb[i] > r) {r = tb[i];} } return r; } public double minimum() { double r = tb[0]; for (int i = 1; i <= tb.length; i++){ if (tb[i]< r){r = tb[i];} } return r; } public static void main(String[] args) { Stat stat = new Stat(); for (int i = 0; i <= (args.length); i++){ stat.tb[i]= Double.parseDouble(args[i]); } System.out.println(stat.somme()+" "+stat.maximum()+" "+stat.minimum()); } }
l'avez vous executé de votre côté ? Peut-être faut-il faire 2 classes ?

MERCI

21/10/2010, 10h38
Eusebius

Citation:

Envoyé par sb35

les erreurs ne me guident guère

Et si tu nous les donnais (avec les numéros de lignes), au lieu de nous faire deviner ?
21/10/2010, 10h40
professeur shadoko
euh ... dejà commencer avec la condition d'arrêt de la boucle for!
Code:

1 2 3 for(int ix = 0 ; ix < tb.length; ix++) { }
inférieur pas inférieur ou égal!
sinon tu plantes à l'exécution ....

erreur

Bon j'ai modifié tous les for avec la nouvelle condition

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
import java.lang.Double;
 
public class Stat {
	public double[] tb;
 
public double somme() {
	double r = 0;
	for (int i = 0; i < tb.length; i++){
		r = r + tb[i];
	};
	return r;
}
 
public double maximum() {
	double r = tb[0];
	for (int i = 1; i < tb.length; i++){
		if (tb[i] > r) {r = tb[i];}
	}
	return r;
}
 
public double minimum() {
	double r = tb[0];
	for (int i = 1; i < tb.length; i++){
		if (tb[i]< r){r = tb[i];}
	}
	return r;
}
 
public static void main(String[] args) {
	Stat stat = new Stat();
	for (int i = 0; i < (args.length); i++){
		stat.tb[i]= Double.parseDouble(args[i]);
	}
	System.out.println(stat.somme()+" "+stat.maximum()+" "+stat.minimum());
}
 
}

Voici l'erreur dans la console :

Exception in thread "main" java.lang.NullPointerException
at com.moi.package1.Stat.main(Stat.java:35)

et en cliquant sur la ligne ...c'est là que ça plante !!!

stat.tb[i]= Double.parseDouble(args[i]);

stat.tb n'est pas initialisé, donc tu ne peux pas accéder à son contenu comme tu le fait. Il fau d'abord créer ce tableau (avec un new, comme pour les objets) avant de pouvoir aller tripatouiller dedans:
Code:

1 2 3 4 5 6 7 8 public static void main(String[] args) { Stat stat = new Stat(); stat.tb = new double[args.length]; // moche, on devrais faire ça dans le constructeur mais j'ai la flemme là for (int i = 0; i < (args.length); i++){ stat.tb[i]= Double.parseDouble(args[i]); } System.out.println(stat.somme()+" "+stat.maximum()+" "+stat.minimum()); }

21/10/2010, 11h46
slim_java

salut,
tu n'a pas alloué de l'espace mémoire pour ton tableau : double[] tb

tu dois faire pr exemple :

Code:

double[] tb = new double[10];
21/10/2010, 11h48
sinok

Pourrais tu nous mettre la trace entière de l'exception?

ça marche !

Citation:
Envoyé par tchize_
stat.tb n'est pas initialisé, donc tu ne peux pas accéder à son contenu comme tu le fait. Il fau d'abord créer ce tableau (avec un new, comme pour les objets) avant de pouvoir aller tripatouiller dedans:
Code:

1 2 3 4 5 6 7 8 public static void main(String[] args) { Stat stat = new Stat(); stat.tb = new double[args.length]; // moche, on devrais faire ça dans le constructeur mais j'ai la flemme là for (int i = 0; i < (args.length); i++){ stat.tb[i]= Double.parseDouble(args[i]); } System.out.println(stat.somme()+" "+stat.maximum()+" "+stat.minimum()); }
effectivement ça marche....Merci !
Par contre tu dit :

// moche, on devrais faire ça dans le constructeur mais j'ai la flemme là

ça se traduit par quoi ?...J'aime bien les choses bien faite! ...Non c'est pour bien comprendre !

21/10/2010, 12h03
slim_java

Citation:

Envoyé par sb35

effectivement ça marche....Merci !
Par contre tu dit :

// moche, on devrais faire ça dans le constructeur mais j'ai la flemme là

dans le constructeur de ta classe Stat
21/10/2010, 12h10
sb35

.....Dans le constructeur

Ok ! ça j'ai compris, mais comment ?
21/10/2010, 12h18
slim_java
Citation:

Envoyé par sb35

Ok ! ça j'ai compris, mais comment ?
Code:

1 2 3 4 5 6 7 public Stat() { tb = new double[nb]; }
ou bien, directement lors de la déclaration du tableau tb :
Code:

1 2 public class Stat { public double[] tb = new double[nb]
puisque tu utilise un constructeur par défaut

nb = nombre de paramètre à fournir à ton programme dans args.
Si ce nombre est variable, tu peux donner une valeur maximale.

Constructeur

Merci

ça marche dans les 2 cas !

Au bout du compte qu'est ce qui est le mieux ?
Car dans le cas :
Code:

1 2 public class Stat { public double[] tb = new double[100];
sans savoir vraiment combien on va avoir de paramètres !!!

ou
Code:

1 2 public class Stat { public double[] tb
et
Code:

1 2 3 Stat stat = new Stat(); stat.tb = new double[args.length]; for (int i = 0; i < (args.length); i++){

21/10/2010, 16h42
slim_java

Pour plus d'informations sur le sujet, je te recommande la lecture de ces deux parties du cour penser en Java :

http://bruce-eckel.developpez.com/li...?chap=5&page=3

http://bruce-eckel.developpez.com/li...&page=4#04.023
21/10/2010, 17h08
tchize_

Citation:

Envoyé par sb35

Merci

ça marche dans les 2 cas !

Au bout du compte qu'est ce qui est le mieux ?

Aucun des deux. Soit tu utilise un List plutot qu'un tableau, comem ca la taille n'a pas d'importance, soit tu passe en paramètre au constructeur la taille que devra avoir le tableau et tu l'initialise dans le constructeur.