Re: Problem with CreateMutex() and CloseHandle() - C / C++ / MFC Discussion Boards

santhoshv8419-Dec-07 19:36

19-Dec-07 19:36

Hi Friends,

I have a MHTML file, i have to Split it in to Number of files,
First from the Header of the file as header.
Then we have a String in as "PAGE-BREAK-AFTER". I have to read this file and find this string, till that string i have to make a new file1, like that there are lot of "PAGE-BREAK-AFTER" String, Could any one tell me.

How to do this, some logic.

Thanks and Regards.
SANTHOSH V

Re: How to Split a txt file.

CPallini19-Dec-07 21:16

CPallini

19-Dec-07 21:16

That can be easily done:

(0) if you have reached endo of file exit
(1) Read a line and append it to your buffer
(2) Search the line for the "PAGE-BREAK-AFTER" separator
(3) if the separator is not found go to point (0)
(4) write buffer content to new file
(5) erase buffer
(6) go to point (0)

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.

error with a /clr option

tasumisra19-Dec-07 18:56

tasumisra

19-Dec-07 18:56

Dear all,

i am writing some simple program that contains few classes and a main..

once i am compiling that application i am getting following error

"Error 8 error C3381: 'sub' : assembly access specifiers are only available in code compiled with a /clr option c:\documents and settings\my documents\visual studio 2005\projects\trtrtr\trtrtr\astreo.cpp 16
"

what wrong is going on here..

please help me..
T@SU

Re: error with a /clr option

CPallini19-Dec-07 21:08

CPallini

19-Dec-07 21:08

tasumisra wrote:
'sub'

Post the releted code.
Smile | :)

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.

Re: error with a /clr option

Mark Salsbery20-Dec-07 5:49

Mark Salsbery

20-Dec-07 5:49

Did you try to specify assembly access on a class,
something like

public class MyClass
{
...

You can only use the "public" access modifier in CLR code.

The above code should be

class MyClass
{
...

Mark

Mark Salsbery
Microsoft MVP - Visual C++
Java | [Coffee]

Solution

Tausifahamad Tamboli5-Dec-10 17:45

Tausifahamad Tamboli

5-Dec-10 17:45

Actually you may not include clr inside your coding while building your project.Thats why you just try to go with project having clr option enabled while making new project.

Problem with CreateMutex() and CloseHandle()

ashishbhatt19-Dec-07 18:18

ashishbhatt

19-Dec-07 18:18

Hi All,

I have little much confusion about CreateMutex() and CloseHandle(). I am using these functions to allow not to create multiple instances of the my application.

For that I have written below code...
Here juct consider that InstanceHandler is one Class and m_mutex is of the HANDLE type.

<br />
InstanceHandler::InstanceHandler():m_mutex(NULL)<br />
{<br />
}<br />
<br />
InstanceHandler::~InstanceHandler()<br />
{<br />
	// make sure mutex is cleared as we exit<br />
	Release();<br />
}<br />
<br />
BOOL RTMPInstanceHandler::Init()<br />
{<br />
	// Create the named mutex<br />
	m_mutex = CreateMutex(NULL,FALSE,LPCWSTR(mutexname));<br />
	if(m_mutex == NULL)<br />
        {<br />
	        return FALSE;<br />
        }<br />
<br />
	// Check that the mutex didn't already exist<br />
	if (GetLastError() == ERROR_ALREADY_EXISTS)<br />
	{<br />
		return FALSE;<br />
	}<br />
	return TRUE;<br />
}<br />
<br />
//<br />
// allow mutex to be explicitely cleared<br />
//<br />
DWORD RTMPInstanceHandler::Release()<br />
{<br />
	try <br />
	{ <br />
		CloseHandle(m_mutex);<br />
	}<br />
	catch (...)<br />
	{<br />
		//<br />
		// Release() should be called once from the user code,<br />
		// and a second time by the object's destructor. <br />
		// the second call will cause an exception.<br />
		// we can just ignore it.<br />
		//<br />
	}<br />
<br />
	return GetLastError();<br />
}<br />

Here When I call these function like below....

<br />
<br />
RTMPInstanceHandler instancehan;<br />
        if(!instancehan.Init())<br />
	{<br />
		// We don't allow multiple instances!<br />
		MessageBox(NULL, CString("Another instance of Application is already running!!"), CString("Server Message"), MB_OK);<br />
		instancehan.Release();<br />
	}<br />

And here is my problem,, what happens everything runs complete but when I call Release() function then it executes but does not reomve from the Process list.

And Every time I run application then it shows the message indicating that another Instance is running but CloseHandle() function does not remove process.

If you have any idea then reply me.

Thanks in advance. Rose | [Rose]

Re: Problem with CreateMutex() and CloseHandle()

Sarath C19-Dec-07 19:18

Sarath C

19-Dec-07 19:18

ashishbhatt wrote:

RTMPInstanceHandler instancehan;
if(!instancehan.Init())
{
// We don't allow multiple instances!
MessageBox(NULL, CString("Another instance of Application is already running!!"), CString("Server Message"), MB_OK);
instancehan.Release();
}

Where you have written the above code snippet? CloseHandle call in the release function will decrement handle count/Close the mutex you have created.

After detecting another instance your main thread should return. Probably you should do in WinMain() or InitInstace() of your Application class.

You should use your InstanceHandler class properly. i.e When first instance of the program creates, the MutexShould be created and valid throughout the life of the program.

Search in CP. it contains article on this issue.

-Sarath.
"Great hopes make everything great possible" - Benjamin Franklin

My blog - Sharing My Thoughts, An Article - Understanding Statepattern

Re: Problem with CreateMutex() and CloseHandle()

ashishbhatt19-Dec-07 19:32

ashishbhatt

19-Dec-07 19:32

Thanks for reply.

I have already called this class from the winmain() function. And I hope it executes properly.
From testing my code I have come to the conclusion that the problem is with CloseHandle().because when I run another instance of my appliction then it creates new Mutex again (means that in Task Manager -> Process Tab ,it shows two instances of my application), but it also generates the error ERROR_ALREADY_EXISTS and then it goes to call the CloseHandle() function and it also executes but it does not close handle(means it does not delete instance from the Task Manager).

Is there ny other idea??

Thanks.

Re: Problem with CreateMutex() and CloseHandle()

Kiran Pinjala19-Dec-07 22:31

Kiran Pinjala

19-Dec-07 22:31

Hai Sarath,
I have a doubt. If i fulfill the same requirement by using CSemaphore as follows...

// CMyApplicationApp.cpp
...
CSemaphore csLock(1,1,"MYSTRING",NULL);
...
BOOL CMyApplication::InitInstance()
{
 ...
 if(csLock.Lock(100))
 {
   AfxMessageBox("One Instance is already running");
   return false;
 }
 ...
}
...
int CMyApplication::ExitInstance()
{
  csLock.Unlock();
  ...
}

Is there anything wrong doing like this?
I would be thankful if you give me suggessions on using this.

Thank you

Saadhinchaali

Re: Problem with CreateMutex() and CloseHandle()

Sarath C19-Dec-07 23:03

Sarath C

19-Dec-07 23:03

Hello,

It's a good idea to make use of available class. There's nothing wrong with the method you have specified. It should work. The idea is only to make use of any named kernel object. CSemaphore is a wrapper which is doing the same operation of CreateSemaphore, WaitForSingleObject and ReleaseSemaphore functions (atleast for the operations you specified).

For the single instance purpose, we only need a mutex. Mutex is really like a binary semaphore (either free/owned state). It's a bad idea to use semaphore as we have mutex (you can also use CMutex instead of win32 APIs ) which is already available. Also you don't have to wait for 100 ms. you can pass 0 to the Lock function which will tests the object's state and returns immediately.

Another thing is advisable to make csLock as the member of App class. Nothing special but just being Object Oriented.

I don't know why Ashish's class is not working as it should work if properly used.

In An MFC App, it can be used as follows

BOOL MyApp::InitInstance()<br />
{<br />
<br />
if( !m_Instance.Init()) // m_instance is a member of app class<br />
        return 0;<br />
....<br />
}

-Sarath.
"Great hopes make everything great possible" - Benjamin Franklin

My blog - Sharing My Thoughts, An Article - Understanding Statepattern

Re: Problem with CreateMutex() and CloseHandle()

Kiran Pinjala19-Dec-07 23:11

Kiran Pinjala

19-Dec-07 23:11

Thank sarath, for your kind reply.
So the CSemaphore should be used whene we want to allow a certain number of processes or threads to allow to access? is't it?

Saadhinchaali

Re: Problem with CreateMutex() and CloseHandle()

Sarath C19-Dec-07 19:41

Sarath C

19-Dec-07 19:41

The following seems to be the problem with the code LPCWSTR(mutexname).

ashishbhatt wrote:
m_mutex = CreateMutex(NULL,FALSE,LPCWSTR(mutexname));

What's the type of mutexname. It's like type casting ASCII string into widestring which may cause unexpected behavior.

Seems this will do. Just try it

LPCTSTR mutexname = _T("MyMutex");
m_mutex = CreateMutex(NULL,FALSE,mutexname);

The following code worked fine for me.

<br />
class RTMPInstanceHandler<br />
{<br />
    HANDLE m_mutex;<br />
public:<br />
    RTMPInstanceHandler():m_mutex(NULL)<br />
    {<br />
    }<br />
<br />
    ~RTMPInstanceHandler()<br />
    {<br />
        // make sure mutex is cleared as we exit<br />
        Release();<br />
    }<br />
<br />
    BOOL Init()<br />
    {<br />
        // Create the named mutex<br />
        m_mutex = CreateMutex(NULL,FALSE,_T("mutexname"));<br />
        if(m_mutex == NULL)<br />
        {<br />
            return FALSE;<br />
        }<br />
<br />
        // Check that the mutex didn't already exist<br />
        if (GetLastError() == ERROR_ALREADY_EXISTS)<br />
        {<br />
            return FALSE;<br />
        }<br />
        return TRUE;<br />
    }<br />
<br />
    //<br />
    // allow mutex to be explicitely cleared<br />
    //<br />
    DWORD Release()<br />
    {<br />
        try<br />
        {<br />
            CloseHandle(m_mutex);<br />
        }<br />
        catch (...)<br />
        {<br />
            //<br />
            // Release() should be called once from the user code,<br />
            // and a second time by the object's destructor.<br />
            // the second call will cause an exception.<br />
            // we can just ignore it.<br />
            //<br />
        }<br />
<br />
        return GetLastError();<br />
    }<br />
};<br />
<br />
int _tmain(int argc, TCHAR* argv[], TCHAR* envp[])<br />
{<br />
    int nRetCode = 0;<br />
<br />
<br />
    RTMPInstanceHandler inst;<br />
    if ( !inst.Init())<br />
    {<br />
        cout<<"instance already exists";<br />
        return nRetCode;<br />
    }<br />
<br />
    cin >> nRetCode;<br />
    return nRetCode;<br />
}<br />

-Sarath.
"Great hopes make everything great possible" - Benjamin Franklin

My blog - Sharing My Thoughts, An Article - Understanding Statepattern

Re: Problem with CreateMutex() and CloseHandle()

ashishbhatt19-Dec-07 20:56

ashishbhatt

19-Dec-07 20:56

Thank you very much for taking that too much interest in my problem.

I used your suggetion as you told to do but the same problem exists.

Thanks.

Re: Problem with CreateMutex() and CloseHandle()

Member 75496020-Dec-07 3:37

Member 754960

20-Dec-07 3:37

"And Every time I run application then it shows the message indicating that another Instance is running but CloseHandle() function does not remove process."

I think this is the source of the problem. The CloseHandle API doesn't control the running process. It is used to tell the running process that a previous instance exists and that it should not run. It is not clear from the code snippet that application is terminated when the first instance of the application is detected.

For a MFC CWinApp this text would occur in the InitInstance member function,

BOOL CMyApp::InitInstance()
{
	m_hMutex = ::CreateMutex(0, FALSE, "TestApp");
	if (::GetLastError() == ERROR_ALREADY_EXISTS) {
		TRACE("TestApp: Already running");
		// explicitly close the mutex handle
		::CloseHandle(m_hMutex);
		
		return FALSE;	// do not run second time
	}
}

A return value of FALSE from CMyApp::InitInstance will terminate the second instance of the program. The important point is that the Mutex is a resource used by the process, not the process itself. When you detect the ERROR_ALREADY_EXISTS condition you must terminate the application yourself. You must close the handle because CreateMutex returned a valid handle.

Re: Problem with CreateMutex() and CloseHandle()

ashishbhatt24-Dec-07 18:18

ashishbhatt

24-Dec-07 18:18

Thanks to you.

Sorry For the very late Reply on this htread becaz I do not want to create new thread for the same problem.

As you told in your post to test the code you show ,, I tested it and works fine.But i want to know some more things,, As you told that I have to terminate my application myself but how can I do that??? D'Oh! | :doh:

My problem is that when I run another instance of application then from my code it shows the message that another instance is running and I also close the Mutex Handle,But it creates new entry into the Process list(in TaskManager->Process tab).So how can I remove this entry when one is already running?? D'Oh! | :doh:

Ashish Bhatt,
System Developer,
Avinashi System Pvt. Ltd.

Re: Problem with CreateMutex() and CloseHandle()

Member 75496026-Dec-07 4:51

Member 754960

26-Dec-07 4:51

It is not clear to me which instance of the process you are trying to terminate. I have been assuming you want to terminate the second instance. I'll address that.

First what are the results we are looking for? The test program I use is the MFC program generated by MSVC App Wizard. The produces a program that will run two instances. When you do the Windows task manager will show both instances in the process list. To make the test results easy to see, sort the Task Manager process list by name in order to see the two processes listed together. If you then terminate one instance it will be removed from the process list after it has terminated. This is the result we are seeking. What we want is a way for the second instance of the program to detect the first instance of the program. When the first instance is detected the second program instance should terminate itself, just as if we had done it from the application. As before, the task manager process list will reflect that the second instance of the program terminated and remove it from the process list, leaving just the first instance running.

The way we detect the second instance of the program is to use a mutex object. Each instance of the program tries to create the mutex. This is doen when the application starts during InitInstance().

BOOL CMfctestApp::InitInstance()
{
	//...

	m_hMutex = ::CreateMutex(0, FALSE, "TestApp");
	if (::GetLastError() == ERROR_ALREADY_EXISTS)
	{
		TRACE("TestApp: Already running");
		// explicitly close the mutex handle
		::CloseHandle(m_hMutex), m_hMutex = 0;
		return FALSE;	// do not run second time
	}
	
	// ...
}

Note the use comma sequence operator and the explicit assignment of the mutex handle. A second instance of the application will get a valid handle to the existing mutex. Once we have closed the handle we do not want to use it again. We declare the m_hMutex handle as a private member variable in the application class,

class CMfctestApp : public CWinApp
{
	// ...
private:
	HANDLE m_hMutex;
};

Finally, we insure that the first instance of the application will properly release the mutex when the application terminates. This is done in ExitInstance()

int CMfctestApp::ExitInstance() 
{
	if (m_hMutex)
		::CloseHandle(m_hMutex);
	
	return CWinApp::ExitInstance();
}

The first instance of the program will have a valid, non null mutex handle and will close it. All other instances will have handle of 0.

If now run the program outside of the debugger and look at the Task Manager process list and you will see the first instance running. In the MSVC IDE place a breakpoint in the InitInstance member function and run the program under the debugger. When the breakpoint is hit look at the process list in the Task Manager and you will see both instances running. Let the second instance of the program, in the debugger, continue and you should see the process get removed from the task list.

Re: Problem with CreateMutex() and CloseHandle()

DevelopmentNoob16-Dec-09 19:56

DevelopmentNoob

16-Dec-09 19:56

Hi, i follow your way to create the program, so far is okay.
but i try to add the code below in

ExitInstance()
{
	if(m_hMutex)
	{
		::CloseHandle(m_hMutex);
                ReleaseMutex(m_hMutex);
	}
       //here suppose the program will not run to following code.
       //but it did now.ReleaseMutex should release the m_hMutex already right?
	if(m_hMutex)
	{
		::CloseHandle(m_hMutex);
                ReleaseMutex(m_hMutex);
	}
		
	return CWinApp::ExitInstance();

Re: Problem with CreateMutex() and CloseHandle()

NehaMishra286846-Dec-10 19:31

NehaMishra28684

6-Dec-10 19:31

The CreateMutex does not work on Win 7 32 bit. Even if the application already exists it returns a success..

Any thought?

Thanks..

Drag Drop

john563219-Dec-07 18:01

john5632

19-Dec-07 18:01

Hi Experts,

Hi have 2 view(both ListView) in my application. One conataind the full path of files. I want to drag drop a file in other ListView from first ListView.

How can I do this ?

Thanx in advance.

Re: Drag Drop

Hamid_RT19-Dec-07 18:17

Hamid_RT

19-Dec-07 18:17

Did you search on the codeproject?

Help Needed for getting record from database

Y_Kaushik19-Dec-07 17:53

Y_Kaushik

19-Dec-07 17:53

Hi..all's
I am making a program. I need to fetch record from Oracle database and
print these record to .txt file . I write a program for this . Although
i fetch record from database and store these record to result set .

Now i can exteract these record from object of CRecordset class and how can i write these record to .txt file although . I have create file successfully. My code is written below.

CDatabase *Dbase = new CDatabase();
CRecordset rs;

int count=0;
if(!Dbase->Open("aixmmapping_72",FALSE,FALSE,"ODBC;DSN=aixmmapping_72;UID=aixmmapping_72;PWD=aixmmapping_72"))
{
MessageBox(" Can Not Connect With Database");
}else
{
rs.Open(CRecordset::dynaset,_T("Select * from vor"));
while(!rs.IsEOF())
{
rs.MoveNext();
}
CFile obj;
obj.Open( "D:\\Yogi\\Test\\abc.txt", CFile::modeCreate|CFile:: modeReadWrite);

char TextData[800];
strcpy(TextData, "rs");
obj.Write(TextData,800);
MessageBox("File Write Successfully");
rs.Close();
}

Please help me

Regard's
Kaushik

Re: Help Needed for getting record from database

Roger Broomfield19-Dec-07 18:57

Roger Broomfield

19-Dec-07 18:57

Refer CRecordset::GetFieldValue[^] and pay particular attention to the Note in the remarks section, which mentions that you should NOT have the ODBC Cursor Library loaded.

Re: Help Needed for getting record from database

Y_Kaushik19-Dec-07 19:22

Y_Kaushik

19-Dec-07 19:22

First of all Thanks for reply.....

I can get record from database and can print it to .txt file.
but all the record print in a single line.

Can you told me how can make formatting of file so that each and every row print in a saprate line.

Again Thanks for help

Regard's
Kaushik

Re: Help Needed for getting record from database

Roger Broomfield19-Dec-07 19:41

Roger Broomfield

19-Dec-07 19:41

Here is what I get from the code you posted:
Open database
Open a recordset of all fields of table "vor"
Walk through all records in the recordset, doing nothing with each record
Open a file
Write the single string "rs" to the file
close the file
close the recordset

the MSDN article I linked includes a piece of sample code that:
walks through all the records of the recordset
walks each of the fields of each record

what I think you need to do is:
figure out how to write each varValue to the file
include a write to file of a cariage-return line-feed pair before the rs.MoveNext()

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