1. C++ / Говнокод #2384

    +144

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    void render_tee(/*<еще аргументы>*/, vec2 dir, vec2 pos)
{
	vec2 direction = dir;
	vec2 position = pos;

                  //еще код
}

    вот что я нашел в сорцах игрушки TeeWorlds

    Запостил: danilissimus, 07 Января 2010

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

    • ava guest 07.01.2010 22:08 # 0

      И что?
      Ответить

    • ava guest 07.01.2010 22:21 # +6

      Возможно, там по ходу в этом методе direction и position изменяются, и при этом потом требуется получить доступ к оригинальным аргументам. По этому автор и скопировал аргументы в локальные переменные.
      Ответить

    • ava guest 07.01.2010 22:22 # 0

      ???
      Ответить

    • ava LuCiFer 08.01.2010 10:14 # +3

      не видя что находиться за
      //еще код
      назвать это говнокодом ну никак нельзя....
      даже если там они не изменеются возможно что при разрботке кто-то хотел их изменить, а потом забыл удалить эти присваивания
      Ответить

    • ava pushkoff 08.01.2010 17:12 # 0

      vec2 dir, vec2 pos надо было передать как const vec2&
      Ответить

      • ava guest 09.01.2010 01:09 # 0

        +1
        Ответить

      • ava guest 10.01.2010 04:34 # 0

        И получить на ровном месте тормоза из-за разыменовываний ссылки. Если значешь что делаешь, то вполне можно передавать переменные по значению. Тем более, что тут vec2 - может быть чем нибудь типа struct { char x,y; }
        Ответить

    • ava Altravert 09.01.2010 08:54 # 0

      А "/* еще аргументы */" так и есть в оригинале? Оно бы скомпилилось?
      Ответить

      • ava Altravert 09.01.2010 08:57 # 0

        Ааааа, ступил. Да, non-говнокод. Если раньше передавались ссылки, то всё понятно и логично, даже если нет -- возможно для страховки.
        Ответить

        • ava guest 09.01.2010 15:35 # 0

          /* комментарии */
          обычный C-стиль :)
          Ответить

          • ava Altravert 09.01.2010 18:34 # 0

            Эм, это ты к чему? Я знаю что такое комментарии. Я подумал сначала, что метод у автора void render_tee(, vec2 dir, vec2 pos). Потом дошло, что это постер вырезал аргументы.
            Ответить

    • ava guest 10.01.2010 00:08 # 0

      Где сдесь С++?
      Ответить

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