HamBook – Diff between revs 53 and 54

Subversion Repositories:

?pathlinks?

Rev 53 – Rev 56 – Go to most recent revision – Details – Blame – Last modification – View Log

-using Configuration;
-using HamBook.Properties;
-using HamBook.Radios;
-using HamBook.Radios.Generic;
-using HamBook.Utilities;
-using HamBook.Utilities.Controls;
-using HamBook.Utilities.Serialization;
-using Serilog;
 using System;
 using System.Collections.Concurrent;
 using System.Collections.Generic;
 using System.ComponentModel;
-using System.Data;
 using System.Drawing;
 using System.Linq;
 using System.Media;
 using System.Reflection;
-using System.Runtime.Remoting.Channels;
 using System.Threading;
 using System.Threading.Tasks;
 using System.Windows.Forms;
+using HamBook.Properties;
+using HamBook.Radios;
+using HamBook.Radios.Generic;
+using HamBook.Utilities.Serialization;
+using Serilog;
-using MemoryChannel = HamBook.Radios.Generic.MemoryChannel;
+using MemoryRadioMode = HamBook.Radios.Yaesu.FT_891.MemoryRadioMode;
 namespace HamBook
+{
     public partial class MemoryOrganizerForm : Form
     {
-        private Configuration.Configuration Configuration { get; set; }
+        private readonly CancellationTokenSource _cancellationTokenSource;
+        private readonly CatAssemblies _catAssemblies;
+        private readonly CancellationToken _localCancellationToken;
         private readonly CancellationTokenSource _localCancellationTokenSource;
-        private CatAssemblies _catAssemblies;
-        private CancellationToken _cancellationToken;
-        private Radios.Generic.MemoryBanks _memoryBanks;
-        private CancellationTokenSource _cancellationTokenSource;
-        private CancellationTokenSource _localCancellationTokenSource;
+        private readonly MemoryBanks _memoryBanks;
+        private CancellationToken _cancellationToken;
+        private List<DataGridViewRow> _clipboardRows;
+        private CancellationTokenSource _writeCancellationTokenSource;
+        private CancellationTokenSource _writeLinkedCancellationTokenSource;
+        private Task _writeMemoryBanksTask;
+        private CancellationTokenSource _readCancellationTokenSource;
-        private CancellationToken _localCancellationToken;
+        private CancellationTokenSource _readLinkedCancellationTokenSource;
-        private List<DataGridViewRow> _clipboardRows;
+        private Task _readMemoryBanksTask;
         public MemoryOrganizerForm()
+        {
             InitializeComponent();
             Utilities.WindowState.FormTracker.Track(this);
+            _localCancellationTokenSource = new CancellationTokenSource();
-            _localCancellationTokenSource = new CancellationTokenSource();
+            _localCancellationToken = _localCancellationTokenSource.Token;
             _localCancellationToken = _localCancellationTokenSource.Token;
+        public MemoryOrganizerForm(Configuration.Configuration configuration, CatAssemblies catAssemblies,
+            CancellationToken cancellationToken) : this()
         public MemoryOrganizerForm(Configuration.Configuration configuration, CatAssemblies catAssemblies, CancellationToken cancellationToken) : this()
-        {
+            Configuration = configuration;
-            Configuration = configuration;
+            _catAssemblies = catAssemblies;
+            _cancellationTokenSource =
-            _catAssemblies = catAssemblies;
+                CancellationTokenSource.CreateLinkedTokenSource(_localCancellationToken, cancellationToken);
+            _cancellationToken = _cancellationTokenSource.Token;
             _cancellationTokenSource =  CancellationTokenSource.CreateLinkedTokenSource(_localCancellationToken, cancellationToken);
-            _cancellationToken = _cancellationTokenSource.Token;
+            _memoryBanks = MemoryBanks.Create(Configuration.Radio);
+        }
             _memoryBanks = Radios.Generic.MemoryBanks.Create(Configuration.Radio);
-        }
+        private Configuration.Configuration Configuration { get; }
-        /// <summary>
-        /// Clean up any resources being used.
-        /// </summary>
+        /// <summary>
-        /// <param name="disposing">true if managed resources should be disposed; otherwise, false.</param>
-        protected override void Dispose(bool disposing)
+        ///     Clean up any resources being used.
         /// </summary>
+        /// <param name="disposing">true if managed resources should be disposed; otherwise, false.</param>
-            if (disposing && (components != null))
+        protected override void Dispose(bool disposing)
+        {
-                if (_cancellationTokenSource != null)
+            if (disposing && components != null)
+            {
-                    _cancellationTokenSource.Cancel();
+                if (_cancellationTokenSource != null) _cancellationTokenSource.Cancel();
                 components.Dispose();
                 components.Dispose();
             base.Dispose(disposing);
             base.Dispose(disposing);
         private async void button1_Click(object sender, EventArgs e)
         private async void button1_Click(object sender, EventArgs e)
             var rows = dataGridView1.Rows.OfType<DataGridViewRow>().OrderBy(row => row.Index).ToList();
-            var rows = dataGridView1.Rows.OfType<DataGridViewRow>().OrderBy(row => row.Index).ToList();
+            var count = rows.Count;
-            var count = rows.Count;
+            toolStripProgressBar1.Minimum = 0;
             toolStripProgressBar1.Maximum = count;
-            toolStripProgressBar1.Minimum = 0;
+            toolStripProgressBar1.Value = toolStripProgressBar1.Minimum;
             toolStripProgressBar1.Maximum = count;
             var progress = new Progress<DataGridViewRowProgress>(rowProgress =>
             var progress = new Progress<DataGridViewRowProgress>(rowProgress =>
-            {
+                try
+                {
-                try
+                    switch (rowProgress)
+                    {
-                    switch (rowProgress)
+                        case DataGridViewRowProgressSuccess<MemoryChannel> rowProgressSuccess:
                             dataGridView1.Rows[rowProgressSuccess.Row.Index].DefaultCellStyle.BackColor =
-                        case DataGridViewRowProgressSuccess<MemoryChannel> rowProgressSuccess:
+                                DefaultBackColor;
                                     break;
+                            }
                             toolStripStatusLabel1.Text = $"{Resources.Read_memory_bank} {rowProgress.Index + 1}";
                             break;
-                        case DataGridViewRowProgressFailure rowProgressFailure:
+                        case DataGridViewRowProgressFailure<int> rowProgressFailure:
                             Log.Error(rowProgressFailure.Exception, $"{Resources.Could_not_read_memory_bank}");
+                            dataGridView1.Rows[rowProgressFailure.Row.Index].DefaultCellStyle.BackColor = Color.Red;
-                            dataGridView1.Rows[rowProgressFailure.Row.Index].DefaultCellStyle.BackColor = Color.Red;
                             toolStripStatusLabel1.Text =
-                            toolStripStatusLabel1.Text = $"{Resources.Could_not_read_memory_bank} {rowProgress.Index + 1}";
+                                $"{Resources.Could_not_read_memory_bank} {rowProgress.Index + 1}";
                             break;
+                    }
             var rows = dataGridView1.Rows.OfType<DataGridViewRow>().OrderBy(row => row.Index).ToList();
             var count = rows.Count;
             toolStripProgressBar1.Minimum = 0;
+            toolStripProgressBar1.Maximum = count;
-            toolStripProgressBar1.Maximum = count;
+            toolStripProgressBar1.Value = toolStripProgressBar1.Minimum;
             var progress = new Progress<DataGridViewRowProgress>(rowProgress =>
+            {
                 try
+                {
                     switch (rowProgress)
                     {
+                        case DataGridViewRowProgressSuccess<bool> rowProgressSuccess:
-                        case DataGridViewRowProgressSuccess<bool> rowProgressSuccess:
+                            dataGridView1.Rows[rowProgressSuccess.Row.Index].DefaultCellStyle.BackColor =
-                            dataGridView1.Rows[rowProgressSuccess.Row.Index].DefaultCellStyle.BackColor = DataGridView.DefaultBackColor;
+                                DefaultBackColor;
                             var success = rowProgressSuccess.Data;
-                            if (success)
                             if (success)
+                            {
                                 toolStripStatusLabel1.Text =
-                                $"{Resources.Wrote_memory_bank} {rowProgress.Index + 1}";
+                                    $"{Resources.Wrote_memory_bank} {rowProgress.Index + 1}";
                                 toolStripProgressBar1.Increment(1);
                                 statusStrip1.Update();
                                 return;
+                            }
                             Log.Error($"{Resources.Could_not_write_memory_bank}");
                             break;
-                        case DataGridViewRowProgressFailure rowProgressFailure:
+                        case DataGridViewRowProgressFailure<int> rowProgressFailure:
                             dataGridView1.Rows[rowProgressFailure.Row.Index].DefaultCellStyle.BackColor = Color.Red;
+                }
                 catch (Exception exception)
+                {
                     Log.Error(exception, Resources.Unexpected_error_while_writing_memory_bank);
+                }
             });
             await Task.Run(() => WriteMemoryBanks(rows, progress, _cancellationToken), _cancellationToken);
+        }
         private void MemoryOrganizerForm_Load(object sender, EventArgs e)
+        {
             // Generate columns based on radio type.
-            switch(Configuration.Radio)
+            switch (Configuration.Radio)
+            {
+                case "Yaesu FT-891":
-                case "Yaesu FT-891":
+                    dataGridView1.Columns.Add(new DataGridViewTextBoxColumn
+                        { Name = "LocationColumn", HeaderText = "Location", ReadOnly = true });
-                    dataGridView1.Columns.Add(new DataGridViewTextBoxColumn() { Name = "LocationColumn", HeaderText = "Location", ReadOnly = true });
+                    dataGridView1.Columns.Add(new DataGridViewTextBoxColumn
+                        { Name = "FrequencyColumn", HeaderText = "Frequency" });
-                    dataGridView1.Columns.Add(new DataGridViewTextBoxColumn() { Name = "FrequencyColumn", HeaderText = "Frequency" });
+                    var clarifierDropDownColumn = new DataGridViewComboBoxColumn
-                    var clarifierDropDownColumn = new DataGridViewComboBoxColumn() { Name = "ClarifierDirectionColumn", HeaderText = "Clarifier" };
+                        { Name = "ClarifierDirectionColumn", HeaderText = "Clarifier" };
                     clarifierDropDownColumn.Items.AddRange(
                         "+",
                         "-"
                     );
+                    dataGridView1.Columns.Add(clarifierDropDownColumn);
-                    dataGridView1.Columns.Add(clarifierDropDownColumn);
+                    dataGridView1.Columns.Add(new DataGridViewTextBoxColumn
-                    dataGridView1.Columns.Add(new DataGridViewTextBoxColumn() { Name = "ClarifierOffsetColumn", HeaderText = "Offset" });
+                        { Name = "ClarifierOffsetColumn", HeaderText = "Offset" });
+                    dataGridView1.Columns.Add(new DataGridViewCheckBoxColumn
-                    dataGridView1.Columns.Add(new DataGridViewCheckBoxColumn() { Name = "ClarColumn", HeaderText = "Clar" });
+                        { Name = "ClarColumn", HeaderText = "Clar" });
-                    var modeComboBoxColumn = new DataGridViewComboBoxColumn() { Name = "ModeColumn", HeaderText = "Mode" };
+                    var modeComboBoxColumn = new DataGridViewComboBoxColumn
-                    foreach (var name in Radios.Yaesu.FT_891.MemoryRadioMode.Names)
+                        { Name = "ModeColumn", HeaderText = "Mode" };
                     foreach (var name in MemoryRadioMode.Names)
                         modeComboBoxColumn.Items.Add(
                             name
-                        );
-                    }
+                        );
                     dataGridView1.Columns.Add(modeComboBoxColumn);
+                    var ctcssComboBoxColumn = new DataGridViewComboBoxColumn
-                    var ctcssComboBoxColumn = new DataGridViewComboBoxColumn() { Name = "CtcssColumn", HeaderText = "CTCSS" };
+                        { Name = "CtcssColumn", HeaderText = "CTCSS" };
                     ctcssComboBoxColumn.Items.AddRange(
                         "Off",
                         "Enc/Dec",
                         "Enc"
                     );
                     dataGridView1.Columns.Add(ctcssComboBoxColumn);
+                    var phaseComboBoxColumn = new DataGridViewComboBoxColumn
-                    var phaseComboBoxColumn = new DataGridViewComboBoxColumn() { Name = "PhaseColumn", HeaderText = "Phase" };
+                        { Name = "PhaseColumn", HeaderText = "Phase" };
                     phaseComboBoxColumn.Items.AddRange(
                         "Simplex",
                         "Plus Shift",
                         "Minus Shift"
                     );
                     dataGridView1.Columns.Add(phaseComboBoxColumn);
+                    dataGridView1.Columns.Add(new DataGridViewCheckBoxColumn
+                        { Name = "TagColumn", HeaderText = "Tag" });
+                    dataGridView1.Columns.Add(new DataGridViewTextBoxColumn
+                    {
-                    dataGridView1.Columns.Add(new DataGridViewCheckBoxColumn() { Name = "TagColumn", HeaderText = "Tag" });
+                        Name = "TextColumn", HeaderText = "Text",
+                        MaxInputLength = Radios.Yaesu.FT_891.Constants.MaxTagCharacters
-                    dataGridView1.Columns.Add(new DataGridViewTextBoxColumn() { Name = "TextColumn", HeaderText = "Text", MaxInputLength = Radios.Yaesu.FT_891.Constants.MAX_TAG_CHARACTERS });
+                    });
                     break;
+            }
+            toolStripProgressBar1.Minimum = 0;
-            toolStripProgressBar1.Minimum = 0;
+            toolStripProgressBar1.Maximum = 98;
-            toolStripProgressBar1.Maximum = 98;
+            toolStripProgressBar1.Value = toolStripProgressBar1.Minimum;
                         return;
+                    }
                     var index = dataGridView1.Rows.Add();
-                    switch(Configuration.Radio)
+                    switch (Configuration.Radio)
+                    {
                         case "Yaesu FT-891":
                             dataGridView1.Rows[index].Cells["LocationColumn"].Value = memoryBank;
+            }
             Application.Idle += IdleHandler;
             try
             {
-                foreach (var memoryBank in _memoryBanks.GetMemoryBanks())
-                {
-                    memoryBankQueue.Enqueue(memoryBank);
                 foreach (var memoryBank in _memoryBanks.GetMemoryBanks()) memoryBankQueue.Enqueue(memoryBank);
                 memoryBankAddRowsTaskCompletionSource.TrySetResult(new { });
+            }
         private void DataGridView1_CurrentCellDirtyStateChanged(object sender, EventArgs e)
+        {
             var dataGridView = (DataGridView)sender;
-            if (dataGridView.CurrentCell is DataGridViewCheckBoxCell)
             if (dataGridView.CurrentCell is DataGridViewCheckBoxCell)
-                dataGridView.CommitEdit(DataGridViewDataErrorContexts.Commit);
                 dataGridView.CommitEdit(DataGridViewDataErrorContexts.Commit);
+        }
         private void DataGridView1_CellEndEdit(object sender, DataGridViewCellEventArgs e)
             dataGridView.CommitEdit(DataGridViewDataErrorContexts.Commit);
+        }
         private void DataGridView1_CellContentClick(object sender, DataGridViewCellEventArgs e)
-        {
+        {
+        }
         private void DataGridView1_CellValueChanged(object sender, DataGridViewCellEventArgs e)
         {
-            var dataGridView = (DataGridView)sender;
-            if (e.RowIndex == -1 || e.ColumnIndex == -1)
             var dataGridView = (DataGridView)sender;
                 return;
             if (e.RowIndex == -1 || e.ColumnIndex == -1) return;
+        {
             var dataGridView = (DataGridView)sender;
             if (e.RowIndex == -1 || e.ColumnIndex == -1 ||
-                !(dataGridView.Columns[e.ColumnIndex] is DataGridViewCheckBoxColumn))
                 !(dataGridView.Columns[e.ColumnIndex] is DataGridViewCheckBoxColumn))
-                return;
                 return;
             dataGridView.EndEdit();
+        }
         private void DataGridView_CellDoubleClick(object sender, DataGridViewCellEventArgs e)
+        {
-            var dataGridView = (DataGridView)sender;
             var dataGridView = (DataGridView)sender;
-            if (e.RowIndex == -1 || e.ColumnIndex == -1 ||
                 !(dataGridView.Columns[e.ColumnIndex] is DataGridViewCheckBoxColumn))
             if (e.RowIndex == -1 || e.ColumnIndex == -1 ||
-                return;
+                !(dataGridView.Columns[e.ColumnIndex] is DataGridViewCheckBoxColumn))
         private static IEnumerable<DataGridViewRow> GetSelectedDataGridViewRows(DataGridView dataGridView)
+        {
             return dataGridView.SelectedRows.OfType<DataGridViewRow>().OrderBy(row => row.Index);
+        }
+        private async Task ReadMemoryBanks(IReadOnlyList<DataGridViewRow> rows,
-        private async Task ReadMemoryBanks(IReadOnlyList<DataGridViewRow> rows, IProgress<DataGridViewRowProgress> progress,
+            IProgress<DataGridViewRowProgress> progress,
-                                 CancellationToken cancellationToken)
+            CancellationToken cancellationToken)
+        {
             var count = rows.Count;
             for (var i = 0; i < count && !cancellationToken.IsCancellationRequested; ++i)
             for (var i = 0; i < count && !cancellationToken.IsCancellationRequested; ++i)
                 try
+                {
-                {
+                    var location = $"{rows[i].Cells["LocationColumn"].Value}";
-                    var location = $"{rows[i].Cells["LocationColumn"].Value}";
+                    var result =
                         await _catAssemblies.CatReadAsync<MemoryChannel>("MT", new object[] { location },
-                    var result = await _catAssemblies.CatReadAsync<MemoryChannel>("MT", new object[] { location }, _cancellationToken);
+                            cancellationToken);
                     progress.Report(new DataGridViewRowProgressSuccess<MemoryChannel>(rows[i], i, result));
+                }
                 catch (UnexpectedRadioResponseException exception)
+                {
-                    progress.Report(new DataGridViewRowProgressFailure(rows[i], i, exception));
+                    progress.Report(new DataGridViewRowProgressFailure<int>(rows[i], i, exception));
+                }
-                catch (Exception exception)
                 catch (Exception exception)
-                    progress.Report(new DataGridViewRowProgressFailure(rows[i], i, exception));
+                {
+                    progress.Report(new DataGridViewRowProgressFailure<int>(rows[i], i, exception));
                 }
+        }
         private async Task WriteMemoryBanks(IReadOnlyList<DataGridViewRow> rows,
-        private async Task WriteMemoryBanks(IReadOnlyList<DataGridViewRow> rows, IProgress<DataGridViewRowProgress> progress,
-                         CancellationToken cancellationToken)
+            IProgress<DataGridViewRowProgress> progress,
             CancellationToken cancellationToken)
             var count = rows.Count;
+            var count = rows.Count;
             for (var i = 0; i < count && !cancellationToken.IsCancellationRequested; ++i)
             for (var i = 0; i < count && !cancellationToken.IsCancellationRequested; ++i)
+                try
-                try
+                {
                     var success = false;
                     var success = false;
+                    switch (Configuration.Radio)
+                    {
+                        case "Yaesu FT-891":
-                    switch(Configuration.Radio)
+                            var memoryChannel =
                                 (Radios.Yaesu.FT_891.MemoryChannel)MemoryChannel.Create(Configuration.Radio);
-                        case "Yaesu FT-891":
+                            memoryChannel.CurrentLocation = $"{rows[i].Cells["LocationColumn"].Value}";
-                            var memoryChannel = (Radios.Yaesu.FT_891.MemoryChannel)MemoryChannel.Create(Configuration.Radio);
+                            memoryChannel.Frequency = int.Parse($"{rows[i].Cells["FrequencyColumn"].Value}");
                             memoryChannel.ClarifierDirection =
-                            memoryChannel.CurrentLocation = $"{rows[i].Cells["LocationColumn"].Value}";
+                                char.Parse($"{rows[i].Cells["ClarifierDirectionColumn"].Value}");
-                            memoryChannel.Frequency = int.Parse($"{rows[i].Cells["FrequencyColumn"].Value}");
+                            memoryChannel.ClarifierOffset =
-                            memoryChannel.ClarifierDirection = char.Parse($"{rows[i].Cells["ClarifierDirectionColumn"].Value}");
+                                int.Parse($"{rows[i].Cells["ClarifierOffsetColumn"].Value}");
+                            memoryChannel.Clar = Convert.ToBoolean(rows[i].Cells["ClarColumn"].Value);
-                            memoryChannel.ClarifierOffset = int.Parse($"{rows[i].Cells["ClarifierOffsetColumn"].Value}");
+                            memoryChannel.MemoryRadioMode = Radios.Generic.MemoryRadioMode.Create(Configuration.Radio,
-                            memoryChannel.Clar = Convert.ToBoolean(rows[i].Cells["ClarColumn"].Value);
+                                $"{rows[i].Cells["ModeColumn"].Value}");
-                            memoryChannel.MemoryRadioMode = Radios.Generic.MemoryRadioMode.Create(Configuration.Radio, $"{rows[i].Cells["ModeColumn"].Value}");
+                            memoryChannel.Ctcss = new Ctcss($"{rows[i].Cells["CtcssColumn"].Value}");
-                            memoryChannel.Ctcss = new Ctcss($"{rows[i].Cells["CtcssColumn"].Value}");
+                            memoryChannel.Phase = new Phase($"{rows[i].Cells["PhaseColumn"].Value}");
-                            memoryChannel.Phase = new Phase($"{rows[i].Cells["PhaseColumn"].Value}");
+                            memoryChannel.Tag = Convert.ToBoolean(rows[i].Cells["TagColumn"].Value);
-                            memoryChannel.Tag = Convert.ToBoolean(rows[i].Cells["TagColumn"].Value);
+                            memoryChannel.Text = $"{rows[i].Cells["TextColumn"].Value,-12}";
-                            memoryChannel.Text = $"{rows[i].Cells["TextColumn"].Value,-12}";
                             success = await _catAssemblies.CatSetAsync<MemoryChannel, bool>("MT",
-                            success = await _catAssemblies.CatSetAsync<MemoryChannel, bool>("MT", new object[] { memoryChannel }, _cancellationToken);
+                                new object[] { memoryChannel }, cancellationToken);
                             break;
                     }
-                    progress.Report(new DataGridViewRowProgressSuccess<bool>(rows[i], i, success));
                     progress.Report(new DataGridViewRowProgressSuccess<bool>(rows[i], i, success));
                 catch (UnexpectedRadioResponseException exception)
                 catch (UnexpectedRadioResponseException exception)
                     progress.Report(new DataGridViewRowProgressFailure(rows[i], i, exception));
             saveFileDialog1.ShowDialog();
+        }
         private async void saveFileDialog1_FileOk(object sender, CancelEventArgs e)
         {
-            if (e.Cancel)
-            {
-                return;
             if (e.Cancel) return;
             var fileName = saveFileDialog1.FileName;
-            var list = new List<MemoryChannelIndexed>();
-            foreach(var row in dataGridView1.Rows.OfType<DataGridViewRow>().OrderBy(row => row.Index))
+            var list = new List<MemoryChannelIndexed>();
             foreach (var row in dataGridView1.Rows.OfType<DataGridViewRow>().OrderBy(row => row.Index))
                 if (row.Tag is MemoryChannel memoryChannel)
+                {
                     var memoryChannelOrganizerBanks = new MemoryChannelIndexed(row.Index, memoryChannel);
-                    list.Add(memoryChannelOrganizerBanks);
                     list.Add(memoryChannelOrganizerBanks);
+                }
+            }
+        }
         private async void openFileDialog1_FileOk(object sender, CancelEventArgs e)
         {
-            if(e.Cancel)
-            {
-                return;
             if (e.Cancel) return;
             var fileName = openFileDialog1.FileName;
             MemoryChannelIndexed[] memoryBanks = null;
+            var deserializationResult =
-            var deserializationResult =
+                await Serialization.Deserialize<MemoryChannelIndexed[]>(fileName,
-                await Serialization.Deserialize<MemoryChannelIndexed[]>(fileName,
+                    "urn:hambook-memorychannelorganizerbank-schema", "MemoryChannelIndexed.xsd",
-                    "urn:hambook-memorychannelorganizerbank-schema", "MemoryChannelIndexed.xsd", CancellationToken.None);
+                    CancellationToken.None);
                     return;
+            }
             toolStripProgressBar1.Minimum = 0;
+            toolStripProgressBar1.Maximum = 98;
-            toolStripProgressBar1.Maximum = 98;
+            toolStripProgressBar1.Value = toolStripProgressBar1.Minimum;
             var memoryBankQueue = new ConcurrentQueue<MemoryChannelIndexed>();
                         dataGridView1.Sort(dataGridView1.Columns["LocationColumn"], ListSortDirection.Ascending);
                         toolStripStatusLabel1.Text = "Done.";
                         return;
                     }
-                    switch(Configuration.Radio)
+                    switch (Configuration.Radio)
+                    {
                         case "Yaesu FT-891":
                             var memoryChannel = (Radios.Yaesu.FT_891.MemoryChannel)memoryBank.MemoryChannel;
+                            dataGridView1.Rows[memoryBank.Index].Cells["FrequencyColumn"].Value =
+                                memoryBank.MemoryChannel.Frequency;
-                            dataGridView1.Rows[memoryBank.Index].Cells["FrequencyColumn"].Value = memoryBank.MemoryChannel.Frequency;
+                            dataGridView1.Rows[memoryBank.Index].Cells["ClarifierDirectionColumn"].Value =
+                                (char)memoryChannel.ClarifierDirection;
-                            dataGridView1.Rows[memoryBank.Index].Cells["ClarifierDirectionColumn"].Value = (char)memoryChannel.ClarifierDirection;
+                            dataGridView1.Rows[memoryBank.Index].Cells["ClarifierOffsetColumn"].Value =
-                            dataGridView1.Rows[memoryBank.Index].Cells["ClarifierOffsetColumn"].Value = memoryChannel.ClarifierOffset;
+                                memoryChannel.ClarifierOffset;
+                            dataGridView1.Rows[memoryBank.Index].Cells["ClarColumn"].Value = memoryChannel.Clar;
-                            dataGridView1.Rows[memoryBank.Index].Cells["ClarColumn"].Value = memoryChannel.Clar;
+                            dataGridView1.Rows[memoryBank.Index].Cells["ModeColumn"].Value =
+                                memoryBank.MemoryChannel.MemoryRadioMode.Name;
-                            dataGridView1.Rows[memoryBank.Index].Cells["ModeColumn"].Value = memoryBank.MemoryChannel.MemoryRadioMode.Name;
+                            dataGridView1.Rows[memoryBank.Index].Cells["CtcssColumn"].Value =
+                                (string)memoryChannel.Ctcss;
-                            dataGridView1.Rows[memoryBank.Index].Cells["CtcssColumn"].Value = (string)memoryChannel.Ctcss;
+                            dataGridView1.Rows[memoryBank.Index].Cells["PhaseColumn"].Value =
+                                (string)memoryChannel.Phase;
-                            dataGridView1.Rows[memoryBank.Index].Cells["PhaseColumn"].Value = (string)memoryChannel.Phase;
+                            dataGridView1.Rows[memoryBank.Index].Cells["TagColumn"].Value =
+                                memoryBank.MemoryChannel.Tag;
-                            dataGridView1.Rows[memoryBank.Index].Cells["TagColumn"].Value = memoryBank.MemoryChannel.Tag;
+                            dataGridView1.Rows[memoryBank.Index].Cells["TextColumn"].Value =
-                            dataGridView1.Rows[memoryBank.Index].Cells["TextColumn"].Value = memoryBank.MemoryChannel.Text;
+                                memoryBank.MemoryChannel.Text;
                             dataGridView1.Rows[memoryBank.Index].Tag = memoryBank.MemoryChannel;
                             break;
+            }
             Application.Idle += IdleHandler;
             try
-            {
-                foreach (var memoryChannel in memoryBanks)
-                {
+            {
-                    memoryBankQueue.Enqueue(memoryChannel);
                 foreach (var memoryChannel in memoryBanks) memoryBankQueue.Enqueue(memoryChannel);
                 snapshotsQueuedTaskCompletionSource.TrySetResult(new { });
+            }
+            }
+        }
         private void MemoryOrganizerForm_FormClosing(object sender, FormClosingEventArgs e)
-        {
-            if(_cancellationTokenSource != null)
-            {
+        {
-                _cancellationTokenSource.Cancel();
             if (_cancellationTokenSource != null) _cancellationTokenSource.Cancel();
+        }
+        private async void readToolStripMenuItem_Click(object sender, EventArgs e)
+        {
-        private async void readToolStripMenuItem_Click(object sender, EventArgs e)
+            if (_readLinkedCancellationTokenSource != null)
+            {
+                _readLinkedCancellationTokenSource.Cancel();
-        {
+                _readLinkedCancellationTokenSource = null;
             var rows = GetSelectedDataGridViewRows(dataGridView1);
+            if (_readMemoryBanksTask != null)
             if(!rows.Any())
+                await _readMemoryBanksTask;
+                _readMemoryBanksTask = null;
+            }
+            _readCancellationTokenSource = new CancellationTokenSource();
+            _readLinkedCancellationTokenSource =
+                CancellationTokenSource.CreateLinkedTokenSource(new[]
+                    { _cancellationTokenSource.Token, _cancellationToken });
+            _readMemoryBanksTask = ReadMemoryBanks(_readLinkedCancellationTokenSource.Token);
+        }
+        private async Task ReadMemoryBanks(CancellationToken cancellationToken)
+        {
             var rows = GetSelectedDataGridViewRows(dataGridView1);
                 return;
             if (!rows.Any()) return;
+            var list = rows.ToList();
             var list = rows.ToList();
             var count = list.Count;
             var count = list.Count;
             toolStripProgressBar1.Minimum = 0;
-            toolStripProgressBar1.Minimum = 0;
+            toolStripProgressBar1.Maximum = count;
-            toolStripProgressBar1.Maximum = count;
+            toolStripProgressBar1.Value = toolStripProgressBar1.Minimum;
             var progress = new Progress<DataGridViewRowProgress>(rowProgress =>
+            {
                 try
                 try
                     switch (rowProgress)
                     switch (rowProgress)
-                    {
+                        case DataGridViewRowProgressSuccess<MemoryChannel> rowProgressSuccess:
+                            dataGridView1.Rows[rowProgressSuccess.Row.Index].DefaultCellStyle.BackColor =
-                        case DataGridViewRowProgressSuccess<MemoryChannel> rowProgressSuccess:
+                                DefaultBackColor;
-                            dataGridView1.Rows[rowProgressSuccess.Row.Index].DefaultCellStyle.BackColor = DataGridView.DefaultBackColor;
+                            switch (rowProgressSuccess.Data)
+                            {
-                            switch (rowProgressSuccess.Data)
+                                case Radios.Yaesu.FT_891.MemoryChannel memoryChannel:
                                     rowProgress.Row.Cells["FrequencyColumn"].Value = memoryChannel.Frequency;
-                                case Radios.Yaesu.FT_891.MemoryChannel memoryChannel:
+                                    rowProgress.Row.Cells["ClarifierDirectionColumn"].Value =
                                     break;
+                            }
                             toolStripStatusLabel1.Text = $"{Resources.Read_memory_bank} {rowProgress.Index + 1}";
                             break;
-                        case DataGridViewRowProgressFailure rowProgressFailure:
+                        case DataGridViewRowProgressFailure<int> rowProgressFailure:
                             Log.Error(rowProgressFailure.Exception, $"{Resources.Could_not_read_memory_bank}");
+                            dataGridView1.Rows[rowProgressFailure.Row.Index].DefaultCellStyle.BackColor = Color.Red;
-                            dataGridView1.Rows[rowProgressFailure.Row.Index].DefaultCellStyle.BackColor = Color.Red;
                             toolStripStatusLabel1.Text =
-                            toolStripStatusLabel1.Text = $"{Resources.Could_not_read_memory_bank} {rowProgress.Index + 1}";
+                                $"{Resources.Could_not_read_memory_bank} {rowProgress.Index + 1}";
                             break;
+                    }
+                {
                     Log.Error(exception, Resources.Unexpected_error_while_reading_memory_bank);
+                }
             });
-            await Task.Run(() => ReadMemoryBanks(list, progress, _cancellationToken), _cancellationToken);
+            await Task.Run(() => ReadMemoryBanks(list, progress, cancellationToken), cancellationToken);
             if (!_cancellationToken.IsCancellationRequested)
+            {
                 toolStripProgressBar1.Value = toolStripProgressBar1.Maximum;
                 toolStripStatusLabel1.Text = "Done.";
+            }
+        }
+        private async void writeToolStripMenuItem_Click(object sender, EventArgs e)
+        {
+            if (_writeLinkedCancellationTokenSource != null)
+            {
-        private async void writeToolStripMenuItem_Click(object sender, EventArgs e)
+                _writeLinkedCancellationTokenSource.Cancel();
                 _writeLinkedCancellationTokenSource = null;
-            var rows = GetSelectedDataGridViewRows(dataGridView1);
+            }
             if (_writeMemoryBanksTask != null)
+            {
+                await _writeMemoryBanksTask;
+                _writeMemoryBanksTask = null;
+            }
+            _writeCancellationTokenSource = new CancellationTokenSource();
+            _writeLinkedCancellationTokenSource =
+                CancellationTokenSource.CreateLinkedTokenSource(new[]
+                    { _cancellationTokenSource.Token, _cancellationToken });
+            _writeMemoryBanksTask = WriteMemoryBanks(_writeLinkedCancellationTokenSource.Token);
+        }
+        private async Task WriteMemoryBanks(CancellationToken cancellationToken)
             if (!rows.Any())
             var rows = GetSelectedDataGridViewRows(dataGridView1);
                 return;
             if (!rows.Any()) return;
             var list = rows.ToList();
             var list = rows.ToList();
             var count = list.Count;
             var count = list.Count;
             toolStripProgressBar1.Minimum = 0;
-            toolStripProgressBar1.Minimum = 0;
+            toolStripProgressBar1.Maximum = count;
-            toolStripProgressBar1.Maximum = count;
+            toolStripProgressBar1.Value = toolStripProgressBar1.Minimum;
+            var progress = new Progress<DataGridViewRowProgress>(rowProgress =>
             var progress = new Progress<DataGridViewRowProgress>(rowProgress =>
                 try
                 try
-                {
                     switch (rowProgress)
                     {
                         case DataGridViewRowProgressSuccess<bool> rowProgressSuccess:
-                            dataGridView1.Rows[rowProgressSuccess.Row.Index].DefaultCellStyle.BackColor = DataGridView.DefaultBackColor;
+                            dataGridView1.Rows[rowProgressSuccess.Row.Index].DefaultCellStyle.BackColor =
-                            var success = rowProgressSuccess.Data;
+                                DefaultBackColor;
                             var success = rowProgressSuccess.Data;
                             if (success)
                             if (success)
+                            {
                                 toolStripStatusLabel1.Text =
-                                $"{Resources.Wrote_memory_bank} {rowProgress.Index + 1}";
+                                    $"{Resources.Wrote_memory_bank} {rowProgress.Index + 1}";
                                 toolStripProgressBar1.Increment(1);
                                 statusStrip1.Update();
+                }
                 catch (Exception exception)
+                {
                     Log.Error(exception, Resources.Unexpected_error_while_writing_memory_bank);
+                }
             });
-            await Task.Run(() => WriteMemoryBanks(list, progress, _cancellationToken), _cancellationToken);
+            await Task.Run(() => WriteMemoryBanks(list, progress, cancellationToken), cancellationToken);
             if (!_cancellationToken.IsCancellationRequested)
+            {
         private async void readFromVFOAToolStripMenuItem_Click(object sender, EventArgs e)
+        {
             var rows = GetSelectedDataGridViewRows(dataGridView1);
-            if (!rows.Any())
-            {
                 return;
             if (!rows.Any()) return;
             var list = rows.ToList();
             try
             {
                 foreach (var row in list)
-                {
                     var fa = await _catAssemblies.CatReadAsync<int>("FA", new object[] { }, _cancellationToken);
                     var fa = await _catAssemblies.CatReadAsync<int>("FA", new object[] { }, _cancellationToken);
                     row.Cells["FrequencyColumn"].Value = $"{ fa }";
                     row.Cells["FrequencyColumn"].Value = $"{fa}";
+                }
         private async void readFromVFOBToolStripMenuItem_Click(object sender, EventArgs e)
+        {
             var rows = GetSelectedDataGridViewRows(dataGridView1);
-            if (!rows.Any())
-            {
                 return;
             if (!rows.Any()) return;
+                {
                     var fb = await _catAssemblies.CatReadAsync<int>("FB", new object[] { }, _cancellationToken);
                     row.Cells["FrequencyColumn"].Value = $"{fb}";
-                }
+                }
+            }
             catch (Exception exception)
+            {
                 Log.Error(exception, Resources.Failed_to_read_VFO_B);
         private async void memoryChannelToolStripMenuItem_Click(object sender, EventArgs e)
+        {
             var rows = GetSelectedDataGridViewRows(dataGridView1);
-            if (!rows.Any())
-            {
                 return;
             if (!rows.Any()) return;
             var row = rows.First();
             try
-            try
+            {
+                using (var soundPlayer = new SoundPlayer(Assembly.GetExecutingAssembly()
                            .GetManifestResourceStream("HamBook.Effects.pot.wav")))
                 using (var soundPlayer = new SoundPlayer(Assembly.GetExecutingAssembly().GetManifestResourceStream("HamBook.Effects.pot.wav")))
                     await _catAssemblies.CatWriteAsync<string>("MC", new[] { row.Cells["LocationColumn"].Value },
-                    await _catAssemblies.CatWriteAsync<string>("MC", new object[] { row.Cells["LocationColumn"].Value }, _cancellationToken);
+                        _cancellationToken);
         private async void frequencyToolStripMenuItem_Click(object sender, EventArgs e)
+        {
             var rows = GetSelectedDataGridViewRows(dataGridView1);
-            if (!rows.Any())
-            {
                 return;
             if (!rows.Any()) return;
             var row = rows.First();
             try
-            try
+            {
-            {
+                using (var soundPlayer = new SoundPlayer(Assembly.GetExecutingAssembly()
-                using (var soundPlayer = new SoundPlayer(Assembly.GetExecutingAssembly().GetManifestResourceStream("HamBook.Effects.pot.wav")))
+                           .GetManifestResourceStream("HamBook.Effects.pot.wav")))
-                {
                     if (await _catAssemblies.CatSetAsync<int, bool>("FA", new object[] { row.Cells["FrequencyColumn"].Value }, _cancellationToken))
-                    {
-                        if (Configuration.Navigation.MouseScrollSound)
                     if (await _catAssemblies.CatSetAsync<int, bool>("FA", new[] { row.Cells["FrequencyColumn"].Value },
-                            soundPlayer.Play();
+                            _cancellationToken))
                         if (Configuration.Navigation.MouseScrollSound)
                             soundPlayer.Play();
+                }
             var hitTest = dataGridView.HitTest(e.X, e.Y);
             switch (e.Button)
+            {
                 case MouseButtons.Right:
-                    if(GetSelectedDataGridViewRows(dataGridView).Count() > 1)
-                    {
-                        return;
                     if (GetSelectedDataGridViewRows(dataGridView).Count() > 1) return;
                     dataGridView.ClearSelection();
-                    dataGridView.Rows[hitTest.RowIndex].Selected = true;
+                    if (hitTest.RowIndex != -1) dataGridView.Rows[hitTest.RowIndex].Selected = true;
                     break;
+            }
+            {
                 var clone = row.Clone() as DataGridViewRow;
                 for (var i = 0; i < row.Cells.Count; ++i)
                 {
-                    if (row.Cells[i].ReadOnly)
-                    {
-                        continue;
                     if (row.Cells[i].ReadOnly) continue;
                     clone.Cells[i].Value = row.Cells[i].Value;
+                }
+                yield return clone;
                 yield return clone;
+        }
-        private void DeleteSelectedRows(DataGridView dataGridView)
         private void DeleteSelectedRows(DataGridView dataGridView)
             foreach (var row in GetSelectedDataGridViewRows(dataGridView))
-            {
+            foreach (var row in GetSelectedDataGridViewRows(dataGridView))
-                for (var i = 0; i < row.Cells.Count; ++i)
-                {
-                    if (row.Cells[i].ReadOnly)
                 for (var i = 0; i < row.Cells.Count; ++i)
                         continue;
-                    }
                     if (row.Cells[i].ReadOnly) continue;
                     row.Cells[i].Value = default;
                     row.Cells[i].Value = default;
+                }
         }
-        private void PasteSelectedRows(DataGridView dataGridView, IEnumerable<DataGridViewRow> pasteRows)
             var rows = pasteRows.ToList();
-            if(rows.Count == 0)
+        private void PasteSelectedRows(DataGridView dataGridView, IEnumerable<DataGridViewRow> pasteRows)
-            {
-                return;
-            }
+        {
-            foreach (var row in GetSelectedDataGridViewRows(dataGridView))
+            var rows = pasteRows.ToList();
             if (rows.Count == 0) return;
-                if (rows.Count == 0)
-                {
-                    return;
-                }
             foreach (var row in GetSelectedDataGridViewRows(dataGridView))
                 var paste = rows[0];
-                for (var i = 0; i < paste.Cells.Count; ++i)
+                if (rows.Count == 0) return;
+            {
                 var clone = row.Clone() as DataGridViewRow;
                 for (var i = 0; i < row.Cells.Count; ++i)
                 {
-                    if (row.Cells[i].ReadOnly)
-                    {
-                        continue;
                     if (row.Cells[i].ReadOnly) continue;
                     clone.Cells[i].Value = row.Cells[i].Value;
                     row.Cells[i].Value = default;
+                }
                 yield return clone;
+            }
+        }
-        private void cutToolStripMenuItem1_Click(object sender, EventArgs e)
-        {
-            _clipboardRows = new List<DataGridViewRow>();
-            foreach (var row in CutSelectedRows(dataGridView1))
+        private void cutToolStripMenuItem1_Click(object sender, EventArgs e)
+        {
-                _clipboardRows.Add(row);
+            _clipboardRows = new List<DataGridViewRow>();
             foreach (var row in CutSelectedRows(dataGridView1)) _clipboardRows.Add(row);
         }
-        private void copyToolStripMenuItem1_Click(object sender, EventArgs e)
-        {
             _clipboardRows = new List<DataGridViewRow>();
-            foreach (var row in CopySelectedRows(dataGridView1))
+        private void copyToolStripMenuItem1_Click(object sender, EventArgs e)
+        {
-                _clipboardRows.Add(row);
+            _clipboardRows = new List<DataGridViewRow>();
             DeleteSelectedRows(dataGridView1);
+        }
         private void dataGridView1_RowPrePaint(object sender, DataGridViewRowPrePaintEventArgs e)
         {
-            DataGridView dataGridView = sender as DataGridView;
+            var dataGridView = sender as DataGridView;
             foreach (DataGridViewCell cell in dataGridView.Rows[e.RowIndex].Cells)
             {
-                if (cell.Selected == false) { continue; }
+                if (cell.Selected == false) continue;
                 var bgColorCell = Color.White;
+                if (cell.Style.BackColor != Color.Empty)
-                if (cell.Style.BackColor != Color.Empty) { bgColorCell = cell.Style.BackColor; }
+                    bgColorCell = cell.Style.BackColor;
-                else if (cell.InheritedStyle.BackColor != Color.Empty) { bgColorCell = cell.InheritedStyle.BackColor; }
+                else if (cell.InheritedStyle.BackColor != Color.Empty) bgColorCell = cell.InheritedStyle.BackColor;
                 cell.Style.SelectionBackColor = MixColor(bgColorCell, Color.FromArgb(0, 150, 255), 10, 4);
+            }
+        }
         //Mix two colors
         //Example: Steps=10 & Position=4 makes Color2 mix 40% into Color1
         /// <summary>
-        /// Mix two colors.
+        ///     Mix two colors.
         /// </summary>
-        /// <param name="Color1"></param>
+        /// <param name="color1"></param>
-        /// <param name="Color2"></param>
+        /// <param name="color2"></param>
-        /// <param name="Steps"></param>
+        /// <param name="steps"></param>
-        /// <param name="Position"></param>
+        /// <param name="position"></param>
         /// <example>Steps=10 & Positon=4 makes Color2 mix 40% into Color1</example>
         /// <remarks>https://stackoverflow.com/questions/38337849/transparent-selectionbackcolor-for-datagridview-cell</remarks>
         /// <returns></returns>
-        public static Color MixColor(Color Color1, Color Color2, int Steps, int Position)
+        public static Color MixColor(Color color1, Color color2, int steps, int position)
         {
-            if (Position <= 0 || Steps <= 1) { return Color1; }
+            if (position <= 0 || steps <= 1) return color1;
-            if (Position >= Steps) { return Color2; }
+            if (position >= steps) return color2;
             return Color.FromArgb(
-                Color1.R + ((Color2.R - Color1.R) / Steps * Position),
+                color1.R + (color2.R - color1.R) / steps * position,
-                Color1.G + ((Color2.G - Color1.G) / Steps * Position),
+                color1.G + (color2.G - color1.G) / steps * position,
-                Color1.B + ((Color2.B - Color1.B) / Steps * Position)
+                color1.B + (color2.B - color1.B) / steps * position
-                );
+            );
         }
+    }