Дать рекомендации по улучшению проекта

Уважаемые разработчики! Прошу кинуть ваш профессиональный взгляд на сей маленький проект.
Его задача сводится к работе одной формы: слева список пользователей (полное имя = фамилия имя, логин), справа: детальная информация о пользователе. Редактирование осуществляется кнопкой Update, данные сохраняются в базе. Находится он по адресу http://users.somee.com/

Прошу придраться ко всему что может быть непрофессиональным (код, комментарии, разметка, интерфейс и т.д.), наверное, кроме шапки сайта соответственно, кому не лень, очень желательно посмотреть исходники.

Я, конечно, далек от профессионала и хочу знать насколько... Никакой материальной выгоды не преследую, консультироваться и набираться опыта просто негде и не от кого. Проект прилагаю.

Вложения

UsersManager.zip (193.1 Кб, 11 просмотров)

0

Мм, не стал вникать в сам код, но по первому взгляду могу привести пару статеек, возможно поможет расширить кругозор. А там уже сами оцените качество своего проекта ).

http://ru.wikipedia.org/wiki/Model-View-Presenter
http://aspnetmvp.codeplex.com/
http://habrahabr.ru/post/113718/
http://ru.wikipedia.org/wiki/%... 0%BA%D0%B0
http://www.masters.donntu.edu.... yered.html
http://citforum.novgorod.ru/SE/project/pattern/
http://msdn.microsoft.com/library/ff632023.aspx
http://msdn.microsoft.com/ru-r... .100).aspx
http://rsdn.ru/article/mag/200401/codestyle.XML

1

Хорошо, много информации про MVP, прочитал, но все равно есть недоумения. А можно теперь просто на практике объяснить что не так с архитектурой моего проекта и как применить шаблон MVP(быть может его скачать надо и им пользоваться, либо это просто набор взглядов, по которым нужно разрабатывать проект)?

Я сделал так:
- Выделил отдельно класс, взаимодействующий с БД
- Бизнес-логика здесь, это серверный код страницы в файле .aspx.cs (который обрабатывает события, получает данные через класс, взаимодействующий с БД и лишь вызывает его методы и получает данные)
- Представление, собственно, разметка страницы и визуальная часть в файле .aspx

Что неверно с точки зрения профессионализма в построении архитектуры?

0

А можно теперь просто на практике объяснить что не так с архитектурой моего проекта

- все так , если нет каких либо особых требований к проекту, по сути можно один раз написать и забыть - и тогда не важно что и как там было сделано ).
Если проект покрупнее и планирует развиваться - масштабироваться и т.п , то уже желательно писать в хорошем стиле с применением различных технологий - выбор которых зависит от требований к проекту. (шаблон MVP лишь небольшая часть)

быть может его скачать надо и им пользоваться, либо это просто набор взглядов, по которым нужно разрабатывать проект ?

- прежде всего это шаблон проектирования UI (просто набор взглядов), есть еще масса подобных шаблонов - MVC, MVVM и т.п. , просто для веб.форм удобнее применять MVP.
Но, всегда есть какие либо фреймворки реализующие тот или иной подход - так что можно скачать и пользоваться, не изобретая велосипед.
Ну и применять его имеет смысл, если вы планируете иметь несколько различных интерфейсов (Скажем, есть еще толстый клиент WinFrorms)

- Бизнес-логика здесь, это серверный код страницы в файле .aspx.cs

- бизнес логика должна быть отвязана таким образом, что бы ее можно было легко использовать независимо от логики и типа UI, а так же тестировать в UnitTest-ах.
(По факту она вообще может быть размазана по различным внешним Workflow сервисам и т.п , ясно что к UI она никакого отношения не имеет)
В данном случае - это логика UI , которую кстати тоже можно вынести и юзать отдельно (если она достаточно абстрагированна от типов элементов управления).
Но тут нужно понимать, что применение громоздких технологий, шаблонов и прочих фич, оправдано только если это упростит развитие проекта в будущем.
Иначе , может получится так , что проект на 90% кода состоит из реализации того или иного подхода и трудозатраты на его поддержку выше чем просто переписать проект по новой.

0