По колено в коде — Говнокод.ру

C++ / Говнокод #27309

+3

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60

template<typename T>
struct [[nodiscard]] InplacePointerGuard {
    InplacePointerGuard(T *& ptr_) : ptr(ptr_) {}
    ~InplacePointerGuard()
    {
        if (ptr != nullptr) {
            ptr->~T();
        }
    }

    T *& ptr;
};

template<typename EventType, typename... Args>
bool publishEmplace(Args &&... args) const
{
    static_assert((std::is_base_of_v<Event, EventType> && std::is_convertible_v<const EventType *, const Event *>),
                  "EventType must be a public subtype of Event");
    auto typeIdx = Event::getStaticIndex<EventType>();

    std::aligned_storage_t<sizeof(EventType), alignof(EventType)> eventStorage;
    EventType *eventConstructed = nullptr;
    auto guard = InplacePointerGuard(eventConstructed);

    publishEmplaceImpl<EventType>(typeIdx, eventStorage, eventConstructed, std::forward<Args>(args)...);

    return eventConstructed != nullptr;
}

template<typename EventType, typename... Args>
static EventType *constructEvent(std::aligned_storage_t<sizeof(EventType), alignof(EventType)> & eventStorage,
                                 Args &&... args)
{
    return std::launder(new(&eventStorage) EventType(std::forward<Args>(args)...));
}

template<typename EventType, typename... Args>
void publishEmplaceImpl(const std::type_index & typeIdx,
                        std::aligned_storage_t<sizeof(EventType), alignof(EventType)> & eventStorage,
                        EventType *& eventConstructed,
                        Args &&... args) const
{
    if (auto it = callbacks.find(typeIdx); it != callbacks.end()) {
        for (const auto & [subId, callback] : it->second) {
            if (!eventConstructed) {
                eventConstructed = constructEvent<EventType>(eventStorage, std::forward<Args>(args)...);
            }
            callback(*eventConstructed);
        }
    }

    auto range = EventsRelation::RelationManager::getParentsRange(typeIdx);
    for (auto it = range.first; it != range.second; ++it) {
        if (!eventConstructed) {
            publishEmplaceImpl<EventType>(it->second, eventStorage, eventConstructed, std::forward<Args>(args)...);
        } else {
            publishImpl(it->second, *eventConstructed);
        }
    }
}

PolinaAksenova, 22 Марта 2021

Комментарии (66)

C++ / Говнокод #27307

+1

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72

namespace detail {
    template<entity_event_t Event>
    struct EventHasEntityStateConstructor {
        // Sanity check
        static_assert(static_cast<int32_t>(Event) >= 0
                      && static_cast<int32_t>(Event) < ENTITY_EVENTS_COUNT);

    private:
        struct TwoChar {
            char a, b;
        };

        template<typename T>
        constexpr static TwoChar _check(
            decltype(
                T(std::declval<const entityState_t &>())
            )*
        );

        template<typename T>
        constexpr static char _check(...);

    public:
        constexpr inline static bool value = (sizeof(_check<EntityEvent<Event>>(nullptr)) == sizeof(TwoChar));
    };

    template<typename BusT, entity_event_t Event>
    bool defaultEntityEventPublisher(const BusT & bus, const entityState_t & eventEntity)
    {
        static_assert(EventHasEntityStateConstructor<Event>::value,
                      "defaultEntityEventFactory<Event>() instantiated for a custom\n"
                      "event that does not have a (const entityState_t & eventEntity) constructor.\n"
                      "This should not happen (you'll get a more obscured compiler error anyway)!");
        return bus.publishEmplace<EntityEvent<Event>>(eventEntity);
    }

    template<typename BusT, typename T, T... Is>
    constexpr std::array<EntityEventPublisherPtr<BusT>, sizeof...(Is)>
        createDefaultEntityEventFactories(std::integer_sequence<T, Is...>)
    {
        return {
            [](auto i) -> EntityEventPublisherPtr<BusT> {
                if constexpr (EventHasEntityStateConstructor<static_cast<entity_event_t>(i.value)>::value) {
                    return &defaultEntityEventPublisher<BusT, static_cast<entity_event_t>(i.value)>;
                } else {
                    return nullptr;
                }
            }(std::integral_constant<T, Is>{})...
        };
    }
}

// An (event_number -> EntityState<event_number> 'publishing factory' function) mapping;
// if event N could not be constructed from a single entityState_t reference
// then this table would contain nullptr at the index N
template<typename BusT>
const std::array<EntityEventPublisherPtr<BusT>, ENTITY_EVENTS_COUNT> & getDefaultEntityEventsPublishers() noexcept
{
    static auto factories = detail::createDefaultEntityEventFactories<BusT>(std::make_integer_sequence<int32_t, ENTITY_EVENTS_COUNT>());
    return factories;
}

template<typename BusT>
EntityEventPublisherPtr<BusT> tryGetDefaultEntityEventPublisher(entity_event_t event) noexcept
{
    auto eventNum = static_cast<int32_t>(event);
    if (eventNum >= 0 && eventNum < ENTITY_EVENTS_COUNT) {
        return getDefaultEntityEventsPublishers<BusT>()[eventNum];
    } else {
        return nullptr;
    }
}

PolinaAksenova, 21 Марта 2021

Комментарии (10)

Java / Говнокод #27306

+1

1

Class HasThisTypePatternTriedToSneakInSomeGenericOrParameterizedTypePatternMatchingStuffAnywhereVisitor

https://javadoc.io/doc/org.aspectj/aspectjweaver/1.8.10/org/aspectj/weaver/patterns/HasThisTypePatternTriedToSneakInSomeGene ricOrParameterizedTypePatternMatchingStu ffAnywhereVisitor.html

6oHo6o, 21 Марта 2021

Комментарии (23)

C++ / Говнокод #27305

−1

1
2
3
4
5
6
7
8
9

enum class NodeFlags : int {
    None               = 0,
    Let                = 1 << 0,
// ...
}

contextFlags &= ~flag;

// error C2675: unary '~': 'NodeFlags' does not define this operator or a conversion to a type acceptable to the predefined operator

Кому нужны такие гавно-компиляторы? которые даже не в состоянии привети четко прописанные типы?

ASD_77, 20 Марта 2021

Комментарии (29)

C++ / Говнокод #27303

0

1
2
3
4
5
6
7
8
9

metrics_key::metrics_key(std::initializer_list<std::string> const& il) {
  TRI_ASSERT(il.size() > 0);
  TRI_ASSERT(il.size() < 3);
  name = *il.begin();
  if (il.size() == 2) {
    labels = *(il.begin()+1);
  }
  _hash = std::hash<std::string>{}(name + labels);
}

просто гавно

gnusmas, 18 Марта 2021

Комментарии (36)

Си / Говнокод #27302

0

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36

if (LD3_state == 0 || LD2_state == 0 || LD1_state == 0)
{
    GPIO_SetBits(GPIOB, LED1); //RED ON
    LD_Status = NO_LOCK;
}
else
{
    LD_Status = LOCK_OK;
}

if (LD4_state == 0 && LD_Status == LOCK_OK)
{
    GPIO_SetBits(GPIOB, LED1); //RED ON
}
else
{
    GPIO_ResetBits(GPIOB, LED1); //RED OF
}

if (LD4_state == 0 && LD_Status == LOCK_OK)
{
    GPIO_ResetBits(GPIOB, LED1); //RED OFF
}

/* Ну можно же было сделать по-человечески, а? */
/* Эквивалентный код: */

LD_Status = LD1_state && LD2_state && LD3_state && LD4_state;
if (LD_Status)
{
    GPIO_ResetBits(GPIOB, LED1); //RED OFF
}
else
{
    GPIO_SetBits(GPIOB, LED1); //RED ON
}

Как же я ненавижу чье-то легаси...

viteo, 18 Марта 2021

Комментарии (33)

C++ / Говнокод #27301

0

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15

// почему это гавно не будет работать?
auto size = buffer.size() - 1;
auto *ptr = new byte[size];
for (auto i = size; i >= 0; i--)
{
    ptr[i] = 0;
}

// a это гавно будет работать :)
auto size = buffer.size() - 1;
auto *ptr = new byte[size];
for (int i = size; i >= 0; i--)
{
    ptr[i] = 0;
}

почему это говно не будет работать?

ASD_77, 17 Марта 2021

Комментарии (133)

JavaScript / Говнокод #27300

0

1
2
3

// These are equivalent:
fn`some string here`;
fn(['some string here']);

Джаваскриптеры переизобрели руби (ладно-ладно, скрестили с FormattableString из шарпа)

https://styled-components.com/docs/advanced#tagged-template-literals

Fike, 16 Марта 2021

Комментарии (17)

JavaScript / Говнокод #27299

0

01
02
03
04
05
06
07
08
09
10
11
12

// Since styled-components allows you to use arbitrary input as interpolations, you must be careful to sanitize that input.
// Using user input as styles can lead to any CSS being evaluated in the user's browser that an attacker can place in your application.

// This example shows how bad user input can even lead to API endpoints being called on a user's behalf.

// Oh no! The user has given us a bad URL!
const userInput = '/api/withdraw-funds'

const ArbitraryComponent = styled.div`
  background: url(${userInput});
  /* More styles here... */
`

в процессе многолетнего пересоздания ruby on rails с нуля фронтендеры умудрились accidentally a bottle of injections

https://styled-components.com/docs/advanced#security

Fike, 16 Марта 2021

Комментарии (61)

Pascal / Говнокод #27297

0

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29

type
.....
TSyncParamsPasser = class
   PassedJSON:String;
   constructor Create(RJS:String);
   procedure ProcessExternalCmd;
  end;
.....
.....
procedure TForm1.TCPServerExecute(AContext: TIdContext);
begin
.....
   SyncParamsPasser:=TSyncParamsPasser.Create(CustomData);
   TIdSync.SynchronizeMethod(SyncParamsPasser.ProcessExternalCmd);
   FreeAndNil(SyncParamsPasser);
.....
end;
.....
constructor TSyncParamsPasser.Create(RJS:String);
begin
 PassedJSON:=RJS;
end;

procedure TSyncParamsPasser.ProcessExternalCmd;
var
 JSON:TJSONObject;
begin
 JSON:=TJSONObject.ParseJSONValue(PassedJSON) as TJSONObject;
.....

Событие OnExecute компонента TCPServer вызывается из рабочего потока, в котором обрабатывается входящее подключение.

Способ передачи параметров в процедуру, выполняющуюся внутри Synchronize, с помощью класса.
Говно или сойдёт?

SmseR, 16 Марта 2021

Комментарии (7)

Говнокод: по колено в коде.