Mody_2004 Ответов: 1

Почему, когда я вызываю эту функцию, появляется это сообщение ?

Function MaxFieldID(ByVal Tabel As String, ByVal Field As String) As Integer

       Try

           'con.Close()
           If con.State = ConnectionState.Closed Then con.Open()
           cmd = New SqlCommand("SELECT MAX(CAST(" + Field + " AS INT))+1 FROM " + Tabel, con)

           If cmd.ExecuteScalar Is DBNull.Value Then
               Return 1
           Else
               Return cmd.ExecuteScalar

           End If

       Catch ex As Exception
           MsgBox(ex.Message)

       Finally
           '  If con.State = ConnectionState.Open Then con.Close()
       End Try

   End Function


Что я уже пробовал:

gmAccCmd.Parameters.Add("@idCard", SqlDbType.BigInt).Value = Zcls.MaxFieldID("empCards", "idCard")th



сообщение предупреждение :
выполните команду scalar requites, чтобы иметь транзакцию, когда con
назначенная команде находится в ожидании локальной транзакции.
свойство transaction команды не было инициализировано.

1 Ответов

Рейтинг:
12

OriginalGriff

Посмотрите на сообщение об ошибке:

Execute scalar requites command to have transaction when the con 
assigned to the command is in pending local transaction.
the transaction property of the command has not been initialized.

Это означает, что "объект SqlConnection, который вы используете, обрабатывает команду, которая открыла транзакцию и не завершила ее".

В принципе, не "делайте SQL" так: создайте свой SqlConnection и SqlCommand объекты внутри Using блокируйте как новые объекты каждый раз, когда они вам нужны, а не "перерабатывайте" их, как это делает ваш существующий код:
Using con As New SqlConnection(strConnect)
	con.Open()
	Using cmd As New SqlCommand("SELECT iD, description FROM myTable", con)
		Using reader As SqlDataReader = cmd.ExecuteReader()
			While reader.Read()
				Dim id__1 As Integer = CInt(reader("iD"))
				Dim desc As String = DirectCast(reader("description"), String)
				Console.WriteLine("ID: {0}" & vbLf & "    {1}", iD, desc)
			End While
		End Using
	End Using
End Using


И сделайте себе одолжение: найдите лучший способ делать таблицы - вы никогда не должны объединять строки для формирования команд SQL, это чрезвычайно опасно. Это оставляет вас широко открытыми для случайной или преднамеренной атаки SQL-инъекции, которая может уничтожить всю вашу базу данных. Вместо этого всегда используйте параметризованные запросы.

Когда вы объединяете строки, вы вызываете проблемы, потому что SQL получает такие команды, как:
SELECT * FROM MyTable WHERE StreetAddress = 'Baker's Wood'
Цитата, добавленная пользователем, завершает строку в том, что касается SQL, и вы получаете проблемы. Но могло быть и хуже. Если я приду и наберу вместо этого: "x';DROP TABLE MyTable;--", то SQL получит совсем другую команду:
SELECT * FROM MyTable WHERE StreetAddress = 'x';DROP TABLE MyTable;--'
Которые SQL видит как три отдельные команды:
SELECT * FROM MyTable WHERE StreetAddress = 'x';
Совершенно правильный выбор
DROP TABLE MyTable;
Вполне допустимая команда "удалить таблицу" 
--'
А все остальное-это комментарии.
Так оно и происходит: выбирает любые совпадающие строки, удаляет таблицу из базы данных и игнорирует все остальное.

Поэтому всегда используйте параметризованные запросы! Или будьте готовы часто восстанавливать свою БД из резервной копии. Вы ведь регулярно делаете резервные копии, не так ли?

Вы не можете использовать параметры с именами таблиц, поэтому найдите лучший способ их обработки - просто полагаться на правильные строки очень опасно!