vous le savez : Oracle me les casse, et royalement. sa dernière lubie : une erreur ORA-00947 qui me sort de je ne sais pas ou. pour info, cette erreur se déclenche quand le nombre de colonnes dans une requete INSERT est différent du nombre de valeurs qu'on insère. sauf que dans mon cas, c'est exactement le même... Oracle, O désespoir...
 
voici le code de mon appli C# qui s'occupe de l'insert, à partir de données contenues dans un fichier csv :
 

[...]
using (StreamReader sr = new StreamReader(file))
                    {
                        using (OracleConnection conn = new OracleConnection(strConnection))
                        {
                            try
                            {
                                conn.Open();
                                command = conn.CreateCommand();
                                String ligne = null;
                                String[] champs = null;
                                String delim = ";";
                                char[] delimTab = delim.ToCharArray();
                                pbImport.Visible = true;
                                pbImport.Minimum = 0;
                                pbImport.Maximum = CountLines(sr);
                                pbImport.Value = 0;
                                pbImport.Step = 1;
                                file.Seek(0, SeekOrigin.Begin);
 
                                transaction = conn.BeginTransaction(IsolationLevel.ReadCommitted);
                                command.Transaction = transaction;
 
                                while ((ligne = sr.ReadLine()) != null)
                                {
                                    champs = ligne.Split(delimTab);
                                    importOracle(champs);
                                    pbImport.PerformStep();
                                }
                            }
                            catch (InvalidOperationException ex)
                            {
                                MessageBox.Show("Connection non établie !" );
                                transaction.Rollback();
                                return;
                            }
                            catch (OracleException ex)
                            {
                                MessageBox.Show("Erreur Oracle : " + ex.Code.ToString());
                                transaction.Rollback();
                                return;
                            }
                            catch (Exception ex)
                            {
                                MessageBox.Show(ex.Message);
                                transaction.Rollback();
                                return;
                            }
                            transaction.Commit();
                        }
                    }
                    MessageBox.Show("Importation terminée avec succés.", "Importation", MessageBoxButtons.OK, MessageBoxIcon.Information);
                }
            }
        }
 
        private void importOracle(string[] lines)
        {
            try
            {
                string req = "INSERT INTO shema.table (col1, col2, col3, col4, col5, col6, col7, col8, col9) VALUES ("
                            + "'STOCK','"
                            + lines[0] + "','"
                            + "CLI',"
                            + "1,"
                            + lines[1] + ","
                            + lines[1] + ",'"
                            + "E','"
                            + txtDateDeb.Text + "','"
                            + txtDateFin.Text + "')";
                command.CommandText = req;
                command.ExecuteNonQuery();
            }
            catch (InvalidOperationException ex)
            {
                throw;
            }
            catch (OracleException ex)
            {
                throw;
            }
        }

rien de bien méchant comme vous le voyez.... une caisse de Bordeaux à celui qui me dépatouille de ce merdier.
je soupçonne un bug de la classe OracleClient de .NET, mais bon, sait on jamais..
 
à vot bon coeur m'sieurs dames
 
edit: les noms des champs ne sont évidemment pas ceux que j'ai indiqués

y'a pas de prepared statements en c#?

(et figure toi que ça a un rapport avec la question, parce que normalement, il devrait refuser d'executer la requete si tous les params ne sont pas bindés, avant qu'oracle ait qqch à y dire)

si si, y'a aussi les requetes paramétrées en C#, j'avais utilisé ça en tout premier lieu, mais ça faisait la même erreur
j'ai donc essayé avec le code que j'ai posté, sans plus de succés

Tu as vérifié que la requête est bien celle qui parvient au bouzin, aka Oracle, et que tous les paramètres sont bien là (log Oracle)
 
Quid si tu exécutes la requête depuis SQLPlus / Toad ?
 
Sans vouloir t'insulter et être TT pour 3 semaines : tu as bien affiché "req" et vérifié qu'elle est bien formée (on sait jamais, hein) ?
 

oui, tout part chez Oracle, aucun souci là dessus, j'ai vérifié (c'est moi l'admin après tout bordel )
la requete marche sous Toad, et req contient la requete correcte ! j'ai vérifié tout ça, tu penses bien

bon, j'ai compris, faut que je me démerde
je vais recoder l'appli en Java histoire de vérifier si ça vient de .NET ou d'Oracle

a tout hasard, t'as testé le statut de "conn" pour voir son état ?

 
essaie de virer ton niveau d'isolation dans ta transaction (ne lui passe pas de parametre)
transaction = conn.BeginTransaction()

Y aurait pas un caractère en trop dans lines[1] ?

une quote qui manque ou une virgule perdue ?
prepared stmt bordel

Dans lines[0] tu n'as pas de valeur avec une ' ? Dans ce cas il faut la doubler.
Et si col8 et col9 ont de types DATE, il faut mieux faire un "TO_DATE("+txtDateFin.Text1",'DDMMYYY')".
 
(J'aime bien le Pomerol d'une 10ène d'années par caisse de 12   )

pardon, mais si il a vraiment essayé avec un prepared statement avant, ta caisse tu peux te la foutre au cucul

 
Une requête SQL avec des paramètres de type DATE sans TO_DATE et des chaînes de caratères sans aucun traitement, tu peux la mettre au même endroit.  

mais pas dans un prepared statement, broucouille
 
(ou alors l'implementation en c# est toute moisie ? )

bon, c'est réglé ! y'avait une tabulation dans le fichier csv qui foutait tout en l'air.
par contre : j'importe 12484 lignes, et c'est super lent. en effet, il faut une bonne heure pour tout importer.
quand je me suis aperçu de la présence du tab, et que le fait de l'enlever permettait au logiciel de refonctionner, j'ai remis ma requete paramétrée.
j'ai lu que ADODB (sur lequel repose la classe OracleConnection de .NET) était super lent et très peu optimisé. ceci explique peut etre celà.
néanmoins, que pensez vous de l'utilité du try/catch dans ma fonction importOracle() ? comme vous voyez, une fois l'exception catchée, je la rebalance directement à l'appelant, sans aucun traitement. ceci pour avoir une stacktrace la plus complète possible en cas de plantage.
par contre, est-ce vraiment utile finalement ? parce que de toute façon, si une exception doit péter, elle sera quand même catchée dans ma fonction principale. que pensez vous de virer le try/catch de la fonction importOracle() en terme de gain de vitesse ?

 
MSDN:

 
perdu

euh, ce serait pas un parfait exemple pour utiliser sql loader, plutôt?

ben oui, mais que veux tu... ordre de mon chef : "ponds moi une appli facile à utiliser dans le langage que tu veux pour balancer un fichier csv de plusieurs milliers de lignes dans la base. pense surtout que ce sont des gens qui ne connaissent rien à l'info qui utiliseront ton programme"
 
donc bon, SQL Loader pour un end user, voilà quoi
(ou alors je code un front end)

 
Le front end me parait pas forcément une mauvaise idée...

bon, jpense effectivement que je vais coder un front end pour sql loader, parce que plus ça va, plus c'est lent...
ce qui me tue, c'est que l'insertion des 2000 premieres lignes environ est hyper rapide, puis ensuite ça rame de plus en plus pour arriver péniblement à 12484 au bout d'une heure et demie
 
questions à propos de sql loader :  
- ça gère les transactions ? (j'ai pas trop envie de péter mes données si jamais ça plante en plein traitement)
- dans mon fichier csv, si je veux insérer que les champs 4, 5 et 9, je spécifie ça comment dans le fichier de controle ?
 
merci
 

je me réponds à moi même : il faut faire suivre toutes les colonnes que je ne veux pas insérer par le mot FILLER dans le fichier de controle
par contre, il ne semble pas possible de déclarer tout le chargement d'un fichier comme une seule transaction, ça craint du boudin
je pense donc que j'importerais tout dans une table temporaire, et si j'ai aucun rejet, alors je ferais l'import réél

j'ai utilisé que ponctuellement, mais oui, il vaut mieux passer par une table temporaire sans aucune contrainte avec ce machin...