Discussion :

[Xanime]

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.

[davcha]

nombreVitres = (long)cmd.ExecuteScalar();

[Xanime]

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

[Thomas Lebrun]

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

[SaumonAgile]

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

[Xanime]

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

[Thomas Lebrun]

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

[Xanime]

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

[SaumonAgile]

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

[Xanime]

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.

[Xanime]

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

[Thomas Lebrun]

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

[Xanime]

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

[Thomas Lebrun]

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

[Xanime]

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

[davcha]

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 :

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

[SaumonAgile]

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.

[SaumonAgile]

Ca c'est bien :

Code :

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

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

[Xanime]

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.

[SaumonAgile]

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.