Discussion :

[lepirlouit]

j'ai un code qui génère mes 130 requ^tes, je me demandais s'il n'étais pas possible de simplifier un peu, en mettant des group by par exemple, mais je ne voit pas très bien comment.

voici mon code qui génère :

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
 
 
   protected void Page_Load(object sender, EventArgs e)
    {
        IList[] yValues = new ArrayList[13];
        for (int i = 0; i < 13; i++) yValues[i] = new ArrayList();
        DebugBox.Text = "";
        const String SELECT_FROM = "SELECT count(t.ticketid) FROM ticket t ";
        String qSelect = "";
        IEnumerator it;
 
        ISession session = new SessionScopeWrapper();
 
        for (int i = 0; i < 10; i++) {
            /*créés dans le mois :*/
            qSelect = SELECT_FROM + "WHERE " + inMonth("t.createdate", i - 10) ;
            DebugBox.Text += qSelect + "\n";
            it= session.CreateSQLQuery(qSelect).List().GetEnumerator();
            if (it.MoveNext()) yValues[0].Add(it.Current);
            /* créé et résolu dans le mois*/
            qSelect = SELECT_FROM + "WHERE " + inMonth("t.createdate", i - 10) + "AND " + inMonth("t.completeddate", i - 10) ;
            DebugBox.Text += qSelect + "\n";
            it = session.CreateSQLQuery(qSelect).List().GetEnumerator();
            if (it.MoveNext()) yValues[1].Add(it.Current);
            for (int j = 1; j < 11; j++)
            {
                qSelect = SELECT_FROM + "WHERE " + inMonth("t.createdate", i - j - 10) + "AND NOT " + lowerMonth("t.completeddate", i - 10 + 1) ;
                DebugBox.Text += qSelect + "\n";
                it = session.CreateSQLQuery(qSelect).List().GetEnumerator();
                if (it.MoveNext()) yValues[1+j].Add(it.Current);
            }
            qSelect = SELECT_FROM + "WHERE " + lowerMonth("t.createdate", i - 10) + "AND " + inMonth("t.completeddate", i - 10) ;
            DebugBox.Text += qSelect + "\n";
            it = session.CreateSQLQuery(qSelect).List().GetEnumerator();
            if (it.MoveNext()) yValues[12].Add(it.Current);
        }
        for (int i = 0; i < 13; i++) Chart1.Series.FindByName("Series" + (i + 1)).Points.DataBindY(yValues[i]);
 
 
    }
 
    private string inMonth(String field, int number) {
        return greaterMonth(field, number) + "AND " + lowerMonth(field, number + 1);
    }
    private string greaterMonth(String field, int number)
    {
        return field+">dateadd(\"mm\","+number+",getdate()) ";
    }
    private string lowerMonth(String field, int number)
    {
        return field + "<dateadd(\"mm\"," + number + ",getdate()) ";
    }

et voici une partie du sql généré :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
SELECT count(t.ticketid) FROM ticket t WHERE t.createdate>dateadd("mm",-10,getdate()) AND t.createdate<dateadd("mm",-9,getdate()) 
SELECT count(t.ticketid) FROM ticket t WHERE t.createdate>dateadd("mm",-10,getdate()) AND t.createdate<dateadd("mm",-9,getdate()) AND t.completeddate>dateadd("mm",-10,getdate()) AND t.completeddate<dateadd("mm",-9,getdate()) 
SELECT count(t.ticketid) FROM ticket t WHERE t.createdate>dateadd("mm",-11,getdate()) AND t.createdate<dateadd("mm",-10,getdate()) AND NOT t.completeddate<dateadd("mm",-9,getdate()) 
SELECT count(t.ticketid) FROM ticket t WHERE t.createdate>dateadd("mm",-12,getdate()) AND t.createdate<dateadd("mm",-11,getdate()) AND NOT t.completeddate<dateadd("mm",-9,getdate()) 
SELECT count(t.ticketid) FROM ticket t WHERE t.createdate>dateadd("mm",-13,getdate()) AND t.createdate<dateadd("mm",-12,getdate()) AND NOT t.completeddate<dateadd("mm",-9,getdate()) 
SELECT count(t.ticketid) FROM ticket t WHERE t.createdate>dateadd("mm",-14,getdate()) AND t.createdate<dateadd("mm",-13,getdate()) AND NOT t.completeddate<dateadd("mm",-9,getdate()) 
SELECT count(t.ticketid) FROM ticket t WHERE t.createdate>dateadd("mm",-15,getdate()) AND t.createdate<dateadd("mm",-14,getdate()) AND NOT t.completeddate<dateadd("mm",-9,getdate()) 
SELECT count(t.ticketid) FROM ticket t WHERE t.createdate>dateadd("mm",-16,getdate()) AND t.createdate<dateadd("mm",-15,getdate()) AND NOT t.completeddate<dateadd("mm",-9,getdate()) 
SELECT count(t.ticketid) FROM ticket t WHERE t.createdate>dateadd("mm",-17,getdate()) AND t.createdate<dateadd("mm",-16,getdate()) AND NOT t.completeddate<dateadd("mm",-9,getdate()) 
SELECT count(t.ticketid) FROM ticket t WHERE t.createdate>dateadd("mm",-18,getdate()) AND t.createdate<dateadd("mm",-17,getdate()) AND NOT t.completeddate<dateadd("mm",-9,getdate()) 
SELECT count(t.ticketid) FROM ticket t WHERE t.createdate>dateadd("mm",-19,getdate()) AND t.createdate<dateadd("mm",-18,getdate()) AND NOT t.completeddate<dateadd("mm",-9,getdate()) 
SELECT count(t.ticketid) FROM ticket t WHERE t.createdate>dateadd("mm",-20,getdate()) AND t.createdate<dateadd("mm",-19,getdate()) AND NOT t.completeddate<dateadd("mm",-9,getdate()) 
SELECT count(t.ticketid) FROM ticket t WHERE t.createdate<dateadd("mm",-10,getdate()) AND t.completeddate>dateadd("mm",-10,getdate()) AND t.completeddate<dateadd("mm",-9,getdate())

[al1_24]

Quel est ton SGBD ?
dateadd et getdate ne sont pas des mots du langage SQL normalisé, il est difficile de comprendre tes requêtes.

As-tu essayé d'exprimer en français courant ton besoin ?
Cela suffit quelquefois à trouver comment simplifier un ensemble de requêtes !

[lepirlouit]

les requêtes fonctionnent, pas de soucis à ce niveau là.

mon sgbd est MSSQL server 2005 Enterprise Edition

getDate renvoie la date d'ajourd'hui

et dateadd permet d'ajouter un certains nombre de "dd" -> jours à une date donnée.

le but est de pouvoir visualiser l'historique de résolution de tickets.

sous forme de chart dont voici une capture:


pour l'instant, ça fonctionne avec les 130 requêtes.

[lepirlouit]

une connaisance m'a donéne une réponse que je vais essayer d'approfondir ...

Très intéressant comme problème dis donc !
En fait, si j'ai bien tout compris, tu cherches à compter chaque 'ticketid' en fonction de son mois d'ancienneté.
En SQL, t'as une fonction qui te donne directement la différence de mois entre deux dates, datediff(month,date1,date2)

Si tu fais la requète suivante :

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
 
SELECT
count(t.ticketid) as Count,
datediff(month,t.createdate,getdate()) as MonthCreate,
datediff(month,t.completeddate,getdate()) as MonthComplete
 
FROM ticket t
 
group by
datediff(month,t.createdate,getdate()),
datediff(month,t.completeddate,getdate())
 
ORDER BY
datediff(month,t.createdate,getdate()),
datediff(month,t.completeddate,getdate())

Ca va te sortir une table avec le compte pour chaque mois d'ancienneté de création et chaque mois d'ancienneté de complétion. Je pense que ça sera beaucoup plus rapide.
Après, tu peux rajouter des contraintes du genre

Code :

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

WHERE (datediff(month,t.createdate,getdate())=11 AND datediff(month,t.completeddate,getdate()<>9) OR (datediff(month,t.createdate,getdate())=18 AND datediff(month,t.completeddate,getdate()<>9)

Et dans ton appli, suffit que tu récupère toute la table, et au final t'as fait qu'un seul appel base de données

[lepirlouit]

it works, j'ai tout ce qui me faut.

un grand merci.