Optimisation requête SQLite

Bonjour,

J'aimerais avoir votre avis concernant le temps de chargement des requêtes de mon applications.

J'initialise mon application par des insert et des Query

Pour une soixantaine de requêtes, je mets 3 secondes. Ca me parait beaucoup ?

Merci de vos avis

Salut,

Effectivement ça peut paraitre long pour un utilisateur lambda. Mais cela dépend bien sûr du type de ton application (c'est à dire, pour un jeu 3D ça me dérange pas de patienter 3 sec, mais pour lancer une calculette, je trouve ça pénible).

Techniquement, peux tu nous décrire pourquoi tu fais des insert / query à chaque coup ? c'est obligatoire de le refaire à chaque lancement ?
Ta base est complexe ?

Bonjour,

merci de ta réponse. Je finalise une application de video-surveillance avec la gestion de plusieurs types de camera et la gestion des mouvements des cameras.

Je suis en train d'optimiser l'application mais sur certaines activités le temps d'attente peut etre de plus de 10 secondes au minimum.

J'ai tracé chaque ligne pr voir ou déconne, je commence donc par le début de mon application, c'est à dire l'initialisation.

Dans cette phase, j'initialise ma BD avec des cameras pour que mon application ne soit pas vide. Cette phase peut déjà durer 3 à 4 secondes. J'appelle une classe que j'ai simplifié mais qui ressemble à cela :

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 40 41

package com.eyeseeyou;   import java.util.ArrayList;   import android.content.Context; import android.util.Log;   public class InitBd {   private DBAdapter db; private Context context; private Profil profil; private Camera camera;   //Connexion BD query private ArrayList<Camera> listCam;   public InitBd(Context _context){ context = _context; init(); }   public void init(){ db = new DBAdapter(context); db.open(); db.insererUneCamera(camera); db.getAllListeDesCameras().add(camera); camera = new Camera(.......................); db.insererUneCamera(camera); db.getAllListeDesCameras().add(camera); camera = new Camera(.......................); db.insererUneCamera(camera); db.getAllListeDesCameras().add(camera); camera = new Camera(........................); db.insererUneCamera(camera); db.getAllListeDesCameras().add(camera); db.close(); }   }

---

Il y a peut etre une mauvaise utilisation mais en tt cas je trouve l'utilisation de sqlite avec Android pas vraiment optimum car on doit vraiment bien gérer la gestion de l'open et du close selon cycle de vie, selon l'activité etc...

Comparer avec le dev pour les iphones ou tu ouvres en début de l'application puis tu fermes à la fin.

En tt cas, j'espére que j'ai bien fait une erreur quelque part, car mes connections sont trop lente à mon gout

Ps: je précise que c'est le remplissage ou la lecture de ma BD qui est lente et non la lecture des flux de cameras qui marche bien.
Merci

J'ai encore une question, en testant un peu je ne comprends une chose. Si je mets cette classe initBd dans le oncreate ou plutot ds le onResume() (si je fais pas erreur c'est preferable a ce niveau là), cela dure entre 3 et 4 secondes.

Maintenant si je le met dans un thread en arriére plan (et dc pas ds le thread de l'interface utilisateur) cela prend 14 seondes.

Comment cela s'explique ?

Merci

Citation:

---

Dans cette phase, j'initialise ma BD avec des cameras pour que mon application ne soit pas vide. Cette phase peut déjà durer 3 à 4 secondes. J'appelle une classe que j'ai simplifié mais qui ressemble à cela

---

Combien en insère tu ?
Si c'est comme sur ton exemple ou tu insère 3 ou 4 caméras ça me parait énorme. Peut être envisager une méthode du type insertCameras(ArrayList Cameras) qui va venir insérer plusieurs camera en une seule fois.

Peut on voir comment tu insères tes données (même un peu édulcoré si c'est sensible).

Cependant il ne faut pas perdre de vu que il ne sera jamais possible d'obtenir les même performance que l'on peut avoir avec un SGBD classique de part la puissance réduite des devices (comparé à un serveur).

Citation:

---

Si je mets cette classe initBd dans le oncreate ou plutot ds le onResume()

---

As tu réellement besoin de réinsérer des données à chaque focus de ton activity ? Dans l'idée , une fois stockée , tu n'as plus besoin de réinsérer tes données.
On pourrait même imaginer que cette intialisation se fait pendant un splashscreen et ne se refera que si la bdd est vide.

Coucou,

j'ai pas tout compris mais pourquoi tu fais ça :

Code:

---

1 2 3

db.insererUneCamera(camera); db.getAllListeDesCameras().add(camera);

---

Si j'ai compris : DBAdapter c'est ta couche DAO qui écrit et lit dans la database.
Donc à l'init tu ajoutes une camera avec le insererUneCamera mais ensuite tu fais un db.getAllListeDesCameras().add(camera); ? Pourquoi ?
Car db.getAllListeDesCameras() n'est pas déjà un appel à ta base pour aller chercher la liste des cameras ? Pourquoi avoir besoin de rajouter dedans ta camera si celui-ci y est déjà puisque tu l'as déjà ajouté au dessus ?
J'ai l'impression que tu fais une requete d'insert puis un select 4 fois de suite ?
Je me trompe peut etre sur la méthode db.getAllListeDesCameras(). Dans ce cas ignore ce message.

Merci à vous 2, pour répondre à vos questions :

Combien en insère tu ?
Une cinquantaine d'insert qui dure donc approximativement 17 secondes. Je me suis trompé dans mes traces :cry:

As tu réellement besoin de réinsérer des données à chaque focus de ton activity ?
C'est déjà fait, j'utilise un SharedPreferences pour voir si mon application a été initialisé. Donc cette classe est appelé uniquement une fois normalement. Comme je l'ai dit c'est pour comprendre et oiptimiser car je fais d'autres requêtes ds mon application. Je commence par le début. C'est qd même vraiment problématique...

J'ai l'impression que tu fais une requete d'insert puis un select 4 fois de suite ?
Tu as raison je suis allé un peu vite, je n'ai pas besoin de lire ma query. Je l'ai modifié mais cela n'a aucun effet sur le temps. Le vrai problème semble être les insert.

Voici le code de l'insert :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14

// ajoute une camera public long insererUneCamera(Camera camera) { ContentValues values = new ContentValues(); values.put(COLONNE_SELECT, champ1); values.put(COLONNE_NOM, champ2); values.put(COLONNE_ACTIVE, champ3); values.put(COLONNE_ADRESSE, champ4); values.put(COLONNE_PORT, champ5); values.put(COLONNE_LOGIN, champ6); ...... ...... return db.insert(TABLE_CAMERAS, null, values); }

---

J'ai à peu près 20 champs.... Je ne sais pas trop quoi faire, quel testes je pourrais faire pour trouver le vrai souci....

Tu peux essayer de placer tes insert dans une transactions du genre :

Code:

---

1 2 3 4 5

sql.execSQL("BEGIN;"); //insert 1 //insert 2 // ... sql.execSQL("END;");

---

Une autre solution est de désactiver la synchronisation en écriture via les PRAGMA :

Code:

---

PRAGMA synchronous=0/1/2

---

Tu va gagner énormément en vitesse mais tu va perdre en fiabilité d'écriture. (perso je l'ai utilisé sur des bases de plusieur Go sans pb).

Plus d'info ici : http://web.utk.edu/~jplyon/sqlite/SQ...Q.html#pragmas et http://www.sqlite.org/pragma.html#pragma_synchronous

Super grunk, avec les transactions ma cinquentaines d'insert + open de ma BD et la fermeture dure 3 à 4 sec au lieu de 15 secondes.

Je vais pouvoir avancer sur le reste mais il y a encore quelques chose qui me chagrine, car 3.5 je trouve ca encore beaucoup car pr android, il préconise 5 sec maximum de chargement sinon elle péte. Tu me diras de la mettre ds un thread en arrière mais qd même.

Quand je regarde encore les traces, c'est l'open de ma BD qui met 2.5 secondes (ce qui est énorme au prorata du reste.)

Donc le problème n'est plus les requête mais l'ouverture de ma BD qui ressemble à ceci :

Code:

---

1 2 3 4 5 6 7 8

public void open() throws SQLiteException{ try{ db = DBHelper.getWritableDatabase(); }catch (SQLiteException e) { db = DBHelper.getReadableDatabase(); } }

---

Aurais tu un tuyau pour améliorer cela, surtout vu que j'ouvre ma BD pour chaque activité, je vais dc perdre 2.5 secondes à chaque fois.

Merci
Et super conseil :ccool:

J'ai une autre question concernant les transactions.

Est ce que si je n'ai qu'une requête à faire, cela est interessant d'utiliser les transactions ou ce n'est plutot que pour de lourd chargement ?

Merci d'avance
Amaury

Pour une seule requête je pense pas que les transactions soient utiles.

Effectivement la durée d'ouverture de ta base est assez énorme, surtout que dans l'absolu une ouverture ne fait rien de spécial si ce n'est retourner un objet capable d'écrire/lire dans la base.

As tu le même résultat sur un émulateur et sur un téléphone ?