Problème pour récupérer un int en C# sur une bdd oracle

Bonjour,
je travaille en ce moment même sur une base de donnée oracle 10.g, je doit coder un web service en C# qui y accede, le problème est que lorsque je désir e récupèrer un long dans la base de donnée le programme s'arrete. voici le code :

[WebMethod]
public long recupereNombreVitrage(int numeroFenetre, int numeroLot)
{
long nombreVitrage;

// Requête SQL a exécuter
string sQuery = "SELECT nombre_vitrage from cadre WHERE numero_fenetre ="
+ Convert.ToString(numeroFenetre) + " AND numero_lot="
+ Convert.ToString(numeroLot);

// Instanciation de l'objet OracleCommand et de ses propriétés
OracleCommand cmd = new OracleCommand(sQuery);
cmd.Connection = OConnexion;
OConnexion.Open();
// Lecture de la requête SQL
OracleDataReader reader = cmd.ExecuteReader();
reader.Read();
String h=reader.GetDataTypeName(0);
nombreVitrage = reader.GetInt64(0);

// Fermeture de la connexion
OConnexion.Close();
// Libération des ressources
OConnexion.Dispose();
return nombreVitrage;

}

lorsque je recupère un string le problème ne survient pas, j'aimerais donc votre avis sur le sujet merci d'avance.

nombreVitres = (long)cmd.ExecuteScalar();

Merci mais après modification le web service ne marche toujours pas :cry: . Je récupère bien la donnée mais c'est au moment de getInt64() que le web service s'arrete.

Vire la ligne nombreVitrage = reader.GetInt64(0); : tu n'en as plus besoin si tu utilises le code de davcha ;)

Plutot que de concaténer ta requête avec les valeurs des paramètres, je te conseille d'utiliser des OracleParameter. Cela te permet de gérer correctement les paramètres liés à la culture (FR, EN, etc).
Autre chose très importante, Oracle analyse les requêtes que tu lui envoies et les met en cache.

En concaténant les variables dans la chaine directement, ça oblige l'instance à analyser la requête à chaque fois, alors que si tu utilises des OracleParameter, tu envoies la même requête à chaque fois avec des valeurs de paramètres différentes. La requête ne sera analysée qu'une fois. Je te laisse imaginer le gain au niveau des performances... :yaisse2:

j'avais retirer nombreVitrage = reader.GetInt64(0); mais sa ne marchais toujours pas

Quelle est l'erreur ? La ligne qui pose problème ?

lorsque je récupère un long ou même un int lorsque le web service arrive a la ligne nombreVitrage = reader.GetInt64(0); il s'arrete directement après l'avoir éxecuter

Peux tu poster le corps de la méthode ? car on ne sait plus ce que tu exécutes exactement... 8O

il fait de même avec les méthodes getInt32(0) et getString(0), alors que dans une autre méthode lorsque j'utilise getString pour récupérer une chaine de caractère contenu dans la bdd oracle ça marche, donc je suis en peu perdu.

cette méthode ne marche pas :

[WebMethod]
public long recupereNombreVitrage( int numeroFenetre, int numeroLot)
{
long nombreVitrage;

// Requête SQL a exécuter
string sQuery = "SELECT nombre_vitrage from cadre WHERE numero_fenetre ="+ Convert.ToString(numeroFenetre) + " AND numero_lot="
+ Convert.ToString(numeroLot);

// Instanciation de l'objet OracleCommand et de ses propriétés
OracleCommand cmd = new OracleCommand(sQuery);
cmd.Connection = OConnexion;
OConnexion.Open();

// Lecture de la requête SQL
OracleDataReader reader = cmd.ExecuteReader();
reader.Read();
nombreVitrage = reader.GetInt64(0);

// Fermeture de la connexion
OConnexion.Close();

// Libération des ressources
OConnexion.Dispose();

return nombreVitrage;
}

alors que celle-ci marche :

[WebMethod]
public String recupereTypeCadre(int numeroLot,int numeroFenetre,int numeroCadre)
{
String typeCadre;

// Requête SQL a exécuter
string sQuery = "SELECT type_cadre FROM cadre WHERE numero_Lot ="+Convert.ToString(numeroLot)+"AND numero_fenetre="+Convert.ToString(numeroFenetre)+"AND numero_cadre ="+Convert.ToString(numeroCadre);

// Instanciation de l'objet OracleCommand et de ses propriétés
OracleCommand cmd = new OracleCommand(sQuery);
cmd.Connection = OConnexion;

//cmd.CommandType = CommandType.Text;
OConnexion.Open();

// Lecture de la requête SQL
OracleDataReader reader = cmd.ExecuteReader();
reader.Read();
typeCadre=reader.GetString(0);


// Fermeture de la connexion
OConnexion.Close();

// Libération des ressources
OConnexion.Dispose();

return typeCadre;
}

Si cela plante, c'est peut-être simplement car ta requête ne renvoit aucun résultats ?

string sQuery = "SELECT nombre_vitrage from cadre WHERE numero_fenetre ="+ Convert.ToString(numeroFenetre) + " AND numero_lot="
+ Convert.ToString(numeroLot);

la methode retourne bien quelque chose car l'attribut reader.VisibleFieldCount=1

Cette propriété retourne : "Obtient le nombre de champs dans DbDataReader qui ne sont pas masqués." donc, d'après moi, pas forcément que tu as des résultats....


Essaye avec la propriété Count

Les attributs reader.m_resultCount et reader.m_fieldCount retourne la valeur 1.

Récupérer une donnée scalaire (sur une seule ligne et une seule colonne, donc) ça se fait avec ExecuteScalar, pas avec ExecuteReader.
Bien sûr, c'est faisable avec ExecuteReader, mais ça bouffe davantage de ressources, ça "pollue" inutilement le code, pas top pour la lisibilité, etc...

Ca c'est bien :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21

public string recupereTypeCadre(int numeroLot, int numeroFenetre, int numeroCadre) { using (OracleCommand command = new OracleCommand(@" SELECT type_cadre FROM cadre WHERE numero_Lot = :pnumero_Lot AND numero_fenetre = :pnumero_fenetre AND numero_cadre = :pnumero_cadre", OConnexion)) { command.Parameters.Add("pnumero_Lot", OracleType.Int).Value = numeroLot; command.Parameters.Add("pnumero_fenetre", OracleType.Int).Value = numeroFenetre; command.Parameters.Add("pnumero_cadre", OracleType.Int).Value = numeroCadre;   try { OConnexion.Open(); return ( string )command.ExecuteScalar(); }catch(....){ throw; } finally { OConnexion.Close(); } } }

---

Dans ton code il y a quelque chose qui me choque beaucoup, c'est : OConnexion.Dispose();
Tu libères un objet dans une méthode qui n'a pas instancié cet objet.

Y'a de bonnes chances que ton problème vienne de là : tu récupères d'abord ton string, ce qui fonctionne puisque OConnexion existe. Et puis tu essaies de récupérer un long, mais OConnexion est, au mieux, marqué comme étant disposé, et donc non utilisable, au pire il n'existe déjà plus, et donc non utilisable, et ça plante.

tu devrais toujours tester si une variable/un champ/ est défini avec par exemple des test de type '!= null' ou 'IsDbNull'.
Dn plus, si une méthode renvoie un booléen pour dire s'il y a des données ou pas (DataReader.Read() par exemple), le minimum c'est déjà de s'en servir, pour éviter le genre de problèmes que tu rencontres. ;)

Citation:

---

Envoyé par davcha

Ca c'est bien :

Code:

---

1 2 3

public string recupereTypeCadre(int numeroLot, int numeroFenetre, int numeroCadre) { ... }

---

---

Ce code est magnifique, ça tirerait une larme à un inspecteur des impots... Je rève de rencontrer ce genre de code dans mon appli au boulot :)

j'ai essayer votre méthode qui retourne un string et ça marche bien mais quand je la modifie pour récupérer un int ça me refait la même erreur voici le code :

[WebMethod]
public int recupereNumeroCadre(int identifiantTAG)
{
using (OracleCommand command = new OracleCommand(@" SELECT numero_cadre FROM tag WHERE identifiant_tag = :pidentifiant_tag", OConnexion))
{
command.Parameters.Add("pidentifiant_tag", OracleDbType.Int32).Value = identifiantTAG;
try
{
OConnexion.Open();
return (int)command.ExecuteScalar();
}
catch (Exception Ex)
{
throw;
}
finally
{
OConnexion.Close();
}
}
}

donc je me demandais si ce ne serais pas possible que se soit un problème dans mes tables plutôt qu'une erreur dans le code.

Tu dois appeler la méthode DataReader.GetQuelquechose() qui correspond au type de la donnée dans la base de données.
Par exemple, tu ne peux pas faire un GetInt32() si le champ dans la table est de type NVARCHAR ou DATETIME.

Si le champ dans la table est un type chaine, tu dois appeler GetString();
Si ensuite tu veux convertir la valeur en un numerique, libre à toi, mais tu ne peux pas le faire en appelant GetInt32(). Il faut convertir à la main après, ou bien utiliser l'instruction T-SQL CONVERT pour convertir depuis la base de données ta valeur VARCHAR en une valeur numérique.

J'ai essayer d'utiliser une autre façon de récupérer mes donnée avec un dataView mais lorsque je lance mon web service j'obtien cette erreur :

Erreur du serveur dans l'application '/webserviceAccesBDD'.
--------------------------------------------------------------------------------

Pour être sérialisables XML, les types qui héritent de ICollection doivent avoir une implémentation de Add(System.Data.DataRowView) à tous les niveaux de leur hiérarchie d'héritage. System.Data.DataView n'implémente pas Add(System.Data.DataRowView).

et voici le code de la methode:

DataView monDataView = new DataView();
OConnexion.Open();
OracleDataAdapter monDataAdapter=new OracleDataAdapter(" SELECT * FROM tag ", OConnexion);
DataSet monDataSet = new DataSet("test");
monDataAdapter.Fill(monDataSet, "test");
monDataView = monDataSet.Tables["tag"].DefaultView;
OConnexion.Close();
return monDataView;

Merci beaucoup pour votre aide, j'ai finalement résolu mon pb en utilisant le type decimal et getdecimal à la place de int et getint32 :king: