Discussion :

[Manu0086]

Bonjour,

Le code d'une app provoque de gros lags et plantages.
J'ai identifié la ligne provoquant cela, il s'agit de
String value = jsonObject.getString(key);

Ce script récupère des milliers de communes de France, donc cette ligne est appelée autant de fois... la mémoire sature peu à peu jusqu'au plantage mais je ne comprends pas pourquoi cela agit ainsi.
(je ne suis pas l'auteur de cette app et suis débutant en dév)

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
    		    		jsonObject = json.init(Config.COMMUNES_URL);
 
    		    		Iterator iter = jsonObject.keys();
 
    		    	    while(iter.hasNext()){
    		    	    	String key = (String)iter.next();
    		    	        String value = jsonObject.getString(key);	// provoque bug, out of memory
    		    	        JSONObject jsonCommunes = new JSONObject(value);
    		    	        Iterator iterCommunes = jsonCommunes.keys();
    		    	        Commune commune = new Commune();
 
    		    	        while(iterCommunes.hasNext()){
    		    	        	String keyCommune = (String)iterCommunes.next();
        		    	        String valueCommune = jsonCommunes.getString(keyCommune);
	    		    	        if(keyCommune.equalsIgnoreCase("com_id"))
	    		    	        	commune.setId(valueCommune);
	    		    	        if(keyCommune.equalsIgnoreCase("com_libelle"))
	    		    	        	commune.setLibelle(valueCommune);
	    		    	        if(keyCommune.equalsIgnoreCase("com_latitude"))
	    		    	        	commune.setLatitude(valueCommune);
	    		    	        if(keyCommune.equalsIgnoreCase("com_longitude"))
	    		    	        	commune.setLongitude(valueCommune);
    		    	        }
    		    	        communesList.add(commune);
    		    	    }

Merci de m'éclairer.

[nicroman]

Code :

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

1
2
3
 
    		    	        String value = jsonObject.getString(key);	// provoque bug, out of memory
    		    	        JSONObject jsonCommunes = new JSONObject(value);

Ce ne serait pas simplement:

Code :

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

1
2
 
JSONObject jsonCommunes = jsonObject.getJSONObject(key);

?

On dirait que le programmeur ne savait pas utiliser JSON.... ou Java5.

J'ai l'impression qu'on a plutôt un truc dans le genre:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
jsonObject = json.init(Config.COMMUNES_URL);
for (String key : jsonObject.keys()) { // si warning/error, caster jsonObject.keys() en Iterator<String>, simple problème de générics
    JSONObject jsonCommune = jsonObject.getJSONObject(key);
    Commune commune = new Commune(); // on perd "key" ici c'est normal ?
    commune.setId(jsonCommune.optString("com_id"));
    commune.setLibelle(jsonCommune.optString("com_libelle"));
    commune.setLatitude(jsonCommune.optString("com_latitude"));
    commune.setLongitude(jsonCommune.optString("com_longitude"));
    communesList.add(commune);
}

Maintenant, la mémoire est limitée à 16Mo pour une application Java... disons 8Mo pour les communes, avec des id de 8 chars, des libellé de 16 des coordonnées de 20 chars... on a en tout 64 chars par commune... (soit 128 octets, ben oui 1 char = 2 octets en java) soit une limite de 65536 communes...

[Manu0086]

Merci beaucoup.

Mais avec la taille du fichier (35 000 communes je crois), ça génère trop de problèmes.

J'ai regardé du côté de SAX mais ai laissé tombé pour le moment.

Il y a d'autres bugs bien plus gênants... une appli qui serait à refaire de 0, mais je n'ai que 4h pour tenter de l'améliorer (et ces 4h sont finies ).