Writing to a table using a stored procedure?

See more:

C#

ASP.NET

SQL

Hey guys, im trying to write to a table through a stored proc, i keep getting an error:

Procedure or function ParseTheString has too many arguments specified.

here is the simple code behind:

it is a text box and a button.

private void button1_Click(object sender, EventArgs e)
        {
            SqlConnection con = new SqlConnection();
            con.ConnectionString = "Data Source=Matthew-PC\\SQLEXPRESS;Initial Catalog=SureViewDB;Integrated Security=True";
            con.Open();
            SqlCommand insertcmd = new SqlCommand("ParseTheString", con);
            insertcmd.Parameters.Add("@AlarmID", SqlDbType.VarChar).Value = textBox1.Text;
            insertcmd.Parameters.Add("@ServerNumber", SqlDbType.VarChar).Value = textBox1.Text;
            insertcmd.CommandType = CommandType.StoredProcedure;
            insertcmd.ExecuteNonQuery();
            con.Close();
        }

And here is the stored procedure running against data entered into text box:

CREATE PROCEDURE [dbo].[ParseTheString]

@Message VARCHAR(1000)

--@message examples

--'The alarm id from server number 1 is 6'

--'Alarm ID 6 has been received from video server number 1'

AS

 

BEGIN

DECLARE

         @str1 VARCHAR(1),@str2 VARCHAR(250),@str3 VARCHAR(250),@str4 VARCHAR(250),

         @str5 VARCHAR(250),@num1 INT,@num2 INT,@loop1 INT, @loop2 INT,@MType INT,

         @str6 VARCHAR(25),@numstart int,@numend int,@now DATETIME

 

 

--Check if the table that stores the data exists, if not create it.

IF (NOT EXISTS (SELECT TABLE_NAME FROM INFORMATION_SCHEMA.TABLES

                          WHERE TABLE_NAME = 'LoggedMessages'  

                            AND TABLE_SCHEMA = 'dbo')) 

        BEGIN

        CREATE TABLE dbo.LoggedMessages (AlarmUnit INT Identity (1,1),ServerNo INT, AlarmID INT, GeneratedOn DATETIME, MessageType INT)      

        END

 

---------------ALTER THE CODE HERE TO SUIT THE STARTING WORD OF THE MESSAGES THAT WILL BE RECIEVED.....................

  IF LOWER(LEFT(LTRIM(@Message),5)) = 'alarm'  --convert to lowercase in case there is a capital letter at the start of the word

  BEGIN                                        --and also remove any whitespace from beginning of string

    SET @MType = 1

  END      

   ELSE

     BEGIN

       SET @MType = 2

     END

--------------------------

 

 

 

SET @loop1 = 1

  SET @str1 = (SUBSTRING(@Message,@loop1,1))

    WHILE ISNUMERIC(@str1)=0  --this condition looks for characters in a string.

      BEGIN

        --scroll through the rest of the string until we get a number instead of a character.

        SET @loop1 = @loop1 + 1

         SET @str1 = (SUBSTRING(@Message,@loop1,1))

         PRINT '@loop1a position = '+CAST(@loop1 AS VARCHAR(20))

      END

SET @numstart = @loop1     

--we now have the starting position of the first set of numbers.

 

--numbers start at point @loop1

  SET @str1 = (SUBSTRING(@Message,@loop1,1))

    WHILE ISNUMERIC(@str1)=1  --this condition looks for numbers in a string.

      BEGIN

        --scroll through the rest of the string until we get a character instead of a number.

        SET @loop1 = @loop1 + 1   --increment the position we are looking at.

         SET @str1 = (SUBSTRING(@Message,@loop1,1))  --check the next character to see if it is a number.

         PRINT '@loop1 position = '+CAST(@loop1 AS VARCHAR(20))  --this is to check the position of the numbers is correct.

      END

SET @numend = @loop1

--now get the number out of the string.

SET @num1 = (SELECT SUBSTRING(@Message,@numstart,(@numend-@numstart)))

PRINT @num1

 

--now get the next set of numbers.

 

  SET @str1 = (SUBSTRING(@Message,@loop1,1))

    WHILE ISNUMERIC(@str1)=0  --this condition looks for characters in a string.

      BEGIN

        --scroll through the rest of the string until we get a number instead of a character.

        SET @loop1 = @loop1 + 1

         SET @str1 = (SUBSTRING(@Message,@loop1,1))

         PRINT '@loop1a position = '+CAST(@loop1 AS VARCHAR(20))

      END

SET @numstart = @loop1         

--we now have the starting position of the first set of numbers.

 

--numbers start at point @loop1a

  SET @str1 = (SUBSTRING(@Message,@loop1,1))

    WHILE ISNUMERIC(@str1)=1  --this condition looks for numbers in a string.

      BEGIN

        --scroll through the rest of the string until we get a character instead of a number.

        SET @loop1 = @loop1 + 1

         SET @str1 = (SUBSTRING(@Message,@loop1,1))

         PRINT '@loop1a position = '+CAST(@loop1 AS VARCHAR(20))

      END

SET @numend = @loop1

--now get the number out of the string.

SET @num2 = (SELECT SUBSTRING(@Message,@numstart,(@numend-@numstart)))  --set the value

PRINT @num2

 

--Now write the data to the table...........

SET @now = GETDATE()  --get the current data and time.

 

      IF @MType = 1  -- the message is in format 1 'The alarm id from server number 1 is 6'

        BEGIN

             INSERT INTO dbo.LoggedMessages VALUES (@num1,@num2,@now,1)

        END

          ELSE   -- the message is in format 2  'Alarm ID 6 has been received from video server number 1'

             BEGIN

             INSERT INTO dbo.LoggedMessages VALUES (@num2,@num1,@now,2)

             END

END

in the text box i am simply typing in 'Alarm ID 6 has been received from video server number 1'

why is it giving the error?


is it because i am specifying the alarm id and servernum?
if so, how do i go about adding records through the sp?

any help is highly appreciated.