diff --git a/projects/Infrastructure/Views/Shared/DisplayTemplates/MultiSelectList.cshtml b/projects/Infrastructure/Views/Shared/DisplayTemplates/MultiSelectList.cshtml
index e721a4e4..ecea8176 100644
--- a/projects/Infrastructure/Views/Shared/DisplayTemplates/MultiSelectList.cshtml
+++ b/projects/Infrastructure/Views/Shared/DisplayTemplates/MultiSelectList.cshtml
@@ -1,8 +1,8 @@
-@model List<Guid>
+@model List<SelectListItem>
 @if (Model != null && Model.Count > 0)
 {
-    foreach (var id in Model)
+    foreach (var item in Model)
     {
-        <a href="javascript:;" class="btn btn-default btn-xs">@ViewData.Get(id)</a>
+        <a href="javascript:;" class="btn btn-default btn-xs">@item.Text</a>
     }
 }
\ No newline at end of file
diff --git a/projects/IoT.Shared/Application/Models/Users/EditUserModel.cs b/projects/IoT.Shared/Application/Models/Users/EditUserModel.cs
index 0f356639..3dfd4c5b 100644
--- a/projects/IoT.Shared/Application/Models/Users/EditUserModel.cs
+++ b/projects/IoT.Shared/Application/Models/Users/EditUserModel.cs
@@ -1,5 +1,5 @@
 using Infrastructure.Application;
-using System;
+using Microsoft.AspNetCore.Mvc.Rendering;
 using System.Collections.Generic;
 using System.ComponentModel;
 using System.ComponentModel.DataAnnotations;
@@ -18,6 +18,6 @@ namespace Application.Models
 
         [DataType("MultiSelectList")]
         [Display(Name = "角色")]
-        public List<Guid> Roles { get; set; } = new List<Guid>();
+        public List<SelectListItem> Roles { get; set; } = new List<SelectListItem>();
     }
 }
\ No newline at end of file
diff --git a/projects/IoT.Shared/Areas/Admin/Controlls/Users/UserController.cs b/projects/IoT.Shared/Areas/Admin/Controlls/Users/UserController.cs
index 5d39bb36..34c7fe9e 100644
--- a/projects/IoT.Shared/Areas/Admin/Controlls/Users/UserController.cs
+++ b/projects/IoT.Shared/Areas/Admin/Controlls/Users/UserController.cs
@@ -6,7 +6,9 @@ using Infrastructure.Extensions;
 using Infrastructure.Web.Mvc;
 using Microsoft.AspNetCore.Authorization;
 using Microsoft.AspNetCore.Mvc;
+using Microsoft.AspNetCore.Mvc.Rendering;
 using Microsoft.EntityFrameworkCore;
+using System;
 using System.Linq;
 
 namespace IoT.Shared.Areas.Admin.Controllers
@@ -30,33 +32,29 @@ namespace IoT.Shared.Areas.Admin.Controllers
 
         public override void ToDisplayModel(User entity, EditUserModel model)
         {
-            model.Roles = entity.UserRoles.Select(o => o.RoleId).ToList();
-            entity.UserRoles.ForEach(o => ViewData.Add(o.RoleId, o.Role.Name));
+            model.Roles = entity.UserRoles.Select(o => new SelectListItem { Value = o.RoleId.ToString(), Text = o.Role.Name }).ToList();
+            //entity.UserRoles.ForEach(o => ViewData.Add(o.RoleId, o.Role.Name));
         }
 
         public override void ToEditModel(User entity, EditUserModel model)
         {
-            if (entity != null)
-            {
-                model.Roles = entity.UserRoles.Select(o => o.RoleId).ToList();
-            }
-            this.ViewData.MultiSelectList(o => model.Roles, () => this._ajax.GetRoleMultiSelectList(model.Roles));
+            this.ViewData.MultiSelectList(o => model.Roles, () => this._ajax.GetRoleMultiSelectList(model.Roles.Select(o => Guid.Parse(o.Value))));
         }
 
         public override void ToEntity(EditUserModel model, User entity)
         {
-            foreach (var id in entity.UserRoles.Select(o => o.RoleId).ToList())
+            foreach (var roleId in entity.UserRoles.Select(o => o.RoleId).ToList())
             {
-                if (!model.Roles.Any(o => o == id))
+                if (!model.Roles.Any(o => o.Value == roleId.ToString()))
                 {
-                    entity.UserRoles.RemoveAll(o => o.RoleId == id);
+                    entity.UserRoles.RemoveAll(o => o.RoleId == roleId);
                 }
             }
-            foreach (var id in model.Roles)
+            foreach (var role in model.Roles)
             {
-                if (!entity.UserRoles.Any(o => o.RoleId == id))
+                if (!entity.UserRoles.Any(o => o.RoleId.ToString() == role.Value))
                 {
-                    entity.UserRoles.Add(new UserRole { RoleId = id });
+                    entity.UserRoles.Add(new UserRole { RoleId = Guid.Parse(role.Value) });
                 }
             }
         }