回答編集履歴

見直しキャンペーン中

2023/08/14 10:10

投稿

TN8001

スコア9350

test CHANGED Viewed

@@ -295,8 +295,8 @@
 	}
 }
 ```
 「WinForms的発想」のほうを一時しのぎのつもりで書きましたが、かえって難しいしトータル行数も大差ない気がします^^;
+![アプリ画像](https://ddjkaamml8q8x.cloudfront.net/questions/2023-08-14/40d47835-a91b-44e4-b2da-a8ac5e0062f7.png)
 ---

見直しキャンペーン中

2023/07/27 16:16

投稿

TN8001

スコア9350

test CHANGED Viewed

@@ -1,605 +1,303 @@
 > コード作成者がすでに不在で既存部分も理解できていないため1から学習を始めるのですが、本件はすぐに対応が必要なため一時的にしのぎたいです。
 本当に何もかもわからない状態だとすると、なかなか厳しいと思いますね。
 少なくともテスト用のミニマムなプロジェクトを作って、試されたほうがいいと思います（提示コードはなんだかさっぱりわかりません）
 `DataGrid`の仮想化を切っても使える程度の少量の件数であれば、WinForms的な発想もありえます。
 具体的には`CellEditEnding`で`cell.Background = Brushes.Yellow`のような（あくまでイメージです。実際はこれではダメです）
-```xaml
+```xml
 <DataGrid
   EnableColumnVirtualization="False"
-  EnableRowVirtualization="False"
+  EnableRowVirtualization="False">
 ```
 こうしてみてとてもじゃないが使いものにならない（読み込み速度やスクロール・カラムサイズ変更等）なら、このアプローチはすっぱり諦めたほうがいいでしょう（手元では1000件程度でもかなり厳しい）
 WPF的には98mateさんの調査通り、おそらく`DataTrigger`でやるのが真っ当なのでしょう。
 > →(疑問)DataTriggerっていつ反映される？
 やってみたらすぐわかるのでは？（WPFでは`INotifyPropertyChanged`は欠かせないので、必ず押さえておいてください）
 > →(疑問)これだと背景を変えるセル(座標)が不明？参考サイトだと元々バインドされたプロパティが変化したときの例。
 発想が逆です。セルごとに状態が必要ならば、セル個数分プロパティが必要です。
-```xaml
+```xml
 <Window
   x:Class="Questions346520.MainWindow"
   xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
   xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
   Width="800"
   Height="450">
   <Window.Resources>
     <Style x:Key="dirtyStyle" BasedOn="{StaticResource {x:Type DataGridCell}}" TargetType="DataGridCell">
       <Setter Property="Background" Value="Yellow" />
     </Style>
   </Window.Resources>
   <Grid>
     <Grid.ColumnDefinitions>
       <ColumnDefinition />
       <ColumnDefinition />
     </Grid.ColumnDefinitions>
     <GroupBox Header="WinForms的発想">
       <DockPanel>
         <Button Click="Button1_Click" Content="更新" DockPanel.Dock="Top" />
         <DataGrid
           x:Name="dataGrid1"
           AutoGenerateColumns="False"
           BeginningEdit="DataGrid1_BeginningEdit"
           CellEditEnding="DataGrid1_CellEditEnding"
           EnableColumnVirtualization="False"
           EnableRowVirtualization="False"
           ItemsSource="{Binding Records1}"
           SelectionUnit="CellOrRowHeader">
           <DataGrid.Columns>
             <DataGridTextColumn Binding="{Binding No}" Header="No" IsReadOnly="True" />
             <DataGridTextColumn Binding="{Binding UshortValue}" Header="ushort" />
             <DataGridTextColumn Binding="{Binding StringValue}" Header="string" />
           </DataGrid.Columns>
         </DataGrid>
       </DockPanel>
     </GroupBox>
     <GroupBox Grid.Column="1" Header="DataTrigger">
       <DockPanel>
         <Button Click="Button2_Click" Content="更新" DockPanel.Dock="Top" />
         <DataGrid AutoGenerateColumns="False" ItemsSource="{Binding Records2}" SelectionUnit="CellOrRowHeader">
           <DataGrid.Columns>
             <DataGridTextColumn Binding="{Binding No}" Header="No" IsReadOnly="True" />
             <DataGridTextColumn Binding="{Binding UshortValue}" Header="ushort">
               <DataGridTextColumn.CellStyle>
                 <Style TargetType="DataGridCell">
                   <Style.Triggers>
                     <DataTrigger Binding="{Binding IsDirtyUshortValue}" Value="True">
                       <Setter Property="Background" Value="Yellow" />
                     </DataTrigger>
                   </Style.Triggers>
                 </Style>
               </DataGridTextColumn.CellStyle>
             </DataGridTextColumn>
             <DataGridTextColumn Binding="{Binding StringValue}" Header="string">
               <DataGridTextColumn.CellStyle>
                 <Style TargetType="DataGridCell">
                   <Style.Triggers>
                     <DataTrigger Binding="{Binding IsDirtyStringValue}" Value="True">
                       <Setter Property="Background" Value="Yellow" />
                     </DataTrigger>
                   </Style.Triggers>
                 </Style>
               </DataGridTextColumn.CellStyle>
             </DataGridTextColumn>
           </DataGrid.Columns>
         </DataGrid>
       </DockPanel>
     </GroupBox>
   </Grid>
 </Window>
 ```
-```C#
+```cs
 using System;
 using System.Collections.Generic;
 using System.ComponentModel;
 using System.Linq;
 using System.Runtime.CompilerServices;
 using System.Windows;
 using System.Windows.Controls;
 using System.Windows.Media;
 namespace Questions346520
 {
 	public partial class MainWindow : Window
 	{
 		public List<Record1> Records1 { get; } = new List<Record1>();
 		public List<Record2> Records2 { get; } = new List<Record2>();
 		private bool isDirty1;
 		private string oldValue;
 		private Style defaultStyle;
 		private Style dirtyStyle;
 		public MainWindow()
-		{
+		{
 			InitializeComponent();
 			DataContext = this;
 			defaultStyle = FindResource(typeof(DataGridCell)) as Style;
 			dirtyStyle = FindResource("dirtyStyle") as Style;
 			for (var i = 0; i < 1000; i++)
-			{
+			{
 				Records1.Add(new Record1()
 				{
 					No = i,
 					UshortValue = (ushort)(i * 10),
 					StringValue = (i * 100).ToString(),
 				});
 				// ↑↓片方ずつコメント化してパフォーマンスチェック
 				Records2.Add(new Record2()
 				{
 					No = i,
 					UshortValue = (ushort)(i * 10),
 					StringValue = (i * 100).ToString(),
 					IsDirtyUshortValue = false,
 					IsDirtyStringValue = false,
 				});
-			}
+			}
-		}
+		}
 		private void DataGrid1_BeginningEdit(object sender, DataGridBeginningEditEventArgs e)
-		{
+		{
 			if (e.EditingEventArgs.Source is TextBlock textBlock)
 				oldValue = textBlock.Text;
 			if (e.EditingEventArgs.Source is DataGridCell cell)
 				oldValue = (cell.Content as TextBlock).Text;
-		}
+		}
 		private void DataGrid1_CellEditEnding(object sender, DataGridCellEditEndingEventArgs e)
-		{
+		{
 			if (e.EditAction != DataGridEditAction.Commit)
-			{
+			{
 				oldValue = null;
 				return;
-			}
+			}
 			var textbox = e.EditingElement as TextBox;
 			var newValue = textbox.Text;
 			if (oldValue == newValue)
-			{
+			{
 				oldValue = null;
 				return;
-			}
+			}
 			var cell = textbox.Parent as DataGridCell;
 			cell.Style = dirtyStyle;
 			isDirty1 = true;
-		}
+		}
 		private void Button1_Click(object sender, RoutedEventArgs e)
-		{
+		{
 			if (!isDirty1) return;
 			foreach (var cell in dataGrid1.Descendants<DataGridCell>())
 				cell.Style = defaultStyle;
 			isDirty1 = false;
-		}
+		}
 		private void Button2_Click(object sender, RoutedEventArgs e)
-		{
+		{
 			if (!Record2.IsDirty2) return;
 			foreach (var record2 in Records2)
-			{
+			{
 				record2.IsDirtyUshortValue = false;
 				record2.IsDirtyStringValue = false;
-			}
+			}
 			Record2.IsDirty2 = false;
-		}
+		}
 		public class Record1
-		{
+		{
 			public int No { get; set; }
 			public ushort UshortValue { get; set; }
 			public string StringValue { get; set; }
-		}
+		}
 		public class Record2 : Observable
-		{
+		{
 			// ダサいがこうするしかないか？
 			public static bool IsDirty2;
 			public int No { get; set; }
 			public ushort UshortValue
-			{
+			{
 				get => _UshortValue;
 				set
 				{
 					if (_UshortValue != value)
 					{
 						_UshortValue = value;
 						IsDirtyUshortValue = true;
 					}
 				}
-			}
+			}
 			private ushort _UshortValue;
 			// ValueはPropertyChangedを発砲する必要はないが、↑は面倒なので使っちゃうw
 			public string StringValue
-			{
+			{
 				get => _StringValue;
 				set { if (SetProperty(ref _StringValue, value)) IsDirtyStringValue = true; }
-			}
+			}
 			private string _StringValue;
 			// こちらはコードから変更されるため、PropertyChangedを発砲しないとViewに反映されない
 			public bool IsDirtyUshortValue
-			{
+			{
 				get => _IsDirtyUshortValue;
 				set { if (SetProperty(ref _IsDirtyUshortValue, value)) IsDirty2 = true; }
-			}
+			}
 			private bool _IsDirtyUshortValue;
 			public bool IsDirtyStringValue
-			{
+			{
 				get => _IsDirtyStringValue;
 				set { if (SetProperty(ref _IsDirtyStringValue, value)) IsDirty2 = true; }
-			}
+			}
 			private bool _IsDirtyStringValue;
-		}
+		}
 	}
 	// [VisualTreeの子孫要素を取得する - xin9le.net](https://blog.xin9le.net/entry/2013/10/29/222336)
 	static class DependencyObjectExtensions
 	{
 		public static IEnumerable<DependencyObject> Children(this DependencyObject obj)
-		{
+		{
 			if (obj == null) throw new ArgumentNullException(nameof(obj));
 			var count = VisualTreeHelper.GetChildrenCount(obj);
 			if (count == 0) yield break;
 			for (var i = 0; i < count; i++)
-			{
+			{
 				var child = VisualTreeHelper.GetChild(obj, i);
 				if (child != null) yield return child;
-			}
+			}
-		}
+		}
 		public static IEnumerable<DependencyObject> Descendants(this DependencyObject obj)
-		{
+		{
 			if (obj == null) throw new ArgumentNullException(nameof(obj));
 			foreach (var child in obj.Children())
-			{
+			{
 				yield return child;
 				foreach (var grandChild in child.Descendants())
 					yield return grandChild;
-			}
+			}
-		}
+		}
 		public static IEnumerable<T> Descendants<T>(this DependencyObject obj) where T : DependencyObject => obj.Descendants().OfType<T>();
 	}
 	public class Observable : INotifyPropertyChanged
 	{
 		public event PropertyChangedEventHandler PropertyChanged;
 		protected bool SetProperty<T>(ref T storage, T value, [CallerMemberName] string propertyName = null)
-		{
+		{
 			if (Equals(storage, value)) return false;
 			storage = value;
 			OnPropertyChanged(propertyName);
 			return true;
-		}
+		}
 		protected void OnPropertyChanged(string propertyName) => PropertyChanged?.Invoke(this, new PropertyChangedEventArgs(propertyName));
 	}
 }
 ```
 「WinForms的発想」のほうを一時しのぎのつもりで書きましたが、かえって難しいしトータル行数も大差ない気がします^^;
 ---
 行ごとの判定でよければ`DataTable`をソースにすれば、簡単にできそうな気はします（おそらく「それじゃダメ」と言われそうなので特に調べていませんが^^;