Solved

Accessing  data from two threads

Posted on 2004-09-13
6
177 Views
Last Modified: 2013-11-20
Hello,

I am writing an application which uses two threads.
I have a list<int> variable in my Document class.
THREAD 1
In this thread, I am filling a list from back i.e pDoc->list.insert(list.begin(),data);
THREAD 2
In this thread, I am extracting each element from top of the list, and displaying it on the UserInterface i.e
data = pDoc->list.pop_front();
Display(data);

THREAD1 cotinously reads and adds data at the bottom of the list while THREAD 2 gets  data from the top of the list and displays it.

Can I directly access the list memeber available in my document class as above in the two threads.
When I tried the above, it worked fine when the list had limited number say 10 to 30 records, but as the size of list was growing to say 10,000 records the program was crashing.

Can anyone please help me to solve this ?

Thanks
Madhavi
0
Comment
Question by:j_madhavi
6 Comments
 
LVL 15

Accepted Solution

by:
lakshman_ce earned 30 total points
ID: 12052897
list::push_back
Adds an element to the end of a list.

list::pop_front
Deletes the element at the beginning of a list.

Plz refer to
http://msdn.microsoft.com/library/default.asp?url=/library/en-us/vcstdlib/html/vclrflist_members.asp

As you are using threads, it is advisable you synchronize between these 2 threads. If the first thread is adding element #10 in the list and the same is being accessed by the second thread, this may lead to crash.You can refer to
http://msdn.microsoft.com/library/default.asp?url=/library/en-us/vccore98/HTML/_core_multithreading.3a_.how_to_use_the_synchronization_classes.asp
0
 
LVL 1

Expert Comment

by:Ilango
ID: 12071822
Fine.. To be specific have a critical section included in your document object and lock it before you make a insertion and release once you are done. The same should be done for reading also. From your explanation, I can say you need a single Critical session and not seperate read and write locks.

If needed use the thread safe InterlockIncrement/InterlockedDecrement for any index based operation.

Thread synchronization is closely associated with the implementation requirement and you have to decide on how better your data will be manipulated.

Cheers !
0
 

Expert Comment

by:l3st4rd
ID: 12091571
to give an example:
first declare a critical section object in your header file:

CRITICAL_SECTION csListAccess;

now initialize the critical section BEFORE you start the threads:

    ::InitializeCriticalSection(&csListAccess);

the new thread code will look like this:
THREAD1:
    ::EnterCriticalSection(&csListAccess);
    pDoc->list.insert(list.begin(),data);
    ::LeaveCriticalSection(&csListAccess);

THREAD2:
    ::EnterCriticalSection(&csListAccess);
    data = pDoc->list.pop_front();
    ::LeaveCriticalSection(&csListAccess);

and don't forget to delete the critical section again (maybe in the destructor):
    ::DeleteCriticalSection(&csListAccess);

of course u need to place i.e. a pointer to the critical section into the thread parameter so that u'll get access to the critical section from within the thread functions.
0
 

Expert Comment

by:curero
ID: 12483157
A simpler (but not the finest) MFC solution:

    CMutex mtxList;

 (you can make it global for easy access)

 for each access to the list:
   mtxList.Lock();
   //access the list
   mtxList.Unlock

 When the application is closed, the kernel will clean it (no need to destroy it). You can add a time-out (just to be sure) in the 'lock' call.
 
0

Featured Post

Problems using Powershell and Active Directory?

Managing Active Directory does not always have to be complicated.  If you are spending more time trying instead of doing, then it's time to look at something else. For nearly 20 years, AD admins around the world have used one tool for day-to-day AD management: Hyena. Discover why

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

Suggested Solutions

Introduction: Displaying information on the statusbar.   Continuing from the third article about sudoku.   Open the project in visual studio. Status bar – let’s display the timestamp there.  We need to get the timestamp from the document s…
Introduction: Finishing the grid – keyboard support for arrow keys to manoeuvre, entering the numbers.  The PreTranslateMessage function is to be used to intercept and respond to keyboard events. Continuing from the fourth article about sudoku. …
This video will show you how to get GIT to work in Eclipse.   It will walk you through how to install the EGit plugin in eclipse and how to checkout an existing repository.
Although Jacob Bernoulli (1654-1705) has been credited as the creator of "Binomial Distribution Table", Gottfried Leibniz (1646-1716) did his dissertation on the subject in 1666; Leibniz you may recall is the co-inventor of "Calculus" and beat Isaac…

821 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question