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

    +64

    1. 1
    delete new T;

    убийство новорожденного

    Запостил: absolut, 19 Сентября 2014

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

    • ava Psionic 19.09.2014 12:14 # +2

      Мать Жана Батиста Гренуя видимо чуяла неладное. По коду - а смысл?
      Ответить

      • ava bormand 19.09.2014 12:16 # +1

        > а смысл
        Ради побочного эффекта конструктора, возможно.
        Ответить

        • ava roman-kashitsyn 19.09.2014 12:18 # +4

          Я вот прям сейчас смотрю на жабий код
          public static void main(String[] args) {
    new Main();
}
          Ответить

          • ava Elvenfighter 19.09.2014 12:46 # +8

            Смотрю я на эти ваши new и думаю, что пора бы завести оператор old. Последний должен возвращать ссылку на уже дешевый (по ресурсам) старый объект в неизвестном состоянии. Дадим старым вещам новую жизнь! Спасём кучу от смерти в мусоре!
            Ответить

            • ava absolut 19.09.2014 12:55 # 0

              old уже есть, это shared_ptr по сути.
              Ответить

            • ava bormand 19.09.2014 15:18 # +2

              > Последний должен возвращать ссылку на уже дешевый (по ресурсам) старый объект в неизвестном состоянии.
              Ну всякие пулы так и работают...
              Ответить

            • ava guest 20.09.2014 05:19 # 0

              В жаве до жопы, т.к. там копирующий сборщик мусора.
              Ответить

              • ava bormand 20.09.2014 07:27 # 0

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

          • ava absolut 19.09.2014 13:04 # +1

            а в конструкторе Main какой-нибудь вызов mAin ... и дальше перебор кейсов.
            Ответить

      • ava absolut 19.09.2014 13:02 # +2

        > а смысл?
        смысла особого нет, кроме явного "передергивания" объекта, влекущего то, о чем сказал bormand.
        недавно просто проплывала ссылка на код тараса (http://govnokod.ru/14093),
        и я подумал, ну нельзя же так срать в куче, надо и убирать иногда. И вот результат.
        Ответить

    • ava roman-kashitsyn 19.09.2014 13:14 # +7 

      {T _;}
      Ответить

      • ava absolut 19.09.2014 13:20 # 0

        стековая версия похожа на франкенштейна
        Ответить

        • ava roman-kashitsyn 20.09.2014 16:29 # +1

          Одноглазый объект плачет в тюрьме из скобок.
          Ответить

          • ava inkanus-gray 20.09.2014 16:31 # +2

            А вроде в тюрьме вилок не дают?
            Ответить

            • ava bormand 20.09.2014 19:24 # +2

              http://static.diary.ru/userdir/4/1/3/0/413061/22806947.jpg
              Ответить

              • ava kegdan 20.09.2014 19:37 # 0

                Ремейк все видели?
                Ответить

                • ava 3.14159265 20.09.2014 22:22 # +1

                  >Ремейк
                  Говноедов тут нет, я полагаю.
                  Ответить

                • ava bormand 20.09.2014 22:26 # 0

                  Что за ремейк?
                  Ответить

                  • ava 3.14159265 20.09.2014 22:30 # +4

                    Гайдаи и Даниэлии повывелись.Вместо них теперь Баскова.
                    Но унылые халтурщики решили снимать сиквелы к старым советским комедиям, пытаясь срубить немного бабла на их известности.
                    Ответить

                • ava absolut 20.09.2014 23:18 # 0

                  >Ремейк все видели?
                  Он уже вышел? Про кавказскую пленницу что-то слышал краем уха.
                  Ответить

                  • ava kegdan 21.09.2014 00:34 # +1

                    вышел. ГГ играет сам Безруков. Это пиздец
                    Ответить

    • ava 1024-- 19.09.2014 13:46 # +12

      Я не устоял и написал свой калькулятор в этой парадигме.
      http://ideone.com/11jE3T

      Упрощёння версия:
      #include <iostream>
#include <stack>

#define OPERATIONS(X) X(Add, +) X(Sub, -) X(Mul, *) X(Div, /)

std::stack<double> stack;

struct Top  { Top  (double& x) { x = stack.top();   } };
struct Push { Push (double  x) {     stack.push(x); } };

struct Pop {
	Pop(double& x) {
		delete new Top(x);
		stack.pop();
	}
};

struct Number { Number (double v) { delete new Push(v); } };

#define OPERATION_CLASS(Name, X) \
struct Name { \
	Name () { \
		double x, y; \
		delete new Pop(y); delete new Pop(x); \
		delete new Push(x X y); \
	} \
};

OPERATIONS(OPERATION_CLASS)

struct Print {
	Print () {
		double x; 
		delete new Top(x);
		std::cout << x << std::endl;
	}
};

int main() {
	delete new Number(3); delete new Number(2);
	delete new Div; delete new Print;
	
	return 0;
}
      Ответить

      • ava 3.14159265 19.09.2014 13:54 # +6

        Знатно упоролся. Я б добавил для "читабельности":
        #define make delete new
        http://ideone.com/CCYnk1

        Кстати а компилятор, он тварь крестящая, иль право имеет на выпил таких вот сайд-эффектов?
        Ответить

        • ava 1024-- 19.09.2014 14:03 # +3

          Там ещё 11 штук delete new осталось.
          А вообще, для "читабельности" надо не make, а exec.
          Ответить

          • ava 3.14159265 19.09.2014 14:04 # +1

            Исполнить Номер 2 vs Сделать Номер 2
            Сделать Номер 3
            Сделать Деление
            >ещё 11 штук delete new осталось.
            То ж библиотечный код. Там можно.
            Ответить

            • ava 1024-- 19.09.2014 14:18 # +1

              Тогда ладно.

              Кстати, замена для for(i=0; i<N; ++i)...: http://ideone.com/yf1ars
              Ответить

    • ava neTyx_npoTKHyTbIu 25.08.2021 08:19 # 0

      - Ладно, можешь сделать салат из свежих овощей - просто порезать их и сбрызнуть оливковым маслом?
      Ответить

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