Выделение памяти (new)

[ImmortalAlexSan]

3 Пепел Феникса, инициализировать вот так:

LPTREENODE pTreeNode;
TREEROOT treeroot;
TREECHIELD treechield;
pTreeNode = new TREENODE();
pTreeNode->pTreeRoot = new TREEROOT(); //или это уже не надо?
pTreeNode->pTreeChield = new TREECHIELD(); //или это уже не надо?
pTreeNode->pTreeChield = &treechield;
pTreeNode->pTreeRoot = &treeroot;

2 Я не знаю, как это назвать, наверное боязнь, если я включу ещё в свой код std, то у меня ехе увеличится в размере.
4 Я пробовал от переменной. Вываливалась ошибка кучи.
1 да нет, не собираюсь в цикле выделять, просто если разницы в выделении при одном применении не было, я и решал в цикл загнать и проверить, результат в три раза выше, а у нас на компах на работе скажется это очень сильно.
Сtrl, у меня дерево - стандартный контрол. Просто к lParam - полю его фйтемов я привязываю указатели на свою структуры. А почему я с typedef намудрил? Майкрософт так использует. В данном случае это не нужно, наверное? Просто хочется, чтоб все по стандартам, всё крамиво было... Какая-то мания началась.
Вы уж все извените, ребят, что я так занудствую...

[Пепел Феникса]

1 да нет, не собираюсь в цикле выделять, просто если разницы в выделении при одном применении не было, я и решал в цикл загнать и проверить, результат в три раза выше, а у нас на компах на работе скажется это очень сильно.

медленно само выделение памяти, а не работа с динамически выделенной памятью.

2 Я не знаю, как это назвать, наверное боязнь, если я включу ещё в свой код std, то у меня ехе увеличится в размере.

а лучше проверить, зато сколько проблем то отпадет(если все хорошо сделаете)
да и лучше чуток увеличить размер, чем городить кучу велосипедов с квадратными колесами.

pTreeNode = new TREENODE();
pTreeNode->pTreeRoot = new TREEROOT(); //или это уже не надо?
pTreeNode->pTreeChield = new TREECHIELD(); //или это уже не надо?

только это(и это все кстати для того варианта структуры где указатели на структуры были)
а указатель на стековый элемент сохранять...очень дурная мысль.

4 Я пробовал от переменной. Вываливалась ошибка кучи.

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

[ImmortalAlexSan]

netrino. спасибо) Мне нужно дерево элементов, куда я могу добавлять элементы и удалять, каждый элемент должен иметь ряд полей типа LPWSTR к примеру. ользователь должен менять содержание каждого поля каждого элемента и иметь возможность посмотреть их при надобности. Может быть вы подскажите что-нибудь менее геморойное. Я использую TreeView Control. Но айтемы данного дерева могут хранить только ограниченное число полей - имя, ссылки на изображения активных и неактивных элементов и ещё парочку чего-то, - не помню. так же у элемента дерева есть поле lParam типа LPARAM, которое может содержать данные программиста. Так вот, чтобы свой класс дерева не писать, я решил написать только структуру, на которую будет ссылаться lParam каждого элемента. Ну и в файл потом записать будет не проблимотично, считав все элементы дерева, и восстановить смогу тоже.
Пепел Феникса, спасибо большое, теперь я разграничил понятие инициализации! Убедили в изучении vector'a. Просто так навскидку, какова вероятность, даже если я все правильно сделаю, вываливания за кучу, при использовании vector'a и как это скажется на быстроте работы по сравнению с вариантом, где все структуры мои определны статически?

[netrino]

Ну тогда да, можно делать как вы и делаете. Не знаю структуру вашей программы, обязательно нужны структуры TREEROOT и TREECHILD?

Просто так навскидку, какова вероятность, даже если я все правильно сделаю, вываливания за кучу, при использовании vector'a и как это скажется на быстроте работы по сравнению с вариантом, где все структуры мои определны статически?

Что значит вываливание за кучу? Компилятор обычно в состоянии оптимизировать работу с вектором так, словно вы работаете с обычным массивом.

[ImmortalAlexSan]

netrino, В проге планируется несколько главных корней и все остальные узлы - второстипенны. Главные должны иметь поля отличные от полей второстипенных (там 4 поля, там 8). У меня структура TREECHIELD называется (ошибся и потащил на протяжении всего проекта, поздно уже исправлять). Я просто решил сделать два указателя на эти структуры в одной главной, с мыслью о том, что если одна из 2-х не будет иметь указателя, то и память она занимать не будет, это верный подход? Или можете подсказать идею получше? Спасибо насчет vector'a.
И еще, обязательно после применения delete присваивать указателю NULL?

[netrino]

Сообщение от ImmortalAlexSan

netrino, В проге планируется несколько главных корней и все остальные узлы - второстипенны. Главные должны иметь поля отличные от полей второстипенных (там 4 поля, там 8). У меня структура TREECHIELD называется (ошибся и потащил на протяжении всего проекта, поздно уже исправлять). Я просто решил сделать два указателя на эти структуры в одной главной, с мыслью о том, что если одна из 2-х не будет иметь указателя, то и память она занимать не будет, это верный подход? Или можете подсказать идею получше? Спасибо насчет vector'a.
И еще, обязательно после применения delete присваивать указателю NULL?

Структура TREENODE всегда будет занимать 8 байт на 32-х битных машинах, не смотря на то, что один из указателей (или оба) будут неинициализированы. Если вы ничего не присваиваете полю, это не значит, что его нет) Есть и даже с каким-то значением.
В целом, можно сделать как вы задумали, используя union, но стоит ли? Ведь если вы как-то собираетесь различать, какую же из структур вы хотите использовать (то есть существуют разные процедуры для работы с TREECHIELD и TREEROOT), то может передавать в них указатель именно на них, а не на рудиментарную TREENODE? Но это конечно, если я верно всё понял, для более точных подсказок нужно смотреть код программы.
Насчёт delete: да, обязательно.

[Пепел Феникса]

И еще, обязательно после применения delete присваивать указателю NULL?

нет, но лучше стоит, а то как определите что указатель пустой?

Я просто решил сделать два указателя на эти структуры в одной главной, с мыслью о том, что если одна из 2-х не будет иметь указателя, то и память она занимать не будет, это верный подход?

если вы веделите память то займет, но помните что сам указатель то занимает 4/8 байт, вне зависимости от того пустой или нет

[ImmortalAlexSan]

А я создал две функции - запись в поле и чтение из поля. Т.е. не зависит от того с какой структурой я работаю. Я в этих функциях только указываю указатель на память и всё. А главный или побочный узел идентифицирую путем ввода переменной типа bool, если 1 -то главный, 0 -побочный. Вот так делаю:

void SetNodeField(WCHAR* pSource, WCHAR* pTaker)
{
	WCHAR* PtrT = pTaker;
	WCHAR* PtrS = pSource;
	while (*pSource != '\0') {
		*pTaker = *pSource;
		pTaker++;
		pSource++;
	};
	*pTaker = '\0';
	pTaker = PtrT;
	pSource = PtrS;
};

Ну а создаю элемент так: MyTree.InsertItem(0 или 1); в зависимости от того что нажал юзер на выпадающей менюшке.
Спасибо, вам ребят большое за терпение и за помощь! =))
P.S. Кстати SetNodeField не косячно написана?)) Вроде работает всё.)

[Сtrl]

А форматирование хорошее у вас.

Уверены? Я бы начал с того, что с помощью табуляции делается исключительно индентация, а не выравнивание.

[netrino]

Сообщение от Сtrl

Уверены? Я бы начал с того, что с помощью табуляции делается исключительно индентация, а не выравнивание.

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