Zuweisen einer Rolle zu einem Benutzer führt zu einer neuen Rolle in der Datenbank zu erstellen

Question

ich diese

[HttpPost, ValidateAntiForgeryToken] 
    public ActionResult Create(UserNew form) 
    { 
     var user = new User(); 
     SyncRoles(form.Roles, user.Roles); 

     if (db.Users.Any(u => u.username == form.username)) 
     { 
      ModelState.AddModelError("Username", "Username must be unique"); 
     } 

     if (ModelState.IsValid) 
     { 
      user.username = form.username; 
      user.email = form.email;     
      user.SetPassword(form.password); 

      db.Users.Add(user); 
      db.SaveChanges(); 

      return RedirectToAction("Index"); 
     } 

     return View(form); 
    } 

und mit dieser Funktion bekommen haben Ich versuche, das Kontrollkästchen zu bekommen in Form geprüft und die geprüften Rollen zuweisen für den Benutzer:

private void SyncRoles(IList<RoleCheckbox> checkboxes, IList<Role> roles) 
    { 
     var selectedRoles = new List<Role>(); 
     RolesContext rdb = new RolesContext(); 


     List <Role> roleList = new List<Role>(); 
     roleList = rdb.Roles.ToList(); 


     foreach (var role in roleList) 
     { 
      var checkbox = checkboxes.Single(c => c.roleId == role.roleId); 
      checkbox.roleName = role.roleName; 

      if (checkbox.IsChecked) 
       selectedRoles.Add(role); 
     } 
     foreach (var toAdd in selectedRoles.Where(t => !roles.Contains(t))) 
     { 
      roles.Add(toAdd); 
     } 
     foreach (var toRemove in roles.Where(t => !selectedRoles.Contains(t)).ToList()) 
     { 
      roles.Remove(toRemove); 
     } 
    } 

Wenn ich versuche, um den Benutzer zu erstellen, habe ich die Rolle „admin“ wählen Sie in der Datenbank, erhalte ich ein neues Feld (das in der Datenbank mit der ID = 1 ist bereits vorhanden) in der Rollen Tabelle mit dem Namen "admin" aber einer anderen ID. Kann mir jemand helfen zu verstehen, was ich falsch mache?

public class User 
{ 
    [HiddenInput(DisplayValue = false)] 
    public int userId { get; set; } 

    [Required] 
    [StringLength(128)] 
    public string username { get; set; } 

    [Required, DataType(DataType.EmailAddress)] 
    [StringLength(256)] 
    public string email { get; set; } 

    [Required] 
    [StringLength(128)] 
    public string password_hash { get; set; } 

    public virtual IList<Role> Roles { get; set; } 

    public User() 
    { 
     Roles = new List<Role>(); 
    } 

    public void SetPassword(string password) 
    { 
     password_hash = BCrypt.Net.BCrypt.HashPassword(password, 14); 
    } 
    public virtual bool CheckPassword(string password) 
    { 
     return BCrypt.Net.BCrypt.Verify(password, password_hash); 
    }   
} 

.

protected override void OnModelCreating(DbModelBuilder modelBuilder) 
    { 
     modelBuilder.Entity<User>() 
      .Property(e => e.username) 
      .IsUnicode(false); 

     modelBuilder.Entity<User>() 
      .Property(e => e.email) 
      .IsUnicode(false); 

     modelBuilder.Entity<User>() 
      .Property(e => e.password_hash) 
      .IsUnicode(false); 

     modelBuilder.Entity<User>().HasMany(r => r.Roles).WithMany().Map(m => 
       { 
        m.ToTable("role_users"); 
        m.MapLeftKey("user_Id"); 
        m.MapRightKey("role_Id"); 
       }); 
    } 

Answer 1

Ich möchte einen Kommentar hinzufügen, ist aber nur nach 50+ Ruf möglich. Von dem, was ich in Ihrem Code sehe, weisen Sie dem Benutzer Rollen von MVC zu und Sie senden direkt an EF, die Sie auf Dubletten in der Datenbank führen. Sie sollten die Logik ändern, und nachdem Sie Benutzer mit Erfolg zu schaffen, sollten Sie UserManager Klasse verwenden Benutzer Rolle hinzufügen:

UserManager.AddToRole(user.Id, role.Name); 

wo Rolle gegen Datenbank überprüft wird. Wenn Sie keine Rolle in der Datenbank finden, sollten Sie sie mit IdentityRol e Klasse