Говнокод #13942 — C++ — Говнокод.ру

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

+13
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
```
if(ortho){
            if( abs(e->x() - line.p1().x() ) * 2 < abs( e->y() - line.p1().y() ) ){
                y = e->y();
                x = line.p1().x();
            }else if(abs(e->x() - line.p1().x() ) > 2 * abs( e->y() - line.p1().y() )){
                x = e->x();
                y = line.p1().y();
            }else if(abs(e->x() - line.p1().x() ) * 2 > abs( e->y() - line.p1().y() ) &&
                     abs(e->x() - line.p1().x() ) < abs( e->y() - line.p1().y() )){
                if((e->x() < line.p1().x() && e->y() < line.p1().y()) || (e->x() > line.p1().x() && e->y() < line.p1().y())){
                    x = e->x();
                    y = line.p1().y() - abs(line.p1().x() - e->x());
                }else if((e->x() > line.p1().x() && e->y() > line.p1().y()) || (e->x() < line.p1().x() && e->y() > line.p1().y())){
                    x = e->x();
                    y = line.p1().y() + abs(line.p1().x() - e->x());
                }
            }else if(abs(e->x() - line.p1().x() ) < 2 * abs( e->y() - line.p1().y() ) &&
                     abs(e->x() - line.p1().x() ) > abs( e->y() - line.p1().y() )){
                if((e->x() < line.p1().x() && e->y() < line.p1().y()) || (e->x() < line.p1().x() && e->y() > line.p1().y())){
                    y = e->y();
                    x = line.p1().x() - abs(line.p1().y() - e->y());
                }else if((e->x() > line.p1().x() && e->y() > line.p1().y()) || (e->x() > line.p1().x() && e->y() < line.p1().y())){
                    y = e->y();
                    x = line.p1().x() + abs(line.p1().y() - e->y());
                }
            }else{
                x = e->x();
                y = e->y();
            }
```
Запостил: Abbath, 14 Октября 2013

Tweet
Комментарии (34) RSS
- Xom94ok 14.10.2013 21:21 # +7
  
  Подсократив чуток код (http://pastebin.com/saYWxTCR), я сделал выводы:
  1. Это, наверное, фрагмент какого-то схемопостроителя.
  2. Некоторые программисты рождаются со встроенным обфускатором.
  3. На C++ 99% алгоритмов и математики выглядят как говно.
  
  P.S. При сокращении отвалилось несколько условий, но, как мне кажется, результат от этого ничуть не испортился.
  Ответить
  - roman-kashitsyn 15.10.2013 08:39 # +8
    
    > C++99%
    как-то я упустил этот стандарт плюсов
    Ответить
    - defecate-plusplus 15.10.2013 09:25 # +11
      
      99% UB, 1% luck
      Ответить
  - roman-kashitsyn 15.10.2013 10:15 # 0
    
    > На C++
    На каком языке этот алгоритм можно записать понятней?
    
    Вообще интересно, что автор хотел выразить этим кодом. Перемещаем некую точку к началу прямой, если разница расстояний слишком несимметрична? Хочется объяснений в контексте предметной области. Ну и if (ortho) немного смущает...
    
    P.S. код будет немного читабельней, если все условия привести к единому виду, т.е.
    if (2 * dist_x < dist_y) if (2 * dist_y < dist_x) if (dist_y < 2 * dist_x && dist_x < dist_y) if (dist_x < 2 * dist_y && dist_y < dist_x)
    Ответить
    - Xom94ok 15.10.2013 10:28 # 0
      
      >>На каком языке этот алгоритм можно записать понятней?
      На естественном, в комментарии :) Фортран из книг по вычмату тоже был более-менее читабельным.
      Ответить
      
      roman-kashitsyn 15.10.2013 10:29 # 0
      
      > На естественном, в комментарии :)
      Ох, это врядли...
      > Фортран
      Писал я на фортране, не верю, что будет понятней.
      Ответить
    - Abbath 15.10.2013 10:29 # 0
      
      Это поиск координаты конца линии, которая должна быть вертикальной или горизонтальной, потому и ortho, а потом были добавлены линии под углом 45 °
      Ответить
      
      roman-kashitsyn 15.10.2013 10:34 # 0
      
      > Это поиск координаты конца линии, которая должна быть вертикальной или горизонтальной
      У нас есть линия line, начинающаяся в p1 и заканчивающаяся в p2, и некая точка e. Что нужно найти?
      Ответить
      
      Abbath 15.10.2013 10:48 # 0
      
      Найти какую линию рисовать, вертикальную, горизонтальную или диагональную в зависимости от того в какой четверти четверти находится e. Из e добываем одну координату p2, а другую считаем.
      Ответить
      
      roman-kashitsyn 15.10.2013 10:54 # 0
      
      а line это координатная ось?
      Ответить
      
      Abbath 15.10.2013 11:14 # 0
      
      line это линия, оси, как таковой нет, начало координат в точке p1, но это все рисуется на прямоугольном окне, и поэтому нужно подгонять координаты. Это все такая херня как в паинте рисовать с зажатым Shift.
      Ответить
      
      Xom94ok 15.10.2013 13:06 # 0
      
      >>в паинте рисовать с зажатым Shift.
      "О сколько нам открытий чудных..."
      Ответить
      
      roman-kashitsyn 15.10.2013 13:16 # 0
      
      Кажется, я понял. Нужно приблизать отрезок (p1, e) другим отрезком, начинающимся в p1, имеющим угол наклона (\pi / 2) * n и заканчивающимся максимально близко к e?
      (пэинта под рукой нет, проверить догадку не могу)
      Ответить
      
      Abbath 15.10.2013 13:21 # 0
      
      Типа того.
      Ответить
      
      roman-kashitsyn 15.10.2013 14:45 # 0
      
      Я почти понял идею. Только проекция на вектора под углами (2n+1)\pi / 2 как-то криво вычисляется.
      Ответить
      
      Abbath 15.10.2013 14:51 # 0
      
      Тем не менее это работает.
      Ответить
    - kipar 16.10.2013 13:13 # +2
      
      >На каком языке этот алгоритм можно записать понятней?
      Ну, как минимум на любом нормальном языке будут свойства вместо рябящих в глазах "()" и точки вместо "->".
      Хотя всех недостатков кода это не отменяет.
      Ответить
- Elvenfighter 15.10.2013 00:55 # +2
  
  Это карта, где клад зарыт. Не зря же там столько стрелочек.
  Ответить
- kegdan 15.10.2013 02:09 # 0
  
  Похоже это работа для... Человека -Оптимизатора!
  Ответить
  - kegdan 15.10.2013 02:57 # +4
    
    if(ortho){ auto X = e->x(); auto Y = e->y(); auto LX = line.p1().x(); auto LY = line.p1().y(); auto DX = abs(X - LX ); auto DY = abs( Y - LY ); x = X; y = Y; if ( DX * 2 == DY || DX == 2 * DY) return; if( DX * 2 < DY ){ x = LX; return; } if(DX > 2 * DY){ y = LY; return; } if ( DX == 0 || DY == 0 || DX == DY) return; if(DX < DY){ y = Y < LY? LY - DX: LY + DX; } else{ x = X < LX? LX - DY: LX + DY; } }
    Ответить
    - kegdan 15.10.2013 13:19 # +1
      
      Почему никто не хвалит Человека-Оптимизатора?(
      Ответить
      
      roman-kashitsyn 15.10.2013 13:21 # +4
      
      дык ведь умножения на 2 не оптимизированы
      Ответить
      
      govnomonad 15.10.2013 13:25 # +1
      
      зато у него будет работать не только на двоичном компьютере
      Ответить
      
      roman-kashitsyn 15.10.2013 13:44 # 0
      
      ну хотя бы не надеяться на компилятор и закешировать
      Ответить
    - spivti 15.10.2013 14:59 # +1
      
      чилавек-рефакторист, поставлю лайк, за одну тока работу. Ибо у автора получилось месиво.
      
      Еще чюю ,это можно еще упростить.
      Ответить
    - Abbath 15.10.2013 15:04 # 0
      
      Спасибо, заменил.
      Ответить
- roman-kashitsyn 15.10.2013 15:24 # +3
  http://ideone.com/z8OvVD
  Наговнявкал свой вариант. Основной код
  point approximate(const point o, const point e) { point points[] = { {e.x, o.y}, // projection to X {o.x, e.y}, // projection to Y project_to_units(o, e) // projection to [(2n+1)*pi / 2] axis }; return *std::min_element(points, points + 3, closer_to(e)); }
  Ответить
  - Abbath 15.10.2013 15:33 # 0
    
    Разосрал на кучу функций, то, что у меня влезлось в одну.
    Ответить
    - roman-kashitsyn 15.10.2013 16:13 # +4
      
      > влезлось в одну
      надеюсь, лось после этого остался жив?
      Ответить
    - 3Doomer 16.10.2013 07:36 # 0
      
      Количество функций влияет только на читабельность. И влияет, как ни странно, ПОЛОЖИТЕЛЬНО
      Ответить
  - defecate-plusplus 16.10.2013 09:41 # 0
    
    на новой работе с++11 ещё не в почёте?
    auto points = { {e.x, o.y}, // projection to X {o.x, e.y}, // projection to Y project_to_units(o, e) // projection to [(2n+1)*pi / 2] axis }; return *std::min_element(std::begin(points), std::end(points), closer_to(e));
    Ответить
    - defecate-plusplus 16.10.2013 09:48 # 0
      
      да и вместо клозер_ту можно и лямбду наговнякать
      вот сразу сколько возможностей
      Ответить
    - roman-kashitsyn 16.10.2013 09:55 # +1
      
      > на новой работе с++11 ещё не в почёте?
      Наоборот, новый стандарт во все поля. Но я решил оставаться в рамках предыдущего стандарта, ведь не всем доступна такая роскошь. Руки чесались воткнуть лямбду и не рассчитывать границы массива самостоятельно, но я сдержал порыв.
      
      P.S. Затупил, [(2n+1)*pi / 4] axis
      Ответить
- ne4eHb 25.08.2021 08:11 # 0
  
  - Мам, ты видела, как он улыбается? - вмешался в разговор Даня.
  Ответить
Добавить комментарий
Ошибка компиляции комментария:

Гости могут высказаться только во вторник, пятницу или субботу
Семь раз отмерь — один отрежь, guest!

А не использовать ли нам bbcode?

[b]жирный[/b] — жирный

[i]курсив[/i] — курсив

[u]подчеркнутый[/u] — подчеркнутый

[s]перечеркнутый[/s] — перечеркнутый

[blink]мигающий[/blink] — мигающий

[color=red]цвет[/color] — цвет (подробнее)

[size=20]размер[/size] — размер (подробнее)

[code=<language>]some code[/code] (подробнее)

Проверочный код: *

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

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

Комментарии (34) RSS

Добавить комментарий