Convertir chaine sql en stringbuilder

**Bluedeep** · 05/08/2010, 15h33

Envoyé par olibara

pour BlueDeep

A bon ?

Je t'informe que chaque Append est une concaténation en vue de produire la chaine SQL résultante

Je t'informe que tu ferais bien dans ce cas de choisir des exemples un peu plus démonstratifs, puisque celui que tu as posté revenait à découper une chaine constante en une succession d'éléments constants. Donc ton exemple démontrait surtout l'inutilité d'un Sb dans ce cas.

car entre

Code :

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

1
2
 
string myString = "AA_BB";

et

Code :

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

1
2
3
4
5
6
 
StringBuilder mySb = new StringBuilder()
mySb.Append("AA");
mySb.Append("_");
mySb.Append("BB");
string myString = mySb.ToString();

Le choix est vite fait !!!!

Comme je l'ai dis l'intéret du stringbuilder et de pouvoir faicilemnt introduire le cas échéant un AppendFormat on manipuler des eléments

Je ne t'ai surtout pas attendu pour me servir des sb, rassure toi.

Dans le cas des requêtes Sql, l'utilité du Sb n'existe que quand on doit rajouter des éléments conditionnels (style "dans un cas j'ai besoin de rajouter un WHERE, dans un autre non"), puisque sur les éléments variants mais non structurants de la requêtes, l'utilisation su Sb en lieu et place des paramètres serait une faute.

**StormimOn** · 05/08/2010, 15h34

Ca me semble étrange comme façon de procéder. Comme Guulh, je trouve qu'une string Verbatim est bien plus simple à utiliser tout en étant plus clair (pas de sb.Append de partout).

Si on reprend ton exemple on obtient :

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
string query = 
@"SELECT
COUNT(*) AS cnt,
PhysicalBag.ID,
PhysicalBag.Number,
PhysicalBag.FillingLevel AS FLev,
PhysicalBag.OpenDate,
PhysicalBag.CloseRequestDate AS CloseNeedDt,
PhysicalBag.CloseRequiredDate AS CloseAskedDt,
PhysicalBag.CloseDate AS CloseDt,
PhysicalBag.ParcelCollectDefer AS ColectDay,
PhysicalBag.ReceivedDate AS ReceivedDt,
PhysicalBag.DispatchedDate,
Affiliate.HRK,
Affiliate.Name,
Affiliate.PhoneNumber,
Affiliate.FaxNumber,
Affiliate.GSMNumber,
Affiliate.Email
FROM
PhysicalBag
 INNER JOIN Affiliate ON PhysicalBag.AffiliateID = Affiliate.ID
 INNER JOIN VirtualBag ON PhysicalBag.ID = VirtualBag.PhysicalBagID
 INNER JOIN TradeIn ON VirtualBag.ID = TradeIn.VirtualBagID
WHERE (PhysicalBag.OpenDate < CONVERT(DATETIME, '2010-07-30 00:00:00', 102))
  AND (PhysicalBag.CloseDate IS NULL)
  AND (PhysicalBag.CloseRequestDate IS NULL)
GROUP BY PhysicalBag.ID,
PhysicalBag.Number,
PhysicalBag.FillingLevel,
PhysicalBag.OpenDate,
PhysicalBag.CloseRequestDate,
PhysicalBag.CloseRequiredDate,
PhysicalBag.CloseDate,
PhysicalBag.ParcelCollectDefer,
PhysicalBag.ReceivedDate,
PhysicalBag.DispatchedDate,
Affiliate.HRK, Affiliate.Name,
Affiliate.PhoneNumber,
Affiliate.FaxNumber,
Affiliate.GSMNumber,
Affiliate.Email
HAVING (COUNT(*)>5)";

C'est quand même plus simple et plus clair non ? Et s'il y a des paramètres, les requêtes paramétrées sont là pour ça, il suffit de placer les @param qui vont bien dans la chaîne.

Sinon utiliser un StringBuilder en l'état n'apporte absolument rien, sauf erreur de ma part, car lorsque tu fais :

Code :

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

"maChaine1" + "maChaine2" + "maChaine3" + ... + "maChaineXXX"

le compilateur est assez malin pour voir que tu concatènes des chaînes de caractères constantes et d'optimiser le code pour n'avoir plus qu'une seule chaîne, et donc pas de concaténation.

**Bluedeep** · 05/08/2010, 15h35

Envoyé par StormimOn

le compilateur est assez malin pour voir que tu concatènes des chaînes de caractères constantes et d'optimiser le code pour n'avoir plus qu'une seule chaîne, et donc pas de concaténation.

Tout à fait; en revanche dans le cas de chaines variantes, la différence de perf est assez importantes (j'ai posté le semaine dernière un bout de code qui illustrait la question), surtout quand on "autoconcatène" la chaine, ça devient du 1 pour 1000.

**olibara** · 05/08/2010, 15h35

Envoyé par Pol63

donc nous partout dans notre code c'est

dim rq as string = "" & VbCrlf & _
"SELECT ch ... " & VbCrLf & _
" FROM table" & VbCrLf & _
" WHERE"

donc pas super

Tu comprends l'intéret du stringBuilder maintenant ?

Je ne suis pas expert en vb mais ca donnerait +/- ceci (c'est quand meme plus lisible)

Code :

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

1
2
3
4
5
Dim sb as new StringBuilder()
 
sb.Append(" SELECT ch ... ")
sb.Append(" FROM table")
sb.Append(" WHERE")

Et tu prends soin de preceder chaque chaine avec un blanc

**Bluedeep** · 05/08/2010, 15h39

Envoyé par olibara

donc nous partout dans notre code c'est

dim rq as string = "" & VbCrlf & _
"SELECT ch ... " & VbCrLf & _
" FROM table" & VbCrLf & _
" WHERE"

donc pas super

Tu comprends l'intéret du stringBuilder maintenant ?

Absolument pas.

Dans cas j'écris :

Code :

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

1
2
3
4
5
 
sql = @"select a, b from tab t
           join tab2 t2
           on t2.Id = t.id
           where t.Lib = @tlib"

encore une fois, je ne vois aucun interêt à utiliser un assemblage de chaine constante quand il n'y en a besoin que d'une.
Et ici on est sur le forum C# donc ton exemple est non relevant (peut être qu'en VB le caractère chaine littérale n'existe pas ? je n'en sais rien).

**StormimOn** · 05/08/2010, 15h39

Envoyé par olibara

donc nous partout dans notre code c'est

dim rq as string = "" & VbCrlf & _
"SELECT ch ... " & VbCrLf & _
" FROM table" & VbCrLf & _
" WHERE"

donc pas super

Pas super en VB.Net, mais en C# il y a les string Verbatim pour ça

**olibara** · 05/08/2010, 15h41

Ca me semble étrange comme façon de procéder. Comme Guulh, je trouve qu'une string Verbatim est bien plus simple à utiliser tout en étant plus clair (pas de sb.Append de partout).

Oui Storminonn

Mais si un jour il est nécessaire (et ca arrivera) d'inserer une variable dans ta requete, toute ta belle construction verbatim s'effondre pour devenir vite un paquet de spagetti

Je l'ai souvent vécu et c'est pourquoi j'utilise maintenant systématiquement le stringbuilder et c'est franchement pratique a l'usage

**StormimOn** · 05/08/2010, 15h44

Envoyé par olibara

Oui Storminonn

Mais si un jour il est nécessaire (et ca arrivera) d'inserer une variable dans ta requete, toute ta belle construction verbatim s'effondre pour devenir vite un paquet de spagetti

Comme je l'ai dit, les requêtes paramétrées sont là pour ça à ce moment, enfin si c'est ce que tu entends par variable.

Code :

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

1
2
3
string query =
@"SELECT * FROM MaTable
WHERE IdTable = @MonId"

@MonId est un paramètre dans la requête que tu pourras préciser à l'exécution de la requête.

**olibara** · 05/08/2010, 15h45

Pour BlueDeep et Stormimonn

Envoyé par Pol63

donc nous partout dans notre code c'est

dim rq as string = "" & VbCrlf & _
"SELECT ch ... " & VbCrLf & _
" FROM table" & VbCrLf & _
" WHERE"
donc pas super
"

Cet exemple etait de Pol63 a qui je répondais

**Pol63** · 05/08/2010, 15h55

c'est pas parce qu'on est sur le forum c# que je ne peux pas parler de vb.net
et nous en vb on est obligé de faire un truc chiant ou d'utiliser un stringbuilder pour tapper moins de code
mais pas de string sur plusieurs ligne de manière simple et lisible ...

je demandais aussi à quoi servait le @ devant un string, chose qui n'a pas l'air d'exister en vb.net
j'ai cru comprendre que ca servait à contrer le caractère \ qui serait chez vous un caracatère d'échapement (ca explique pourquoi je vois des c#iens écrires str = "c:\\monfichier.txt" alors qu'en vb.net un seul "\" suffit

**olibara** · 05/08/2010, 15h58

Pour Rassurer Stormimonn

Meme en cas de requetre parametrée j'utilise un StringBuilder

Exemple

Code :

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

1
2
3
4
5
6
7
8
9
      sb.Append(" update Address set ");
      sb.Append(" PersonContactName=@Name");
      sb.Append(" where ID=@ID", ID);
 
      SqlCommand MyCmd = new SqlCommand();
 
      MyCmd.CommandText = sb.ToString();
      MyCmd.Parameters.AddWithValue("@ID", ID);
      MyCmd.Parameters.AddWithValue("@Name", name);

Mais bon c'est une question de petite manies de chacun
Personnellement pour la construction de requete Sql je ressens plus de confort avec le stringbuilder qu'avec le String verbatim

**Pol63** · 05/08/2010, 16h06

c'est une question de gout c'est sur, m'enfin utiliser append plutot qu'appendline par contre c'est pas terrible, pour du debug ta requete devient illisible

**Guulh** · 05/08/2010, 16h51

Envoyé par olibara

Pour Rassurer Stormimonn

Meme en cas de requetre parametrée j'utilise un StringBuilder

Exemple

Code :

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

1
2
3
      sb.Append(" update Address set ");
      sb.Append(" PersonContactName=@Name");
      sb.Append(" where ID=@ID", ID);

Que fait ce ",ID" ici ? Tu vois, cette façon de faire te perturbe

Sans compter que si t'oublies un espace au début des tes chaînes, t'es foutu.(ou alors t'utilises appendline)

On a certes tous nos manies, mais celle-là est inutilement compliquée. Elle t'oblige même à développer un add-in pour la satisfaire...

Les StringBuilder ne sont pas faits pour concaténer un ensemble de chaînes connues dès la compil.

**ppphil** · 05/08/2010, 17h34

on ne peut pas faire de string multiligne en vb.net

Sincères condoléances... Il fait si bon en c#...

**olibara** · 05/08/2010, 17h47

Que fait ce ",ID" ici ? Tu vois, cette façon de faire te perturbe

Bien vu !
Non non rassure toi pas du tout perturbé
Je voulais illustrer une maniere de faire avec l'AppendFormat et je me suis emmelé les pinceaux en allant trop vite

Bien vu aussi Pol63 le AppendLine serait plus approprié ici !

On a certes tous nos manies, mais celle-là est inutilement compliquée. Elle t'oblige même à développer un add-in pour la satisfaire...

Non !

L'addin c'est surtout pour reformater les chaines sql pourries générees par SQL Mng Studio au autres

Et tu peux facilement l'adapter pour generer un string verbatim

Tiens le voilà

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
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
 
namespace SqlFormatter
{
  class C_SqlFormat
  {
    // ********************************************************************
    public StringBuilder ParseSql(string src, bool StormiGuulh)
    {
      StringBuilder sb = new StringBuilder();
 
      if (StormiGuulh)
      {
        sb.Append("string StormiGuulh=@\"");
      }
      else
      {
        sb.AppendFormat("StringBuilder sb=new StringBuilder();");
        sb.Append(Environment.NewLine);
      }
      src = PrepareSplitRight(src, "SELECT");
      src = PrepareSplitRight(src, "FROM");
      src = PrepareSplitRight(src, "WHERE");
 
      src = PrepareSplitLeft(src, "AND");
      src = PrepareSplitLeft(src, "INNER");
      src = PrepareSplitLeft(src, "RIGHT");
      src = PrepareSplitLeft(src, "LEFT");
      src = PrepareSplitLeft(src, "FULL");
 
      string x = src.Replace(Environment.NewLine, "|").Replace(",", ",|");
 
      string[] item = x.Split('|');
      int i;
      for (i = 0; i < item.Length; i++)
      {
        string st = item[i].Trim();
        if (st.Length <= 0)
        {
          continue;
        }
        if (StormiGuulh)
        {
          sb.Append("   ");
          sb.AppendLine(st);
        }
        else
        {
          sb.AppendFormat("sb.Append(\" {0}\");{1}", st, Environment.NewLine);
        }
      }
      if (StormiGuulh)
      {
        sb.AppendLine("\";");
      }
      return sb;
    }
    // ********************************************************************
    private string PrepareSplitRight(string src, string key)
    {
      int pos = 0;
      while (true)
      {
        pos = src.IndexOf(key,pos, StringComparison.InvariantCultureIgnoreCase);
        if (pos < 0)
        {
          return src;
        }
        pos += key.Length;
        src=src.Insert(pos, "|");
        pos++;
      }
    }
    // ********************************************************************
    private string PrepareSplitLeft(string src, string key)
    {
      int pos = 0;
      while (true)
      {
        pos = src.IndexOf(key, pos, StringComparison.InvariantCultureIgnoreCase);
        if (pos < 0)
        {
          return src;
        }
        src = src.Insert(pos, "|");
        pos++;
        pos += key.Length;
      }
    }
  }
}

**Guulh** · 05/08/2010, 18h04

Envoyé par olibara

public StringBuilder ParseSql(string src, bool StormiGuulh)

J'utilisais Toad lors d'une mission, il le fait assez bien.

Convertir chaine sql en stringbuilder

C#

Discussions similaires

Partager

Partager