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

• guest 7 лет ago # 0

  Не очень понятно, где тут С++. Оператор деления перегружен?
  
  Впрочем, писать одно и тоже два раза точно смысла нет

  Ответить

  • DyKaJluC 7 лет ago # 0 ⇈

    Конечно это не может быть C++.

    Ответить

    • raMagPuJI 7 лет ago # 0 ⇈

      почему?

      Ответить

  • guest 7 лет ago # 0 ⇈

    Починил:

    if (res < ans * 2) {
        ans = res / 2;
    }

    Теперь дублирования кода нет.

    Ответить

    • Web_Monkey 7 лет ago # 0 ⇈

      ans = min(res / 2, ans);

      Ответить

      • raMagPuJI 7 лет ago # 0 ⇈

        а чтобы было интереснее, пусть res будет unsigned

        Ответить

      • DECAHTHblu_nemyx 7 лет ago # 0 ⇈

        Чего смешного то?

        Ответить

    • kegdan 7 лет ago # 0 ⇈

      if (res < ans << 1) {
          ans = res >> 1;
      }

      
      
      улучшено, не благодари

      Ответить

      • gem 7 лет ago # 0 ⇈

        Даже с тарасоформатированием!

        Ответить

  • Steve_Brown 7 лет ago # 0 ⇈

    А как есть смысл?
    

    // Я оптимизироваль
    auto r = res / 2;
    if (r < ans)
      ans = r;

    Ответить

    • Dpyrou_Pycoqpo6_CEMA 7 лет ago # 0 ⇈

      как есть

      Ответить

• rss 7 лет ago # 0

  Проведена дезинфекция

  Ответить

• rss 7 лет ago # 0

  Проведена дезинфекция

  Ответить

• rss 7 лет ago # 0

  Проведена дезинфекция

  Ответить

• OlegUP 7 лет ago # 0

  А где тут говнокод то?

  Ответить

  • awk 7 лет ago # 0 ⇈

    Возможно, кому-то не понравилось, что здесь деление два раза. Хотя компиляторы оптимизировать такое уже научились:
    https://gcc.godbolt.org/z/55RRRc
    
    А может быть, стиль имени магистра Йоды в условии не понравился (ans на первом месте было бы понятнее).

    Ответить

    • awk 7 лет ago # 0 ⇈

      Кстати, интересная подготовка к делению:

      shr     eax, 31
      add     eax, edi

      
      Если установлен знаковый бит, то к делимому добавляется единица. Это чтобы округление всегда было в сторону нуля.

      Ответить