Problem With '{ }'

[Saba02]

Hi all, i used this code,but it has this error:{ expected OR } expected in,i bold place of this error occured ,i tried resolve this errors but i couldn't. :doh:

public bool ValidateNewUser(string strAlias, string strFirst, string strLast)
{

    SqlConnection MyConn = new SqlConnection(ConfigurationSettings.AppSettings\["MyOnlineShoppingConnectionString2"\]);

    SqlCommand MyCmd = new SqlCommand("sp\_CheckForDuplicates", MyConn);

    MyCmd.CommandType = CommandType.StoredProcedure;

    SqlParameter objParam1;
    SqlParameter objParam2;
    SqlParameter objParam3;

    SqlParameter objReturnParam;

    objParam1 = MyCmd.Parameters.Add("@UserName", SqlDbType.VarChar);
    objParam2 = MyCmd.Parameters.Add("@FirstName", SqlDbType.VarChar);
    objParam3 = MyCmd.Parameters.Add("@LastName", SqlDbType.VarChar);
    objReturnParam = MyCmd.Parameters.Add("@Duplicates", SqlDbType.Int);
    objReturnParam.Direction = ParameterDirection.ReturnValue;

    objParam1.Value = strAlias;
    objParam2.Value = strFirst;
    objParam3.Value = strLast;

    try
    {
        if (MyConn.State == ConnectionState.Closed)
        {
            MyConn.Open();
            MyCmd.ExecuteNonQuery();
            
            if (Convert.ToInt32(objReturnParam.Value )> 0)
        {
            lblResult.Text = "UserName already exists or you are already a registered user!";
            return false;
        }
       
        else
        {
            return true;
        } 
            MyConn.Close();
        **}**
   
    catch (Exception ex)
    {
        lblError.Text = "Error Connecting to Database!";

    **}**
**}** 
}

Please Help Me,
Thank You,

Hoda

modified on Monday, May 19, 2008 12:00 PM

[Gareth H]

Saba02, I think you could of solved this your self if you took 2mins to think about it...

public bool ValidateNewUser(string strAlias, string strFirst, string strLast)
{

        SqlConnection MyConn = new SqlConnection(ConfigurationSettings.AppSettings\["MyOnlineShoppingConnectionString2"\]);

        SqlCommand MyCmd = new SqlCommand("sp\_CheckForDuplicates", MyConn);

        MyCmd.CommandType = CommandType.StoredProcedure;

        SqlParameter objParam1;
        SqlParameter objParam2;
        SqlParameter objParam3;

        SqlParameter objReturnParam;

        objParam1 = MyCmd.Parameters.Add("@UserName", SqlDbType.VarChar);
        objParam2 = MyCmd.Parameters.Add("@FirstName", SqlDbType.VarChar);
        objParam3 = MyCmd.Parameters.Add("@LastName", SqlDbType.VarChar);
        objReturnParam = MyCmd.Parameters.Add("@Duplicates", SqlDbType.Int);
        objReturnParam.Direction = ParameterDirection.ReturnValue;

        objParam1.Value = strAlias;
        objParam2.Value = strFirst;
        objParam3.Value = strLast;

        try
        {
            if (MyConn.State == ConnectionState.Closed)
            {
                MyConn.Open();
                MyCmd.ExecuteNonQuery();

                if (Convert.ToInt32(objReturnParam.Value) > 0)
                {
                    lblResult.Text = "UserName already exists or you are already a registered user!";
                    return false;
                }
            }
            else
            {
                return true;
            } 

            MyConn.Close();
        }
        catch (Exception ex)
        {
            lblError.Text = "Error Connecting to Database!";
        }
    }

Regards, Gareth. (FKA gareth111)

[Peter Vertes]

Well for starters you have an extra '}' at the end. In Visial Studio (or almost every good text editor) there is a way to match braces, brackets and parenthesis. Give it a shot; you'll find the culprit in no time.

--- http://iheartdotnet.blogspot.com

[realJSOP]

You should do what I do. If I have a real stumper on my hands, I start writing a message here, but I ALWAYS go back and look a little harder at my code before posting the message. It doesn't always work out, but I have discovered quite a few "stupid programmer tricks" that saved me from having to ask a question. BTW, your try block starts too late - it really should be around ANY code that can throw an exception.

"Why don't you tie a kerosene-soaked rag around your ankles so the ants won't climb up and eat your candy ass..." - Dale Earnhardt, 1997
-----
"...the staggering layers of obscenity in your statement make it a work of art on so many levels." - Jason Jystad, 10/26/2001