C# erreur : chaîne d'Entrée n'était pas dans un format correct

Je recevais ce message d'erreur: "la chaîne d'Entrée n'était pas dans un format correct."

Voici mon Code:

    private void UpdatePOdetailBalance(int Qty)
    {
        int newbal;

        SqlCommand com = new SqlCommand();

        com.Connection = cn;

        newbal = Convert.ToInt16(txtQtyOrdered.Text) - Qty;
        com.CommandText =
            "UPDATE PODetail SET BalanceQty="+ newbal +" WHERE OrderID=" +
             Convert.ToInt16(txtPONumber.Text) + "AND ItemID=" +
             Convert.ToInt16(txtItemNo.Text);


        com.ExecuteNonQuery();

    }

    private void btnOK_Click(object sender, EventArgs e)
    {

            UpdatePOdetailBalance(Convert.ToInt16(txtQuantity.Text));

    }

Je veux calculer la newbal qui est égal à txtQtyOrdered moins Qté mais j'obtiens cette erreur s'il vous plaît m'aider avec cela. Merci.

Veuillez utiliser à l'aide de blocs autour de vos commandes et apprendre à utiliser de lier des paramètres. Pour le court terme, l'impression de votre chaîne sql, il vous manque un vide dans le bon endroit.

InformationsquelleAutor user2874217 | 2013-10-12

c#format sql visual-studio

4

Le problème indiqué par votre message d'erreur est probablement l'une des lignes que d'essayer de convertir la valeur dans les zones de texte pour un entier court. Sans aucune vérification, la valeur saisie par l'utilisateur de votre pourrait être n'importe quoi, mais un certain nombre et vous obtenez ce message d'erreur (par exemple, si l'utilisateur quitte la zone de texte vide).

Vous devriez essayer de vérifier si les zones de texte contenu peut être convertie en un court valide entier en utilisant TryParse avant de tenter d'exécuter la requête
```
int ordered;
if(!int16.TryParse(txtQtyOrdered.Text, out ordered))
{
    MessageBox.Show("Invalid number for Ordered quantity");
    return;
}
int orderID;
if(!int16.TryParse(txtPONumber.Text, out orderID))
{
    MessageBox.Show("Invalid number for OrderId");
    return;
}
int itemID;
if(!int16.TryParse(txtItemNo.Text, out itemID))
{
    MessageBox.Show("Invalid number for ItemID");
    return;
}
```
À ce stade, vous pouvez exécuter votre calcul à l'aide de la conversion d'entiers courts, puis écrivez votre requête de cette façon (en ajoutant un espace avant le ET)
```
  com.CommandText =
        "UPDATE PODetail SET BalanceQty="+ newbal.ToString() +
        " WHERE OrderID=" + orderID.ToString() + 
        " AND ItemID=" + itemID.ToString();
```
Mais la concaténation de chaîne de texte de la requête et de l'entrée utilisateur n'est jamais conseillé comme une bonne pratique (dans votre cas est sans danger, car si la conversion est réussie, vous n'avez pas à vous soucier de l'Injection Sql, mais ne prenez pas l'habitude de le faire).

Donc, la meilleure façon d'écrire cette requête est grâce à l'utilisation d'une requête paramétrée
```
  com.CommandText =
        "UPDATE PODetail SET BalanceQty=@newbal " +
        " WHERE OrderID=@orderID " + 
        " AND ItemID= @itemID"

  com.Parameters.AddWithValue("@newbal", newBal);
  com.Parameters.AddWithValue("@orderID", orderID);
  com.Parameters.AddWithValue("@itemID", itemID);
  com.ExecuteNonQuery();
```
Comme un bon article sur requête Paramétrée et pourquoi les utiliser, je vous suggère de la lecture de ces vieux mots de Jeff Atwood

InformationsquelleAutor Steve
2

Je vous recommande de faire des changements en fonction de la suite de la révision du code des suggestions basées sur le code (énumérées dans l'ordre de valeur (coût/bénéfice de la "fixation")):
1. Cette méthode, qui est de l'accès à une base de données ne devrait pas être la lecture de contrôles pour obtenir ses valeurs. Au lieu de cela, il faut un gestionnaire d'événements, comme un clic sur un bouton, qui analyse les valeurs des autres commandes, à l'aide de TryParse, comme gregjer répondu. En séparant l'INTERFACE utilisateur et code de Données, la couche d'accès aux données est plus facile à tester et par l'analyse de la surface (à la couche d'INTERFACE utilisateur) exceptions portant sur la mauvaise saisie de l'utilisateur sera pris dès que possible.
2. SQL dynamique via des chaînes de caractères dans la base de données ou dans la couche d'accès aux données w/je .NET est ouvert à injection SQL. Vous êtes en train de résoudre cette question par l'analyse du texte, si impressionnant travail par vous. MAIS, cela a déjà été traitée par le .NET de l'équipe en fournissant des commandes paramétrées. Consultez le MSDN SqlCommand.Paramètres ou voir ici pour un bref, y compris la façon dont un en consommant développeur groks ce sujet: Quand faut "SqlDbType" et "taille" être utilisé lors de l'ajout SqlCommand Paramètres?
3. L'appellation Variable. Au lieu de la Quantité, de la norme .NET conventions de nommage ferait appel à la quantité, en camelcase, car il est un paramètre et la totalité de l'homme au nom de la langue, pas une abréviation ou d'une abréviation, en particulier pour les visibles bits. IntelliSense fait longtemps noms de variables n'est pas un problème. Depuis .NET est difficile à manier en utilisant simplement le bloc-notes, il faut supposer que les autres développeurs à l'aide d'un IDE comme VisualStudio ou SharpDevelop, afin d'utiliser des noms significatifs.
4. Procédures stockées doivent être utilisés. Chaque fois que SQL est exécutée, le Serveur SQL doit vérifier sa commande cache très peu, mais si la commande a été vidés de la mémoire cache, la commande SQL doit être interprété et encached (mise en cache). Ainsi que le fait que l'utilisation d'une procédure stockée "expédition" moins d'octets à chaque appel à la base de données.
- Son code n'a rien à faire du SQL Dynamique (il est ouvert à l'injection SQL à partir d'un autre raison)
InformationsquelleAutor paegun
1

Vous avez besoin de mettre un espace avant le "ET" et que vous tentez de convertir une chaîne en un entier qui n'est pas un entier.
- Et c'est ça? Qu'en dire quelque chose sur les mauvaises pratiques de l'été utilisé ici?
- Oui il y a de mauvaises pratiques, mais il est apparemment un jeune programmeur de l'apprentissage, donc au lieu de casser quelqu'un en bas en pointant du doigt les mauvaises pratiques, pourquoi ne pas l'aider à obtenir par le biais de ses besoins immédiats, et puis il peut apprendre sur les bonnes/mauvaises pratiques.
- Pas de. Une ligne n'est pas une réponse adéquate dans ce cas. Vous pouvez remarquer les mauvaises pratiques sans casser quelqu'un en bas!!
- D'accord, vous pouvez aider les gens à apprendre avec les briser. Ma réponse a été vite pendant la cuisson déjeuner pour toute la famille. Il existe d'autres bien pensé en profondeur des réponses sur ce fil. Merci pour vos commentaires. Je le ferai dans l'avenir, être plus réfléchie sur mes réponses.
- Désolé si j'ai été à la dure. C'est une question posée dans les méta-hier et se rapporte à cette discussion- meta.stackexchange.com/questions/200420/...
InformationsquelleAutor David Yancey
0

Cette erreur signifie que la chaîne que vous essayez de convertir n'est pas un entier.
Essayez d'utiliser des int.TryParse
```
int newbal;

if(int.TryParse(txtQtyOrdered.Text, out newbal))
    newbal = newbal - Qty;
```
de même avec les autres textes que vous essayez de convertir

... et ajouter un espace avant " ET qui va générer d'erreur suivante

InformationsquelleAutor gregjer
0

Je pense que vous avez besoin de déboguer votre code. Lors du débogage copie de votre requête à partir de la "com".CommandText" et collez-le dans SQL Server, vous trouver l'erreur

Il n'existe qu'une requête d'erreur rien d'autre...
Peut-être txtQtyOrdered valeur n'est pas entier, il y a aussi besoin d'espace vide "ET ItemID=" pour " ET ItemID="

Merci,

Taha

InformationsquelleAutor taha ahmed
0

Première il Vous manque un espace avant "ET"
1. Vous devriez essayer pour analyser les valeurs avant la mise à jour de la déclaration.
2. Vous devez décider ce que vous voulez faire dans le cas où l'entrée de la zone de texte n'était pas dans le bon format, plutôt que de simplement obtenir une exception lorsque vous essayez de mettre à jour.
3. Ce n'est pas la bonne façon de chaînes de format, Vous devez utiliser string.Format
- Vous avez raison de cette chaîne.le format est mieux que la concaténation de chaîne, cependant lorsque vous traitez avec les instructions de requête et de l'aide de la commande objet, il est toujours préférable d'utiliser des paramètres.
InformationsquelleAutor Yosi Dahari
0

vous pouvez parfois rencontrer ce problème lorsque vous avez de multiples paramètres et utilisez Oracle ou DB2 bases de données. Ils ne pas en charge les paramètres nommés ou il n'est pas allumé.

Oracle:
```
    Dim cmd As OracleCommand = DirectCast(connection.CreateCommand, OracleCommand)
    cmd.BindByName = True
```
Assurez-vous de paramètres sont ajoutés à l'objet de commande dans le même ordre que l'instruction sql

InformationsquelleAutor Eric Schneider

Vous devez vous connecter pour publier un commentaire.