Комментарии (20)

1. 

   sidristij

   28.04.2016 08:56

   #8874276

   +7

   Если вы понимаете «разницу» между (1), (2), (3), то итоговая разница во времени случайна. (6) https://habrahabr.ru/post/281294/#comment_8847910

1. 

   areht

   28.04.2016 13:45

   #8874792

   +2

   > Использование аоп тормознет приложение на прочь
   
   Там проблема не в АОП. Это АОП курильщика.
   
   Возьмите Fody/PropertyChanged, KindOfMagic.

   
   

   1. 

      agulyj

      29.04.2016 14:38

      #8876400

      Есть еще такой проект — AspectInjector разрабатыемый нами. В гитхабе как раз есть пример с INotifyPropertyChanged

   
   

   1. 

      InWake

      29.04.2016 15:13

      #8876460

      речь идет о динамическом АОП, конкретная реализация АОП
      KindOfMagic например, это статическое АОП

      
      

      1. 

         xXxVano

         04.05.2016 11:29

         #8880170

         Некоторое время назад хотел добавить немного АОП в свой код, но не устраивала постобработка скомпиленного кода и совместимость с текущим пайплайном. В результате для этого делал свой велосипед.
         Попробовал локально ваш тест прогнать, различия в районе 10% от 1-го решения. Мне кажется это более верная реализация АОП, если не хочется статического.
         Если захочется проверить, то в вся документация в тестах. Но что бы не копаться можно использовать такую реализацию:
         

         public class TestModel : NotificationObject, IData
         {
           [NotifyProperty]
           public virtual int Value { get; set; }
         
           public TestModel()
           {
             Notificator.Create(this);
           }
         }
         

1. 

   Ateist602

   28.04.2016 14:16

   #8874874

   +1

   Тяжело читать из-за колоссального количества орфографических и пунктуационных ошибок в тексте.

   
   

   1. 

      zodchiy

      29.04.2016 13:57

      #8876326

      Что странно, я отписался в личку автору еще вчера утром, но до сих пор висят ошибки.

      
      

      1. 

         InWake

         29.04.2016 14:36

         #8876398

         Только сейчас удалось выйти.

   
   

   1. 

      InWake

      29.04.2016 14:42

      #8876414

      часть ошибок исправил, спасибо zodchiy

1. 

   Ky7m

   29.04.2016 01:48

   #8875888

   Ещё хотелось бы посмотреть код, с помощью которого получились данные результаты.

   
   

   1. 

      InWake

      29.04.2016 14:43

      #8876418

      Выложу, чуть позже.

   
   

   1. 

      InWake

      30.04.2016 21:52

      #8878000

      Обещанного три года ждут…
      но мы ж не улитки
      
      вот код программы
      
      

      class Program
      	{
      		[STAThread]
      		static void Main( string[] args )
      		{
      			IUnityContainer container = new UnityContainer();
      
      			container.RegisterViewModel<DataAop>();
      			var dataAop = container.Resolve<DataAop>();
      
      			List<IData> listOfData = new List<IData>();
      			listOfData.Add( dataAop );
      
      			var asm = System.Reflection.Assembly.GetExecutingAssembly();
      
      			var excludes = new HashSet<Type> { typeof( IData ), typeof( DataAop ), typeof( ProxyData ), typeof( DataObservableObject ) };
      
      			listOfData.AddRange( asm.GetTypes()
      				.Where( t => typeof( IData ).IsAssignableFrom( t ) && !excludes.Contains( t ) ).Select( t => Activator.CreateInstance( t ) as IData ) );
      
      			foreach (var notify in listOfData.Cast<INotifyPropertyChanged>())
      				notify.PropertyChanged += OnPropertyChanged;
      
      			DataObservableObject observableData = new DataObservableObject();
      
      			var propDesp = DependencyPropertyDescriptor.FromProperty( ObservableObject<int>.ValueProperty, typeof( ObservableObject<int> ) );
      			propDesp.AddValueChanged( observableData.RealValue, OnPropertyChanged );
      
      			listOfData.Add( observableData );
      
      			Console.WriteLine( string.Format( "Start ---> {0}", DateTime.Now ) );
      
      			Stopwatch sw = new Stopwatch();
      			foreach (var data in listOfData)
      			{
      				sw.Restart();
      				for (int i = 0; i < 10000000; i++)
      					data.Value = i;
      				sw.Stop();
      
                                      Console.WriteLine();
      
      				Console.WriteLine( string.Format( "Iteration date {0}, data name {1}, elapsed {2}",
      					DateTime.Now, data.GetType().Name, sw.ElapsedMilliseconds ) );
      			}
      
      			Console.WriteLine( string.Format( "Stop ---> {0}", DateTime.Now ) );
      
      			Console.ReadKey();
      		}
      
      		static void OnPropertyChanged( object sender, EventArgs e ) { }
      
      		static void OnPropertyChanged( object sender, PropertyChangedEventArgs e ) { }
      	}
      

      
      
      интерфейс улиток (эх название у интерфейса подкачало)
      

      public interface IData
          {
              int Value { get; set; }
          }
      

      
      
      пример реализации
      

       class DataString : BindableBase, IData
          {
              int _value;
      
              public int Value
              {
                  get
                  {
                      return _value;
                  }
      
                  set
                  {
                      if (_value != value)
                      {
                          _value = value;
                          OnPropertyChanged("Value");
                      }
                  }
              }
          }
      

      
      
      особо опасная реализация
      

       class DataObservableObject : IData
          {
              public DataObservableObject()
              {
                  RealValue = new ObservableObject<int>();
              }
              public ObservableObject<int> RealValue { get; private set; }
      
              public int Value
              {
                  get
                  {
                      return RealValue.Value;
                  }
      
                  set
                  {
                      RealValue.Value = value;
                  }
              }
          }
      

      
      
      запустил тест на другом компьютере, и вот результат
      
      разница не велика

1. 

   ZimM

   29.04.2016 14:20

   #8876358

   Что такое [CallerMemberName]? Как происходит определение имени текущего свойства?

   
   

   1. 

      InWake

      29.04.2016 14:57

      #8876434

      Атрибут, появился в с#5 https://msdn.microsoft.com/ru-ru/library/system.runtime.compilerservices.callermembernameattribute(v=vs.110).aspx
      имя подставляется во время выполнения
      можно посмотреть https://msdn.microsoft.com/en-us/library/system.runtime.compilerservices(v=vs.110).aspx

      
      

      1. 

         ZimM

         29.04.2016 16:36

         #8876666

         Во время выполнения, или таки во время компиляции? Судя по отсутствию разницы в производительности между методами 1, 2, и 3, я бы предположил, что подстановка происходит прямо в вызове метода во время компиляции.

         
         

         1. 

            InWake

            30.04.2016 21:18

            #8877980

            Во время запуска при анализе байт кода, это делается только 1 раз

1. 

   Urbem

   29.04.2016 14:38

   #8876402

   От куда вы все знаете?
   Вот как вы пришли к этим знаниям?
   И какие интерфейсы стоит изучать в первую очередь(.NET)?

   
   

   1. 

      InWake

      29.04.2016 15:05

      #8876446

      Ну знаю я далеко не все
      Стандартный путь дзена — быдлокодинг -> шаблоны -> solid -> быдлокодинг -> просветление
      на самом деле очень важно работать в команде и обмениваться знаниями, а так же критика
      

      И какие интерфейсы стоит изучать в первую очередь(.NET)?

      что есть интерфейс?

1. 

   xmetropol

   29.04.2016 14:39

   #8876404

   Методы 1, 2 и 3 генерируют одинаковый IL код, разница лишь в предпочтении автора. 1-й может привести к ошибке, в случае переименования свойства,. 2-й метод в этом плане удобнее, т.к. инструмент вроде ReSharper автоматически переименует и свойство, и конструкцию nameof(Property). 5-й метод может быть удобен для реализации дополнительной логики: логирование, сравнение значений и генерация события лишь на действительном изменении значения, генерация события PropertyChanging а потом PropertyChanged. 4,6,7 не вижу смысла, да и вообще велосипеды облегчения реализации INotifyPropertyChanged не читаю, будто других проблем в WPF нет, как сэкономить 1 строку кода на 1-м свойстве. «Дабпиэф ничего не поделаешь» при отрисовке списка говорят, наверное, только джуны, никогда не читавшие исходников VirtualizingStackPanel и ItemsControl и не писавших свои панели, реализующие виртуализацию отображения, там в принципе нечему тормозить.

   
   

   1. 

      InWake

      29.04.2016 14:48

      #8876422

      В целом поддерживаю, за исключением четвертой улитки, хорошая альтернатива nameof в условиях отсутствия c# 6 версии. Ну и про тормознутый список говорил не юниор, а разработчик которому было искрнни лень исправлять проблемный код