problem while freeing a link list within a vector structure [modified] - C / C++ / MFC Discussion Boards

Re: Sort the list of files available in a folder.

Richard MacCutchan21-Jun-11 2:03

Richard MacCutchan

21-Jun-11 2:03

This control[^] should help you get what you want.

The best things in life are not things.

Re: Sort the list of files available in a folder.

lucky_122121-Jun-11 2:10

lucky_1221

21-Jun-11 2:10

i want code for ,two string arrays,if one element in the string array is compared to the another all elements in th enext array,if it is findout discard the element go to the next element.

Re: Sort the list of files available in a folder.

Richard MacCutchan21-Jun-11 2:59

Richard MacCutchan

21-Jun-11 2:59

Sorry, but I don't understand what you are asking for here. If you need to sort a list of strings then either use one of the STL[^] types that provides for sorting, or use the CRT quick sort[^] function. If you are trying to reduce an array to unique entries, then once again one of the STL templates should work for you.

The best things in life are not things.

Re: Sort the list of files available in a folder.

వేంకటనారాయణ(venkatmakam)21-Jun-11 3:05

వేంకటనారాయణ(venkatmakam)

21-Jun-11 3:05

Check this example implementation which sorts CStringArray.

http://support.microsoft.com/kb/120961

http://www.mono-project.com/Main_Page

Question about Getting Main Icon using Findresource()

lek25820-Jun-11 21:51

lek258

20-Jun-11 21:51

When I know the exe path of a file:

hExeModule = LoadLibrary(exe);

...blah..blah..

FindResource(hExeModule, "#N", RT_ICON);

While there are so many icons in the exe file, how can I pass the N parameter as the icon index used for application's representitive main icon which appear on explorer ?

Thanks!!

Re: Question about Getting Main Icon using Findresource()

Richard MacCutchan21-Jun-11 2:01

Richard MacCutchan

21-Jun-11 2:01

If you know the value of N then you should send it as described here[^]. However if you do not know the names then you will need to use the EnumResourceNamesEx()[^] function to get them.

The best things in life are not things.

Re: Question about Getting Main Icon using Findresource()

MicroVirus21-Jun-11 5:50

MicroVirus

21-Jun-11 5:50

First result on Google that I code, CodeProject article[]

Might be interesting to read. I suspect that the first Icon in the file is the one used for the application, but I am not sure.

problem while freeing a link list within a vector structure [modified]

Madan Chauhan20-Jun-11 21:27

Madan Chauhan

20-Jun-11 21:27

Hi all.
I have a link list in a vector structure. It throws exception while freeing first node of the list. This is the sample code-


//////////Declaration//////
typedef struct _TEST_STRUCT
{
	TCHAR _tchTemp[MAX_PATH];
	__int64 i64Temp;
	DWORD dwTemp;
	_TEST_STRUCT *Next;
}TEST_STRUCT;

typedef struct _TEST_VECTORST
{
	int nVal;
	TEST_STRUCT m_ListInVector;

}TEST_VECTORST;

std::vector<TEST_VECTORST> m_StVector; 
////////////////////////////////////////////////////
/////////Fill Structure//////////////
        
	CString csVal = L"";
	csVal         = L"hello";
	for (int i = 0; i < 2; i++)
	{
		memset (&wVectorSt, 0x00, sizeof(TEST_VECTORST));
		ptrList = NULL;
		wVectorSt.nVal = i + 1;
		int j = 0;
		while (j<10)
		{
			if (ptrList == NULL)
			{
				ptrTemp = (TEST_STRUCT*)calloc(1,sizeof(TEST_STRUCT));
				_tcscpy(ptrTemp->_tchTemp, csVal);
				ptrTemp->dwTemp = 10+j;
				ptrTemp->i64Temp = 100+j;
				ptrTemp->Next = NULL;
				ptrCurrent = ptrList = ptrTemp;
			}
			else
			{
				ptrTemp = (TEST_STRUCT*)calloc(1,sizeof(TEST_STRUCT));
				_tcscpy(ptrTemp->_tchTemp, csVal);
				ptrTemp->dwTemp = 10+j;
				ptrTemp->i64Temp = 100+j;
				ptrTemp->Next = NULL;
				ptrCurrent->Next = ptrTemp;
				ptrCurrent       = ptrTemp;

			}
			j ++;
		}
		memcpy(&wVectorSt.m_ListInVector, ptrList, sizeof(TEST_STRUCT));
		m_StVector.push_back (wVectorSt);
	}
	//////Free m_StVector.m_ListInVector////
	for (int i = 0; i<m_StVector.size(); i++)
	{
		ptrList = m_StVector[i].m_ListInVector.Next;
		while (ptrList != NULL)
		{
			ptrTemp = ptrList;
			ptrList = ptrList->Next;
			free(ptrTemp);
			ptrTemp = NULL;
		}
	}  
All the nodes become free but one that is very first  node in the list does not free.
if i take  start to free from first node like- ptrList = &m_StVector[i].m_ListInVector;

then it throws exception immidiately and does not free any node...

Please give me some idea...


Thanks

modified on Tuesday, June 21, 2011 4:17 AM

Re: problem while freeing a link list within a vector structure

CPallini20-Jun-11 22:12

CPallini

20-Jun-11 22:12

Please use the 'code block' button to format properly your code.

If the Lord God Almighty had consulted me before embarking upon the Creation, I would have recommended something simpler.
-- Alfonso the Wise, 13th Century King of Castile.

This is going on my arrogant assumptions. You may have a superb reason why I'm completely wrong.
-- Iain Clarke

[My articles]

Re: problem while freeing a link list within a vector structure

Stefan_Lang21-Jun-11 0:35

Stefan_Lang

21-Jun-11 0:35

You did not dynamically allocate the first member, so you cannot dynamically free it either.

More to the point: when you push_back() a struct object into that vector of yours, the vector will allocate the required memory and copy the struct you passed into that memory location. The vector is thus responsible for the releaese of that storage, not you. Since you didn't allocate the vector on the heap, it will be automatically deallocated when the variable leaves the scope. But if you defined it globally, then you might want to enforce this earlier, by calling

m_StVector.clear()

Re: problem while freeing a link list within a vector structure

Madan Chauhan21-Jun-11 1:07

Madan Chauhan

21-Jun-11 1:07

thanks for the response.
Yes i have declared m_StVector in App class. So i need to clear it.

but even i do m_StVector.clear() then still i see those memory leaks that are due to the first node of the link list...

well I am still stuck how to free those.

Re: problem while freeing a link list within a vector structure

Stefan_Lang21-Jun-11 1:27

Stefan_Lang

21-Jun-11 1:27

I just noticed that of course you need to free the strings within your struct, and that is something your vector will not take care of.

The best way would be to define both a constructor and destructor for your struct, and use new/delete instead of calloc/free. In the destructor you can then just take care of the string:

struct TestStruct {
   TestStruct() : tchTemp(0) {} // constructor initializes string pointer to 0
   ~TestStruct() { delete [] tchTemp; } // destructor frees string if needed
   void set(const TCHAR* stringvalue) {
      delete [] tchTemp; // release previously allocated memory
      if (stringvalue != 0) {
         // create a copy
         tchTemp = new TCHAR[_tcslen(stringvalue)+1];
         _tcscpy(stringvalue, tchTemp);
      }
      else {
         // reset pointer to 0
         tchTemp = 0;
      }
   }
   TCHAR* tchTemp;
};

void test() {
   std::vector<TestStruct> struct_vect;
   // now fill vector and work with it
   // ...
   struct_vect.clear(); // will delete and deconstruct all objects
}

Note that in order to actually deconstruct (i. e. call the constructor of) the TestStruct objects, you must call delete, not free, on any dynamically allocated objects, and thus you must call new to allocate it. Also you must use new to allocate your strings if you want to use delete to free them. I added a set function to simplify the process of assigning this string properly, using new.

Of course, if you don't want to change your struct definition, you can also loop over the element vectory and free() tchTemp from your first nodes manually.

Re: problem while freeing a link list within a vector structure

CPallini21-Jun-11 1:15

CPallini

21-Jun-11 1:15

Why are you using your own list, cannot you use the STL one?
Why are you using C allocation functions?
What is wVectorSt?

Re: problem while freeing a link list within a vector structure

David Crow21-Jun-11 4:04

David Crow

21-Jun-11 4:04

Madan Chauhan wrote:
All the nodes become free but one that is very first node in the list does not free.

How are you verifying this?

Madan Chauhan wrote:
Please give me some idea...

Just a tad bit of cleanup:

void main( void )
{
    for (int i = 0; i < 2; i++)
    {
        TEST_STRUCT *ptrCurrent;
        TEST_VECTORST wVectorSt = {0};
        wVectorSt.nVal = i + 1;
   
        for (int j = 0; j < 10; j++)
        {
            TEST_STRUCT *ptrTemp = (TEST_STRUCT *) calloc(1, sizeof(TEST_STRUCT));
            _tcscpy_s(ptrTemp->_tchTemp, _countof(ptrTemp->_tchTemp), _T(""));
            ptrTemp->dwTemp = 10 + j;
            ptrTemp->i64Temp = 100 + j;
            ptrTemp->Next = NULL;
  
            if (wVectorSt.m_ListInVector == NULL)
                wVectorSt.m_ListInVector = ptrTemp;
            else
                ptrCurrent->Next = ptrTemp;
  
            ptrCurrent = ptrTemp;
        }
  
        m_StVector.push_back(wVectorSt);
    }
  
    for (unsigned int i = 0; i < m_StVector.size(); i++)
    {
        TEST_STRUCT *ptrList = m_StVector[i].m_ListInVector;
        while (ptrList != NULL)
        {
            TEST_STRUCT *ptrTemp = ptrList;
            ptrList = ptrList->Next;
            free(ptrTemp);
        }
    } 
}

"One man's wage rise is another man's price increase." - Harold Wilson

"Fireproof doesn't mean the fire will never come. It means when the fire comes that you will be able to withstand it." - Michael Simmons

"Some people are making such thorough preparation for rainy days that they aren't enjoying today's sunshine." - William Feather

I am using VC++ coding,Can any one help me?

lucky_122120-Jun-11 21:14

lucky_1221

20-Jun-11 21:14

I have the List o f names(if list type of list is CString),How to sort the list of names.Anybody knows please send me the syntax of the coding.

Thanks&Regards,
Lucky.

Re: I am using VC++ coding,Can any one help me?

CPallini20-Jun-11 22:10

CPallini

20-Jun-11 22:10

What kind of container holds the CStrings?
If you are using MFC containers, then have a look at Newcomer's "A guide to sorting"[^].

Re: I am using VC++ coding,Can any one help me?

Cool_Dev20-Jun-11 22:18

Cool_Dev

20-Jun-11 22:18

List classes (std::list or MFC CList etc) are temlate classes and can't have a sort function in a general basis. You have to iterate through the elements in list and implement any of the sorting algorithms as you do with noraml arrays. Since you are dealing with CString, you can use the overloaded < and > operators for comaparing string objects in list.

Converting char array to CSting

VCProgrammer20-Jun-11 1:43

VCProgrammer

20-Jun-11 1:43

Hi all,

I want to convert a char array into CString.

i have tried various ways like

char array[620];
CString str(array,620);
It is giving a compiler error.

Can anybody help me with this?

thanks in advance

Re: Converting char array to CSting

Pete O'Hanlon20-Jun-11 1:54

Pete O'Hanlon

20-Jun-11 1:54

I don't think you need to specify the length in the CString constructor, so you would have

CString str(array);

Forgive your enemies - it messes with their heads

My blog | My articles | MoXAML PowerToys | Mole 2010 - debugging made easier - my favourite utility

Re: Converting char array to CSting [modified]

Albert Holguin20-Jun-11 4:36

Albert Holguin

20-Jun-11 4:36

Shouldn't have to as long as the array contents is null terminated.

modified on Monday, June 20, 2011 10:43 AM

Re: Converting char array to CSting

Pete O'Hanlon20-Jun-11 6:10

Pete O'Hanlon

20-Jun-11 6:10

Good point.

Forgive your enemies - it messes with their heads

My blog | My articles | MoXAML PowerToys | Mole 2010 - debugging made easier - my favourite utility

Re: Converting char array to CSting

CPallini20-Jun-11 1:55

CPallini

20-Jun-11 1:55

Please let we know the error message ('compiler error' is a bit vague). Anyway I guess you are making a UNICODE build hence the ctor CString::CString(const char *, int) is not available. If the character array is zero-terminated (or you can make it zero-terminated and there aren't other '\0' characters inside) then you could use CString::CString(const char *) instead, namely

C++

CString str(array);

Re: Converting char array to CSting

Rajesh R Subramanian20-Jun-11 3:54

Rajesh R Subramanian

20-Jun-11 3:54

VCProgrammer wrote:
char array[620];
CString str(array,620);

Why are you writing that piece of code?

If you want the str object to be initialised with the array contents, you just have to say CString str(array);

Also, if you're talking about a compiler error or so, you should provide the exact details so that others don't have to guess what would that be.

"Real men drive manual transmission" - Rajesh.

[Win32]Add context menu to edit control

Member 296547120-Jun-11 1:24

Member 2965471

20-Jun-11 1:24

How can i add a context menu to an edit control?

I try to handle WM_RBUTTONDOWN in the main window, but don't work if the mouse click is on the edit control.

Re: [Win32]Add context menu to edit control

CPallini20-Jun-11 1:58

CPallini

20-Jun-11 1:58

You have to subclass the Edit Control, see, for instance Create your own controls - the art of subclassing[^].
Smile | :)

Use Ctrl+Left/Right to switch messages, Ctrl+Up/Down to switch threads, Ctrl+Shift+Left/Right to switch pages.