Procédure ou fonction sp_select_companydetails a trop d'arguments spécifiés

Question

Je suis vraiment déprimé à propos de cette erreur tout le code de fonction est correct mais toujours son me donnant l'erreur, j'essaye de sélectionner l'information de la base de données de SQL Server.

procédure stockée:

create procedure sp_select_companydetails 
    @id varchar(5) 
as 
begin 
    select company_name, company_address 
    from CompanyDetails 
end 

code C#:

2) sur le bouton formulaire clic événement

string id = "1"; 
cmd.CommandType = CommandType.StoredProcedure; 
cmd.CommandText = "sp_select_companydetails"; 
cmd.Parameters.Add("@id", id); 
FillDataset(); 

En classe

public DataSet FillDataset() 
{ 
    try 
    { 
     using (cmd) 
     { 
      DataSet ds = new DataSet(); 
      cmd.Connection = con; 

      SqlDataAdapter da = new SqlDataAdapter(cmd); 
      da.Fill(ds); 

      cmd.Parameters.Clear(); 
      return ds; 
     } 
    } 
    catch (Exception) 
    { 
     throw; 
    } 
} 

Lorsque je clique sur le bouton de formulaire J'ai cette erreur:

Procedure or function sp_select_companydetails has too many arguments specified.

me suggérer une bonne solution

Nous vous remercions à l'avance

Answer 1

Au lieu d'utiliser un SqlCommand global, créez comme nouveau chaque fois que vous en avez besoin. Cette approche est recommandée lorsque vous traitez avec des objets jetables

using (SqlConnection con = new SqlConnection(GetConnectionString()) 
using (SqlCommand myCmd = new SqlCommand("sp_select_companydetails", con)) 
{ 
    cmd.CommandType = CommandType.StoredProcedure; 
    cmd.Parameters.Add("@id", SqlDbType.VarChar).Value = "1"; 
    DataSet ds = new DataSet(); 
    SqlDataAdapter da = new SqlDataAdapter(cmd); 
    da.Fill(ds); 
    return ds; 
} 

Par ailleurs, nous ne pouvons pas voir comment créer SqlConnection, mais il semble que vous avez un autre objet jetable maintenu au niveau mondial. Ceci est particulièrement désagréable avec une SqlConnection parce que cet objet garde des références aux ressources du système à la fois sur le client et sur le serveur. Ne créez pas d'objets de connexion globaux, créez simplement une méthode globale qui renvoie la chaîne de connexion en cours à utiliser dans la création de SqlConnection locale (GetConnectionString() dans mon exemple ci-dessus). Si vous pensez que c'est un tueur de performance, je vous suggère de lire à propos de the concept of connection pooling