Discussion :

[F.Saad]

Salut, je dois maintenant faire face a une commande SQL Qui plante, probablement(read 99.99%) a cause de .. sa taille

Code :

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

1
2
3
4
5
6
7
8
9
            foreach (string s in array)
            {
 
                SQLCMD.CommandText += "INSERT INTO MANGA (Name) VALUES ('" + s + "');";
                i += SQLCMD.ExecuteNonQuery();
                SQLCMD.CommandText = "";
                Program.proggy.statusStrip1.Items[0].Text = "Updated " + i + " Querries";
            }
            SC.Close();

l'array en question fait plus de 1200 Entrées toutes de gros strings.
Ma question est :
Comment traiter les grosses querry? Au début j'avais pensé a mettre tout les insert d'un seule coup mais ca avait l'air vachement gros, après j'ai pensé a tout divisé mais ça ne marche pas non plus. Donc si quelqu'un s'y connait en performance question SQLite...

Après ma deuxième question se porte sur mon formulaire, comme vous pouvez le voir le StatsStrip1 est censé se mettre a jour a chaque querry mais la forme plante

[olibara]

Deux conseils déja

1-

Utilise un StringBuilder plutot que de faire des concaténation de string avec +=

2- Construit UNE SEULE chaine de requete et execute en UNE SEULE fois

Code :

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

1
2
3
4
5
6
7
8
StringBuilder sb=new StringBuilder()
 
foreach (string s in array)
{
  sb.AppendFormat("INSERT INTO MANGA (Name) VALUES ('{0}');",s);
}
SQLCMD.CommandText = sb.ToString();
SQLCMD.ExecuteNonQuery();

[tomlev]

Après ma deuxième question se porte sur mon formulaire, comme vous pouvez le voir le StatsStrip1 est censé se mettre a jour a chaque querry mais la forme plante

Ca plante comment ? quelle est l'exception exactement ?
Tu exécutes ce code dans un thread ? si oui, il faut passer par la méthode Invoke pour mettre à jour l'interface graphique :

Code :

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

1
2
3
this.Invoke(
    new MethodInvoker(
        () => program.proggy.statusStrip1.Items[0].Text = "Updated " + i + " Querries"));

Tu peux aussi utiliser un BackgroundWorker, et mettre à jour l'interface dans l'évènement ProgressChanged (qui est déclenché sur le thread de l'interface graphique)

[F.Saad]

Deux conseils déja

1-

Utilise un StringBuilder plutot que de faire des concaténation de string avec +=

2- Construit UNE SEULE chaine de requete et execute en UNE SEULE fois

Code :

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

1
2
3
4
5
6
7
8
StringBuilder sb=new StringBuilder()
 
foreach (string s in array)
{
  sb.AppendFormat("INSERT INTO MANGA (Name) VALUES ('{0}');",s);
}
SQLCMD.CommandText = sb.ToString();
SQLCMD.ExecuteNonQuery();

Merci pour le conseil, quelqu'un pourrait m'expliquer le raisonnement derrière par contre? le but étant d'apprendre et non de faire marcher

Ca plante comment ? quelle est l'exception exactement ?
Tu exécutes ce code dans un thread ? si oui, il faut passer par la méthode Invoke pour mettre à jour l'interface graphique :

Code :

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

1
2
3
this.Invoke(
    new MethodInvoker(
        () => program.proggy.statusStrip1.Items[0].Text = "Updated " + i + " Querries"));

Tu peux aussi utiliser un BackgroundWorker, et mettre à jour l'interface dans l'évènement ProgressChanged (qui est déclenché sur le thread de l'interface graphique)

Non, tout est fait dans le thread principale, j'ai pensé a avoir recours au parallélisme, mais je n'arrive tjrs pas a passer des paramètres efficacement dans a mes fonction dans les threads

[tomlev]

on ne sait toujours pas comment ça plante... quelle est l'erreur obtenue ?

[olibara]

Merci pour le conseil, quelqu'un pourrait m'expliquer le raisonnement derrière par contre? le but étant d'apprendre et non de faire marcher

1- Les concaténation de chaine s1+=s; sont tres gourmandes en ressource, le string builder est dédié a ce genre d'operation, je te conseille de faire une petite recherche a ce sujet

2- Le deuxieme point me semble evident, plutot que d'executer 1200 requetes sequentiellement tu execute une seule requete et c'est le SQL engine qui se débrouille pour effectuer les 1200 insert le mieux qu'il peut

Mais comme le rappelle tres bien TomLev tu n'est pas tres precis sur le problème rencontré !