Member 13604810 Ответов: 2

Дайте мне ошибку system.data.sqlclient.sqlexception: 'неправильный синтаксис рядом с '01/12/2017'.' я ищу дату и даю мне ошибку

частная btn_search_Click недействительным(объект отправителя, EventArgs в электронной)
{
Коннектикут.Открыть();

SqlDataAdapter SDA = new SqlDataAdapter("выберите dbo.Альтерако.Данные, ДБО.Funcionarios.numero как нет, dbo.Funcionarios.Нома, ДБО.Альтерако.Фальта как фальтас, ДБО.Альтерако.Coluna2 как horasextras от dbo.Alteracoes inner join dbo.Функционеры на ДБО.Альтерако.IDFuncionario = dbo.Функционеры.IDFuncionario inner join dbo.Departamentos на dbo.Функционеры.IDDepartamento = dbo.Апартаменты departamentos.IDDepartamento где dbo.Альтерако.Данных '" +dateTimePicker1.Значение.ToString("dd/MM/yyyy") + "'", conn);

Набор ДТ = новый набор данных();
SDA.Fill(dt, "dbo.Альтерако.Данные");
dataGridView1.Источник данных = ДТ.Столы["ДБО.Альтерако.Данные"];
Коннектикут.Закрывать();
}

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

private void btn_search_Click(object sender, EventArgs e)
        {
            conn.Open();

            SqlDataAdapter SDA = new SqlDataAdapter("select dbo.Alteracoes.Data, dbo.Funcionarios.numero as no, dbo.Funcionarios.nome, dbo.Alteracoes.Falta as faltas, dbo.Alteracoes.Coluna2 as horasextras from dbo.Alteracoes inner join dbo.Funcionarios on dbo.Alteracoes.IDFuncionario = dbo.Funcionarios.IDFuncionario inner join dbo.Departamentos on dbo.Funcionarios.IDDepartamento = dbo.Departamentos.IDDepartamento WHERE dbo.Alteracoes.Data '" +dateTimePicker1.Value.ToString("dd/MM/yyyy") + "'", conn);

            DataSet dt = new DataSet();
            SDA.Fill(dt, "dbo.Alteracoes.Data");
            dataGridView1.DataSource = dt.Tables["dbo.Alteracoes.Data"];
            conn.Close();
        }

2 Ответов

Рейтинг:
0

OriginalGriff

Просто: не делай этого так.
Никогда не объединяйте строки для построения команды 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;
Вполне допустимая команда "удалить таблицу" 
--'
А все остальное-это комментарии.
Так оно и происходит: выбирает любые совпадающие строки, удаляет таблицу из базы данных и игнорирует все остальное.

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

Запрос, который вы показываете, не уязвим для SQL - инъекции, но остальная часть вашего кода, вероятно, такова, и конкатенация здесь не помогает проблеме-исправьте весь ваш код, добавьте знак равенства в предложение WHERE, и проблема исчезнет в то же время.

Рейтинг:
0

Pradeep Arthanari

Похоже, вы упустили самое главное. equal(=) в where пункт:

Конкатенация строки sql-запроса-это уязвимый к SQL-инъекция[^] атаки
всегда использовать Параметризованные запросы для предотвращения атак SQL-инъекций в SQL Server[^]


string query = "select dbo.Alteracoes.Data, dbo.Funcionarios.numero as no, dbo.Funcionarios.nome, dbo.Alteracoes.Falta as faltas, dbo.Alteracoes.Coluna2 as horasextras from dbo.Alteracoes inner join dbo.Funcionarios on dbo.Alteracoes.IDFuncionario = dbo.Funcionarios.IDFuncionario inner join dbo.Departamentos on dbo.Funcionarios.IDDepartamento = dbo.Departamentos.IDDepartamento WHERE dbo.Alteracoes.Data = @date";
                SqlCommand cmd = new SqlCommand(query, conn);
                cmd.CommandType= CommandType.Text;
                cmd.Parameters.Add("@date", dateTimePicker1.Value.ToString("dd/MM/yyyy"));
                SqlDataAdapter SDA = new SqlDataAdapter(query, conn);