Итерация по массиву с помощью Array.FindAll()

Question

Я пытаюсь вернуть подмножество записей из массива на основе даты.

Мой код выглядит следующим образом:

var SomeAccounts = Array.FindAll(Accounts, x => x.modification_date > modification_date); 

if (SomeAccounts == null) 
{ 
    return NotFound(); 
} 

return Ok(JsonConvert.SerializeObject(SomeAccounts)); 

Учетные записи массив имеет modification_date поле, но то, что приходит обратно все счета. В моем тестовом образце, 2 имеют ультрасовременные даты> 15 мая 2016 г. Этот вид кода работает в других областях, но не на дату, которая просто:

public DateTime modification_date { get; set; } 

Что я делаю не так?

Answer 1

Обеспечение того, чтобы массива фактически пустой

Во-первых, метод Array.FindAll() никогда не должен фактически вернуть null как он возвращает пустой массив, если совпадений не найдено, так что вы, вероятно, хотите, чтобы проверить, если какие-либо значения были найдены:

if (SomeAccounts.Length == 0) 
{ 
    return NotFound(); 
} 

Если результаты возвращаются массив (без значений), то это было бы правильно (как бы не ударить NotFound() ранее, как он никогда не был нулевым).

экспертизы Даты

Как Cory упоминалось в комментарии, сравнивающие объекты DateTime может быть крайне сложно. Вы захотите убедиться, что любая информация, такая как TimeZones, учитывается при сравнении значений.

Вы можете также рассмотреть возможность использования отладчика вручную, чтобы посмотреть, как выглядят ваши объекты DateTime (как свойство modification_date, так и ваша действительная переменная modification_date). Попробуйте сравнить их в окне «Часы», чтобы убедиться, что сравнения работают правильно.