Convertir chaine sql en stringbuilder

Version imprimable

05/08/2010, 11h34
olibara

Convertir chaine sql en stringbuilder

Bonjour

Il m'arrive souvent de transposer une requete SQL en StringBuilder

Trop souvent a mon gout !
Et c'est chaque fois asser pénible de redecouper la chaine honteusement déstructuée par SQL Management Studio pour une remise en forme humainement lisible a coup des Append dans un stringbuilder

Donc j'ai décidé s'écrire un petit outil qui fait ca automatiquement.

Mais avant de me lancer, avez-vous déja vu qq chose dans ce genre ?

Merci
05/08/2010, 13h11
Mehdi_ab

Bonjour,

Tu peux stp illustrer ton propos avec un exemple concret ?

Merci
05/08/2010, 13h28
Pol63

je ne vois pas du tout ce que tu fais ou alors je ne vois pas l'utilité ...
05/08/2010, 13h31
Pol63

si finalement je crois avoir compris
tu fais une requete dans sql server et tu la colles dans ton code et tu pleures voyant l'étendu des dégats ^^

alors le mieux c'est d'ajouter un nouveau fichier au projet, fichier de type texte (.txt)
tu colles ta requetes dedans
l'ajouter ensuite dans les ressources (drag drop sur les ressources du projet)
et ensuite

Code:

moncommand.commandtext = my.Resources.nom_du_fichier_texte

un fichier texte en ressources étant transformé en string à l'utilisation

(en espérant que ca existe en c#, enfin en vb.net my.resources existe)

Merci

Voici un exemple concret

Ceci

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
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

Peut devenir cela...
(Mais entretemps j'ai déja ecris la procédure qui permet en plus de passer par le clipboard

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

05/08/2010, 13h58
Bluedeep

Oui, et pourquoi tu utilises un StringBuilder ici?
J'avoue ne pas du tout comprendre : tu ne rajoutes pas d'élements conditionel nécessitant l'AppendFormat.
05/08/2010, 13h59
Pol63

on a du se croiser au niveau des posts
ma réponse devrait te convenir, vu que dans le fichier texte tu fais la mise en forme que tu veux comme dans l'ide sql

et le fichier txt va etre compilé dans l'exe
05/08/2010, 14h00
SetaSensei

Bonjour,

Dans un soucis de lisibilité, n'avez-vous jamais pensé passer par une vue, voire une procédure stockée sur votre base ?
05/08/2010, 14h00
Pol63

Citation:

Envoyé par Bluedeep

Oui, et pourquoi tu utilises un StringBuilder ici?
J'avoue ne pas du tout comprendre : tu ne rajoutes pas d'élements conditionel nécessitant l'AppendFormat.

un stringbuilder est des centaines (des millions ?) de fois plus performant qu'un string pour de la concaténation

Citation:

Envoyé par SetaSensei

Bonjour,

Dans un soucis de lisibilité, n'avez-vous jamais pensé passer par une vue, voire une procédure stockée sur votre base ?

les vues c'est pas mal sauf que pour une modif simple du code il faut ouvrir le sgbdr pour aller voir ce que fait la vue au lieu d'avoir le code sous la main
et puis une vue c'est trop simple, on peut modifier la requete sans avoir à recompiler l'exe ... c'est pas drole :lol:
05/08/2010, 14h13
Guulh
Ce que tu dis est vrai quand le compilo ne sait pas ce que tu concatènes ; mais il est assez malin pour traiter statiquement string bidule = "a + "b" (qui devient string bidule = "ab" dans le bytecode). Donc l'argument de la perte de perfs n'est pas valide :)

Et même, plutôt que de préfixer chaque ligne par " et la suffixer par " + , il est bien plus simple d'utilser ce cher @, de cette façon :
Code:

1 2 3 4 5 string request = @" SELECT trucmuche FROM TABLE_MACHIN ";
C'est tout de même plus simple.
05/08/2010, 14h22
tomlev

Citation:

Envoyé par Pol63

un stringbuilder est des centaines (des millions ?) de fois plus performant qu'un string pour de la concaténation

Oui et non... si c'est pour un nombre limité de concaténations (a priori quelques dizaines grand max dans un cas comme celui là), ça ne fait pas une grosse différence.

Par contre, si tu fais des concaténations en boucle, avec potentiellement un grand nombre d'itérations, là il faut absolument utiliser StringBuilder pour éviter de créer inutilement des tonnes d'instances de String de plus en plus longues...
05/08/2010, 14h27
Bluedeep

Citation:

Envoyé par Pol63

un stringbuilder est des centaines (des millions ?) de fois plus performant qu'un string pour de la concaténation

Je le sais, mais ici il n'a pas de concaténation à faire justement.
05/08/2010, 14h32
olibara

Pour repondre a l'avalanche des derniers post

1- Oui les vues et les procédures stocquées c'est bien, mais ca n'exclus pas de pouvoir aussi dans certains cas utiliser des requetes dans le code

2- Ce que j'ai montré est un exemple pour Mehdi_ab qui ne comprenait pas, mais il est evident que le but etait de pouvoir generer un code facilement manipulable et ne pas cammoufler un texte sql dans les ressources

2- Pas besoin de faire des AppendFormat pour utiliser un stringbuilder, l'intéret d'un stringbuilder, ormis la performance, c'est precisemént la facilité le cas echéant de pouvoir ajouter un AppendFormat sans devoir faire d'horribles manipulations sur le string.
Que ce soit sous la forme
""+""+""+""
ou sous la forme
@"
sdf
sdf
sdf
sdf";

Croyez moi, un stringbuilder rends les choses beaucoup plus simple et lisible

Et comme je l'ai dis, j'ai maitenant fait un petit outil a qui je peux donner une chaine SQL et qui me genere un code comme illusté
05/08/2010, 14h44
olibara

pour BlueDeep

Citation:

Je le sais, mais ici il n'a pas de concaténation à faire justement.

A bon ?

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

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

Moi quand je vois du code style

Code:

string machaine="abc"+a.Tostring()+" - "+"un bazar "+"+"+Mafonction().Tostring()+"\"quote\""+" "+DateTime.Now.Year.Tostring()+"/"+DateTime.Now.Month.Tostring();

J'attrappe la migraine et je cours chercher une aspirine ou j'envoie une paire de claque selon l'humeur ;)

Personnellement j'ai ajouter cette fonction dans un AddIn que j'ai codé pour la boite qui m'emploie. En l'occurence je construit une simple string, pas un stringbuilder, mais tu pourrais l'adapter.
Tu peux taper ca en macro plutot qu'en addin. Le principe étant simplement de prendre le texte du presse papier et de le modifier en ajoutant les "sb.AppendLine(", ....
Il faut également prendre en compte que la chaine collée peut contenir des caractères qu'il faut échaper (typiquement les guillemets).
Ma fonction est faite pour coller n'importe quel texte dans une variable string, ca n'est pas dédié au SQL, par contre après tu peux jouer avec les snippets pour qu'en 3 clique tu colle une requete et ca te sort un datatable prêt a l'emploie.
Et fait des AppendLine, pas seulement des Append. Ca parait con, mais au debug ca change tout :aie:
Avec un Append tu perd les retours a la ligne et la requete devient très vite illisible.

Mon code a adapter pour toi :
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 42 43 44 45 46 47 48 49 50 51 52 /// <summary> /// Colle du texte en tant que variable string. /// </summary> private void PasteToString() { string ToPaste = string.Empty; if (Clipboard.ContainsText() == false) return; string VariableDeclaration = "string strCode = "; string ConcatenationOperator = " + Environment.NewLine +"; string EndLine = Environment.NewLine; string EndDeclaration = ";"; string QuoteProtection = "\\\""; // Si on est en VB : if (String.Compare(_DTE.ActiveWindow.Document.Language, "Basic", true) == 0) { VariableDeclaration = "dim strCode as String = "; ConcatenationOperator = " & Environment.NewLine & "; EndLine = "_" + Environment.NewLine; EndDeclaration = ""; QuoteProtection = "\"\""; } ToPaste = Clipboard.GetText(); string[] lines = Regex.Split(ToPaste, "$", System.Text.RegularExpressions.RegexOptions.Multiline); // Supprime la dernière ligne qui est vide Collection<string> text = new Collection<string>(); foreach (string l in lines) text.Add(l); text.RemoveAt(text.Count - 1); // Génère le code ToPaste = string.Empty; for (int i = 0; i < text.Count; i++) { string modified = "\"" + text[i].Trim("\r\n".ToCharArray()).Replace("\\", "\\\\").Replace("\"", QuoteProtection) + "\""; if (i == 0) modified = VariableDeclaration + modified; if (i == text.Count - 1) modified += EndDeclaration; else modified += ConcatenationOperator + EndLine; ToPaste += modified; } TextDocument td = (TextDocument)_DTE.ActiveDocument.Object(""); td.Selection.Insert(ToPaste, (int)vsInsertFlags.vsInsertFlagsContainNewText); _DTE.ExecuteCommand("Edit.FormatSelection", ""); }
C'est pas parfait évidemment, mais ca dépanne bien ;)

EDIT: Ah oui, comme je disais plus haut, c'est dans un addin visual studio ca, donc a toi d'adapter, soit tu créer un addin aussi, soit tu transforme en macro. Le code d'une macro est quasi identique a celui d'un addin. Si tu as besoin de précisions sur mon code, je suis pas loin.

Pour répondre a toutes les bonnes volonté ici présente voici le petit bout de code
Ce n'est pas de la haute sophistication mais j'ai pondu ca en 10 minutes et c'est tres utile

N.B. : J'ai emballé cela dans une petite form qui gere les IO et perment de passer par le clipboard

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
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
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
 
namespace SqlFormatter
{
  class C_SqlFormat
  {
    // ********************************************************************
    public StringBuilder ParseSql(string src)
    {
      StringBuilder sb = new StringBuilder();
      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;
        }
        sb.AppendFormat("sb.Append(\" {0}\");{1}", st, Environment.NewLine);
      }
      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;
      }
    }
  }
}

05/08/2010, 15h01
olibara

Merci CtxNop

Nos codes se sont croisés je pense
05/08/2010, 15h07
ctxnop

Oui, pile poil quand j'ai cliqué sur "Envoyer la réponse" j'ai reçut le mail m'indiquant que tu avais posté ton code :aie:
05/08/2010, 15h21
Pol63

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

enfin on peut avec _ en bout de ligne mais c'est vraiment pas pratique et puis il faut aussi mettre les environment.newline partout

donc nous partout dans notre code c'est

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

donc pas super
j'ai entendu dire qu'avec vb2010 on aurait pas besoin de mettre de _ m'enfin de là à pouvoir écrire un string sur plusieurs lignes comme le montre guulh, je pense pas

d'aileurs à quoi sert le @que vous mettez devant les string chez vous ?

(tiens, une des rares fois où j'envie les c#iens ^^)
05/08/2010, 15h24
ctxnop

En vérité, même si le C# permet les \n pour faire des sauts de lignes, je ne les conseils pas.
Il ne faut pas oublier que via mono notemment le .Net est portable et que le retour à la ligne ne se fait pas partout pareil. Environment.NewLine est une string contenant la séquence propre au système sur lequel il s'exécute.

EDIt:
Je testerai ce que tu dis sur le VB2010.
Sinon le @ devant une string empêche la résolution des caractères d'échapement.
C'est pratique pour un chemin par exemple ou une expression rationnelle, on a pas besoin de faire \\ pour ajouter un simple \.
De même écrire @"\t" ca ne donnera pas une tabulation mais bien \t, comme si on avait écris "\\t"
05/08/2010, 15h33
Bluedeep
Citation:

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:

1 2 string myString = "AA_BB";
et
Code:

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 !!!!

Citation:

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.

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:

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:

"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.

05/08/2010, 15h35
Bluedeep

Citation:

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.
05/08/2010, 15h35
olibara
Citation:

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:

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
05/08/2010, 15h39
Bluedeep
Citation:

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:

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).
05/08/2010, 15h39
StormimOn

Citation:

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 :)
05/08/2010, 15h41
olibara

Citation:

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 :aie:

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
05/08/2010, 15h44
StormimOn
Citation:

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 :aie:

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:

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.
05/08/2010, 15h45
olibara

Pour BlueDeep et Stormimonn

Citation:

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
05/08/2010, 15h55
Pol63

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

Pour Rassurer Stormimonn

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

Exemple
Code:

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

05/08/2010, 16h06
Pol63

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
05/08/2010, 16h51
Guulh
Citation:
Envoyé par olibara

Pour Rassurer Stormimonn

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

Exemple

Code:

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.
05/08/2010, 17h34
ppphil

Citation:

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

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

Citation:

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 !

Citation:

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:

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;
      }
    }
  }
}

05/08/2010, 18h04
Guulh

Citation:

Envoyé par olibara

public StringBuilder ParseSql(string src, bool StormiGuulh)

:ccool:

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