Convertir chaine sql en stringbuilder

**olibara** · 05/08/2010, 11h34

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

**Mehdi_ab** · 05/08/2010, 13h11

Bonjour,

Tu peux stp illustrer ton propos avec un exemple concret ?

Merci

**Pol63** · 05/08/2010, 13h28

je ne vois pas du tout ce que tu fais ou alors je ne vois pas l'utilité ...

**Pol63** · 05/08/2010, 13h31

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 :

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

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)

**olibara** · 05/08/2010, 13h33

Merci

Voici un exemple concret

Ceci

Code sql :

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

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
      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)");

**Bluedeep** · 05/08/2010, 13h58

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.

**Pol63** · 05/08/2010, 14h00

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

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

**Guulh** · 05/08/2010, 14h13

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 :

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

1
2
3
4
5
  string request = 
@"
SELECT trucmuche
FROM TABLE_MACHIN
";

C'est tout de même plus simple.

**tomlev** · 05/08/2010, 14h22

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

**Bluedeep** · 05/08/2010, 14h27

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.

**Pol63** · 05/08/2010, 13h59

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

**SetaSensei** · 05/08/2010, 14h00

Bonjour,

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

Convertir chaine sql en stringbuilder

C#

Vue hybride

Discussions similaires

Partager

Partager