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

    +144.8

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    A* arr = new A[size];
std::vector<A*> vec(size);
for(size_t i=0; i<size; ++i)
	vec.push_back(&a[i])
// Do something with vec
delete[] arr;

    Запостил: kokorins, 08 Апреля 2010

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

    • ava guest 08.04.2010 19:11 # −1.6

      fill(vec.begin(), vec.end(), new A);
      Ответить

      • ava guest 08.04.2010 20:17 # 0

        Ваш код не эквивалентен вышеприведенному, не так ли?
        Ответить

        • ava guest 08.04.2010 21:38 # 0

          Нет, но он короткий и исправляет говнистость исходного =)
          Ответить

    • ava guest 08.04.2010 19:19 # +2

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

      • ava guest 08.04.2010 21:43 # −1

        все засунутые в вектор указатели были delete [] => в векторе wild pointers =)
        Ответить

        • ava guest 09.04.2010 07:15 # +2.2

          Если вектор после приведенного кода прекращает житие - никаких проблем нет.
          Удаление указателей не ведет к вызову деструкторов объектов, на которые они указывают.
          Ответить

          • ava guest 09.04.2010 17:33 # −0.2

            Как это не ведет? Как по вашему вызывается деструктор объекта? A::~A() чтоли?
            Ответить

            • ava guest 10.04.2010 16:04 # 0

              А вот так вот и не ведет.
              В STL при деструкции контейнера вызываются деструкторы того что в контейнере расположено.
              Для объектов - вызываются деструкторы объектов.
              Для указателей - ничего не вызывается.
              Ответить

      • ava kokorins 09.04.2010 23:42 # 0

        Код почти чистый:
        1. код бросает исключения как минимум
        2. ради красоты delete[] дублировать массив указателей, это по моему сликом.
        3. Зная длину массива, делать push_back ну как-то "расточительно" чтоли.
        4. Мне потребовалось минут 5 чтобы понять, что хотел сказать автор.
        Ответить

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