SabreTools/SabreTools.Serialization
1
0
Fork 0
mirror of https://github.com/SabreTools/SabreTools.Serialization.git synced 2026-07-08 18:06:41 +00:00
Code Issues 56 Packages Projects Releases 20 Wiki Activity

Invert some if statements to be clearer

Browse Source
This commit is contained in:
Matt Nadareski
2026-04-05 00:35:09 -04:00
parent 285f3a267e
commit 0ed13167ba
1 changed files with 29 additions and 36 deletions
Download Patch File Download Diff File Expand all files Collapse all files

65
SabreTools.Metadata.Filter/FilterKey.cs
View File

@@ -103,21 +103,19 @@ namespace SabreTools.Metadata.Filter
{
// Get the set of properties
var properties = TypeHelper.GetProperties(typeof(Header));
if (properties is not null)
{
// Get if there's a match to a property
string localFieldName = fieldName;
string? propertyMatch = Array.Find(properties, c => string.Equals(c, localFieldName, StringComparison.OrdinalIgnoreCase));
if (propertyMatch is not null)
{
// Return the sanitized ID
itemName = "header";
fieldName = propertyMatch.ToLowerInvariant();
return true;
}
}
if (properties is null)
return false;
return false;
// Get if there's a match to a property
string localFieldName = fieldName;
string? propertyMatch = Array.Find(properties, c => string.Equals(c, localFieldName, StringComparison.OrdinalIgnoreCase));
if (propertyMatch is null)
return false;
// Return the sanitized ID
itemName = "header";
fieldName = propertyMatch.ToLowerInvariant();
return true;
}
/// <summary>
@@ -127,21 +125,19 @@ namespace SabreTools.Metadata.Filter
{
// Get the set of properties
var properties = TypeHelper.GetProperties(typeof(Machine));
if (properties is not null)
{
// Get if there's a match to a property
string localFieldName = fieldName;
string? propertyMatch = Array.Find(properties, c => string.Equals(c, localFieldName, StringComparison.OrdinalIgnoreCase));
if (propertyMatch is not null)
{
// Return the sanitized ID
itemName = "machine";
fieldName = propertyMatch.ToLowerInvariant();
return true;
}
}
if (properties is null)
return false;
return false;
// Get if there's a match to a property
string localFieldName = fieldName;
string? propertyMatch = Array.Find(properties, c => string.Equals(c, localFieldName, StringComparison.OrdinalIgnoreCase));
if (propertyMatch is null)
return false;
// Return the sanitized ID
itemName = "machine";
fieldName = propertyMatch.ToLowerInvariant();
return true;
}
/// <summary>
@@ -205,15 +201,12 @@ namespace SabreTools.Metadata.Filter
// Get the set of properties
var properties = TypeHelper.GetProperties(itemType);
if (properties is not null)
{
// Get if there's a match to a property
string? propertyMatch = Array.Find(properties, c => string.Equals(c, fieldName, StringComparison.OrdinalIgnoreCase));
if (propertyMatch is not null)
return propertyMatch.ToLowerInvariant();
}
if (properties is null)
return null;
return null;
// Get if there's a match to a property
string? propertyMatch = Array.Find(properties, c => string.Equals(c, fieldName, StringComparison.OrdinalIgnoreCase));
return propertyMatch?.ToLowerInvariant();
}
}
}