vk icon indicating copy to clipboard operation
vk copied to clipboard

Исправляет маппинг отсутствующих значений enum

Open Panuchi opened this issue 8 months ago • 4 comments

#1556

Panuchi avatar Apr 03 '25 00:04 Panuchi

Нужно проверить везде ли у StringEnum'ов тип nullable. Либо добавить во все StringEnum:

	/// <summary>
	/// Необработанное значение.
	/// </summary>
	Unknown = 0,

Возвращаться будет либо Unknown, либо null.

ghost avatar Apr 03 '25 00:04 ghost

Sao ạ

Vào 07:08, T.5, 3 Th4, 2025 Timofei Jääger @.***> đã viết:

Нужно проверить везде ли у StringEnum'ов тип nullable. Либо добавить во все StringEnum:

///

/// Необработанное значение. /// Unknown = 0,

Возвращаться будет либо Unknown, либо null.

— Reply to this email directly, view it on GitHub https://github.com/vknet/vk/pull/1634#issuecomment-2774017211, or unsubscribe https://github.com/notifications/unsubscribe-auth/ALWSO5MPW5JF3R6KMUOIEDT2XR3Y3AVCNFSM6AAAAAB2K2JY7OVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDONZUGAYTOMRRGE . You are receiving this because you are subscribed to this thread.Message ID: @.***> [image: Panuchi]Panuchi left a comment (vknet/vk#1634) https://github.com/vknet/vk/pull/1634#issuecomment-2774017211

Нужно проверить везде ли у StringEnum'ов тип nullable. Либо добавить во все StringEnum:

///

/// Необработанное значение. /// Unknown = 0,

Возвращаться будет либо Unknown, либо null.

— Reply to this email directly, view it on GitHub https://github.com/vknet/vk/pull/1634#issuecomment-2774017211, or unsubscribe https://github.com/notifications/unsubscribe-auth/ALWSO5MPW5JF3R6KMUOIEDT2XR3Y3AVCNFSM6AAAAAB2K2JY7OVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDONZUGAYTOMRRGE . You are receiving this because you are subscribed to this thread.Message ID: @.***>

nhan7878 avatar Apr 03 '25 00:04 nhan7878

Так получается это решает проблему тем, что просто выбрасывает значение полностью? На как костыль это, конечно, хорошее решение, но как минимум внутри enum-а где-то должно остаться оригинальное значение Это не годится для ПР

CaCTuCaTu4ECKuu avatar Apr 03 '25 02:04 CaCTuCaTu4ECKuu

Так получается это решает проблему тем, что просто выбрасывает значение полностью? На как костыль это, конечно, хорошее решение, но как минимум внутри enum-а где-то должно остаться оригинальное значение Это не годится для ПР

Здесь уже задача сделать "хоть что-то", чтобы не падал код при каждом изменении от ВК.

А дальше, кому интересно, можете думать как довести до идеала.

ghost avatar Apr 03 '25 09:04 ghost

https://github.com/vknet/vk/pull/1634#issue-2967823986

cptthura-alt avatar Sep 21 '25 08:09 cptthura-alt

Я в похожей ситуации для StringEnum-a добавил свойство Defined и для предопределенных значений оно было true, а при парсинге неизвестного false, но не падало

CaCTuCaTu4ECKuu avatar Sep 21 '25 14:09 CaCTuCaTu4ECKuu