Discussion :

[Yokooo]

Bonjour, je possède une classe Statistique dans laquelle je calcule la moyenne glissante et la variance d'un tableau nommé "data". Le tableau data en question est dans une autre classe.

Le but ensuite va être de pouvoir afficher ma moyenne et ma variance a partir de l'autre classe.

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
20
21
22
23
24
25
26
27
28
29
30
31
package testjedf;
 
import java.util.ArrayList;
 
public class Statistique {
 
    private ArrayList<short[]> data=null;
 
    private static Statistique moyenne(ArrayList<short[]> data) {
 
        System.out.println("Moyenne");
        // Calcul Moyenne glissante
 
        throw new RuntimeException();
    }
 
    private static Statistique variance(ArrayList<short[]> data) {
 
        System.out.println("Variance");
        // Calcul Variance
 
        throw new RuntimeException();
    }
 
 
    public static void main(String[] args) {
        System.out.println("ok");
        Statistique maMoyenne=Statistique.moyenne(data);
        Statistique maVariance=Statistique.variance(data);
    }  
}

Merci.

[tchize_]

Quelle est la question?

[Yokooo]

Oups.. désolé de pas avoir été clair, en fait je signalais que cela ne marchais pas j'ai une erreur au niveau de :

Code :

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

1
2
       Statistique maMoyenne=Statistique.moyenne(data);
        Statistique maVariance=Statistique.variance(data);

Il est marqué "non-static variable data cannot be referenced from a static context"

Désolé.

[tromaltsec]

Bonjour juste une petite question ta variable data tu la remplis où ??

[Yokooo]

Dans une autre classe, il fonctionne correctement car j'ai fait dans une autre classe un array qui fonctionne.

[darrylsite]

Le probleme vient du fait que la propriete data de la classe statistique n'est pas static.
Pour supprimer le probleme, il faut faire la declaration comme suit :

Code :

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

1
2
 
private static ArrayList<short[]> data=null;

[tchize_]

data n'est pas statique. Ca veux dire que c'est une champ d'instance, il est différent pour chaque instance de ta classe Statistique. Par conséquent, il n'est aussi accessible que depuis les instances de Statistique. Hors, dans ton main, tu n'as aucune instance courant, tu ne peux donc pas déterminer la valeur de cette variable. Ce qui se résume assez bien par le message d'erreur "on ne peux pas accéder à un champ non statique depuis un contexte statique".

[Yokooo]

Merci beaucoup pour ces explications ! Je commence à comprendre...

Après avoir bien cherché je n'ai pas réussi à trouver ma deuxième erreur qui se situe au niveau du

Code :

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

throw new RuntimeException();

L'erreur est : Exception in thread "main" java.lang.RuntimeException

Alors que j'ai vu qu'il n'était pas obligé de faire un throws dans le main avec ce type de fonction

[tchize_]

Après avoir bien cherché je n'ai pas réussi à trouver ma deuxième erreur qui se situe au niveau du

Code :

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

throw new RuntimeException();

L'erreur est : Exception in thread "main" java.lang.RuntimeException

C'est pas une erreur c'est le comportement voulu, au vu du code de ta fonction. Elle renvoie systématiquement un exception (probablement pour signifier "je suis pas encore implémentée, implémentez moi avant de pouvoir utiliser ce logiciel")

[BugFactory]

Code :

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

throw new RuntimeException();

demande à Java de lancer une exception. Pourquoi avez vous mis cette ligne?

[Yokooo]

J'ai mis cette ligne car sinon j'ai une erreur au niveau du

Code :

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

private static Statistique moyenne(ArrayList<short[]> data) {

Il me demande de mettre quelque chose en retour mais je n'ai pas trouvé comment renvoyer le résultat.

[tromaltsec]

J'ai mis cette ligne car sinon j'ai une erreur au niveau du

Code :

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

private static Statistique moyenne(ArrayList<short[]> data) {

Il me demande de mettre quelque chose en retour mais je n'ai pas trouvé comment renvoyer le résultat.

ben il te demande de mettre un objet Statistique en retour normal car c'est le but de ta fonction.

Si ta fonction ne doit rien renvoyer il faut remplacer Statistique par void

[tchize_]

Et si tu n'a juste rien à renvoyer "pour le moment", tu peux te contenter d'un

Code :

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

return null;

à la place du throw.

[BugFactory]

Code :

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

private static Statistique moyenne(ArrayList<short[]> data) {

Vous déclarez ici une fonction nommée moyenne qui renvoie un objet de type "Statistique". Forcément, il faut ... renvoyer un objet de type statistique, via la commande return.

Ajouter

Code :

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

throw new RuntimeException();

fait certes taire le compilateur, mais vous ne faites qu'ajouter une erreur qui masque la précédente.

En règle générale, la seule bonne méthode pour faire taire le compilateur quand il râle, est de corriger l'erreur. Le coup du RuntimeException est une très mauvaise idée.

[Edit]Grillé.[/Edit]

[Yokooo]

Ah je n'y comprend plus rien

En fait je souhaite qu'il renvoie le résultat et que je puisse le récupérer dans le main

Excusez moi pour ma médiocrité en java...

[tchize_]

c'est le mot clé "return" pour renvoyer un résultat. Par contre, pourquoi moyenne renvoie un objet Statistique? D'habitude, une moyenne c'est juste un chiffre, donc un double aurait probablement fait l'affaire....

[Yokooo]

Si je n'instancie pas l'objet donc que j'utilise pas le static est ce que se serait plus facile ? Je ne peux pas passer par une autre méthode ?

Le but étant de calculer la moyenne glissante d'un tableau et sa variance.

[tchize_]

le static n'a rien a voir avec le return.

[Yokooo]

Bon je viens de retoucher à tout afin d'avoir quelque chose de plus basique :

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
20
21
22
23
24
25
26
package testjedf;
 
import java.util.ArrayList;
 
public class Statistique {
 
    private ArrayList<short[]> data=null;
    private double moy;
    private double var;
 
    public Statistique(ArrayList<short[]> data){
        this.data = data;
 
        // Exemple de calculs
        moy = 7 + 2;
        var = 100 - 50;
    }
 
    public void affiche() {
 
        System.out.println("Moyenne =" + moy);
        System.out.println("Moyenne =" + var);
 
    }
 
}

Voila, pourriez vous me dire comment appeler cette classe dans mon autre classe car j'ai utilisé

Code :

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

Statistique a = new Statistique(tabData);

dans l'autre classe mais ça n'a pas l'air de fonctionner.

Merci

[tchize_]

mais ça n'a pas l'air de fonctionner.

Tu pourrais développer un peu plus?