1. Список говнокодов пользователя Antervis

    Всего: 15

  2. C++ / Говнокод #19987

    +2

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    9. 9
    template <typename T, typename ...Args>
std::future<T> looped_thread::add_task(std::function<T(Args...)> func, Args ...args) {
    std::packaged_task<T()> task(std::bind(func,args...));
    std::future<T> fut = task.get_future();
    std::lock_guard<std::mutex> lock(_mutex);
    /*std::queue<std::function<void()>>*/ _tasks.push(/*std::packaged_task<T()> -> std::function<void()> ??*/);
    /*std::condition_variable*/ _cv.notify_one(); 
    return std::move(fut);
}

    Задача: написать собственный пул потоков с блекджеком и шлюхами
    Подзадача: реализовать метод, добавляющий функтор в очередь команд и возвращающий std::future
    Что не так: std::packaged_task не конвертируется в std::future. Он не копируется, а поэтому:
    а. его нельзя передать через std::bind;
    б. при захвате через с++14 lambda capture expressions (аля 

    auto f = [t = std::move(task)](){/**/};
    ) мы получаем t типа const std::packaged_task (мб это особенность mingw, конечно, но один фиг работать должно везде). Ни выполнить, ни мувнуть, ни скопировать. mutable в спецификации лямбды не помогает
    в. Примерно все те же самые проблемы возникают при попытке обернуть packaged_task в unique_ptr
    г. если чуть-чуть погуглить, можно найти кучу страшных решений через виртуальные методы, наследование, оборачивание packaged_task в shared_ptr и пр. Например: https://rsdn.ru/forum/cpp/5824551.all

    Но есть куда более простое и изящное решение. Угадаете?

    Antervis, 12 Мая 2016

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

  3. C++ / Говнокод #19970

    +1

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    29. 29
    30. 30
    31. 31
    32. 32
    33. 33
    34. 34
    35. 35
    36. 36
    37. 37
    38. 38
    39. 39
    40. 40
    41. 41
    42. 42
    43. 43
    44. 44
    45. 45
    46. 46
    47. 47
    48. 48
    49. 49
    #include <iostream>
#include <map>

template <typename T1, typename T2, typename F>
void operator ^= (std::map<T1,T2> &m, F func) {
    for (typename std::map<T1,T2>::iterator i = m.begin();  i != m.end(); ) {
        if (func(i))
            i = m.erase(i);
        else
            ++i;
    }
}

template <typename T1, typename T2>
void operator ^= (std::map<T1,T2> &m, T2 value) {
    for (typename std::map<T1,T2>::iterator i = m.begin();  i != m.end(); ) {
        if (value == i->second)
            i = m.erase(i);
        else
            ++i;
    }
}

int main() {
    std::map<int,bool> m = {
        {0,false},
        {1,true},
        {2,false},
        {3,true},
        {4,false},
        {5,true},
        {6,false},
        {7,true},
        {8,false},
        {9,true},
        {10,false}
    };

    for (auto i = m.begin(); i != m.end(); ++i)
        std::cout << "{" << i->first << "," << i->second << "} ";
    std::cout << std::endl;

    m ^= false;
    m ^= [](auto &i) { return i->first == 5; };

    for (auto i = m.begin(); i != m.end(); ++i)
        std::cout << "{" << i->first << "," << i->second << "} ";
    std::cout << std::endl;
}

    Для std::vector'a запись не особо-то кошерная:

    std::vector<int> v = {1,2,3,4,5};
    
v.erase(std::remove_if(v.begin(),v.end(),[](int &i){ return i <= 3;}), v.end());
    А для std::map даже такого не придумали. Пора исправлять

    Antervis, 10 Мая 2016

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

  4. C++ / Говнокод #19630

    0

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    QMutableListIterator<AISObjectQML *> it(m_visibleList);
if (it.hasNext()) {
    AISObjectQML *v = it.next();
    if (v) { // здесь должен быть while, но, признаться, на это уже посрать
        delete v;
    }
}
m_visibleList.clear();

    Есть люди, которые придумали концепцию итераторов для реализации generic алгоритмов. А есть люди, которые любят итераторы, но не любят generic алгоритмы. Ведь с ними код не внушает ужаса и рвотных позывов

    Antervis, 15 Марта 2016

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

  5. C++ / Говнокод #19326

    0

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    QGeoTiledMap *map = new QGeoTiledMap(this,nullptr);
QGeoMapController *controller = map->mapController();
qDebug() << rotation << controller->bearing(); // 30 0
controller->setBearing(rotation);
qDebug() << controller->bearing(); // 0
return map;

    because fuck you that's why

    Antervis, 19 Января 2016

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

  6. C++ / Говнокод #18815

    +9

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    int turn = m_mapObject->getTurnSpeed();
//НЕ НАДО ТУТ ОПТИМИЗИРОВАТЬ /= 45 !!!!
if (turn != 0) {
    if (turn > 0) {
    //                    qDebug() << "Turn plus";
        turn /= 45;
        if (turn < 4) turn = 4;
        if (turn > 8) turn = 8;
    } else {
    //                    qDebug() << "Turn minus";
        turn /= 45;
        if (turn > -4) turn = -4;
        if (turn < -8) turn = -8;
    }
    paint.drawLine(QPoint(0,-speed),QPoint(turn,-speed));
}

    Вы б знали, как руки чешутся...

    Antervis, 05 Октября 2015

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