Discussion :

[brayanrabat]

bonjour,

Je voulais obtenir la somme where KEY_CONTENT20=prixallo:

basedonne.java:

Code :

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

1
2
3
4
5
public Cursor total_allo(String prixallo) 
{
	return sqLiteDatabase.query(FOUR_CLIENT, new String[] {"SUM(" + KEY_CONTENT22 +")"}, 
			KEY_CONTENT20,  new String[] {prixallo}, null, null, null);
}

main.java:

Code :

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

double sumA= Math.round(my_operations_mois.total_allo(datefinal)*100.0)/100.0;

mais erreur avant exécuter : the operator * is undefined for the argument type(s) Cursor, double

comment corriger l'erreur

Merci d'avance

[Hephaistos007]

Code java :

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

1
2
3
4
Cursor c = my_operations_mois.total_allo(datefinal);
cursor.moveToFirst();
double sumA= Math.round(c.getDouble(0)*100.0)/100.0;
cursor.close();

[stelincat]

Personnellement, je te conseillerai de faire en sorte que ta méthode renvoi directement ton résultat déjà sous forme de double (étant donné que le but de la méthode c'est juste de renvoyer un montant, donc aucun besoin de renvoyer un curseur).

En partant du code de Hephaistos007 et en sécurisant un peu plus le code je te propose cette 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
 
    public double total_allo(String prixallo)
    {
        double result = 0 ;
        Cursor cur = null ;
 
        try
        {
            cur  = sqLiteDatabase.query(FOUR_CLIENT, new String[]{"SUM(" + KEY_CONTENT22 + ")"}, KEY_CONTENT20, new String[]{prixallo}, null, null, null);
 
            if  (( cur != null) && ( cur.getCount() > 0) )
            {
                cur.moveToFirst();
                result  = cur.getDouble(0);
            } 
        } 
        finally {
            if (mCursor != null) {
                mCursor.close();
            }
        }
 
        return result ;
    }

Et l'appelant n'a juste qu'a faire ceci :

Code :

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

1
2
 
double sumA= Math.round(my_operations_mois.total_allo(datefinal)*100.0)/100.0;

[brayanrabat]

bonjour,

j'ai trouvé la solution comme ceci :

Code :

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

1
2
3
4
5
6
7
8
public double Mois_total_client(String mois)
{
 
 
Cursor CCursor = sqLiteDatabase.rawQuery("SELECT  SUM(solde_client) FROM four_cl WHERE mois_annee = ?", new String[] {mois});
if(CCursor.moveToFirst()) {return CCursor.getDouble(0);}
return CCursor.getDouble(0);
}

encore une fois merci Hephaistos007 etstelincat pour l'idée

[nicroman]

Attention...
le cursor n'est pas fermé...
les noms de variables / fonction en java commencent par une minuscules (camelCase) et le "_" est "interdit"

et le "if" ne sert à rien:

Code :

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

1
2
3
 
if(CCursor.moveToFirst()) {return CCursor.getDouble(0);}
return CCursor.getDouble(0);

Si la fonction ne retourne pas de curseur, celui-ci doit être fermé par la fonction:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
public double getTotalMoisClient(String mois)
{
    double ret = 0;
    Cursor curs = null;
    try {
       curs = sqLiteDatabase.rawQuery("SELECT  SUM(solde_client) FROM four_cl WHERE mois_annee = ?", new String[] {mois});
       if (! curs.moveToFirst()) throw new SQLiteException("No data available !");
       ret = curs.getDouble(0);
     } finally {
       if (curs != null) curs.close();
     }
     return ret;
}

Ensuite pour chipoter, je suis toujours étonné de voir un "mois" passé sous forme de "string".
Les mois étant "fixes" (enfin, dans le calendrier grégorien tout du moins), un "enum" permettrait d'éviter les erreurs de programmation (vive le typage fort ! ).
Et si "mois" est en fait un formatage "complexe" (ou non) d'un mois et d'une année (comme la requête le laisse penser), alors une "classe" stockant le mois et l'année, et produisant le formatage (fonction toString() ) voulu sera certainement plus adaptée.