Database Fix Code smell
continuous-integration/drone/push Build is passing Details

pull/19/head
Matheo THIERRY 2 years ago
parent 7a89f4cff5
commit efaa4b878b

@ -37,10 +37,7 @@ namespace Biblioteque_de_Class
{
List<User> searchedUsers = new List<User>();
string search = name.ToLower();
foreach (User user in UserList)
{
if (user.GetUsername().ToLower().Contains(search)) { searchedUsers.Add(user); }
}
searchedUsers.AddRange(UserList.Where(user => user.GetUsername().ToLower().Contains(search)));
return searchedUsers;
}
@ -74,10 +71,7 @@ namespace Biblioteque_de_Class
/// <summary>
/// comparer le mot de passe entré avec celui de l'utilisateur
/// </summary>
public bool ComparePassword(User user, string password)
{
return user.GetPassword() == password;
}
public static bool ComparePassword(User user, string password) => user.GetPassword() == password;
/// <summary>
/// rechercher un mail dans la liste d'utilisateur

@ -83,7 +83,7 @@ while (boucle == 0)
Umail = u.GetEmail();
Upassword = u.GetPassword();
db.FindEmail(Umail);
db.ComparePassword(u,Upassword);
Database.ComparePassword(u,Upassword);
UserListe = db.GetUserList();
if (UserListe.Contains(u))
{
@ -119,7 +119,7 @@ while (boucle == 0)
Umail = u.GetEmail();
db.FindEmail(Umail);
Upassword = u.GetPassword();
db.ComparePassword(u,Upassword);
Database.ComparePassword(u,Upassword);
UserListe = db.GetUserList();
if (UserListe.Contains(u))
{

@ -26,7 +26,7 @@ namespace Notus_UnitTest_Database
{
User user = database.GetUserList()[0];
string password = "password123";
bool result = database.ComparePassword(user, password);
var result = Database.ComparePassword(user, password);
Assert.That(result, Is.True);
}
@ -35,7 +35,7 @@ namespace Notus_UnitTest_Database
{
User user = database.GetUserList()[0];
string password = "incorrectPassword";
bool result = database.ComparePassword(user, password);
var result = Database.ComparePassword(user, password);
Assert.That(result, Is.False);
}
}

Loading…
Cancel
Save