Краткий ответ

Код некорректен, потому что параметр замыкания, переданного в map, неизменяем: фактически user — это локальная константа. Нужно создать изменяемую локальную копию элемента, изменить её и вернуть из замыкания.

let uppercased = users.map { user in
    var copy = user
    copy.name = copy.name.uppercased()
    return copy
}

Исторический контекст

Функциональные операции над коллекциями проектируются как преобразования: map не изменяет исходную коллекцию, а строит результат, применяя функцию к каждому элементу. Такой подход уменьшает количество скрытых изменений состояния и делает цепочки преобразований предсказуемее.

В Swift это дополнительно согласуется с моделью значимых типов: экземпляр структуры передаётся в замыкание как значение, а не как доступная для изменения ссылка на элемент исходного массива.

Постановка проблемы

Элемент массива типа User — структура, то есть value type. Вызов map передаёт его в замыкание по значению, а параметр замыкания нельзя изменять напрямую.

Если попытаться изменить user.name, компилятор сообщит об ошибке вроде Cannot assign to property: 'user' is a 'let' constant. Исходный массив при этом не должен изменяться: цель операции — получить отдельный преобразованный результат.

Подробное решение

Параметры замыканий в Swift неизменяемы по умолчанию. Поэтому строка user.name = ... запрещена независимо от того, является ли свойство name объявленным как var.

Исправление с локальной копией работает так: copy — изменяемая переменная типа User, изменение происходит только в ней, а return copy добавляет новую версию элемента в результирующий массив.

Альтернативный вариант — сразу создать новый экземпляр:

let uppercased = users.map {
    User(name: $0.name.uppercased())
}

Для структур оба варианта сохраняют исходный users неизменным. Если бы массив содержал экземпляры класса, параметр также был бы неизменяемой ссылкой, но изменение свойства объекта через эту ссылку обычно было бы разрешено и затронуло бы общий объект. Это важное различие между изменением значения структуры и изменением состояния объекта класса.

Ситуация из практики

Предположим, приложение формирует отображаемые модели из полученных данных. Вариант с изменением исходного массива через цикл даёт побочный эффект и усложняет повторное использование исходных данных. map с локальной копией явно показывает, что создаётся новая коллекция.

Можно создать новый экземпляр модели вручную: это максимально явно, но при добавлении полей придётся обновлять конструктор преобразования. Можно скопировать значение и изменить только нужное свойство: этот вариант лучше сохраняет остальные поля и обычно удобнее для структур с большим количеством свойств.

Практический выбор — локальная копия или специальный метод вроде uppercasedName(), если преобразование повторяется. В обоих случаях исходный массив остаётся доступным для других операций, а результат содержит независимые значения структур.

Что кандидаты часто упускают

Можно ли написать var user прямо в списке параметров замыкания?

Нет, параметры замыканий нельзя объявлять изменяемыми таким способом. Используют локальную копию внутри тела замыкания или создают новый экземпляр результата. Это не связано с тем, что свойство объявлено как var: изменяемость свойства не делает сам параметр изменяемым.
Изменится ли исходный объект, если элементы массива имеют тип класса?

Да, если замыкание изменяет свойство экземпляра класса. Массив копирует ссылки на объекты, а не сами объекты, поэтому map создаст новый массив ссылок, но объекты могут остаться общими. Для независимого результата потребуется явно скопировать объект, например через инициализатор копирования или отдельный метод клонирования.
Почему нельзя заменить map на изменение элементов исходного массива внутри map?

map предназначен для построения нового результата и не предоставляет замыканию изменяемый доступ к элементам исходной коллекции. Если требуется изменить массив на месте, используют подходящий мутационный API, например forEach с индексами или enumerated() и присваиванием по индексу, когда это безопасно для конкретной коллекции. Для обобщённого алгоритма такое изменение дополнительно требует возможности записи по индексам, которой обладает не любая Collection.