Sqlserver
 sql >> Datenbank >  >> RDS >> Sqlserver

fügen Sie mehrere Datensätze dynamisch bei einem einzigen Klick auf die Schaltfläche ein

Wenn Sie die genaue Anzahl der Fächer für die Eingabe der Noten nicht kennen - wie sollen wir dafür eine Abfrage generieren?

Nichtsdestotrotz, um Ihnen zu zeigen, wie Sie sich vor SQL-Injection-Angriffen schützen können, legen Sie Ihr SQL in Stored Procs ab:

create PROCEDURE [dbo].[pr_GetAssignedSubjectsByFacultyIdAndSemester]
@FacultyID int,
@Semester nvarchar(MAX)
AS
BEGIN
SET NOCOUNT ON;
SELECT [Faculty], [Subjects],[CreatedBy],[CreatedDate],[ModifiedBy],[ModifiedDate]
 FROM [dbo].[tblNotSure]
WHERE [FacultyID] = @FacultyID
AND [Semester] = @Semester
AND [IsDeleted] = 0
END

Dann rufen wir im Code die gespeicherte Prozedur auf, beachten Sie die parametrisierten Befehle, dies verhindert SQL-Injection-Angriffe. Angenommen, wir haben in das Semester-DDL/Textfeld (oder mit FireBug zum Bearbeiten des Elementwerts) 1 UNION SELECT * FROM Master.Users eingegeben – die Ausführung dieser Ad-hoc-SQL könnte die Liste der SQL-Benutzerkonten zurückgeben, aber einen parametrisierten Befehl übergeben vermeidet das Problem:

public static aClassCollection GetAssignedSubjectsByFacultyIdAndSemester(int facultyId, string semester)
{
var newClassCollection = new aClassCollection();
    using (var connection = new SqlConnection(ConfigurationManager.ConnectionStrings["sqlConn"].ConnectionString))
    {
        using (var command = new SqlCommand("pr_GetAssignedSubjectsByFacultyIdAndSemester", connection))
        {
            try
            {
                command.CommandType = CommandType.StoredProcedure;
                command.Parameters.AddWithValue("@facultyId", facultyId);
                command.Parameters.AddWithValue("@semester", semester);
                connection.Open();
                SqlDataReader dr = command.ExecuteReader();
                while (dr.Read())
                {
                    newClassCollection.Add(new Class(){vals = dr["vals"].ToString()});
                }
            }
            catch (SqlException sqlEx)
            {
             //at the very least log the error
            }
            finally
            {
             //This isn't needed as we're using the USING statement which is deterministic                    finalisation, but I put it here (in this answer) to explain the Using...
                connection.Close();
            }
        }
    }

    return newClassCollection;
}