Маппинг значений

@Alex-D-Holliday · Регистрация: 05.02.2021

Author24 — интернет-сервис помощи студентам

Доброго времени суток.
Вот такая есть интересная задача, надеюсь зайдет.

Предположим, у нас есть некий реквест:

JSON

{
    "Singles": [
        12,
        13,
        14,
        15
    ],
    "Objects": [
        1,
        2,
        3,
        4
    ],
    "Integers": [
        9,
        8,
        7,
        6,
        5
    ],
    "Booleans": [
        true,
        false,
        true,
        false
    ]
}

так же есть модель:

C#

public class Props
    {
        public Single[] Singles { get; set; }
        public Single[] Objects { get; set; }
        public Int16[] Integers { get; set; }
        public Boolean[] Booleans { get; set; }
    }

и есть модель из БД (так же есть таблица с в БД):

C#

public class DbProps
    {
        public Single SS1 { get; set; }
        public Single OBJ1 { get; set; }
        public Int16 INT1 { get; set; }
        public Boolean BOOL1 { get; set; }
        
        public Single SS2 { get; set; }
        public Single OBJ2 { get; set; }
        public Int16 INT2 { get; set; }
        public Boolean BOOL2 { get; set; }
        
        public Single SS3 { get; set; }
        public Single OBJ3 { get; set; }
        public Int16 INT3 { get; set; }
        public Boolean BOOL3 { get; set; }
        
        public Single SS4 { get; set; }
        public Single OBJ4 { get; set; }
        public Int16 INT4 { get; set; }
        public Boolean BOOL4 { get; set; }
    }

при чем таких значений может быть до 10

А теперь самое интересное, как мапить значения из реквеста в соответствующие поля таблицы в БД (естественно в ручную прописывать все значения не ок)
То есть что бы SS1 = Singles[0], SS2 = Singles[1], OBJ1 = Objects[0] и так далее

@Dr9vik · 21.07.2022, 20:17

C#

public class DbProps
    {
        public Single SS1 { get; set; }
        public Single OBJ1 { get; set; }
        public Int16 INT1 { get; set; }
        public Boolean BOOL1 { get; set; }
        
        public Single SS2 { get; set; }
        public Single OBJ2 { get; set; }
        public Int16 INT2 { get; set; }
        public Boolean BOOL2 { get; set; }
        
        public Single SS3 { get; set; }
        public Single OBJ3 { get; set; }
        public Int16 INT3 { get; set; }
        public Boolean BOOL3 { get; set; }
        
        public Single SS4 { get; set; }
        public Single OBJ4 { get; set; }
        public Int16 INT4 { get; set; }
        public Boolean BOOL4 { get; set; }
    }

переделывайте таблицу

@Alex-D-Holliday · 21.07.2022, 22:16 **[ТС]**

не вариант

@Dr9vik · 22.07.2022, 00:59

C#

public class Props
    {
        public Single Single { get; set; }
        public Single Object { get; set; }
        public Int16 Integer { get; set; }
        public Boolean Boolean { get; set; }
    }

предположу возвращать List<Props> длиной в 4 Props, так проще свалидировать, а не багать по куче разным массивам
подрубить автомапер где ручками прописать что куда мапить на DbProps

@Usaga · 22.07.2022, 02:52

Alex-D-Holliday, почему single и int16, а не float и int? Это вы так память экономить на спичках пытаетесь?

Сообщение от Alex-D-Holliday

и есть модель из БД (так же есть таблица с в БД):

И это кривая модель. Правильный вариант тут либо массивы прямо в полях хранить (если база позволяет, то натурально, если нет - то в виде строки, через запятую), либо разносить наборы данных по разным записям.

C#

public class DbProps
    {
        public Single SS { get; set; }
        public Single OBJ { get; set; }
        public Int16 INT { get; set; }
        public Boolean BOOL { get; set; }
}

Последнее работает на любой СУБД и довольно широко применяется.

Сообщение от Alex-D-Holliday

не вариант

Так и скажите, что просто лениво)

Сообщение от Dr9vik

подрубить автомапер где ручками прописать что куда мапить на DbProps

Не рекомедуйте этот инструмент никому. Взять автомапер, чтобы потом РУЧКАМИ прописывать что-то. Не проще ли всё тоже самое прописать ручками БЕЗ автомапера?)

@Alex-D-Holliday · 22.07.2022, 10:28 **[ТС]**

Dr9vik

Сообщение от Dr9vik

предположу возвращать List<Props> длиной в 4 Props, так проще свалидировать, а не багать по куче разным массивам

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

Usaga, вы не о том думаете

Сообщение от Usaga

почему single и int16, а не float и int? Это вы так память экономить на спичках пытаетесь?

это обычный пример каких то там типов данных и акцент на том, что они могут быть разные и я не спрашивал, кривая это модель или нет, вопрос задачи состоит в другом

Сообщение от Usaga

Так и скажите, что просто лениво)

причина в другом

Сообщение от Usaga

Не рекомедуйте этот инструмент никому

к сожалению, он подрублен и боюсь, что придется еще что то доп мапить(

@Dr9vik · 22.07.2022, 11:22

Сообщение от Usaga

Не рекомедуйте этот инструмент никому.

в нем есть свои плюсы и минусы

Добавлено через 4 минуты

Сообщение от Alex-D-Holliday

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

ну если не массивы они будут типа общие
просто создайте тогда 2 класса где в первом будут поля не массивы и собственно 1 поле то что я сверху написал

C#

class A
{
    public string A1{get;set;}
    public string A2{get;set;}
    ...
    public List<Props> Items{get;set;}
}

а если выкинуть на мороз автомапер, как впрочем некоторые и делают...да ручками прописывать

@Usaga · 23.07.2022, 12:30

Сообщение от Dr9vik

в нем есть свои плюсы и минусы

И какие для себя вы плюсы видите в нём?

@Dr9vik · 23.07.2022, 13:47

Сообщение от Usaga

И какие для себя вы плюсы видите в нём?

2 одинаковые модели и между ними надо замапить
проще 1 строчку написать и больше ничего

@sau · 28.07.2022, 18:35

Сообщение от Usaga

Не рекомедуйте этот инструмент никому. Взять автомапер, чтобы потом РУЧКАМИ прописывать что-то. Не проще ли всё тоже самое прописать ручками БЕЗ автомапера?)

не проще , если модель нужно отражать туда сюда обратно в различных местах приложения , проще вместо этого прописать var dst = mapper.Map(src); Да и сам маппер может содержать логику отражения чуть сложнее нежели простое присваивание , ее тоже многократно повторять не придется.
Из минусов - это может быть достаточно медленно , особенно если логика маппинга сложная , нужно применять и конфигурировать аккуратно в чувствительных местах.

@Usaga · 29.07.2022, 11:25

Сообщение от Dr9vik

2 одинаковые модели и между ними надо замапить
проще 1 строчку написать и больше ничего

Это когда у вас внатуре две одинаковые модели. Именно для такого сценария Automapper и создавался. Но в реальных проектах модели далеко не одинаковые.

Сообщение от sau

не проще , если модель нужно отражать туда сюда обратно в различных местах приложения , проще вместо этого прописать var dst = mapper.Map(src); Да и сам маппер может содержать логику отражения чуть сложнее нежели простое присваивание , ее тоже многократно повторять не придется.

Так вручную написанный маппер тоже в одну строчку вызывается. И в разных местах его многократно можно вызывать. И логику сложную в него пихать, включая инжектирование сервисов. Прикол в том, что с Automapper писанины такой будет БОЛЬШЕ. Ведь нужно будет эту логику заворачивать в API самого Automapper'а.

Я не просто так это говорю. Уже проходили.

@Alex-D-Holliday · 29.07.2022, 18:08 **[ТС]**

Usaga, Dr9vik, sau,
граждане, вопрос был не об плюсах и минусах Automapper, вы ошиблись темой
не можете ничего конкретного предложить, проходите мимо

@Usaga · 29.07.2022, 20:19

Alex-D-Holliday, вроде бы на ваш вопрос ответили уже.

@Alex-D-Holliday 0 / 0 / 0 Регистрация: 05.02.2021 Сообщений: 11
	21.07.2022, 22:16 [ТС]	3
	не вариант 0

@Alex-D-Holliday 0 / 0 / 0 Регистрация: 05.02.2021 Сообщений: 11
	22.07.2022, 10:28 [ТС]	6
	Dr9vik Сообщение от Dr9vik предположу возвращать List<Props> длиной в 4 Props, так проще свалидировать, а не багать по куче разным массивам сорри, мой косяк... на самом в модели могут быть и другие поля(не обязательно массивы), но длина массивов этих основных будет всегда одинаковой Usaga, вы не о том думаете Сообщение от Usaga почему single и int16, а не float и int? Это вы так память экономить на спичках пытаетесь? это обычный пример каких то там типов данных и акцент на том, что они могут быть разные и я не спрашивал, кривая это модель или нет, вопрос задачи состоит в другом Сообщение от Usaga Так и скажите, что просто лениво) причина в другом Сообщение от Usaga Не рекомедуйте этот инструмент никому к сожалению, он подрублен и боюсь, что придется еще что то доп мапить( 0

@Usaga 12505 / 8692 / 1310 Регистрация: 21.01.2016 Сообщений: 32,661
	23.07.2022, 12:30	8
	Сообщение от Dr9vik в нем есть свои плюсы и минусы И какие для себя вы плюсы видите в нём? 0

@Dr9vik 403 / 265 / 69 Регистрация: 12.04.2020 Сообщений: 1,406
	23.07.2022, 13:47	9
	Сообщение от Usaga И какие для себя вы плюсы видите в нём? 2 одинаковые модели и между ними надо замапить проще 1 строчку написать и больше ничего 0

@sau 2756 / 2059 / 384 Регистрация: 22.07.2011 Сообщений: 7,781
	28.07.2022, 18:35	10
	Сообщение от Usaga Не рекомедуйте этот инструмент никому. Взять автомапер, чтобы потом РУЧКАМИ прописывать что-то. Не проще ли всё тоже самое прописать ручками БЕЗ автомапера?) не проще , если модель нужно отражать туда сюда обратно в различных местах приложения , проще вместо этого прописать var dst = mapper.Map(src); Да и сам маппер может содержать логику отражения чуть сложнее нежели простое присваивание , ее тоже многократно повторять не придется. Из минусов - это может быть достаточно медленно , особенно если логика маппинга сложная , нужно применять и конфигурировать аккуратно в чувствительных местах. 0

@Usaga 12505 / 8692 / 1310 Регистрация: 21.01.2016 Сообщений: 32,661
	29.07.2022, 11:25	11
	Сообщение от Dr9vik 2 одинаковые модели и между ними надо замапить проще 1 строчку написать и больше ничего Это когда у вас внатуре две одинаковые модели. Именно для такого сценария Automapper и создавался. Но в реальных проектах модели далеко не одинаковые. Сообщение от sau не проще , если модель нужно отражать туда сюда обратно в различных местах приложения , проще вместо этого прописать var dst = mapper.Map(src); Да и сам маппер может содержать логику отражения чуть сложнее нежели простое присваивание , ее тоже многократно повторять не придется. Так вручную написанный маппер тоже в одну строчку вызывается. И в разных местах его многократно можно вызывать. И логику сложную в него пихать, включая инжектирование сервисов. Прикол в том, что с Automapper писанины такой будет БОЛЬШЕ. Ведь нужно будет эту логику заворачивать в API самого Automapper'а. Я не просто так это говорю. Уже проходили. 0

@Alex-D-Holliday 0 / 0 / 0 Регистрация: 05.02.2021 Сообщений: 11
	29.07.2022, 18:08 [ТС]	12
	Usaga, Dr9vik, sau, граждане, вопрос был не об плюсах и минусах Automapper, вы ошиблись темой не можете ничего конкретного предложить, проходите мимо 0

@Usaga 12505 / 8692 / 1310 Регистрация: 21.01.2016 Сообщений: 32,661
	29.07.2022, 20:19	13
	Alex-D-Holliday, вроде бы на ваш вопрос ответили уже. 0