el_tot93 Ответов: 2

Приложение посещаемости с использованием qr-кода C# winform

Я создал таймер в своей форме, он запускается при нажатии кнопки камеры. Я хочу проверить, есть ли у пользователя истекший идентификатор или нет, и вставить идентификатор в таблицу (checkin), приведенную ниже, - это мой код до сих пор, пожалуйста, просмотрите и дайте обратную связь.

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

private void Timer1_Tick(object sender, EventArgs e)
        {
            BarcodeReader Reader = new BarcodeReader();
            Result result = Reader.Decode((Bitmap)pictureBox1.Image);
            try
            {
                string decoded = result.ToString().Trim();
                if (decoded != "")
                {
                    timer1.Stop();
                    MessageBox.Show(decoded);

                    using (SqlConnection con = new SqlConnection("Data Source=SQL5037.site4now.net;Initial Catalog=DB_A448D1_Dragon;User Id=***********;Password=******"))
                    {
                        con.Open();
                        try
                        {
                            using (SqlCommand com = new SqlCommand("select count(*)from enddate where ID=@ID and startdate <=@C1 and endDate >=@C2", con))
                            {

                                com.Parameters.AddWithValue("@ID", decoded.Trim());
                                com.Parameters.AddWithValue("@C1", DateTime.Now);
                                com.Parameters.AddWithValue("@C2", DateTime.Now);
                                
                                int count = (int)com.ExecuteScalar();
                                if (count > 0)
                                {
                                    using (SqlCommand com1 = new SqlCommand("INSERT INTO [checkin] (ID,time) VALUES (@ID,@time)", con))
                                    {
                                        com1.Parameters.AddWithValue("@ID", decoded.Trim());

                                        com1.Parameters.AddWithValue("@time", txttime.Text);


                                        com1.ExecuteNonQuery();
                                    }
                                    Form2 form2 = new Form2();
                                    form2.Show();
                                    //MetroFramework.MetroMessageBox.Show(this, "Check In Sucssesfuly ", "Message", MessageBoxButtons.OK, MessageBoxIcon.Information);
                                }
                                else
                                {
                                    //MetroFramework.MetroMessageBox.Show(this, "this ID not Exist ", "Message", MessageBoxButtons.OK, MessageBoxIcon.Warning);
                                    Form3 form3 = new Form3();
                                    form3.Show();

                                }
                            }
                        }

                        catch (Exception ex)
                        {
                            MessageBox.Show(ex.Message);
                        }
                        finally
                        {
                            if (con.State == ConnectionState.Open)
                                con.Close();
                        }
                       
                    }
                }
            }
            catch (Exception ex)
            {
                MessageBox.Show(ex.Message);
            }
        }


Я думаю ,что может быть проблема с (декодированием), что будет правильным способом сделать это .

MadMyche

Какой считыватель штрих-кодов вы используете?

el_tot93

у тебя есть какие-нибудь идеи

MadMyche

BarcodeReader не является собственным классом, проект/пакет должен быть добавлен в ваше решение, чтобы разрешить вызов этого класса (вместе с результатом)

el_tot93

Qr-код

2 Ответов

Рейтинг:
6

el_tot93 

private void Timer1_Tick(object sender, EventArgs e)
       {
           BarcodeReader Reader = new BarcodeReader();
           Result result = Reader.Decode((Bitmap)pictureBox1.Image);
           try
           {
               //string decoded = result.ToString().Trim();
               if (result != null)
               {

                   //MessageBox.Show(result.Text);

                   using (SqlConnection con = new SqlConnection("Data Source=SQL5037.site4now.net;Initial Catalog=DB_A448D1_Dragon;User Id=************;Password=**********"))
                   {
                       con.Open();
                       try
                       {
                           using (SqlCommand com = new SqlCommand("select count(*)from enddate where ID=@ID and startdate <=@C1 and endDate >=@C2", con))
                           {

                               com.Parameters.AddWithValue("@ID", result.Text);
                               com.Parameters.AddWithValue("@C1", DateTime.Now);
                               com.Parameters.AddWithValue("@C2", DateTime.Now);

                               int count = (int)com.ExecuteScalar();
                               if (count > 0)
                               {
                                   using (SqlCommand com1 = new SqlCommand("INSERT INTO [checkin] (ID,time) VALUES (@ID,@time)", con))
                                   {
                                       com1.Parameters.AddWithValue("@ID", result.Text);

                                       com1.Parameters.AddWithValue("@time", txttime.Text);


                                       com1.ExecuteNonQuery();
                                   }
                                   timer1.Stop();
                                   Form2 form2 = new Form2();
                                   form2.Show();
                                   this.Close();
                                   //MetroFramework.MetroMessageBox.Show(this, "Check In Sucssesfuly ", "Message", MessageBoxButtons.OK, MessageBoxIcon.Information);
                               }
                               else
                               {
                                   timer1.Stop();
                                   //MetroFramework.MetroMessageBox.Show(this, "this ID not Exist ", "Message", MessageBoxButtons.OK, MessageBoxIcon.Warning);
                                   Form3 form3 = new Form3();
                                   form3.Show();
                                   this.Close();

                               }

                           }
                       }

                       catch (Exception ex)
                       {
                           MessageBox.Show(ex.Message);
                       }
                       finally
                       {
                           if (con.State == ConnectionState.Open)
                               con.Close();
                       }

                   }
               }
           }
           catch (Exception ex)
           {
               MessageBox.Show(ex.Message);
           }
       }

Рейтинг:
2

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

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

el_tot93

плз как я могу получить результат и нам его

OriginalGriff

Сколько раз мы уже говорили вам о SQL-инъекции?
И вы все еще делаете это. Так что вы нас не слушаете, вы ничему не учитесь. Вы все еще находите биты кода, которые делают то, что вы хотите, но вместо того, чтобы пытаться f=решить это, вы просто закрепляете их в своем приложении и просите нас разобраться в этом.

Если вы собираетесь игнорировать то, что мы говорим, вместо того, чтобы слушать или учиться, почему мы должны хотеть помочь вам?

el_tot93

проверьте сейчас

OriginalGriff

Так почему же вы не сделали этого в первую очередь?

el_tot93

я научился использовать этот способ но теперь у меня есть еще одна проблема все это только тестирование

OriginalGriff

Нет, вы не "научились использовать этот способ" - если бы вы это сделали, вы бы до сих пор не делали этого!

el_tot93

я просто тестирую, чтобы получить идею для этого кода

el_tot93

и я остановился на том, как нам (расшифровать) или (результат) получить идентификатор