|
Покритикуйте, пожалуйста, код.
|
|||
---|---|---|---|
#18+
DI пока прошу не учитывать. Модель: Код: c# 1. 2. 3. 4. 5. 6. 7. 8. 9. 10. 11. 12. 13. 14. 15. 16. 17. 18. 19. 20. 21. 22. 23. 24. 25. 26. 27. 28. 29. 30. 31. 32. 33. 34. 35. 36. 37. 38. 39. 40. 41. 42. 43.
Контроллер: Код: c# 1. 2. 3. 4. 5. 6. 7. 8. 9. 10. 11. 12. 13. 14. 15. 16. 17. 18. 19. 20. 21. 22. 23. 24. 25. 26. 27. 28. 29. 30. 31. 32. 33. 34. 35. 36. 37. 38. 39. 40. 41. 42. 43. 44. 45. 46. 47. 48. 49. 50. 51. 52. 53. 54. 55. 56. 57. 58. 59. 60. 61. 62. 63. 64. 65. 66. 67. 68. 69. 70. 71. 72. 73. 74. 75. 76. 77. 78. 79. 80. 81. 82. 83. 84. 85. 86. 87. 88. 89. 90. 91. 92. 93. 94. 95. 96. 97. 98. 99. 100. 101. 102.
Index.cshtml: Код: plaintext 1. 2. 3. 4. 5. 6. 7. 8. 9. 10. 11. 12. 13. 14. 15. 16. 17. 18. 19. 20. 21. 22. 23. 24. 25. 26. 27. 28. 29. 30. 31. 32. 33. 34. 35. 36. 37. 38. 39. 40. 41. 42. 43. 44. 45. 46. 47. 48. 49. 50.
Create.cshtml: Код: html 1. 2. 3. 4. 5. 6. 7. 8. 9. 10. 11. 12. 13. 14. 15. 16. 17. 18. 19. 20. 21. 22. 23. 24. 25. 26. 27. 28. 29. 30. 31. 32. 33. 34. 35. 36. 37. 38. 39. 40. 41. 42. 43. 44. 45. 46.
Edit.cshtml: Код: html 1. 2. 3. 4. 5. 6. 7. 8. 9. 10. 11. 12. 13. 14. 15. 16. 17. 18. 19. 20. 21. 22. 23. 24. 25. 26. 27. 28. 29. 30. 31. 32. 33. 34. 35. 36. 37. 38. 39. 40. 41. 42. 43. 44. 45. 46. 47.
Что меня смущает: 1) наличие ViewBag.Roles = GetRolesList(); 2) наличие flagDeleted, из-за неполучается сделать например db.Entry(rme).State = EntityState.Modified; при редактировании. 3) поле Owner не помечено атрибутом Required но всегда заполняется. Наверное нужно устанавливать его на гет версиях, тогда валидация модели пройдет при сохранении. 4) Create.cshtml и Edit.cshtml очень похожи. Разница только отсутствия Id на форме создания. Может стоит оставить одну страницу Edit.cshtml, пусть id там нулем инициализируется? как правильно поправить эти проблемы? спасибо ... |
|||
:
Нравится:
Не нравится:
|
|||
08.05.2020, 17:05 |
|
Покритикуйте, пожалуйста, код.
|
|||
---|---|---|---|
#18+
listtoview, после первой строки уже скучно: public partial class RoleMapEmps... "Роль, карта, апчхи" - так это выглядит для меня :) ... |
|||
:
Нравится:
Не нравится:
|
|||
08.05.2020, 20:05 |
|
Покритикуйте, пожалуйста, код.
|
|||
---|---|---|---|
#18+
listtoview в дотнете я не джун, 9 лет опыта ... |
|||
:
Нравится:
Не нравится:
|
|||
08.05.2020, 20:25 |
|
Покритикуйте, пожалуйста, код.
|
|||
---|---|---|---|
#18+
listtoview как правильно поправить эти проблемы? спасибо Видимо, перестать использовать Entity-классы в качестве вью-моделей. Это касается всех пунктов. ... |
|||
:
Нравится:
Не нравится:
|
|||
09.05.2020, 01:16 |
|
Покритикуйте, пожалуйста, код.
|
|||
---|---|---|---|
#18+
Я уже что-то конкретно подзабыл MVC но, ЕМНИП, нет смысла писать Edit(int? id) и тут же на пустой id возвращать 404. Он и так на пустом id вернет 404 - роут нужный не найдет просто. ... |
|||
:
Нравится:
Не нравится:
|
|||
09.05.2020, 03:47 |
|
Покритикуйте, пожалуйста, код.
|
|||
---|---|---|---|
#18+
fkthat Я уже что-то конкретно подзабыл MVC но, ЕМНИП, нет смысла писать Edit(int? id) и тут же на пустой id возвращать 404. Он и так на пустом id вернет 404 - роут нужный не найдет просто. Да, конструкция Edit(int? id) максимально глупая. Так делать не нужно, конечно. ... |
|||
:
Нравится:
Не нравится:
|
|||
10.05.2020, 00:42 |
|
Покритикуйте, пожалуйста, код.
|
|||
---|---|---|---|
#18+
Дмитрий Мух listtoview, после первой строки уже скучно: public partial class RoleMapEmps... "Роль, карта, апчхи" - так это выглядит для меня :) EF так генерит, а что не так? ... |
|||
:
Нравится:
Не нравится:
|
|||
10.05.2020, 22:23 |
|
Покритикуйте, пожалуйста, код.
|
|||
---|---|---|---|
#18+
hVostt listtoview как правильно поправить эти проблемы? спасибо Видимо, перестать использовать Entity-классы в качестве вью-моделей. Это касается всех пунктов. да, спасибо за совет. вообщем то это решает все 3 пункта ... |
|||
:
Нравится:
Не нравится:
|
|||
10.05.2020, 22:24 |
|
Покритикуйте, пожалуйста, код.
|
|||
---|---|---|---|
#18+
listtoview Дмитрий Мух listtoview, после первой строки уже скучно: public partial class RoleMapEmps... "Роль, карта, апчхи" - так это выглядит для меня :) EF так генерит, а что не так? В программировании существует лишь два характерных затруднения: инвалидация кеша и наименование сущностей :) ... |
|||
:
Нравится:
Не нравится:
|
|||
10.05.2020, 22:42 |
|
Покритикуйте, пожалуйста, код.
|
|||
---|---|---|---|
#18+
listtoview, модель: 1. решительно непонятно, что за RoleMapEmps, если это типа таблицы соответствия ролей и пользователей, то почему там user string? это что логин? а если юзер логин поменяет, то все сломается? 2. с Owner аналогично 3. Created / Deleted повторяются, либо делать отдельную таблицу, где хранить когда что и кем было сделано или хотя бы наследоваться от общей модели с этими полями 4. почему нет даты обновления? обычно делают дату создания, дату последнего обновления, и если есть софт-делит - тогда флаг IsDeleted. тогда если он true, то дата обновления является датой удаления. а сейчас выходит - дата удаления есть, но запись будет удалена физически, смысл тогда в этом поле? 5. s на конце названия модели, выходит что virtual Roles Roles { get; set; } содержит Рол и , а она там одна, wtf? контроллер: 1. bool flagDeleted - когда в модели будет нормальный флаг IsDeleted - все встанет на свои места 2. ViewBag.Roles = GetRolesList(); - это зависит от сложности и предпочтений, можно сделать ViewModel<TModel> и сложить туда все сопутствующие данные 3. и да, лучше ентити отдельно, модели отдельно вьюхи: 1. Create.cshtml и Edit.cshtml очень похожи - зависит от, сейчас объединишь, а потом придется разъединять обратно ... |
|||
:
Нравится:
Не нравится:
|
|||
11.05.2020, 08:59 |
|
Покритикуйте, пожалуйста, код.
|
|||
---|---|---|---|
#18+
Дмитрий Мух listtoview, после первой строки уже скучно: public partial class RoleMapEmps... "Роль, карта, апчхи" - так это выглядит для меня :) Что такое "апчхи"? ... |
|||
:
Нравится:
Не нравится:
|
|||
11.05.2020, 11:24 |
|
Покритикуйте, пожалуйста, код.
|
|||
---|---|---|---|
#18+
Дмитрий Мух listtoview пропущено... EF так генерит, а что не так? В программировании существует лишь два характерных затруднения: инвалидация кеша и наименование сущностей :) +, какой-то уважаемый перец сформулировал ... |
|||
:
Нравится:
Не нравится:
|
|||
11.05.2020, 11:26 |
|
Покритикуйте, пожалуйста, код.
|
|||
---|---|---|---|
#18+
listtoview hVostt пропущено... Видимо, перестать использовать Entity-классы в качестве вью-моделей. Это касается всех пунктов. да, спасибо за совет. вообщем то это решает все 3 пункта нет, это решает только "перестать использовать Entity-классы в качестве вью-моделей" ... |
|||
:
Нравится:
Не нравится:
|
|||
11.05.2020, 11:27 |
|
Покритикуйте, пожалуйста, код.
|
|||
---|---|---|---|
#18+
а вообщето, этот товарищ listtoview, просто пытается разжечь срачь. он давний мембер, и так развлекается ... |
|||
:
Нравится:
Не нравится:
|
|||
11.05.2020, 11:30 |
|
Покритикуйте, пожалуйста, код.
|
|||
---|---|---|---|
#18+
love_bach а вообщето, этот товарищ listtoview, просто пытается разжечь срачь. он давний мембер, и так развлекается скажем так, я понимал что нужно использовать модель представления а не доменную модель для решения этой задачи просто задача простейшая и обычно обходятся доменной моделью сгенерированной какой ниб ОРМ ... |
|||
:
Нравится:
Не нравится:
|
|||
11.05.2020, 14:29 |
|
Покритикуйте, пожалуйста, код.
|
|||
---|---|---|---|
#18+
listtoview, для домена "представление", "модель представления" является "доменной моделью". ... |
|||
:
Нравится:
Не нравится:
|
|||
11.05.2020, 15:47 |
|
Покритикуйте, пожалуйста, код.
|
|||
---|---|---|---|
#18+
listtoview love_bach а вообщето, этот товарищ listtoview, просто пытается разжечь срачь. он давний мембер, и так развлекается скажем так, я понимал что нужно использовать модель представления а не доменную модель для решения этой задачи просто задача простейшая и обычно обходятся доменной моделью сгенерированной какой ниб ОРМ Это обманчивое впечателние, проще не будет. Используйте вью-модели. ... |
|||
:
Нравится:
Не нравится:
|
|||
11.05.2020, 21:50 |
|
Покритикуйте, пожалуйста, код.
|
|||
---|---|---|---|
#18+
listtoview obj.RoleId = rme.RoleId; obj.User = rme.User; Если там будет 50 полей? obj.Deleted = !flagDeleted ? null : (DateTime?)DateTime.Now; Почему этот класс выглядит как структура с public полями? ... |
|||
:
Нравится:
Не нравится:
|
|||
12.05.2020, 05:34 |
|
Покритикуйте, пожалуйста, код.
|
|||
---|---|---|---|
#18+
crutchmaster listtoview obj.RoleId = rme.RoleId; obj.User = rme.User; Если там будет 50 полей? obj.Deleted = !flagDeleted ? null : (DateTime?)DateTime.Now; Почему этот класс выглядит как структура с public полями? Потому что фактически это DTO, а они так и выглядят. Паттерн "anemic model". Классы сущностей тоже так обычно выглядят - никого же это не смущает. ... |
|||
:
Нравится:
Не нравится:
|
|||
12.05.2020, 09:56 |
|
Покритикуйте, пожалуйста, код.
|
|||
---|---|---|---|
#18+
fkthat crutchmaster пропущено... Если там будет 50 полей? пропущено... Почему этот класс выглядит как структура с public полями? Потому что фактически это DTO, а они так и выглядят. Паттерн "anemic model". Классы сущностей тоже так обычно выглядят - никого же это не смущает. я если руками делаю, то стараюсь максимально закрыть, а сетить только чз конструктор не знаю насколько это правильно, само получается ... |
|||
:
Нравится:
Не нравится:
|
|||
12.05.2020, 11:06 |
|
Покритикуйте, пожалуйста, код.
|
|||
---|---|---|---|
#18+
listtoview fkthat пропущено... Потому что фактически это DTO, а они так и выглядят. Паттерн "anemic model". Классы сущностей тоже так обычно выглядят - никого же это не смущает. я если руками делаю, то стараюсь максимально закрыть, а сетить только чз конструктор не знаю насколько это правильно, само получается DTO чз конструктор? ... |
|||
:
Нравится:
Не нравится:
|
|||
12.05.2020, 17:41 |
|
Покритикуйте, пожалуйста, код.
|
|||
---|---|---|---|
#18+
love_bach DTO чз конструктор? да мозгов нет, вот понапридумали всякую ерунду ... |
|||
:
Нравится:
Не нравится:
|
|||
12.05.2020, 17:46 |
|
Покритикуйте, пожалуйста, код.
|
|||
---|---|---|---|
#18+
fkthat Классы сущностей тоже так обычно выглядят - никого же это не смущает. Копипаста полей никого не смущает? Ну ок. ... |
|||
:
Нравится:
Не нравится:
|
|||
13.05.2020, 04:44 |
|
Покритикуйте, пожалуйста, код.
|
|||
---|---|---|---|
#18+
crutchmaster fkthat Классы сущностей тоже так обычно выглядят - никого же это не смущает. Копипаста полей никого не смущает? Ну ок. Не смущает. ... |
|||
:
Нравится:
Не нравится:
|
|||
13.05.2020, 04:55 |
|
|
start [/forum/topic.php?fid=18&msg=39955834&tid=1354707]: |
0ms |
get settings: |
11ms |
get forum list: |
13ms |
check forum access: |
3ms |
check topic access: |
3ms |
track hit: |
58ms |
get topic data: |
11ms |
get forum data: |
3ms |
get page messages: |
65ms |
get tp. blocked users: |
1ms |
others: | 284ms |
total: | 452ms |
0 / 0 |