Russian Qt Forum
Ноябрь 24, 2024, 20:50 *
Добро пожаловать, Гость. Пожалуйста, войдите или зарегистрируйтесь.
Вам не пришло письмо с кодом активации?

Войти
 
  Начало   Форум  WIKI (Вики)FAQ Помощь Поиск Войти Регистрация  

Russian Qt Forum > Forum > Qt > Общие вопросы > Причины утечки памяти и способы борьбы с ними
Страниц: 1 [2] 3 4   Вниз
« предыдущая тема следующая тема »
  Печать  
Автор Тема: Причины утечки памяти и способы борьбы с ними  (Прочитано 26187 раз)
Bepec
Гость
« Ответ #15 : Февраль 15, 2014, 16:55 »
оффтоп: и потом образуется по привычке
typedef int fint;
typedef std::string textstring;
typedef bool variable;

PS проекты после этого нечитабельные на 200% Веселый

PPS хотел добавить пример такого кода, но вспомнил что подписывал бумажку :/
« Последнее редактирование: Февраль 15, 2014, 16:56 от Bepec » Записан
Akon
Гость
« Ответ #16 : Февраль 15, 2014, 18:53 »
Old: Спасибо. Типичная реализация, удобная для автора либы, но может оказаться неудобной для пользователя, поскольку навязывается смарт-поинтер.

Код:
NonQObject* createObject()
{
    NonQObject* result = new NonQObject;
    try {
        functionThatCanThrowException();  // может сделать, throw std::runtime_error(), например
    }
    catch(....) {
        delete result;
        throw;
    }
    return result;
}

void foo()
{
    QScopedPointer<NonQObject> localObject = createObject();
    // do something 
}
В функции foo для локального объекта вы вынуждены были бы использовать QSharedPointer, что избыточно/снижает читабельность кода.


Записан
Old
Джедай : наставник для всех
*******
Offline Offline

Сообщений: 4350



Просмотр профиля
« Ответ #17 : Февраль 15, 2014, 18:57 »
Цитата: Akon от Февраль 15, 2014,  18:53
но может оказаться неудобной для пользователя, поскольку навязывается смарт-поинтер.
Не навязывает. Просто все поинтеры в программе должны быть смарт. Улыбающийся Они для этого и сделаны.

Цитата: Akon от Февраль 15, 2014,  18:53
В функции foo для локального объекта вы вынуждены были бы использовать QSharedPointer, что избыточно/снижает читабельность кода.
Это не избыточно и тем более не снижает читабельность. Но это как я уже написал, если все поинтеры в программе смарт. А к этому нужно стремиться и по возможности использовать.

А вот ваш костылец заметно длинней и такие чудеса нужно вставлять везде, где может возникнуть исключение. Вот она избыточность и нечитаемость. Улыбающийся

И это только создание объекта, а если мы проследим все костыли и исправления в программе, что бы его корректно удалять, то лишних строк будет еще больше. А с shared_ptr ничего менять не придется.
« Последнее редактирование: Февраль 15, 2014, 19:04 от Old » Записан
Igors
Джедай : наставник для всех
*******
Offline Offline

Сообщений: 11445


Просмотр профиля
« Ответ #18 : Февраль 15, 2014, 19:16 »
Цитата: Old от Февраль 15, 2014,  18:57
А вот ваш костылец заметно длинней и такие чудеса нужно вставлять везде, где может возникнуть исключение. Вот она избыточность и нечитаемость. Улыбающийся
Я гораздо чаще действую как Akon и "костылем" это не считаю - ведь и Вам придется потратиться на обработку исключения, только в др, месте. И как Вы туда подадите данные исключения/восстановления - хз. А тут все под рукой.

А вот такой вопросик. Есть много "shared", но мне часто нужно знать не просто "удален или нет" а "кто его держит" - есть ли такая возможность?. А в Qt не знаю даже как посмотреть "а сколько ссылок"  Плачущий
Записан
Old
Джедай : наставник для всех
*******
Offline Offline

Сообщений: 4350



Просмотр профиля
« Ответ #19 : Февраль 15, 2014, 19:29 »
Цитата: Igors от Февраль 15, 2014,  19:16
Я гораздо чаще действую как Akon и "костылем" это не считаю
Вы знаете, а я не удивлен.

Цитата: Igors от Февраль 15, 2014,  19:16
ведь и Вам придется потратиться на обработку исключения, только в др, месте. И как Вы туда подадите данные исключения/восстановления - хз. А тут все под рукой.
Вы код Akon не поняли? Он здесь только удаляет указатель, а на обработку он потратится в другом месте, где не все под рукой. Улыбающийся

Цитата: Igors от Февраль 15, 2014,  19:16
А вот такой вопросик. Есть много "shared", но мне часто нужно знать не просто "удален или нет" а "кто его держит" - есть ли такая возможность?
Так вы пишите программу, вы должны знать тот единственный объект, который является владельцем данных. Все остальные только пользователи.

Цитата: Igors от Февраль 15, 2014,  19:16
А в Qt не знаю даже как посмотреть "а сколько ссылок"  Плачущий
Зачем вам эта бесполезная информация?
Записан
Igors
Джедай : наставник для всех
*******
Offline Offline

Сообщений: 11445


Просмотр профиля
« Ответ #20 : Февраль 15, 2014, 20:20 »
Цитата: Old от Февраль 15, 2014,  19:29
Вы код Akon не поняли? Он здесь только удаляет указатель, а на обработку он потратится в другом месте, где не все под рукой. Улыбающийся
Необязательно, часто можно и не испускать вторичное а вернуть NULL. Да и пример слишком прост - если понадобится напр установить флаг/переменную (хоть код ошибки), то лучше сделать это по месту и потом вторичное throw типа "на свободу с чистой совестью".

Цитата: Old от Февраль 15, 2014,  19:29
Так вы пишите программу, вы должны знать тот единственный объект, который является владельцем данных. Все остальные только пользователи.
Ну допустим и так (не суть), вот смотрю - ресурс жив, а я рассчитывал на его удаление/выгрузку. Вероятно др объект его держит, но вот какой? А у вумного указателя ничего для этого нет, счетчик ссылок - и то не везде
Записан
Old
Джедай : наставник для всех
*******
Offline Offline

Сообщений: 4350



Просмотр профиля
« Ответ #21 : Февраль 15, 2014, 20:28 »
Цитата: Igors от Февраль 15, 2014,  20:20
Необязательно, часто можно и не испускать вторичное а вернуть NULL.
Это вы дальше с Akon уже обсуждайте. Подмигивающий

Цитата: Igors от Февраль 15, 2014,  20:20
вот смотрю - ресурс жив, а я рассчитывал на его удаление/выгрузку.
Где смотрите, в отладчике?

Цитата: Igors от Февраль 15, 2014,  20:20
Вероятно др объект его держит, но вот какой?
Вы хотите, что бы умный указатель еще и всех клиентов сохранял? Не можете разобраться со своими объектами - используйте логи или  набросайте свой вариант  умного указателя (или враппер для него), что бы он сохранял всех клиентов. И пользуйтесь им для отладки.
Или вы хотели бы что бы он в боевом коде всех сохранял?
Записан
Akon
Гость
« Ответ #22 : Февраль 15, 2014, 21:43 »
Цитировать
Не навязывает. Просто все поинтеры в программе должны быть смарт.  Они для этого и сделаны.
Я согласен, что когда все поинтеры в вашей программе смарт и программа задумана под это - это хорошо. Не навязывает - это значит смарт поинтера просто нет, и пользователь сам решит: использовать голый указатель или тут же обернуть его своим смартом. Рассмотрите такой пример:
Код:
QList<QSharedPointer<NonQObject> >
В данном случае sizeof(QSharedPointer<NonQObject>) > sizeof(void*), что вызовет инстанцирование QList с менее эффективной реализацией по сравнению с голым указателем. Вы автор библиотеки и не знаете, как будет использоваться объект. Фактически, вы навязали QSharedPointer и менее эффективный QList, последнее запросто может не понравиться пользователю.

Цитировать
Это не избыточно и тем более не снижает читабельность.
Ну как вам сказать... тогда зачем scoped pointer нужен вообще?

Цитировать
А вот ваш костылец заметно длинней и такие чудеса нужно вставлять везде, где может возникнуть исключение. Вот она избыточность и нечитаемость.
Очевидно, такой код писать неприятно. RAII - вот где элегантность. Но это необходимо, если предоставлять голый указатель.

Цитировать
И это только создание объекта, а если мы проследим все костыли и исправления в программе, что бы его корректно удалять, то лишних строк будет еще больше. А с shared_ptr ничего менять не придется.
Я не говорю, что дальше должен использоваться голый указатель. Моя мысль, иллюстрированная примером, следующая: если на стыке (библиотека - программа) интерфейс в смартах, то это может быть не всегда удобно! Мне просто было интересно ваше мнение.
Записан
Old
Джедай : наставник для всех
*******
Offline Offline

Сообщений: 4350



Просмотр профиля
« Ответ #23 : Февраль 15, 2014, 22:14 »
Цитата: Akon от Февраль 15, 2014,  21:43
если на стыке (библиотека - программа) интерфейс в смартах, то это может быть не всегда удобно! Мне просто было интересно ваше мнение.
Мое мнение, что смарты нужно запихивать везде и в библиотеки в первую очередь, что бы заставлять ими пользоваться. Улыбающийся
Я уверен, что большинство их противников просто никогда не имели с ними дела (или имели в тестовых примерчиках), и не очень представляют что это такое.
Записан
kambala
Джедай : наставник для всех
*******
Offline Offline

Сообщений: 4747



Просмотр профиля WWW
« Ответ #24 : Февраль 15, 2014, 22:27 »
2Old: можешь посоветовать хороший материал по умным указателям? максимум, что я использовал — это QPointer.
Записан
Изучением C++ вымощена дорога в Qt.

UTF-8 has been around since 1993 and Unicode 2.0 since 1996; if you have created any 8-bit character content since 1996 in anything other than UTF-8, then I hate you. © Matt Gallagher
Old
Джедай : наставник для всех
*******
Offline Offline

Сообщений: 4350



Просмотр профиля
« Ответ #25 : Февраль 15, 2014, 22:50 »
Цитата: kambala от Февраль 15, 2014,  22:27
2Old: можешь посоветовать хороший материал по умным указателям? максимум, что я использовал — это QPointer.
К сожалению ничего фундаментального посоветовать не могу, давно я не интересовался этим вопросом.
Поэтому, с удовольствием пообщаюсь по этому вопросу и расскажу свое видение использования этих удобных инструментов. Улыбающийся
Записан
Akon
Гость
« Ответ #26 : Февраль 15, 2014, 23:01 »
Old: Я не противник смартов, наоборот. Мне просто хотелось иметь дискуссию и в более глубоком русле, и сос своей стороны я показал, что смарты в некоторых аспектах не безупречны.
Записан
Old
Джедай : наставник для всех
*******
Offline Offline

Сообщений: 4350



Просмотр профиля
« Ответ #27 : Февраль 15, 2014, 23:05 »
Цитата: Akon от Февраль 15, 2014,  23:01
Old: Я не противник смартов, наоборот. Мне просто хотелось иметь дискуссию и в более глубоком русле, и сос своей стороны я показал, что смарты в некоторых аспектах не безупречны.
Как и все у нас. Подмигивающий
Идеальных контейнеров тоже не существует и мы вынуждены все время выбирать чем пожертвовать что бы что-то получить. Все имеет свою цены и смарты в том числе.
И для меня они дают значительно больше, чем забирают. )
Записан
kambala
Джедай : наставник для всех
*******
Offline Offline

Сообщений: 4747



Просмотр профиля WWW
« Ответ #28 : Февраль 15, 2014, 23:06 »
Цитата: Old от Февраль 15, 2014,  22:50
Цитата: kambala от Февраль 15, 2014,  22:27
2Old: можешь посоветовать хороший материал по умным указателям? максимум, что я использовал — это QPointer.
К сожалению ничего фундаментального посоветовать не могу, давно я не интересовался этим вопросом.
Поэтому, с удовольствием пообщаюсь по этому вопросу и расскажу свое видение использования этих удобных инструментов. Улыбающийся
так может уж лучше тогда тему-статью-туториал? Подмигивающий думаю не мне одному будет интересно.
Записан
Изучением C++ вымощена дорога в Qt.

UTF-8 has been around since 1993 and Unicode 2.0 since 1996; if you have created any 8-bit character content since 1996 in anything other than UTF-8, then I hate you. © Matt Gallagher
Akon
Гость
« Ответ #29 : Февраль 15, 2014, 23:10 »
Имхо, необходимость использовать смарты должна придти из опыта - если бы не было распространенных реализаций, вы бы сделали свою. Для глубокого изучения чешите сорцы.
Записан
Страниц: 1 [2] 3 4   Вверх
  Печать  
« предыдущая тема следующая тема »
 
Перейти в:  


Страница сгенерирована за 0.058 секунд. Запросов: 22.