ycwong99 Ответов: 1

Как получить имя пользователя при входе в систему и сохранить его в базе данных, когда пользователи нажмут кнопку голосования


When the users click the vote button, the candidateID that being displayed in the gridview will be store to the database.However, I want to store who vote for the particular candidate (store they studentID). I'd like capture thier username when they login, and when they click the vote button, insert their studentID to the database


I store the session in the user login page. How can I store it to the database using the insert command(when they click the vote button)?


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

<pre><pre lang="c#">
protected void LinkButton_Click(Object sender, EventArgs e)
    {
        String MyConnection2 = "Server=localhost;database=ovs;Uid=root;password=; Convert Zero Datetime=True";
     
        MySqlConnection MyConn2 = new MySqlConnection(MyConnection2);      
        GridViewRow grdrow = (GridViewRow)((LinkButton)sender).NamingContainer;
        Label lblStudentId = (Label)grdrow.Cells[0].FindControl("lblID");
        string studentId = lblStudentId.Text;
        String query = "insert into voting (studentID,)values ('" + lblStudentId.Text + "')";  
        MySqlCommand MyCommand2 = new MySqlCommand(query, MyConn2);
        MySqlDataReader MyReader2;
        MyConn2.Open();
        MyReader2 = MyCommand2.ExecuteReader();

        while (MyReader2.Read())
        {
        }
        MyConn2.Close();
    }

    }



<pre lang="c#">
 Session["UserName"] = txtID.Text;
txtID.Text = Session["UserName"].ToString();

BillWoodruff

Вот вам и анонимное голосование.

1 Ответов

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

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

Кроме того, вы никогда не должны жестко кодировать строки соединений - всегда храните их в конфигурационном файле, чтобы вам не приходилось менять их каждый раз, когда вы выпускаете новую версию! И хранить их в каждом методе? Это просто безумие - и рецепт серьезных проблем, когда вы пытаетесь исправить ошибку и обнаруживаете, что работаете с производственной базой данных в методе, который случайно удаляет всю базу данных...
Прочитайте его из вашего конфигурационного файла один раз и повторно используйте строку оттуда.

Затем посмотрите на исправление проблемы, которую вы заметили: уберите запятую после "StudentID" в вашей команде INSERT:
string query = "INSERT INTO Voting (StudentID) VALUES ( ... )";