Question

我使用以下存储过程来呼叫句号以使其与教师匹配。

我们正在编写一个考勤应用程序，从下拉菜单中选择教师的名字，＆＃34;期间＆＃34;下拉菜单填充。出于某种原因，我们没有填写期间数据。

存储过程：

ALTER PROCEDURE spGetPeriod @Period varchar(10)
AS
SELECT Period 
FROM tmTeacher
WHERE Period = @Period

RETURN

Teacher Table 
TeacherId   varchar(10) Unchecked
FName           varchar(20) Checked
LName           varchar(20) Checked
StudentId   varchar(10) Unchecked
Period          varchar(10) Unchecked
TeacherName varchar(50) Checked

Period Table
PeriodId    varchar(10) Unchecked
Period          varchar(50) Unchecked
ClassId         varchar(10) Unchecked

提前致谢！

调用SP的代码：

'*****  SETUP THE CONNECTION ***
    SQL = "spGetPeriod '" + (cboTeacher.Text) + "'"
    'SQL = "select * from teacher"
    Try
        With SqlCmd
            .CommandText = SQL
            '.CommandType = CommandType.StoredProcedure
            .Connection = objConnection
            '.Parameters.AddWithValue("@TeacherId", cboTeacher.SelectedIndex)
        End With
        SqlCmd = New SqlCommand(SQL, objConnection)
    Catch ex As Exception
        Call ErrorControl(ex)
    End Try

    '*** MAKE THE CONNECTION ***

    Try
        HandleConnection(objConnection)
        'MsgBox("Connection Established!")
        ToolStripStatusLabel1.Text = "Connection Established!"


    Catch ex As Exception
        Call ErrorControl(ex)

    End Try

    '*** EXECUTE THE COMMAND AND FILL THE COMBO ***
    Dim Reader As SqlDataReader = SqlCmd.ExecuteReader()
    While Reader.Read()
        cboPeriod.Items.Add(Reader(0).ToString)

    End While
    ToolStripStatusLabel1.Text = "Periods Loaded!"




End Sub

Answer 1

1）您在代码中添加了一个名为TeacherId的参数，但在您的proc上它并不存在。您必须将正确的参数从接口

传递给过程

2）您的过程称为spGetPeriod，但在您的SQL上，您已选择*来自教师</ p>

Answer 2

您已在sproc中将参数定义为“@Period”，但是当您将该参数添加到命令时，您将其称为“@TeacherId”。您的代码一目了然，尝试改变程序......

ALTER PROCEDURE spGetPeriod @TeacherId varchar(10)
AS
SELECT Period 
FROM tmTeacher
WHERE TeacherId = @TeacherId

RETURN

对于没有任何数据的问题，可能尝试将组合框的数据源绑定到阅读器而不是循环读取器并添加每个数据，保留ExecuteReader语句并尝试用此结束替换你的while循环...

cboPeriod.DataSource = Reader
cboPeriod.DataTextField = "Period"
cboPeriod.DataValueField = "Period"
cboPeriod.DataBind()

Answer 3

我发现您的架构存在一些问题：

你的各种ID应该是整数，而不是varchar。
教师表具有FName，LName和TeacherName。这没有规范化。如果FName和LName适用于学生，则应将其保存在带有StudentId主键的单独学生表中。
StudentId可能不应该在Teacher表中。如果您想存储哪些学生在哪个时期，您应该使用连接表，这是多对多关系的标准。例如，您可以将其称为“PeriodStudent”，它将存储StudentId和PeriodId。

花时间了解数据库规范化概念，如First，Second和Third Normal形式。如果你不遵守基本的标准化原则，你就会在未来遇到重大问题，以后修复这些问题将非常昂贵。

SQL Server存储过程的问题

3 个答案: