Мне кажется или с этим C++ кодом что-то не так?

А ты имя автора видел? Pavan Kumar Sunkara - типичное индусское имя, а ты еще про код спрашиваешь

Автор типичный индус, удивляться нечему. Ну есть прямо в сердце закололо от такого, отправь ему патч.

Это ты очень тонко намекнул, что winapi не торт :)

С любым кодом на С++ что-то всегда не так.

union не может в не POD типы.

Штоу?

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

union тут никаким боком. Ибо он не гарантирует обращение к реальному типу. Скорее std::variant, которого ещё нету.

Но в целом код так себе. Особенно доставляют бессмысленные комментарии. Документация ради документации.

С этим кодом всё не так, но union плохая альтернатива для этого. Union - это привет из Си со всеми соответствующими граблями. В C++ для этого придумали (уже стандартные) variant и any.

... которые являются всего лишь обертками над union

может тут так принято и это нормально?

Ну в общем да. Как ни крутись, а Variant на крестах это всегда либо такой ручной выбор типа, либо тормозной dynamic_cast.

А кто это сказал, что dynamic_cast тормозной? Что касается выбора типа, то, если типов много, в С++, как правильно заметил OP, используется ООП.

ну по сравнению с switсh он таки тормозной. Другое дело, что если у тебя он прям критичен, то вообще все по другому нужно делать.

нет

Щито?

там указатель, скорее всего, безо всяких юнинонов

наслеование какое-нибудь

This. Кому-то было очень лениво описывать 6 классов вместо одного.

Вообще-то автор явно в курсе про Sum types, возможно хаскеллист. Зря вы так