das ist alles gar nicht so einfach

This commit is contained in:
Daniel Schick 2023-03-22 10:54:52 +01:00
parent ce5327821b
commit 98b2b0adba
2 changed files with 22 additions and 13 deletions

View File

@ -375,17 +375,18 @@ namespace ENI2.Controls
List<MaerskData> importData = new List<MaerskData>();
bool isFirstRow = true;
int currentRow = 1;
int currentRow = 0;
bool imosAreOkay = true;
try
{
while (reader.Read())
{
currentRow++;
if (isFirstRow)
{
isFirstRow = false; // this must be a header row, skip
isFirstRow = false; // this must be a header row, skip
continue;
}
@ -435,22 +436,30 @@ namespace ENI2.Controls
{
if (Int32.TryParse(md.ColF, out int imo))
{
if ((imo < 1000000) && (imo > 9999999))
if ((imo < 1000000) || (imo > 9999999))
{
imosAreOkay = false;
}
}
if(!imosAreOkay)
}
else
{
MessageBox.Show($"Invalid IMO in row {currentRow}, aborting import", Properties.Resources.textCaptionError, MessageBoxButton.OK, MessageBoxImage.Error);
break;
}
}
imosAreOkay = false;
}
}
else
{
imosAreOkay = false;
}
if (!imosAreOkay)
{
MessageBox.Show($"Invalid IMO in row {currentRow}, aborting import", Properties.Resources.textCaptionError, MessageBoxButton.OK, MessageBoxImage.Error);
break;
}
importData.Add(md);
if (isFirstRow) isFirstRow = false;
currentRow++;
if (isFirstRow) isFirstRow = false;
}
}
catch (Exception ex)

View File

@ -36,7 +36,7 @@
<MinimumRequiredVersion>5.4.0.0</MinimumRequiredVersion>
<CreateWebPageOnPublish>true</CreateWebPageOnPublish>
<WebPage>publish.html</WebPage>
<ApplicationRevision>5</ApplicationRevision>
<ApplicationRevision>8</ApplicationRevision>
<ApplicationVersion>7.10.0.%2a</ApplicationVersion>
<UseApplicationTrust>false</UseApplicationTrust>
<CreateDesktopShortcut>true</CreateDesktopShortcut>