Discussion :

[LenyEdwarx]

Bonjour à tous,

Le but de mon code est de récupérer les données (id, nom, adresse, tel, longitude, latitude) de défibrilateurs autour d'un individu dont on connait sa longitude et latitude. L'idée est ensuite de comparer les distances entre chacun des defibrilateurs et l'individu pour finalement donner en sortie le nom du défibrilateur le plus proche.

Néanmoins, je me heurte à l'erreur suivante que je n'arrive pas à solutionner :
Erreur de compilation
error: non-static variable this cannot be referenced from a static context
defibrilateurs.put(id, new Defibrilateur(id,nom,adresse,tel,dLonRad,dLatRad));


Merci d'avance pour votre aide ! Je me remets au Java et pour le coup élucider la source de mon erreur me dépasse encore un peu.

Voici mon code complet :

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
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
import java.util.*;
import java.io.*;
import java.math.*;
 
/**
 * Auto-generated code below aims at helping you parse
 * the standard input according to the problem statement.
 **/
class Solution {
 
 
    public class Defibrilateur
	{
		int id;
		String nom, adresse, tel;
		Double dLon, dLat;
 
		public Defibrilateur (int id, String nom, String adresse, String tel, double dLon, double dLat)
		{
			this.id = id;
			this.nom = nom;
			this.adresse = adresse;
			this.tel = tel;
			this.dLon = dLon;
			this.dLat = dLat;
		}
 
 
		public double distIndividu2Defib(double lon_indiv, double lat_indiv)
		{
			// Calcul du x
			double x = (this.dLon - lon_indiv)*Math.cos((this.dLat+lat_indiv)/2);
			// Calcul du y 
			double y = this.dLat - lat_indiv;
			// Calcul de d 
			double d = Math.sqrt(x*x + y*y);
 
			return d;
		}
	}
 
 
 
	public static void main(String[] args) throws FileNotFoundException {
		// TODO Auto-generated method stub
 
		Scanner in = new Scanner (System.in);
 
 
		// Récupération de la position (en degré) de l'individu et conversion en radians
		String lonIndivDeg = in.next();
        Double lonIndivRad = deg2Rad(Double.parseDouble(lonIndivDeg.replace(',', '.')));
		in.nextLine();
		String latIndivDeg = in.next();
        Double latIndivRad = deg2Rad(Double.parseDouble(latIndivDeg.replace(',', '.')));
        in.nextLine();
 
        // Récupération du nombre de défibrilateur
        int N = in.nextInt();
        in.nextLine();
 
        // Création du HashMap
        HashMap<Integer,Defibrilateur> defibrilateurs = new HashMap<>();
 
 
        // Itération sur les N défibrilateurs et Stockage des informations dans un HashMap
        for (int i=0;i<N;i++)
        {
        	String[] defibTab = in.nextLine().split(";");
        	int id = Integer.parseInt(defibTab[0]);
        	String nom = defibTab[1];
        	String adresse = defibTab[2];
        	String tel = defibTab[3];
        	Double dLonRad = deg2Rad(Double.parseDouble(defibTab[4]));
        	Double dLatRad = deg2Rad(Double.parseDouble(defibTab[5]));
 
        	defibrilateurs.put(id, new Defibrilateur(id,nom,adresse,tel,dLonRad,dLatRad));
        }
 
        double min = -1;
        int idMin = 0;
 
        for(Defibrilateur def : defibrilateurs.values())
        {
        	double d = def.distIndividu2Defib(lonIndivRad, latIndivRad);
        	if (min = -1 || d<min)
        	{
        		min = d;
        		idMin = def.id;
        	}
        }
 
        System.out.println(defibrilateurs.get(idMin).nom);
	}
 
	public static double deg2Rad(double angle)
	{
		return angle*Math.PI/180;
	}
 
}

[joel.drigo]

Salut,

La class Defibrilateur est une classe interne non statique de la classe Solution : elle ne peut donc être instanciée qu'à partir d'une instance de la classe solution. Dans la méthode main(), on se trouve dans un contexte statique, en dehors donc de toute instance, qui ne peut donc instancier la classe Defibralateur.

Solution 1 : mettre la classe Defibrilateur dans un fichier .java à part. Java est un langage orienté objet : il est préférable de faire des classes séparées pour répartir les traitements, plutôt que de tout mettre dans la même classe. Surtout si tu dois par la suite ajouter d'autres traitements : la classe Defibrilateur sera indépendante et pourra être manupulée par des méthodes d'autres classes.

Solution 2 : moins souple, mais suffisant si ton programme est limité juste à ce code que tu as donné, tu peux rendre la classe Defibrilateur statique. Ainsi, elle pourra être manipulée depuis un contexte statique. Par ailleurs, tu devrais rendre ta classe Solution public, car si tu multiplies le nombre de classes pour ton application, il te sera préférable d'organiser tout ça en package, et du coup, ouvrir la portée de la classe Solution, pour que la classe statique Defibrilateur soit accessible depuis un maximum de classes (cependant, il sera beaucoup mieux dans ce cas de mettre la classe Defibrilateur dans son propre fichier .java)

[LenyEdwarx]

Merci beaucoup Joël pour tes explications claires !

J'ai déjà testé la Solution 2 que tu proposes et cela marche. Pour être plus propre, je testerai également la solution 1 d'ici peu.
Bon sang, ce que je peux passer du temps sur des coquilles comme ça !

Une fois la solution 1 testée et validée, je mettrai la discussion en résolue.

[plawyx]

Il y a un problème ligne 86: min = -1 n'est pas un booléen mais une affectation. Dans ce cas, pour des types double, Double.doubleToLongBits(min) == Double.doubleToLongBits(limite) avec Double limite = new Double("-1");

Sinon, il est aussi conseillé de fermer tous les flux (Scanner, etc..). Utilisation d'un try-with-resources.

[LenyEdwarx]

En effet plawyx et Eclipse n'a pas manqué de me le faire remarquer également .