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

    +6

    1. 1
    2. 2
    3. 3
    4. 4
    list* down_if_valid_me(void)
{
	return this ? this->down() : NULL;
}

    Запостил: LispGovno, 12 Июля 2013

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

    • ava LispGovno 12.07.2013 09:55 # 0

      Я вот размышляю, подсунули ли мне код с UB? Нужно ли пожурить? На данный момент все эти методы не виртуальны, пока.
      Ответить

    • ava guest 12.07.2013 11:10 # +1

      Он даун, если я нормальный
      Ответить

      • ava guest6 15.05.2024 21:35 # 0

        АХАХХА ПРИКОЛ ПРИКОЛ АНЕКДОТ СМЕШНОЙ



        РУСКИЙ И АМЕРИКАНЕЦ ЗАХОДЯТ В ЛИФТ АМЕРИКАНЕЦ ГОВОРИТ ДАУН А РУСКИЙ ТАКОЙ САМ ТЫ ДАУН

        А ДАУН ПО АМЕРИКАНСКИ ВНИЗ А РУСКИЙ ДУМАЛ ЕГО ОБОЗВАЛИ АХАХА ДОШЛО ДОШЛО ДА ВОТ ПРИКОДЛ
        Ответить

    • ava guest 12.07.2013 11:44 # +1

      Это UB. this не может быть равен NULL
      Ответить

      • ava bormand 12.07.2013 11:58 # +1

        > this не может быть равен NULL
        Может. Но это UB :)
        SomeClass *p = NULL;
p->down_if_valid_me();
        Ответить

      • ava guest 12.07.2013 13:10 # +1

        this - по существу, дополнительный параметр-указатель, передающийся при вызове метода.
        Если к параметру this не происходит явных или неявных обращений (e.g. не производится доступа к полям), то код выполняется, как и должен.
        Ответить

        • ava guest 12.07.2013 13:18 # +1

          http://ideone.com/m97C8j
          Ответить

          • ava guest 12.07.2013 13:23 # +1

            Т.е.
            Pituh *p = NULL;
p->Golos();

            в данном случае всего лишь
            Pituh_Golos(NULL);
            Ответить

            • ava inkanus-gray 12.07.2013 14:13 # +1

              Верно, но требуется уточнение: если используются только статические методы, т. к. для доступа к виртуальным потребуется инициализировать ТВМ.

              http://ideone.com/nbBOPs — к виртуальным без this не достучаться.
              Ответить

              • ava bormand 12.07.2013 16:35 # +3

                http://ideone.com/TnQJll
                Ответить

                • ava inkanus-gray 12.07.2013 17:44 # 0

                  А так падает: http://ideone.com/vzpqvI
                  Ответить

                  • ava bormand 12.07.2013 21:55 # 0

                    А вот тут уже ниче не сделать. Вызов Golos из Golosok с кривым this не пролезет...
                    Ответить

                    • ava guest 12.07.2013 22:24 # 0

                      А откастить?
                      Ответить

                      • ava bormand 12.07.2013 22:29 # +1

                        Ну так трабла в чем - Pituh::Golosok то вызывается нормально, но внутри него скрыт вызов виртуального метода, который на нулле сфейлится. Само собой можно написать return (*((Kokoko**)&tmp))[1], чтобы дернуть Golos, вместо него, но это уже не то ;)
                        Ответить

      • ava someone 14.07.2013 16:40 # +1

        Кстати, в MFC везде стоят проверки на нулёвость this.

        Ну на то он и MFC. Это пособие по тому, как не нужно проектировать объектную библиотеку.
        Ответить

    • ava bormand 12.07.2013 14:06 # +1

      Я понял зачем это! Автор хочет, чтобы работало как в Objective C:
      some *p = a->b->c->d;
if (p) { .. }
      И никаких лишних проверок на null'ы :)
      Ответить

    • ava guest 12.07.2013 18:44 # −8

      показать все, что скрытососу хуи даю в жопу
      Делаю минеты
      Ответить

    • ava Lure Of Chaos 12.07.2013 23:58 # +1

      this может быть null и не упасть?
      Ответить

      • ava bormand 13.07.2013 00:04 # 0

        Может. Не падает, если не дергать виртуальные методы и не лезть в поля. Но делать так - ССЗБ.
        Ответить

        • ava Lure Of Chaos 13.07.2013 00:21 # +1

          ого. в той же джаве не может. ну там все методы виртуальные
          Ответить

          • ava bormand 13.07.2013 00:28 # +2

            Да просто невиртуальный метод - это обычная функция, первым параметром которой идет this. А проверки на NULL кресты автоматом не вставляют. Поэтому ничего и не падает.

            А в джаве, кстати не все виртуальные - есть же статики, и final'ы вроде как оптимизируются в невиртуальный вызов.
            Ответить

            • ava Lure Of Chaos 13.07.2013 00:37 # 0

              у статиков this нет по определению, даже код с указанием this в статиках не скомпилится
              а final только запрещает переопределение, но всегда упает с NPE на nullевом объекте (может инлайнить по своему усмотрению)
              Ответить

              • ava someone 14.07.2013 16:44 # 0

                В Java есть invokevirtual, invokeinterface, invokestatic и invokespecial.

                Вот последний используется для вызова private-методов, ибо там класс известен и полиморфное поведение не нужно.

                Но он всё равно валится по NPE, даже если в private-методе this никак не используется.
                Ответить

          • ava barop 03.11.2016 18:39 # −63

            показать все, что скрытоVanished
            Ответить

      • ava inkanus-gray 13.07.2013 00:05 # 0

        bormand даже виртуальные методы вызывает с нулевым this. См. зелёную ссылку.
        Ответить

      • ava barop 03.11.2016 20:26 # −63

        показать все, что скрытоVanished
        Ответить

    • ava guest 13.07.2013 01:31 # −5

      показать все, что скрыто???????
      Ответить

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