Заполнить выделенную динамическую память нулями ('\0')?
например буфер можно обнулить так:

Можно ли делать так?

Заполниться ли выделенная память нулями?

Это сообщение отредактировал(а) Alca - 21.8.2008, 16:01

Естественно можно

А можно и так:



В случае с new стандарт позволяет записывать так:


Но не все современные компиляторы соблюдают стандарт в этом плане (например, MinGW 3.4.5 тупо не инициализирует массив).

кстати, вопрос немного филогсофский

обычно не рекомендуют смешивать сишный стиль и плюсовой
кто из программистов тогда использует вместо memset std::uninitialized_fill?

std::uninitialized_fill, применяемый к уже существующим объектам, - это типа C++-стиль? 

лучше чем memset

О-о-о, да, мы избавились от необходимости умножения на sizeof(тип элемента массива). Аллилуйя! Мегадостижение. А что мсье будет делать, если тип элементов массива сменится с POD на не POD?

Просто немного не так:

Я про второй свой пример говорил (с new который). Первый, к счастью, работает и в MinGW.

Добавлено через 12 минут и 43 секунды

Кстати, по стандарту, в отличие от


код


гарантированно инициализирует лишь первые два элемента символьного массива.

ну тогда исчезнет необходимость в обнулении, спор то как раз о POD типах

Это сообщение отредактировал(а) Alca - 21.8.2008, 19:16

... и ради достижения корректности программы возникнет необходимость в устранении существующего обнуления, причём компилятор на эту необходимость не укажет. Коли уж наша программа разваливается, стоит нам на шаг отойти от POD-типов, о каком C++-стиле может идти речь?


Я тут никакого спора не вижу. Если некое средство считается более крутым лишь потому, что оно из STL, а не из стандартной библиотеки C, то это не что иное, как детский сад. Над псевдоинтеллектуальными рассуждениями я обычно просто стебусь, а не спорю с ними.

Это сообщение отредактировал(а) UnrealMan - 21.8.2008, 19:37

Этот код может вызвать ошибку Access Violation или Segmentation fault, в зависимости от ОС.
Это типичная ошибка новичка - выход за границы массива.

Это сообщение отредактировал(а) bsa - 21.8.2008, 20:53

немного оффтоп

если не заметил, у меня стоял знак вопроса
а это значит, что я задавал вопрос и ожидал ответа а не ржачки
(кстати нормального ответа, почему все так любят memset и ни разу не используют std::uninitialized_fill)


а поскольку ты над всеми стебешься, кто с тобой не согласен, то... 

и по теме


насколько я вижу в примерной реализации std::uninitialized_fill там вообще не идет речи ни о ПОД ни о каком-то конкретном куске памяти. есть только 2 итератора и значения. что ты имел в виду, когда писал о ПОД? (только, пожалуйста, если можно, ответь серьезно, без клоунады)

Респект и уважуха  

Подозреваю, что об uninitialized_fill мало кто знает, поэтому и не используют. Все-таки это низкоуровневая функция. С другой стороны, она может быть удобна в случае "нестандартного" для С++ выделения памяти (malloc) - не нужно городить огород с размещающими new. Но я бы остерегся от использования этих нестандартных методов без крайней необходимости.

По поводу uninitialized_fill, могу высказать свое имхо
memset работать должен все таки быстрее

думаю, так и есть - в дебаге
в релизе с оптимизацией с указателями должно быть то же самое
хотя - кто знает (пробовать лень  )

Создание массива из n объектов определённого типа и выделение n байт сырой памяти - это разные задачи, и у каждой будет своё наиболее подходящее (хотя бы с точки зрения самодокументируемости кода) решение. В первом случае подходящим решением будет использвание new-expression, std::vector или иного контейнера, если первые два не подходят. Во втором случае подходящим решением будет явный вызов operator new. В первом случае при использовании new-expression для заполнения массива n элементами подходящим средством будет std::fill_n. Во втором случае для размещения n объектов в сырой памяти подходящим средством будет std::uninitialized_fill_n. При следовании такому принципу (каждой задаче - соответствующий метод решения) по коду по крайней мере будет видно, какая задача решается.

unitialized_fill - более общее решение, т.к. работает и для подов, и для не-подов.

Тем более, для типа signed/unsigned char unitialized_fill (по крайней мере в STLPort) превращается в memset
Значит, эффективность она тоже обеспечивает.

Изначальный пример выглядел так:


Если заменить здесь memset на std::uninitialized_fill, а после char заменить на не POD класс, то получишь двойное конструирование объекта в одной и той же области памяти. Затея смешивания относительно высокоуровневого new-expression с низкоуровневым конструированием объекта поверх памяти в качестве замены "смешиванию плюсового и сишного стиля" достойна похвал.

Это сообщение отредактировал(а) UnrealMan - 22.8.2008, 00:27

тогда моя ошибка
я говорил не об этом примере, а вообще
стоило создать новую тему...


в общем, насколько я понял, в тех случаях когда надо, программисты используют memset вместо std::uninitialized_fill в основном потому, что о таком не знают.


а в случае, если не менять memset на std::uninitialized_fill, а сразу менять чар на не ПОД класс?
вроде тогда вообще ж... будет

Обычно таких проблем не возникает. Они решаются на стадии проектирования, и там где нужен char - он нужен. Если в последствии приходится менять тип и замена memset на uninitialized_fill окажется трудоемкой, то проект хренов по определению.

Добавлено через 21 секунду
Все вышесказанное опять же мое ИМХО

А если структура объявленатак:

Как ее обнулить?

Это сообщение отредактировал(а) Alca - 22.8.2008, 13:04

Alca, смотря как ты её создаёшь. Выше было приведено как минимум два характерных способа. Попробуй. 

Это сообщение отредактировал(а) W4FhLF - 22.8.2008, 12:22

Alca, старайся не смешивать С и С++
что значит "обнулить" структуру?
привести к дефолтному состоянию?

тогда тебе поможет доюбавление в структуру функции Reinit(), которая будет вызываться также и в конструкторе.

и структура потихоньку начнет превращаться в класс 

Обнулить (сорри за такие термины) - значит инициализировать выделяемую память '\0'.
Создаю так:

Надо через memset.

Это сообщение отредактировал(а) Alca - 22.8.2008, 13:07

PHOSTENT - Это что? макрос? Что он определяет? Это указатель или структура? POD или не POD? Язык программирования Си или С++?

Плюсы.


Это сообщение отредактировал(а) Alca - 22.8.2008, 13:24

Я вообще предпочтаю обнулять все через макрос ZeroMemory(); в винде.

И как через ZeroMemory()?

Обнулить поля POD-структуры можно не прибегая к библиотечным функциям:

Alca, 
к примеру 


Добавлено через 1 минуту и 36 секунд
так же можно обнулить любую структуру

Надо заполнить структуру:

Alca, согласен но где это ты у меня видел?  

Добавлено через 1 минуту и 48 секунд
есесно что указатель вернет 4 байта. 

Никогда не понимал тех, кто так делает в плюсовых проектах.



Какой вариант короче?

Не досмотрел, сорри... А можно получить размер структуры через указатель?

Это сообщение отредактировал(а) Alca - 22.8.2008, 15:28

но я то беру размер основной струтуры.

Добавлено через 1 минуту и 27 секунд

наврятли, но 100% утверждать не буду

Добавлено через 2 минуты и 43 секунды
UnrealMan, в   твой вариант короче, но для меня мой более наглядней для меня же   

UnrealMan,  имхо но для этого нужно прикрутить boost! А это целая эпопея. По крайней мере у \me так было.

Boost, конечно вещь классная, но тягать её за собой чёта нехо....

Это сообщение отредактировал(а) Alca - 22.8.2008, 15:37

Alca, вы объявили указатель, а память не выделили...

можно и проще:Но надо иметь в виду, что если структура определена так:А затем под нее выделена память так:То sizeof (*p) возвратит то же, что и sizeof(SomeDynamicSizeStruct)

Существует не так уж много объективных причин, чтобы его не прикручивать.
Добавлено @ 15:46

Можно.



Это сообщение отредактировал(а) UnrealMan - 22.8.2008, 15:48

UnrealMan, имхо ты прав. я и спорить не буду. boost это та вещь которую стоит осваивать.  

Я память динамически кажись не выделяю...

Я не объявлял указатель, тока дефайнил.

Это сообщение отредактировал(а) Alca - 22.8.2008, 15:54

Alca, любопытно если переменну pHost объявить глобально - варнинг проподает.

Добавлено @ 15:54
либо как 

  

Это сообщение отредактировал(а) mrbrooks - 22.8.2008, 15:55

Ворнинг улетел!   

Это сообщение отредактировал(а) Alca - 22.8.2008, 16:01

кстати если не ошибаюсь глобальные и статические переменные по умолчанию инициализируются нулем.

так оно и есть

  




PHOSTENT pHost; - это чтоли дефайн?

Ворнинг улетел!  А прога слетела!!!
У меня 2 потока юзают эту функцию...
Дефайнил так:


Это сообщение отредактировал(а) Alca - 22.8.2008, 16:12

пардон за офтопик, ИМХО рано тебе еще потоки использовать  

Надо использовать! Заодно и научюсь.

Данный код содержит ошибку, даже если pHost - глобальная переменная. прежде чем работать со стуктурой, на которую указывает указатель, нужно выделить под нее память.

и не забыть освободить ее когда нужно 

Добавлено через 22 секунды
хотя глобальную иногда можно не освобождать...