C# formularz logowania - niepoprawna nazwa użytkownika lub hasło.

Question

Cześć, w ramach nauki tworze formularz logowania, po utoworzeniu bazy danych(service based database) mam problem z zalogowaniem do konta. po naciśnięciu buttona dostaje komunikat, że dane użytkownika lub hasło są niepoprawne.

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using System.Windows;
using System.Windows.Controls;
using System.Windows.Data;
using System.Windows.Documents;
using System.Windows.Input;
using System.Windows.Media;
using System.Windows.Media.Imaging;
using System.Windows.Navigation;
using System.Windows.Shapes;
using System.Data.SqlClient;
using System.Data;

namespace FormLog
{
    /// <summary>
    /// Interaction logic for MainWindow.xaml
    /// </summary>
    public partial class MainWindow : Window
    {
        public MainWindow()
        {
            InitializeComponent();
        }


        private void Button_Click(object sender, RoutedEventArgs e)
        {
            string con_str = Properties.Settings.Default.con_String;
            SqlConnection con = new SqlConnection(con_str);

            try
            {
                if (con.State == ConnectionState.Closed)
                {
                    con.Open();
                    string User = UsernameTxt.Text;
                    string Password = PasswordTxt.Text;
                    SqlCommand cmd = new SqlCommand("select User,Password from Tbl_Users where User='" + UsernameTxt.Text + "'and Password='" + PasswordTxt.Text + "'", con);
                    SqlDataAdapter da = new SqlDataAdapter(cmd);
                    DataTable dt = new DataTable();
                    da.Fill(dt);
                    if (dt.Rows.Count > 0)
                    {
                        Window Calendary = new Window();
                        Calendary.Show();
                        Close();

                    }
                    else
                    {
                        MessageBox.Show("Incorrect login or password. Please, try again");
                    }

                }

            }
            catch (Exception ex)
            {

                MessageBox.Show(ex.Message);
            }
            finally
            {
                con.Close();
            }



            
        }
    }
}

 

Comments

Brakuje przekształcenia hasła w szyfr, może to jest problemem? W bazie jest zaszyfrowane, a próbuje porównać z plaintextem.

---

Nie tylko hasła gołe w bazie, ale też sprawa, że ogólnie tu jest podręcznikowy case na SQL injection...

Answer 1

Z tego co tu jest napisane ,trudno powiedzieć co jest źle.

Zaczni  od tego że wyselekcjonuj najpierw jedną zmienną np user i zobacz co zwraca , jeśli ok ,to drugą...

to jedno ,  drugie  ... twój kod wygląda ..fatalnie :-/ popraw ,a na pewno szybciej znajdziesz błąd.

np po co to masz  : 

 string User = UsernameTxt.Text;
 string Password = PasswordTxt.Text;

?

po drugie jak zrobiłeś baze danych ?

nchar czy nvarchar ?