Говнокод #14470 — Java — Говнокод.ру

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

+64
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
```
balance2N(nodeGet(0), nodeGet(0).getLine(0).getNodeConnectedWith(nodeGet(0)));

class Node {
    private ArrayList<Line> children = new ArrayList<Line>();
    public Line getLine (int index) {
        return children.get(index);
    }
}

class Line {
    private Node[] parents = new Node[2];
    public Node getNodeConnectedWith(Node nodeA) {
            if (parents[0] == nodeA) {
                return parents[1];  //return node another from this node (node on the other side)
            } else {                //if    parents[1] == nodeA     equals  !(parents[0] == nodeA)
                return parents[0];
            }
        }
}
```
Метод balance2N делает что-то с двумя нодами.

Запостил: energetix_user, 02 Февраля 2014

Tweet
Комментарии (41) RSS
- bormand 02.02.2014 18:26 # 0
  
  И в чем говно кроме незакешированного nodeGet(0)?
  Ответить
  - energetix_user 02.02.2014 18:28 # −1
    
    Хотя бы в том что не перегрузки для balance
    Ответить
    - bormand 02.02.2014 18:29 # −1
      
      А если ли в ней смысл? getLine(0) это же первая попавшаяся линия, исходящая из ноды. Почему balance должен работать именно с ней, а не с getLine(1) или вообще со всеми линиями, исходящими из ноды?
      
      Тут надо задачу смотреть. Но заворачивать эту хрень из первой строки в отдельную функцию я бы не рискнул. Вдруг там вообще ни одной линии к этой ноде не подключено, или их 100500, а обрабатывать мы будем только первую.
      
      P.S. Ок, говно нашлось - getLine(0).
      Ответить
      
      energetix_user 02.02.2014 18:32 # −1
      
      getLine(0) - единственная линия если ноды две. Если ноды три - уже три линии. Если нод четыре - от четырех до шести
      Ответить
      
      bormand 02.02.2014 18:33 # −1
      
      Ну а что тогда должна делать перегрузка balance с одной нодой, которой вам так не хватает?
      
      UPD: Более менее осмысленный вариант, который приходит в голову - она должна в цикле балансить ВСЕ линии исходящие из указанной ноды.
      Ответить
      
      bormand 02.02.2014 18:36 # +1
      
      UPD: Ааа! Перегрузка должна принимать Line и отдавать оба конца в balance2N. Это же имелось в виду?
      public void balance(Line line) { balance2N(line.getParent(0), line.getParent(1)); }
      Ответить
      
      energetix_user 02.02.2014 23:32 # 0
      
      Да.
      Ответить
      
      bormand 02.02.2014 23:51 # −1
      
      Понятно...
      
      Ну в общем-то отсутствие convenience методов является не говном, а просто неудобством.
      
      Вот если бы там не было какого-то важного метода, без которого задача бы вообще не решалась - вот тогда да, говноабстракция детектед.
      Ответить
- kegdan 03.02.2014 03:06 # −4
  
  Ну охуеть теперь архитектура. Попытка сделать утечку памяти через кольцевые ссылки?
  Ответить
  - anonimb84a2f6fd141 03.02.2014 06:01 # +3
    
    Жаве на кольцевые ссылки насрать
    Ответить
    - kegdan 03.02.2014 06:03 # −2
      
      разве там не инкриментный GC?
      Ответить
      
      bormand 03.02.2014 06:19 # +1
      
      Не бойся, не подсчет ссылок там. Иначе все проги на жабе уже давно утекли бы.
      Ответить
      
      anonimb84a2f6fd141 03.02.2014 08:12 # +2
      
      В путхоне подсчет ссылок с детектором циклических ссылок.
      Ответить
  - bormand 03.02.2014 06:18 # 0
    
    Что не так с архитектурой то? Всем GC (кроме подсчета ссылок) всегда было насрать на кольцевые ссылки.
    Ответить
    - kegdan 03.02.2014 06:36 # −1
      
      Я CG только начинаю изучать. так что простите мне мою глупость) Есть какие нибудь Хорошие статьи на этот счет? Можно на буржуйском)
      Ответить
      
      anonimb84a2f6fd141 03.02.2014 08:13 # 0
      
      Из того что я нагуглил - http://www.ibm.com/developerworks/ru/library/j-jtp10283/ и другие статьи про яву оттуда.
      Ответить
      
      bormand 03.02.2014 08:30 # +6
      
      Технология утилизации памяти.
      
      Как красиво назвали, и главное как точно. Сколько памяти не суй - а жаба всю ее утилизирует ;)
      Ответить
      
      anonimb84a2f6fd141 03.02.2014 09:13 # −1
      
      Ну это ЫНТЫРПРАЙЗ же, где программа запускается в одном экземпляре и час работы программиста стоит как 8 гигов памяти. Только ЫНТЫРПРАЙЗ и развивается и его либы соответственно суются во все места.
      Ответить
      
      LispGovno 03.02.2014 10:24 # 0
      
      Я вот жрал когда узнал про то, что наш сервер на Java требует хотя бы 128 GB памяти чтобы обрабатывать хоть сколько-то приличное кол-во запросов (хотя бы 100ню если правильно помню) одновременно.
      Ответить
      
      defecate-plusplus 03.02.2014 10:43 # +4
      
      ну это черезчур
      просто скажи, что ты это придумал
      Ответить
      
      LispGovno 03.02.2014 12:11 # −1
      
      Не люблю когда слова расходятся с делом. Слава богу мне вовремя сообщили что это так. Переписываем сервер на кресты.
      Ответить
      
      bormand 03.02.2014 13:04 # +1
      
      Что ж она у вас там жрет такими темпами? По полной копии базы в памяти каждого воркера?
      Ответить
      
      Horse3 03.02.2014 14:47 # −7
      
      показать все, что скрытоСоси хуец.
      Ответить
      
      kegdan 03.02.2014 16:10 # −3
      
      Немного не в тему - читал статейку про геймдев типа майнкрафтика. Прикол в том. что большенство деймдевщиков в собственном соку не занимается распределением памяти, они - просто дают проге свободу и ничего никогда не выгружают. Поэтому инди игры обычно жрут оперативу по полной. Майнкрафт с модами вообще сразу отьедает себе 4 оперативы и стабильно работает 10 минут, потом дохнет. грусть - печаль
      Ответить
      
      TarasB 03.02.2014 16:17 # −1
      
      > Прикол в том. что большенство деймдевщиков в собственном соку не занимается распределением памяти, они - просто дают проге свободу и ничего никогда не выгружают.
      
      лолшто
      
      а, я понял
      просто в геймдеве принято делать так, чтобы все нужные данные изначально были загружены и заниматься мудизмом с памятью вообще не надо было
      а какой-то то ли гсм-журнашлюшка, то ли гсм-читатель написал это так, что якобы они "дают проге свободу и не выгружают память".
      Ответить
      
      kegdan 03.02.2014 16:26 # −3
      
      >>дают проге свободу и не выгружают память
      
      это я написал своими словами - ты сначала это как то иначе понял?
      Ответить
      
      Horse3 03.02.2014 16:37 # −16
      
      показать все, что скрытоЯ насрал в унитаз и сунул туда мордой твоего батю с зубной щёткой.
      Ответить
      
      TarasB 04.02.2014 09:45 # +1
      
      ты разницы не видишь?
      охлол...
      Ответить
      
      kegdan 04.02.2014 10:03 # −1
      
      Ты разницу видишь?
      охлол...
      Ответить
      
      bormand 04.02.2014 11:29 # +1
      
      Твой вариант - желтушно-журналистский стиль: сенсация, инди игры не освобождают память! Т.е. предполагается, что память течет.
      
      Вариант Тараса - игры загружают ресурсы заранее, чтобы не было фризов во время самой игры. Предполагается, что памяти юзается сколько нужно.
      Ответить
      
      kegdan 04.02.2014 11:37 # 0
      
      >> не занимается распределением памяти, они - просто дают проге свободу и ничего никогда не выгружают.
      
      и
      
      >>желтушно-журналистский стиль: сенсация, инди игры не освобождают память!
      
      идентичны?
      
      >>игры загружают ресурсы заранее, чтобы не было фризов во время самой игры.
      
      Ну а че. Вот в ом же крузисе загрузить все ресурсы сразу. Пусть комп встанет на колени и проблюется радугой
      
      Я не понимаю ваших претензий, тем более в форме охлол и лолшто
      Ответить
      
      laMer007 04.02.2014 11:30 # +2
      
      Ти різницю бачиш?
      хохол...
      Ответить
      
      roman-kashitsyn 03.02.2014 16:19 # +1
      
      > читал статейку
      очередные breaking news с хабры?
      Ответить
      
      wvxvw 03.02.2014 21:48 # 0
      
      Попробуй загрузить атлас текстур не выгрузив предыдущий /0.
      Ответить
      
      kegdan 04.02.2014 01:14 # 0
      
      легко
      Ответить
      
      bormand 04.02.2014 05:28 # +1
      
      Смотря какой атлас. Если он всего десяток метров весит - то еще как загрузится ;) Правда потом рано или поздно все равно игрушка вылетит от недостатка памяти, но мы это спишем на кривое железо.
      Ответить
      
      kegdan 04.02.2014 05:36 # 0
      
      обычно это списывают на платформу.Жаба она и есть жаба
      Ответить
      
      eth0 04.02.2014 18:49 # 0
      
      Я читал где-то, что если в минекрафте долго-долго-долго идти в одну сторону можно в Африку, никуда не сворачивая, то сервер съест вообще всю память и умрёт.
      Ответить
      
      kegdan 04.02.2014 18:59 # 0
      
      ну да. это называется принудительный вайп. ждешь копа интузиасты наснут лепить мегабашню или замок, достоял наполовину - и бежишь в ебеня. карту распидорасивает, игра начинает блевать радугой и админу ничего не остается как вайпнуть мирок
      Ответить
      
      bormand 04.02.2014 19:19 # 0
      
      Дык на пабликах ограничение на размер мира ставят. А если не ставят - ну ССЗБ, что поделать.
      Ответить
      
      kegdan 04.02.2014 19:22 # 0
      
      когда как. там где я игрался ограничения не было. ну хотя я игрался когда еще бетка 1.4 была последним писком
      Ответить
Добавить комментарий
Ошибка компиляции комментария:

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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