C / C++ / MFC

Re: To Christian Graus & Nish

28-Nov-01 7:21

Hello, it's me again.

I did all your advises but still have nothing changes ...
If it's possible, can you, please look my code, where is mistake?

all best ..
-----------------------------

void CRegListDlg::QueryKey(HWND hDlg, HKEY hKey)
{
CHAR achKey[MAX_PATH];
CHAR achClass[MAX_PATH] = ""; // buffer for class name
DWORD cchClassName = MAX_PATH; // length of class string
DWORD cSubKeys; // number of subkeys
DWORD cbMaxSubKey; // longest subkey size
DWORD cchMaxClass; // longest class string
DWORD cValues; // number of values for key
DWORD cchMaxValue; // longest value name
DWORD cbMaxValueData; // longest value data
DWORD cbSecurityDescriptor; // size of security descriptor
FILETIME ftLastWriteTime; // last write time

DWORD i, j;
DWORD retCode, retValue;

CHAR achValue[MAX_VALUE_NAME];
DWORD cchValue = MAX_VALUE_NAME;
CHAR achBuff[80];

// Get the class name and the value count.
RegQueryInfoKey(hKey, // key handle
achClass, // buffer for class name
&cchClassName, // length of class string
NULL, // reserved
&cSubKeys, // number of subkeys
&cbMaxSubKey, // longest subkey size
&cchMaxClass, // longest class string
&cValues, // number of values for this key
&cchMaxValue, // longest value name
&cbMaxValueData, // longest value data
&cbSecurityDescriptor, // security descriptor
&ftLastWriteTime); // last write time

// SetDlgItemText(hDlg, IDE_CLASS, achClass);
// SetDlgItemInt(hDlg, IDE_CVALUES, cValues, FALSE);

::SendMessage(GetDlgItem(hDlg, m_list),
LB_ADDSTRING, 0, (LONG) "..");

DWORD dwLength = MAX_PATH;

// Enumerate the child keys, until RegEnumKeyEx fails. Then
// get the name of each child key and copy it into the list box.

SetCursor(LoadCursor(NULL, IDC_WAIT));
for (i = 0, retCode = ERROR_SUCCESS;
retCode != ERROR_NO_MORE_ITEMS; i++)
{
retCode = RegEnumKeyEx(hKey,
i,
achKey,
&dwLength,
NULL,
NULL,
NULL,
&ftLastWriteTime);
if (retCode != (DWORD)ERROR_NO_MORE_ITEMS)
{
::SendMessage(GetDlgItem(hDlg, m_list),
LB_ADDSTRING, 0, (LONG) achKey);
}

dwLength = MAX_PATH;
}
SetCursor(LoadCursor (NULL, IDC_ARROW));

// Enumerate the key values.
SetCursor(LoadCursor(NULL, IDC_WAIT));

if (cValues)
{
for (j = 0, retValue = ERROR_SUCCESS;
j < cValues; j++)
{
cchValue = MAX_VALUE_NAME;
achValue[0] = '\0';
retValue = RegEnumValue(hKey, j, achValue,
&cchValue,
NULL,
NULL, // &dwType,
NULL, // &bData,
NULL); // &bcData

if (retValue != (DWORD) ERROR_SUCCESS &&
retValue != ERROR_INSUFFICIENT_BUFFER)
{
wsprintf (achBuff,
"Line:%d 0 based index = %d, retValue = %d, "
"ValueLen = %d",
__LINE__, j, retValue, cchValue);
// MessageBox (m_hWnd, achBuff, "Debug", MB_OK);
}

achBuff[0] = '\0';

// Add each value to a list box.
if (!lstrlen(achValue))
lstrcpy(achValue, "<no name="">");
wsprintf(achBuff, "%d) %s ", j, achValue);
::SendMessage(GetDlgItem(hDlg,m_list2),
LB_ADDSTRING, 0, (LONG) achBuff);
}
}
SetCursor(LoadCursor(NULL, IDC_ARROW));
}

-------------------------------------------------
and it makes this mistake
: error C2664: 'void __thiscall CWnd::GetDlgItem(int,struct HWND__ ** ) const' : cannot convert parameter 1 from 'struct HWND__ *' to 'int'
This conversion requires a reinterpret_cast, a C-style cast or function-style cast

each time when it goes to "SendMessage" ..

Confused | :confused:

==============
www.design.kg

C^hris L^osinger28-Nov-01 7:46

C^hris L^osinger

28-Nov-01 7:46

GetDlgItem doesn't take a handle to a window as its first parameter, it takes a control ID (ID_MY_BUTTON or something).

-c

Smaller Animals Software, Inc.

Re: Who are these guys?

Masaaki Onishi28-Nov-01 9:47

28-Nov-01 9:47

Hello, the codegurus around the world;)

The poster simply misunderstands between CWnd function and Windows API function.

Windows API

HWND GetDlgItem(
  HWND hDlg,       // handle to dialog box
  int nIDDlgItem   // control identifier
);

And, MFC

CWnd* GetDlgItem( int nID ) const;
void CWnd::GetDlgItem( int nID, HWND* phWnd ) const;

Also, if the poster wants to use WinAPI function in MFC,
we need the global resultion ::GetDlgItem(...); Roll eyes | :rolleyes:

Please, don't send me your email about your questions directly.
Have a nice day!
Sonork - 100.10571:vcdeveloper Wink | ;)

-Masaaki Onishi-

Re: Who are these guys?

Karavaev Denis28-Nov-01 17:20

Re: To Christian Graus & Nish

28-Nov-01 17:20

Yah! And as you can see exactly like this I called this function:
::GetDlgItem

Or I did smf wrong?

==============
www.design.kg

Karavaev Denis28-Nov-01 17:23

Re: OK, This is a tricky one!

28-Nov-01 17:23

I found it here:
http://msdn.microsoft.com/library/en-us/sysinfo/regapi_0tq1.asp?frame=true
is it MSDN mistake, that GetDlgItem take a handle to a window as its first parameter?

==============
www.design.kg

Masaaki Onishi28-Nov-01 17:34

I think, I'm an idiot ....

28-Nov-01 17:34

Hello, the codegurus around the world.;)

First of all, you must understand the dif between C and C++ code.

In msdn example, this is written by C code.
In this case, we don't need the global resolution "::".

VOID QueryKey(HWND hDlg, HANDLE hKey)
{
   .....
   //GetDlgItem means the Windows API function.
   SendMessage(GetDlgItem(hDlg, IDL_LISTBOX), 
        LB_ADDSTRING, 0, (LONG) ".."); 
   ......
}

However, if you use this one in MFC,

void CMyDialog::QueryKey(...)
{
   .....
   //In this case, this->GetDlgItem(...) is called.
  // this means CMyDialog's pointer.
  // So, you got the compiler error!
   SendMessage(GetDlgItem(hDlg, IDL_LISTBOX), 
        LB_ADDSTRING, 0, (LONG) ".."); 
   ......
   //As a result, this is right.
   //
   .....
   SendMessage(::GetDlgItem(hDlg, IDL_LISTBOX), 
        LB_ADDSTRING, 0, (LONG) ".."); 
   ......
   or GetDlgItem(IDL_LISTBOX);

}

Please, don't send me your email about your questions directly.
Have a nice day!
Sonork - 100.10571:vcdeveloper Wink | ;)

-Masaaki Onishi-

Karavaev Denis28-Nov-01 17:47

Re: I think, I'm an idiot ....

28-Nov-01 17:47

Like this:

::SendMessage(::GetDlgItem(m_hWnd, m_list),
LB_ADDSTRING, 0, (LONG) "..");

The result is:
: error C2664: 'GetDlgItem' : cannot convert parameter 2 from 'class CString' to 'int'
No user-defined-conversion operator available that can perform this conversion, or the operator cannot be called

==============
www.design.kg

Masaaki Onishi28-Nov-01 18:44

Re: I think, I'm an idiot ....

28-Nov-01 18:44

Hello, the codegurus around the world.;)

::SendMessage(::GetDlgItem(m_hWnd, m_list),
LB_ADDSTRING, 0, (LONG) "..");

How did you declare m_list? CString m_list?
GetDlgItem has the second parameter as ID of the control.
So, this should take something like IDC_LIST1.
But, are you sure that you add ListBox (ID: IDC_LIST1) on your dialog box?

Simply copying the code never works unless you have enough knowledge
of both MFC and Win32 program in this case. Cool | :cool:

Or, to be sure, you add ListBox on your dialog box.
This works.
If you declare CListBox m_list;

::SendMessage(m_list.m_hWnd, LB_ADDSTRING, 0, (LONG) "..");
or
m_list.AddString ("..");
or
m_list.SendMessage(LB_ADDSTRING, 0, (LONG)"..");

Please, don't send me your email about your questions directly.
Have a nice day!
Sonork - 100.10571:vcdeveloper Wink | ;)

-Masaaki Onishi-

Karavaev Denis28-Nov-01 19:01

Re: I think, I'm an idiot ....

28-Nov-01 19:01

Oh, ÎÃÐÎÌÍÎÅ ÑÏÀÑÈÁÎ!
thats in my language mean THANK YOU VERY MUCH! Masaaki Smile | :)

==============
www.design.kg

Nish Nishant28-Nov-01 18:45

Nish Nishant

28-Nov-01 18:45

"::GetDlgItem(m_hWnd, m_list)" <--- error line

The second parameter should be the control id.
replace m_list with the control id,something like IDLIST01

Nish

Sonork ID 100.9786 voidmain
www.busterboy.org
If you don't find me on CP, I'll be at Bob's HungOut

Re: Ooops, I forgot to say...

Masaaki Onishi29-Nov-01 7:03

CFileDialog and CFile::Open

29-Nov-01 7:03

Hello, the codegurus around the world.;)

Yesterday, I forgot to say that if we use the class wziard... Dead | X|

Simply if we declare CListBox listbox, listbox will never get the right stuff.
If we declare CListBox listbox by the class wizard, DoDataExchange(...) assign
the right stuff to listbox.

Or, we can use Create or CreateEx function like listbox.Create(...)
Dead | X|

Please, don't send me your email about your questions directly.
Have a nice day!
Sonork - 100.10571:vcdeveloper Wink | ;)

-Masaaki Onishi-

28-Nov-01 5:34

RobJones28-Nov-01 5:23

28-Nov-01 5:23

Hello,
In the following code, I open a MP3 and read the last 128 bytes of information (to see if the MP3 has a ID3 tag).. This code works fine most of the time.. I click the open button on my app which calls this function.. it works 95% of the time, but every now and then it gives me an error.. "an unnamed file was not found", the file does exist and its in the same directory as all the previous files that I have opened.. if I move or copy the file to another location and try to open it, it opens fine.. weird! Any ideas why??

CFileDialog MP3(TRUE,NULL,NULL,OFN_HIDEREADONLY,"MP3 Files (*.mp3)|*.mp3|");
	
	if(MP3.DoModal() == IDOK)
	{
		strPath = MP3.GetPathName();
		
		ID3TAG.Open(strPath, CFile::modeRead);
		ID3TAG.Seek(-128,CFile::end);
		ID3TAG.Read(tagData,128); //after this line I do a bunch of other stuff with the data..

Thanks,
Rob

Nish Nishant28-Nov-01 5:26

Nish Nishant

28-Nov-01 5:26

perhaps the file is open by some other process or thread already?

Nish

Sonork ID 100.9786 voidmain
www.busterboy.org
If you don't find me on CP, I'll be at Bob's HungOut

RobJones28-Nov-01 5:38

28-Nov-01 5:38

It shouldn't be open by any other processes. It's on my local machine, I don't have any other apps that are using the file and I don't have any shares open on my PC. I wonder if the strPath string is getting corrupted some how.. I'll do some traces.

If you have any other ideas please let me know.

Thanks!!

Rob

RobJones28-Nov-01 6:35

28-Nov-01 6:35

It appears that the CFile::Seek function is throwing the error.. Any ideas why? It's like it can't find the end of the file. Is there any trouble shooting methods that I can use to determine the cause of the exception?

Thanks again for you help!
Rob

RobJones28-Nov-01 9:19

28-Nov-01 9:19

I don't usually reply to my own msg's but I figured everything out.. so disregard my last few posts..

Thanks,

Rob