Kald af stored procedure?

Hvis du ikke skal have et resultat sæt tilbage så

PageTextCmd.ExecuteNonQuery()

Ellers

SqlDataReader reader = PageTextCmd.ExecuteReader()

Du mangler at execute din stored procedure.

Det gøres enter ved en

PageTextCmd.ExecuteReader el.lign. resultatet af denne gives til en datareader

SqlDataReader dr = PageTextCmd.ExecuteReader();

While(dr.read());

UPs -

de enkelte felt i resultatet kan du så give til en StringCollection eller andet.

Lars

Men jeg får en fejl i denne linie hver gang jeg prøver at builde....

SqlCommand PageTextCmd = new SqlCommand("ViewPageTexts", ConnStr);

Hvilken build fejl får du?

The best overloaded method match for 'System.Data.SqlClient.SqlCommand.SqlCommand(string, System.Data.Sql.Client.SqlConnection)' has some invalide arguments

For at angive en SP skal du have [] rundt om strengen -

SqlCommand PageTextCmd = new SqlCommand("[ViewPageTexts]", ConnStr);

Lars

Hmm jeg får den samme fejl besked?

Min SP...

CREATE PROCEDURE ViewPageTexts (
  @Language INT,
  @PageName CHAR
)
AS
SELECT [dbo].[PageText].[Field], [dbo].[PageText].[Text]
FROM [dbo].[PageText]
WHERE PageName = @PageName
AND Language = @Language
AND Status = 1
GO

Det har intet med din stored procedure at gøre. Hvis du læser dokumentationen for SqlCommand, så er der ingen constructor der tager to strenge som argumenter, men derimod en der tager en streng og en SqlConnection. Det du skal gøre er....

SqlConnection connection = new SqlConnection(dinConnnectionStrengHer);
SqlCommand command = new SqlCommand("ViewPageTexts", connection);

/*
Du har ganske ret, jeg glemte at instatiere min connection string som en SqlConnection
Her er hvad jeg har gjort....
*/

private static string ConnStr = System.Configuration.ConfigurationSettings.AppSettings["ConnStr"];

public static DataSet getTexts(int Language, string PageName)
{
    SqlConnection Conn = new SqlConnection(ConnStr);
   
    DataSet ds = new DataSet();
   
    SqlCommand PageTextCmd = new SqlCommand("[getPageTexts]", Conn);
    PageTextCmd.CommandType = CommandType.StoredProcedure;
    PageTextCmd.Parameters.Add("@Language", SqlDbType.Int).Value = Language;
    PageTextCmd.Parameters.Add("@PageName", SqlDbType.Char).Value = PageName;

    Conn.Open();
   
    SqlDataAdapter da = new SqlDataAdapter(PageTextCmd);
   
    Conn.Close();
   
    return ds;
}

/*
Tak for hjælpen!

Kan du eventuelt guide mig i en ordenlig exception håndtering i ovenstående?
(Jeg er stadig grøn! :)
*/

Forresten points til dig 'z42cool' ! Tak!

Jeg smider normalt en Try Catch udenom open delen.

Jeg bruger Finally til at smide Command objektet (cmd.Dispose) væk og lukke forbindelsen igen.

/*
Jeg havde forstillt mig....
*/

private static string ConnStr = System.Configuration.ConfigurationSettings.AppSettings["ConnStr"];

public static DataSet getTexts(int Language, string PageName)
{
    try
    {
        SqlConnection Conn = new SqlConnection(ConnStr);
   
        DataSet ds = new DataSet();
   
        SqlCommand PageTextCmd = new SqlCommand("[getPageTexts]", Conn);
        PageTextCmd.CommandType = CommandType.StoredProcedure;
        PageTextCmd.Parameters.Add("@Language", SqlDbType.Int).Value = Language;
        PageTextCmd.Parameters.Add("@PageName", SqlDbType.Char).Value = PageName;

        Conn.Open();
   
        SqlDataAdapter da = new SqlDataAdapter(PageTextCmd);
    }
    catch (Exception e)
    {
        // do something
        string exceptionCaught = e.ToString();
    }
    Finally
    {
        PageTextCmd.Dispose();
        Conn.Close();
    }
    return ds;
}

/*
Men er ikke helt sikker???
*/

Præcist. Jeg bruger dog normalt en Trace.Write i catch-delen. Enten det eller smider jeg fejlen ned i database, det gør det nemmere at konstere fejl.

Tjo... Min ide var sådan set at vise brugeren er venlig fejlside, smide fejlen i db, og en sende mig en email.
Tak for hjælpen!