Add add/remove/edit/moveup/movedown code

Author: Coding-Enthusiast

WatchOnlyBitcoinWallet/MainWindow.axaml

@@ -34,7 +34,7 @@
 
   <Grid RowDefinitions="auto,*,auto">
 
-    <Menu Background="LightGray" Grid.Row="0">
+    <Menu Background="GhostWhite" Grid.Row="0">
       <MenuItem Header="_Options">
         <MenuItem Header="_Settings"
                   InputGesture="F2"
@@ -105,7 +105,11 @@
         </TextBlock>
       </Grid>
 
-      <ListBox ItemsSource="{Binding AddressList}" Margin="3" Grid.Column="1">
+      <ListBox ItemsSource="{Binding AddressList}"
+               SelectedItem="{Binding SelectedAddress}"
+               SelectedIndex="{Binding SelectedIndex}"
+               Margin="3" Grid.Column="1">
+        
         <ListBox.Template>
           <ControlTemplate>
             <DockPanel LastChildFill="True">
@@ -143,19 +147,19 @@
           </Style>
         </StackPanel.Styles>
 
-        <Button ToolTip.Tip="Add New">
+        <Button ToolTip.Tip="Add New" Command="{Binding AddCommand}">
           <Image Source="/Assets/Add.png"/>
         </Button>
-        <Button ToolTip.Tip="Remove Selected">
+        <Button ToolTip.Tip="Remove Selected" Command="{Binding RemoveCommand}">
           <Image Source="/Assets/Remove.png"/>
         </Button>
-        <Button ToolTip.Tip="Edit Selected">
+        <Button ToolTip.Tip="Edit Selected" Command="{Binding EditCommand}">
           <Image Source="/Assets/Edit.png"/>
         </Button>
-        <Button ToolTip.Tip="Move Up">
+        <Button ToolTip.Tip="Move Up" Command="{Binding MoveUpCommand}">
           <Image Source="/Assets/MoveUp.png"/>
         </Button>
-        <Button ToolTip.Tip="Move Down">
+        <Button ToolTip.Tip="Move Down" Command="{Binding MoveDownCommand}">
           <Image Source="/Assets/MoveDown.png"/>
         </Button>
       </StackPanel>

WatchOnlyBitcoinWallet/ViewModels/MainWindowViewModel.cs

@@ -5,11 +5,11 @@
 
 using Avalonia.Input.Platform;
 using System;
-using System.Collections.Generic;
 using System.Collections.ObjectModel;
-using System.ComponentModel;
+using System.Diagnostics;
 using System.Linq;
 using System.Reflection;
+using System.Threading.Tasks;
 using WatchOnlyBitcoinWallet.Models;
 using WatchOnlyBitcoinWallet.MVVM;
 using WatchOnlyBitcoinWallet.Services;
@@ -24,9 +24,7 @@ public MainWindowViewModel()
             WindowMan = new WindowManager();
             FileMan = new FileManager();
 
-            AddressList = new BindingList<BitcoinAddress>(FileMan.ReadWalletFile());
-            AddressList.ListChanged += AddressList_ListChanged;
-
+            AddressList = new(FileMan.ReadWalletFile());
             SettingsInstance = FileMan.ReadSettingsFile();
 
             GetBalanceCommand = new BindableCommand(GetBalance, () => !IsReceiving);
@@ -37,37 +35,53 @@ public MainWindowViewModel()
             ImportFromTextCommand = new BindableCommand(ImportFromText);
             ImportFromFileCommand = new BindableCommand(ImportFromFile);
 
+            AddCommand = new(Add);
+            RemoveCommand = new(Remove, () => SelectedAddress is not null);
+            EditCommand = new(Edit, () => SelectedAddress is not null);
+            MoveUpCommand = new(MoveUp, () => SelectedIndex > 0);
+            MoveDownCommand = new(MoveDown, () => SelectedIndex >= 0 && SelectedIndex < AddressList.Count - 1);
+
             Version ver = Assembly.GetExecutingAssembly().GetName().Version ?? new Version();
             VersionString = string.Format("Version {0}.{1}.{2}", ver.Major, ver.Minor, ver.Build);
         }
 
 
 
-        void AddressList_ListChanged(object sender, ListChangedEventArgs e)
+        public ObservableCollection<BitcoinAddress> AddressList { get; set; }
+
+        private BitcoinAddress? _selAddr;
+        public BitcoinAddress? SelectedAddress
         {
-            if (e.ListChangedType == ListChangedType.ItemChanged)
+            get => _selAddr;
+            set
             {
-                BitcoinAddress addr = ((BindingList<BitcoinAddress>)sender)[e.NewIndex];
-                if (addr.Address != null)
-                {
-                    addr.Validate(addr.Address);
-                }
-                if (!addr.HasErrors)
+                if (SetField(ref _selAddr, value))
                 {
-                    DataManager.WriteFile(AddressList, DataManager.FileType.Wallet);
+                    RemoveCommand.RaiseCanExecuteChanged();
+                    EditCommand.RaiseCanExecuteChanged();
                 }
             }
-            else if (e.ListChangedType == ListChangedType.ItemDeleted || e.ListChangedType == ListChangedType.ItemAdded)
+        }
+
+        private int _selIndex = -1;
+        public int SelectedIndex
+        {
+            get => _selIndex;
+            set
             {
-                DataManager.WriteFile(AddressList, DataManager.FileType.Wallet);
+                if (SetField(ref _selIndex, value))
+                {
+                    MoveUpCommand.RaiseCanExecuteChanged();
+                    MoveDownCommand.RaiseCanExecuteChanged();
+                }
             }
         }
 
-
-        public IWindowManager WindowMan { get; set; }
         public IClipboard Clipboard { get; set; }
         public IFileManager FileMan { get; set; }
-
+        public IWindowManager WindowMan { get; set; }
+        public SettingsModel SettingsInstance { get; }
+        public string VersionString { get; }
 
         /// <summary>
         /// Indicating an active connection.
@@ -86,18 +100,6 @@ public bool IsReceiving
         }
         private bool isReceiving;
 
-        public string VersionString { get; private set; }
-
-
-        public BindingList<BitcoinAddress> AddressList { get; set; }
-
-
-        private SettingsModel settingsInstance;
-        public SettingsModel SettingsInstance
-        {
-            get { return settingsInstance; }
-            set { SetField(ref settingsInstance, value); }
-        }
 
         public decimal BitcoinBalance
         {
@@ -126,6 +128,36 @@ public decimal BitcoinBalanceLC
         }
 
 
+        private void SaveWallet()
+        {
+            FileMan.WriteWallet(AddressList.ToList());
+        }
+
+        private readonly object lockObj = new();
+        private bool isSavePending = false;
+        private async void QueueSaveWallet()
+        {
+            lock (lockObj)
+            {
+                if (isSavePending)
+                {
+                    return;
+                }
+                else
+                {
+                    isSavePending = true;
+                }
+            }
+
+            await Task.Delay(TimeSpan.FromSeconds(10));
+            lock (lockObj)
+            {
+                SaveWallet();
+                isSavePending = false;
+            }
+        }
+
+
         public BindableCommand OpenAboutCommand { get; private set; }
         private async void OpenAbout()
         {
@@ -255,5 +287,92 @@ private async void GetBalance()
             IsReceiving = false;
         }
 
+
+        public BindableCommand AddCommand { get; }
+        public async void Add()
+        {
+            AddEditViewModel vm = new();
+            await WindowMan.ShowDialog(vm);
+            if (vm.IsChanged)
+            {
+                BitcoinAddress t = new()
+                {
+                    Address = vm.AddressString,
+                    Name = vm.Tag
+                };
+                AddressList.Add(t);
+                SaveWallet();
+            }
+        }
+
+        public BindableCommand RemoveCommand { get; }
+        private void Remove()
+        {
+            if (SelectedAddress is not null)
+            {
+                AddressList.Remove(SelectedAddress);
+                SaveWallet();
+            }
+            else
+            {
+                Errors = "Nothing is selected!";
+            }
+        }
+
+        public BindableCommand EditCommand { get; }
+        public async void Edit()
+        {
+            if (SelectedAddress is not null)
+            {
+                AddEditViewModel vm = new()
+                {
+                    AddressString = SelectedAddress.Address,
+                    Tag = SelectedAddress.Name
+                };
+
+                await WindowMan.ShowDialog(vm);
+                if (vm.IsChanged)
+                {
+                    SelectedAddress.Address = vm.AddressString;
+                    SelectedAddress.Name = vm.Tag;
+
+                    SaveWallet();
+                } 
+            }
+            else
+            {
+                Errors = "Nothing is selected!";
+            }
+        }
+
+        public BindableCommand MoveUpCommand { get; }
+        private void MoveUp()
+        {
+            Debug.Assert(SelectedIndex != -1); // selected is not null
+            Debug.Assert(SelectedIndex != 0); // selected is not first item
+
+            // When items are swapped, the selected item turns into null so SelectedIndex value will turn into -1.
+            // Store it here to use later when setting the new selected item
+            int i = SelectedIndex;
+            (AddressList[i - 1], AddressList[i]) = (AddressList[i], AddressList[i - 1]);
+            SelectedAddress = AddressList[i - 1];
+
+            QueueSaveWallet();
+        }
+
+        public BindableCommand MoveDownCommand { get; }
+        private void MoveDown()
+        {
+            Debug.Assert(SelectedIndex != -1); // selected is not null
+            Debug.Assert(SelectedIndex != AddressList.Count - 1); // selected is not last item
+
+            // When items are swapped, the selected item turns into null so SelectedIndex value will turn into -1.
+            // Store it here to use later when setting the new selected item
+            int i = SelectedIndex;
+            (AddressList[i + 1], AddressList[i]) = (AddressList[i], AddressList[i + 1]);
+            SelectedAddress = AddressList[i + 1];
+
+            QueueSaveWallet();
+        }
     }
 }