Discussion :

[abbescr7]

salut a tous

j'ai une erreur que j'arrive pas a résoudre !

j'ai déclarer un tableaux globalement

Code :

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

double[][] StationTram1;

et je l'ai initialisé et remplie a l'aide d'une méthode depuis une base de donnée

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
 public void onResponse(String response) {
 
                        try {
 
                            JSONObject jsonObj = new JSONObject(response);
                            JSONArray jsonArray = jsonObj.getJSONArray("markers");
 
 
                            double [][] StationTram1 = new double [jsonArray.length()][2];
 
                            for (int i = 0; i < jsonArray.length(); i++) {
                                jresponse = jsonArray.getJSONObject(i);
                                StationTram1[i][0]=Double.parseDouble(jresponse.getString("lng"));
                                StationTram1[i][1]=Double.parseDouble(jresponse.getString("lat"));
                            }
                            for ( double[] point1 : StationTram1){
                                Log.i("lng",String.valueOf(point1[0]) );
                            }
 
 
 
 
                        } catch (JSONException e) {
                            e.printStackTrace();
                        }
                        //pDialog.dismiss();
 
                    }
 
                },
                new Response.ErrorListener() {
                    @Override
                    public void onErrorResponse(VolleyError error) {
                    }
                }) {
            @Override
            protected Map<String, String> getParams() {
                Map<String, String> params = new HashMap<String, String>();
 
                return params;
            }
 
        };
 
        RequestQueue requestQueue = Volley.newRequestQueue(MapsActivity.this);
        requestQueue.add(stringRequest);

je n peux pas l'inisitailisé "globalement" vu qu'il prend la taille de la table qu'elle est en base
maintenant quand je veux calculé la distance

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
 for(double[] point : StationTram1) {
 
            LatLng to = new LatLng(point[0], point[1]);
            //Calculating the distance in meters
            int distanceLambda = (int) SphericalUtil.computeDistanceBetween(from, to);
 
 
 
            if (distanceLambda < distance) { // cette distance est donc plus petite que la plus petite distance déjà trouvée
                distance = distanceLambda;
                Latitude1 = point[0];
                Longitude1 = point[1];
 
            }
 
        }

j'ai une erreur : java.lang.NullPointerException: Attempt to get length of null array

[autran]

il doit aussi te dire quelle est la ligne qui lève une erreur

[abbescr7]

il doit aussi te dire quelle est la ligne qui lève une erreur

l'erreur est au niveau de la boucle for etendu

Code :

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

  for(double[] point : StationTram1)

[medzila]

Salut,

StationTram1 doit être null. Faut peut être vérifier la façon dont tu le fais.

[abbescr7]

Salut,

StationTram1 doit être null. Faut peut être vérifier la façon dont tu le fais.

je l'ai remplie

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
   JSONObject jsonObj = new JSONObject(response);
                            JSONArray jsonArray = jsonObj.getJSONArray("markers");
 
 
                            double [][] StationTram1 = new double [jsonArray.length()][2];
 
                            for (int i = 0; i < jsonArray.length(); i++) {
                                jresponse = jsonArray.getJSONObject(i);
                                StationTram1[i][0]=Double.parseDouble(jresponse.getString("lng"));
                                StationTram1[i][1]=Double.parseDouble(jresponse.getString("lat"));
                            }
                            for ( double[] point1 : StationTram1){
                                Log.i("lng",String.valueOf(point1[0]) );
                            }

le soucis c'est qu'il déclarer localement du coup comment pourrai-je faire la liaison ! ?

[Lady]

je l'ai remplie

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
   JSONObject jsonObj = new JSONObject(response);
                            JSONArray jsonArray = jsonObj.getJSONArray("markers");
 
 
                            double [][] StationTram1 = new double [jsonArray.length()][2];
 
                            for (int i = 0; i < jsonArray.length(); i++) {
                                jresponse = jsonArray.getJSONObject(i);
                                StationTram1[i][0]=Double.parseDouble(jresponse.getString("lng"));
                                StationTram1[i][1]=Double.parseDouble(jresponse.getString("lat"));
                            }
                            for ( double[] point1 : StationTram1){
                                Log.i("lng",String.valueOf(point1[0]) );
                            }

le soucis c'est qu'il déclarer localement du coup comment pourrai-je faire la liaison ! ?

Tu ne remplis pas la bonne variable.

Vu que tu fait "double [][] StationTram1 = new double [jsonArray.length()][2];" c'est une version locale à ta méthode de StationTram1 que tu instancie et non ta variable globale.

Enlève la déclaration de type de cette ligne et ça ira mieux ( StationTram1 = new double [jsonArray.length()][2]; ), Tu n'aura au moins plus 2 versions différentes même si là je ne vois pas trop pourquoi il n'utilise pas la bonne (ou alors tu n'a pas retranscrit exactement ton code et la boucle for est dans une autre méthode)

[medzila]

C'est difficile de te répondre si on ne peut pas voir la construction de tes méthodes etc.
StationTram1 que tu utilise pour calculer les distances est celui qui est déclaré localement ou globalement ?
Le soucis est que quand tu sors de ta méthode qui rempli ton tableau local, celui-ci n'est plus utilisé...

[abbescr7]

Tu ne remplis pas la bonne variable.

Enlève la déclaration de type de cette ligne et ça ira mieux ( StationTram1 = new double [jsonArray.length()][2]; ), Tu n'aura au moins plus 2 versions différentes même si là je ne vois pas trop pourquoi il n'utilise pas la bonne (ou alors tu n'a pas retranscrit exactement ton code et la boucle for est dans une autre méthode)

même si j’enlève la déclaration toujours la même erreur et effectivement la boucle for est dans une autre méthode

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
 public void drawPath() {
 
        StringRequest stringRequest = new StringRequest(Request.Method.POST, AppConfig.URL_Marker,
                new Response.Listener<String>() {
 
                    @Override
                    public void onResponse(String response) {
 
                        try {
 
                            JSONObject jsonObj = new JSONObject(response);
                            JSONArray jsonArray = jsonObj.getJSONArray("markers");
 
 
                            double [][] StationTram1 = new double [jsonArray.length()][2];
 
                            for (int i = 0; i < jsonArray.length(); i++) {
                                jresponse = jsonArray.getJSONObject(i);
                                StationTram1[i][0]=Double.parseDouble(jresponse.getString("lng"));
                                StationTram1[i][1]=Double.parseDouble(jresponse.getString("lat"));
                            }
 
                        } catch (JSONException e) {
                            e.printStackTrace();
                        }
                        //pDialog.dismiss();
 
                    }
 
                },
                new Response.ErrorListener() {
                    @Override
                    public void onErrorResponse(VolleyError error) {
                    }
                }) {
            @Override
            protected Map<String, String> getParams() {
                Map<String, String> params = new HashMap<String, String>();
 
                return params;
            }
 
        };
 
        RequestQueue requestQueue = Volley.newRequestQueue(MapsActivity.this);
        requestQueue.add(stringRequest);
 
 
        //Getting both the coordinates
        LatLng from = new LatLng(Latitude1,Longitude1);
 
        int distance = 999999999;
 
        for(double[] point : StationTram1) {
 
            LatLng to = new LatLng(point[0], point[1]);
            //Calculating the distance in meters
            int distanceLambda = (int) SphericalUtil.computeDistanceBetween(from, to);
 
 
            if (distanceLambda < distance) { // cette distance est donc plus petite que la plus petite distance déjà trouvée
                distance = distanceLambda;
                Latitude1 = point[0];
                Longitude1 = point[1];
 
            }
 
        }
 
        Toast.makeText(this,String.valueOf(distance+" Meters"),Toast.LENGTH_LONG).show();
        moveMap();
 
    }

[medzila]

Comme a dit Lady, fais plutôt

Code :

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

StationTram1 = new double [jsonArray.length()][2];

au lieu de

Code :

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

double [][] StationTram1 = new double [jsonArray.length()][2];

et tu déclare StationTram1 à l'extérieur.

[abbescr7]

Comme a dit Lady, fais plutôt

Code :

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

StationTram1 = new double [jsonArray.length()][2];

au lieu de

Code :

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

double [][] StationTram1 = new double [jsonArray.length()][2];

et tu déclare StationTram1 à l'extérieur.

même en faisant ça , toujours la même erreur !
java.lang.NullPointerException: Attempt to get length of null array

le souci est au niveau de la boucle for etendu

Code :

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

 for(double[] point : StationTram1)

[wax78]

Bah il faut passer par le debuggeur et voir ligne par ligne ou ca "foire" parait.

[abbescr7]

Bah il faut passer par le debuggeur et voir ligne par ligne ou ca "foire" parait.

j'ai verfié avec le log.i

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
 public void onResponse(String response) {
 
                        try {
 
                            JSONObject jsonObj = new JSONObject(response);
                            JSONArray jsonArray = jsonObj.getJSONArray("markers");
 
 
                            StationTram1 = new double [jsonArray.length()][2];
                            for (int i = 0; i < jsonArray.length(); i++) {
                                jresponse = jsonArray.getJSONObject(i);
                                StationTram1[i][0]=Double.parseDouble(jresponse.getString("lng"));
                                StationTram1[i][1]=Double.parseDouble(jresponse.getString("lat"));
                            }
                            for ( double[] point1 : StationTram1){
                                Log.i("lng",String.valueOf(point1[0]) );
                            }
 
 
 
 
                        }

le tableau est bien remplie , si je met

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
for(double[] point : StationTram1) {
 
            LatLng to = new LatLng(point[0], point[1]);
            //Calculating the distance in meters
            int distanceLambda = (int) SphericalUtil.computeDistanceBetween(from, to);
 
 
 
            if (distanceLambda < distance) { // cette distance est donc plus petite que la plus petite distance déjà trouvée
                distance = distanceLambda;
                Latitude1 = point[0];
                Longitude1 = point[1];
 
            }
 
        }

le tableaux ne se remplie pas , et j'ai l'erreur java.lang.NullPointerException: Attempt to get length of null array

[medzila]

Avant ta boucle met un log qui montre la taille de ton tableau StationTram1.length, ça te donne quoi ?

[abbescr7]

Avant ta boucle met un log qui montre la taille de ton tableau StationTram1.length, ça te donne quoi ?

Code :

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

Log.i("taille", String.valueOf(StationTram1.length));

erreur : java.lang.NullPointerException: Attempt to get length of null array

[medzila]

Je ne suis pas pro en Android mais je pense que ce qu'il se passe est que ton tableau se remplit de manière asyncrone donc il se peut que ton tableau soit null quand tu l'appelles ensuite dans ta boucle...
Jette un coup d’œil à ça: http://arnab.ch/blog/2013/08/asynchr...-using-volley/

[abbescr7]

Je ne suis pas pro en Android mais je pense que ce qu'il se passe est que ton tableau se remplit de manière asyncrone donc il se peut que ton tableau soit null quand tu l'appelles ensuite dans ta boucle...
Jette un coup d’œil à ça: http://arnab.ch/blog/2013/08/asynchr...-using-volley/

c'est quoi la solution alors ?

[abbescr7]

Je ne suis pas pro en Android mais je pense que ce qu'il se passe est que ton tableau se remplit de manière asyncrone donc il se peut que ton tableau soit null quand tu l'appelles ensuite dans ta boucle...
Jette un coup d’œil à ça: http://arnab.ch/blog/2013/08/asynchr...-using-volley/

t'avais aison c'etais sa le soucis du coup au lieu de remplir le tableau dans la methode

Code :

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

public void drawPath()

que je l'appelé en cilquant sur un boutton

j'ai remplis le tableaux dans la methode

Code :

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

]public void onMapReady(GoogleMap googleMap)

Merci a tous

[medzila]

Le soucis est que ta réponse est donnée de manière asyncrone, du coup le code qui vient après peut être exécuté avant que la réponse soit traitée.
En gros, le traitement que tu veux faire pour calculer les distance peut être fais juste après que tu remplisse ton tableau.
Tu peux faire par exemple une méthode en dehors du genre (je te le fais en pseudo-code):

Code :

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

1
2
3
4
 
void doSomething(ton tableau){
     ta boucle qui calcule des distances;
}

Et cette méthode tu l'appelles dans ton onResponse, à la suite:

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
public void onResponse(String response) {

                        try {

                            JSONObject jsonObj = new JSONObject(response);
                            JSONArray jsonArray = jsonObj.getJSONArray("markers");


                            double [][] StationTram1 = new double [jsonArray.length()][2];

                            for (int i = 0; i < jsonArray.length(); i++) {
                                jresponse = jsonArray.getJSONObject(i);
                                StationTram1[i][0]=Double.parseDouble(jresponse.getString("lng"));
                                StationTram1[i][1]=Double.parseDouble(jresponse.getString("lat"));
                            }
                            doSomething(StationTram1);
                        } catch (JSONException e) {
                            e.printStackTrace();
                        }
                        //pDialog.dismiss();

                    }

                }

EDIT:

t'avais aison c'etais sa le soucis du coup au lieu de remplir le tableai dans la methode

Code :

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

public void drawPath()

que je l'appelé en clquant sur un boutton

j'ai remplis le tableaux dans la methode [CODE]]public void onMapReady(GoogleMap googleMap)[/CODE

Merci a tous

OK si ça marche
Bon courage.