2

На сервер отправляются файлы на основе которых формируется отчет. Для создания отчета необходимо минимум 3 и максимум 5 файлов.

public IHttpActionResult PostFiles()
{
    HttpFileCollection httpFileCollection = HttpContext.Current.Request.Files;

    var streams = new List<Stream>();

    var countAddedStream = 0;            
    foreach (var formDataKey in httpFileCollection.AllKeys)
    {
        var httpPostedFile = httpFileCollection[formDataKey];
        if (httpPostedFile != null)                
            streams.Add(httpPostedFile.InputStream);                             
        if (++countAddedStream == 5)
            break;                                    
    }

    // Fail
    if (countAddedStream < 3)            
        return BadRequest("Не достаточно корректных файлов для создания отчета");
    // Success
    //...
    return Ok();
}

1) Интересует: правильно ли я делаю валидацию и извлекаю содержимое согласно моему условию и практике хорошего дизайна? К сожалению, что-то типа LINQ для HttpFileCollection я не смог обнаружить, чтобы сделать что-то вроде.Where(x => x[key] != null && x[key].InputStream != null).

2) Также интересует: действительно ли httpFileCollection[formDataKey] может вернуть null. Я делаю валидацию в этом месте if (httpPostedFile != null) т.к. ReSharper намекнул на это. А к примеру, когда я через Postman попытался добавить запись в form-data, указав только key, то такая запись просто не добавилась.

1 ответ 1

1

Частичный ответ:
httpFileCollection[] надо привести к ожидаемому типу с помощью .Cast<T>() или отфильтровать .OfType<T>(), например
httpFileCollection.Cast<HttpPostedFile>().Where(hpf => ...).

Именно .Where(x => x[key] != null && x[key].InputStream != null) не получиться, т.к. у вас не перечисление экземпляров httpFileCollection (у которых есть доступ по индексу и ключи), а внутри класса определены массивы ключей и файлов, см.

public sealed class HttpFileCollection : NameObjectCollectionBase {
        // cached All[] arrays
        private HttpPostedFile[] _all;
        private String[] _allKeys;
        ... 

т.е. будет вроде .Where(hpf => (hpf != null)&&(hpf.InputStream != null))

4
  • Т.е., если капнуть глубже, правильно ли я понимаю, что метод httpFileCollection.Cast<HttpPostedFile>()будет приводить каждый объект, который ему будет приходит из NameObjectCollectionBase.IEnumerator GetEnumerator()? 17 янв 2018 в 8:17
  • 1
    @adamshakhabov "Элементы объекта HttpFileCollection имеют тип HttpPostedFile", т.е., как я понимаю, приведение/фильтр будет выполняться над HttpPostedFile[] _all;; проще наверное фильтр .OfType<T>(), хотя бы формально.
    – Alias
    17 янв 2018 в 9:39
  • А вот эта проверка hpf.InputStream != null не лишняя? Мне кажется в коллекции такого элемента так и так просто не будет. Или все-таки можно отправить в запросе файлы с null значением? 18 янв 2018 в 6:58
  • @adamshakhabov тут я просто исходил от вашего .Where(x => ...), когда хотелось что-то вроде :) КМК, проверять или нет все объекты на не-NULL перед использованием определяется исключительно здравым смыслом.
    – Alias
    18 янв 2018 в 11:21

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими условиями использования и подтверждаете, что прочитали и поняли наши политику конфиденциальности и нормы поведения.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.