ReSQL.ru
     
powered by simpleCommunicator - 2.0.51     © 2025 Programmizd 02
Мобильная версия    Контакт    Правила    FAQ    Помощь
Доб. в избранное | Игнор. тему | Прикреп. тему | Пометить прочит. / непрочит. | Фильтр
Форумы / WPF, Silverlight [игнор отключен] [закрыт для гостей] / Покритикуйте пожалуйста код
8 сообщений из 33, страница 2 из 2
  2  все
Покритикуйте пожалуйста код
    #38144545
Фотография iscrafm
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
iscrafm
Участник
wcfwpfсказали что нашли баг что мол после добавления кастомера, другой кастомер не добавить.
Ну что тут могу сказать... в медицине есть хорошее определение слабоумия: это когда человек неспособен фокусироваться на главных вещах и делает экстремальный фокус на каких-то несущественных моментах вместо главных.
Поэтому вместо того чтобы оценить что сделано за 4 дня, архитектуру, паттерны, технологии, они начинают искать вдруг найдется баг к которому можно придраться.
Марат, оценивают именно ТВОЮ работу, поэтому проверяли именно твой код, а не чью-то архитектуру, технологии и чьи-то паттерны, которые ты выбрал. В твоем случае оценивали умение воспользоваться чужими мыслями в своих целях. Нашли ошибку - указали на нее.
...
Рейтинг: 0 / 0
09.02.2013, 18:31
| Ответить | Цитировать | Написать  
Покритикуйте пожалуйста код
    #38144556
Фотография EDUARD SAPOTSKI
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
EDUARD SAPOTSKI
Участник
wcfwpf, во первых начинать нужно топик с четкой формулировки требований в ТЗ. Если сказали собрать винегрет со всех извесных тебе технологий, то возможно ты и правильно сделал, хотя это пусть SeVa скажет, он спец по передовым методологиям программирования, я только посмотрел с точки зрения рациональности - нифига не рационально.
Больше интересно, что такое в твоем понимании Model, DAL, Domain и Entity Framework, и что такое BLL "дополнительная бизнес логика"?
...
Рейтинг: 0 / 0
09.02.2013, 18:43
| Ответить | Цитировать | Написать  
Покритикуйте пожалуйста код
    #38144561
Фотография EDUARD SAPOTSKI
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
EDUARD SAPOTSKI
Участник
wcfwpfМожно ли ссылку на предложение мелкософта?
http://msdn.microsoft.com/ru-ru/library/ee707344(v=vs.91).aspx
...
Рейтинг: 0 / 0
09.02.2013, 18:48
| Ответить | Цитировать | Написать  
Покритикуйте пожалуйста код
    #38144608
SeVa
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
SeVa
Участник
EDUARD SAPOTSKIwcfwpfМожно ли ссылку на предложение мелкософта?
http://msdn.microsoft.com/ru-ru/library/ee707344(v=vs.91).aspx

Банальный datasource для таскателей контролов и необходимостью перелопачивать все при смене дизайна.

Это полное УГ с отсутствием нормальных бизнес-объектов с ленивой загрузкой и лапидарными интерфейсами, про возможности xaml по полной программе можно забыть.
...
Рейтинг: 0 / 0
09.02.2013, 19:52
| Ответить | Цитировать | Написать  
Покритикуйте пожалуйста код
    #38144627
SeVa
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
SeVa
Участник
SeVawcfwpfПрокоментируй плз третий пункт:
а) viewmodel не должны содержать загрузку классификаторов
б) "дублировать функционал combobox в коде - совершенно лишнее" - что тут имелось в виду?

Сейчас ухожу, отвечу позже


Для загрузки models можно сделать markup extensions, который будет грузить нужные данные автоматически по их типу.

Тогда, например, для твоих State данные подтянутся без viewmodel, список будет показан в combobox, который на автомате изменит нужно свойство в редактируемой Model
<UserControl.Resources>
<ViewModelProvider x:Key="States" Type="StateModels" />

<UserControl.Resources>


<ComboBox Grid.Row="3"
HorizontalAlignment="Stretch"
ItemsSource="{Binding Source={StaticResource States}, Path=Value, Mode=TwoWay}"
DisplayMemberPath="bla"
SelectedValue="{Binding Path=Model.Value.blabla,
Mode=TwoWay}"
SelectedValuePath="bla" />

ViewModelProvider - markup extensions c незамысловатым кодом за счет паттрена Repository и Rx
Код: c#
1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
11.
12.
13.
14.
15.
16.
17.
18.
19.
20.
21.
22.
23.
 public sealed class ViewModelProvider : MarkupExtension
    {
        public //Type 
            string Type { get; set; }

        public override object ProvideValue( IServiceProvider serviceProvider )
        {
            if (IsInDesingMode() || (ServiceLocator.Current == null))
                return null;

            if (this.Type != null)
            {
                var xamlTypeResolver = serviceProvider.GetService( typeof( IXamlTypeResolver ) ) as IXamlTypeResolver;
                Type type = xamlTypeResolver.Resolve( this.Type );
                var entityType = typeof( ObservableEntity<> ).MakeGenericType( type );
                var entity =(dynamic)Activator.CreateInstance( entityType );

                var res = entity.Repository().Fetch().ToUnit() as IObservable<Unit>;
                res.Subscribe<Unit>();
              return (object)entity;
            }
            return null;
        }
...
Рейтинг: 0 / 0
09.02.2013, 20:30
| Ответить | Цитировать | Написать  
Покритикуйте пожалуйста код
    #38144630
Фотография EDUARD SAPOTSKI
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
EDUARD SAPOTSKI
Участник
SeVaБанальный datasource для таскателей контролов и необходимостью перелопачивать все при смене дизайна.

Это полное УГ с отсутствием нормальных бизнес-объектов с ленивой загрузкой и лапидарными интерфейсами, про возможности xaml по полной программе можно забыть.
SeVa, ответь мне тогда на один простой вопрос! В мелкософтовских мануалах нигде не встрачаются "нормальные" бизнес-объекты с ленивой загрузкой с лапидарными интерфейсами, если и встрачается то явно не в основном контексте. Почему? Неужели в мелкософте работают полные долпоепы? Или эти навороты просто нах не нужны?
...
Рейтинг: 0 / 0
09.02.2013, 20:34
| Ответить | Цитировать | Написать  
Покритикуйте пожалуйста код
    #38144633
SeVa
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
SeVa
Участник
EDUARD SAPOTSKISeVaБанальный datasource для таскателей контролов и необходимостью перелопачивать все при смене дизайна.

Это полное УГ с отсутствием нормальных бизнес-объектов с ленивой загрузкой и лапидарными интерфейсами, про возможности xaml по полной программе можно забыть.
SeVa, ответь мне тогда на один простой вопрос! В мелкософтовских мануалах нигде не встрачаются "нормальные" бизнес-объекты с ленивой загрузкой с лапидарными интерфейсами, если и встрачается то явно не в основном контексте. Почему? Неужели в мелкософте работают полные долпоепы? Или эти навороты просто нах не нужны?

Они не нужны для шабашников с олд скульными подходами и тупыми интерфейсами на гридах.
А у ms много решений, посмотри prism и konaguidance.
...
Рейтинг: 0 / 0
09.02.2013, 20:49
| Ответить | Цитировать | Написать  
Покритикуйте пожалуйста код
    #38146292
Фотография @k@DElpher
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
@k@DElpher
Участник
wcfwpfEDUARD SAPOTSKI,


А Entity Framework в схеме для красоты чтоль?

Entity Framework - это технология, в проекте с ним у меня работают 3 library: Model, DAL, Domain



Очень даже верно, EF с виду напоминает DAL, но это ОRM, предназначена для перевода таблиц в объекты ООП.
DAL же решает задачу предоставления бизнес-логике объектов, которых сможет воспринять BLL не вникая в то, что там EF нагенерирует. Если топик стартеру понадобится изменить тип источника данных, скажем с SQLServer на текстовые файлы, другой сервис, или генерировать тестовые данные на ходу, то ему придется лишь заменить
Код: c#
1.
2.
3.
4.
5.
6.
7.
8.
9.
public List<Country> GetCountries()
        {
            using (var context = new TestEntities())
            {
                var list = context.Countries.OrderBy(p => p.CountryName).ToList();

                return list;
            }
        }


и List<Country> доставать откуда угодно. Больше ничего менять не придется.
Если убрать его DAL из проекта, то при просьбе воспользоваться источником данных, отличным от того, что поддерживает Entity Framework - придется туго.
Хотя если присмотреться, то у него Domain генерируются шаблонами на основе edmx. Мне интересно, по каким руководствам это все сделано?)
Я не шибко хороший советчик, но мне кажется, что автору для DAL не хватает интерфейсов. Тогда бы он с легостью сможет сделать в настройках механизм смены источника данных (кому-то SQL Server, кому-то My SQL, а кому-то и текстовых файлов хватит). Ну это если автор хочет сделать идеальный сферический шаблон для создания корпоративных приложений.
...
Рейтинг: 0 / 0
11.02.2013, 15:52
| Ответить | Цитировать | Написать  
8 сообщений из 33, страница 2 из 2
  2  все
Форумы / WPF, Silverlight [игнор отключен] [закрыт для гостей] / Покритикуйте пожалуйста код
Целевая тема:
Создать новую тему:
Автор:
Закрыть
Цитировать
Найденые пользователи ...
Разблокировать пользователей ...
Читали тему (0):
Читали форум (0):
Пользователи онлайн (0):
Польз. соглашение
NoSQL.ru       Новости, Чат       Фотоальбом: Участники, Встречи
Закрыть
start [/forum/topic.php?fid=21&msg=38144561&tid=1441496]:
 0ms
get settings:
 8ms
get forum list:
 14ms
check forum access:
 4ms
check topic access:
 4ms
track hit:
 55ms
get topic data:
 11ms
get forum data:
 3ms
get page messages:
 52ms
get tp. blocked users:
 1ms
others: 307ms
total:  459ms
созд. / загр.: 459ms
x
x
Закрыть


Просмотр
0 / 0
Close
Debug Console [Select Text]