Move FillPosition logic to separate method

This commit is contained in:
Sergey Chebotar 2022-01-28 15:21:58 +03:00
parent b4763bd03c
commit ca575bef2d

View File

@ -37,12 +37,27 @@ namespace RehauSku.PriceListTools
object[,] groupColumn = TargetFile.groupCell.EntireColumn.Value2; object[,] groupColumn = TargetFile.groupCell.EntireColumn.Value2;
foreach (var kvp in dictionary) foreach (var kvp in dictionary)
{
FillPosition(kvp, columns, ref missing, ref groupColumn);
}
if (missing.Count > 0)
{
System.Windows.Forms.MessageBox.Show
($"{missing.Count} артикулов отсутствует в таблице заказов {RegistryUtil.PriceListPath} Попробовать найти новый вариант?",
"Отсутствует позиция в конечной таблице заказов",
System.Windows.Forms.MessageBoxButtons.YesNo,
System.Windows.Forms.MessageBoxIcon.Information);
}
}
protected private void FillPosition(KeyValuePair<Position, double> kvp, int[] columns, ref List<KeyValuePair<Position, double>> missing, ref object[,] groupColumn)
{ {
Range foundCell = TargetFile.skuCell.EntireColumn.Find(kvp.Key.Sku); Range foundCell = TargetFile.skuCell.EntireColumn.Find(kvp.Key.Sku);
if (foundCell == null) if (foundCell == null)
{ {
missing.Add(kvp); missing.Add(kvp);
continue; return;
} }
string foundCellGroup = groupColumn[foundCell.Row, 1].ToString(); string foundCellGroup = groupColumn[foundCell.Row, 1].ToString();
@ -76,16 +91,6 @@ namespace RehauSku.PriceListTools
} }
} }
if (missing.Count > 0)
{
System.Windows.Forms.MessageBox.Show
($"{missing.Count} артикулов отсутствует в таблице заказов {RegistryUtil.PriceListPath} Попробовать найти новый вариант?",
"Отсутствует позиция в конечной таблице заказов",
System.Windows.Forms.MessageBoxButtons.YesNo,
System.Windows.Forms.MessageBoxIcon.Information);
}
}
protected private void FilterByAmount() protected private void FilterByAmount()
{ {
AutoFilter filter = TargetFile.Sheet.AutoFilter; AutoFilter filter = TargetFile.Sheet.AutoFilter;