C / C++ / MFC

Re: Sending iterator items to other functions from in a for block....

8-Oct-12 21:32

Here is some code from the draw function in my custom control. I am using a vector to contain data concerning portions of the control.

This code works. You will surely agree with me, though, that using the iterator 'x' to identify vector members in called functions is probably a tad on the clunky side. I have had a go at sending pointers to vector members. I gave up with this. I think this might not be the right way to go either. What it is the correct, elegant way to do this?

C++

vector<sliderUnit>::iterator slider;
for ( slider=sliderVector.begin() ; slider != sliderVector.end(); slider++ )
{

    theSliderRect.left = theDrawRect.left+(x-2)*sliderWidth;
    theSliderRect.right = theSliderRect.left + sliderWidth;

    (*slider).m_TextBoxSize = m_TextBoxSize;
    (*slider).sliderRect = (RECT)theSliderRect;

    tempRect=theSliderRect;
    tempRect.right=tempRect.right-4;

    DrawBackground(&theMemDC,tempRect);

    if (m_currentSlider != x-1){
        (*slider).m_bFlashFocus = FALSE;
    }
    else{
        (*slider).m_bFlashFocus = TRUE;
    }

    DrawButtonsByID(&theMemDC, theSliderRect, x-1);
    DrawThumbSliderByID(&theMemDC, theSliderRect, x-1);
    DrawSliderLabelByID(&theMemDC, theSliderRect, x-1);
    x++;
}

modified 9-Oct-12 16:11pm.

Maximilien9-Oct-12 3:53

Maximilien

9-Oct-12 3:53

Huh?

What type is "X"? I assume it is an int (or long) since you add it to a CRect internal value.

In what way does it identify a vector element? it is not clear.

Nihil obstat

Re: Sending iterator items to other functions from in a for block....

Ben Aldhouse9-Oct-12 10:16

Re: Sending iterator items to other functions from in a for block....

9-Oct-12 10:16

Sorry, I should have clearer. The function calls I refer to are at the bottom of that block of code

C++

DrawButtonsByID(&theMemDC, theSliderRect, x-1);
DrawThumbSliderByID(&theMemDC, theSliderRect, x-1);
DrawSliderLabelByID(&theMemDC, theSliderRect, x-1);

The 'x' becomes 'id' in the called function below which dereferences the vector member that is the same as the iterator in that first for loop block I posted. I know this is murky. That's why I want to make nicer.

C++

void CScrollBarEx::DrawThumbSliderByID(CDC* inDC, CRect inDrawRect, short id)
{
	// Override this to draw the thumb slider

	CRect		theThumbSliderRect;
	COLORREF	theThumbSliderColor = GetSysColor(COLOR_3DFACE);
	COLORREF	theThumbSliderTextColor = GetSysColor(COLOR_WINDOWTEXT);

	ASSERT(inDC != NULL);

	if (sliderVector[id].m_bThumbSliderDownFlag && sliderVector[m_currentSlider].m_bTrackThumbSliderFlag)
	{	
		//theThumbSliderColor = sliderVector[m_currentSlider].m_ThumbSliderTrackColor;
		theThumbSliderColor = sliderVector[id].m_ThumbSliderTrackColor;
		theThumbSliderTextColor = m_ThumbSliderTextColor;
	}

...

Stefan_Lang15-Oct-12 22:33

Stefan_Lang

15-Oct-12 22:33

So you're accessing a vector with an index that comes from a variable of unknown type and origin. Not a lot of information to work with. Why do you think this is a problem? Is there a reason why you think x is not the appropriate index to use?

Until you explain what x is and where you got it from, it's hard for us to understand your problem - if there is one. Since the code works, the only suggestion I can make at this point is to rename x to something more ... self-explanatory, and add a few comments to explain what's going on (while you're still able to understand it). That may be helpful if you need to fix it in a year or two.

P.S.: There's one other suggestion I can think of: instead of passing x to those functions, pass a reference to the vector element. While technically that is the same as passing a vactor index, it removes the dependency between those functions and the entire vector.

Re: Sending iterator items to other functions from in a for block....

Ben Aldhouse16-Oct-12 21:35

Re: Sending iterator items to other functions from in a for block....

16-Oct-12 21:35

I have spent some time trying to write code to send a reference to the vector element. I think I might have had trouble extracting this from the iterator. I must have somehow persuaded myself that there was some reason as to why passing a reference to a vector element from an iterator was a bad idea. I was just wondering if anyone knew if there is some sort of trap here - . I would probably be happier doing it this way if it is straightforward and safe. It seems more elegant. I don't really want to get bogged down in making the changes, though, if there is some inherent problem.

The functions in the iterator block use and possibly change the data in the vector element but don't call any further functions that use the vector element. There is only a single thread accessing the vector.

Stefan_Lang16-Oct-12 22:20

Stefan_Lang

16-Oct-12 22:20

You didn't use an actual iterator anywhere in the code that you posted, so I don't see the problem. When a function argument is declared as a reference, you can just pass a vector element, such as sliderVector[id]. If you do have an iterator, then you can just dereference it to get the vector element and pass that instead. If you wish to pass a pointer instead, just use the 'address-of' operator:

C++

void bar_ref(double& element) {
   element *= 2.0;
}
void bar_ptr(double* p_element) {
   *p_element += 3.0;
}
void foo() {
   std::vector<double> myvec(2); // size = 2
   myvec[0] = 3.1415;
   myvec[1] = 2.718;
   for (std::size_t i = 0; i < myvec.size(); ++i) {
      bar_ref(myvec[i]); // passing element by reference
      bar_ptr(&(myvec[i])); // passing element by pointer
   }
   for (auto it = myvec.begin(); it != myvec.end(); ++it) {
      bar_ref(*it); // passing vector element by reference from iterator
      bar_ptr(&(*it)); // passing vector element by pointer from iterator
   }
}

The only possible trap I see is trying to pass an iterator directly when a pointer is expected. An iterator may behave like a pointer, but, while - depending on the implementation - it may actually auto-convert to a valid pointer (I know VS 2003 did that), you shouldn't rely on that. (e. g. VS2010 doesn't accept it)

Re: Sending iterator items to other functions from in a for block....

Ben Aldhouse18-Oct-12 9:16

Re: Sending iterator items to other functions from in a for block....

18-Oct-12 9:16

Thanks for this! I will certainly give this a go and let you know how I get on. It might not be for a few weeks, though...

Ben Aldhouse12-Nov-12 21:47

Re: Sending iterator items to other functions from in a for block....

12-Nov-12 21:47

Thanks very much for showing me how this can be done, Stefan. I have tried your method and it works perfectly.

I have thought some more about what I'm trying to achieve, though, and have come to the conclusion that since the vector array has class scope and that its members don't themselves know, or need to know, whether they are the current item in the 'for' block (where I've got the iterator) - the more elegant solution would, I guess, involve just sending an (integer) identifier for the required element.

Your help is very much appreciated. Your correspondence has helped my thought process a lot.

Cheers,

Ben

Stefan_Lang12-Nov-12 22:38

Stefan_Lang

12-Nov-12 22:38

Glad I could be of help.

As for your conclusion, keep in mind that sending an ID of the element requires you to also pass the container that this ID refers to! Otherwise the function won't know what to do with that ID. Whether that ID is an iterator or index value doesn't matter in this regard - neither have an inherent knowledge of the container they refer to.

If the container is class scope, and you pass an ID to a class member function, that would work. But from a design view, doing so would still be questionable as the function actually shouldn't need to care about the rest of the container - you have introduced an unnecessary dependency. If at some point you realize you want to use that same function for elements not related to this class, you cannot easily do it, due to that dependency.

help

thebebo188-Oct-12 11:12

thebebo18

8-Oct-12 11:12

hi all I wonder how comfigurar the code blocks for borland ide bcc55 and download the library QTK and c + + qt hope your help.

Re: help

Peter_in_27808-Oct-12 13:22

Peter_in_2780

8-Oct-12 13:22

Get a time machine.
Dial up 1996.
Ask again when you get there.

Seriously, Borland C++ 5.5 is so out of date it's not worth thinking about. (But I loved it in its day.) There are much better modern compilers available for free.

Peter

Software rusts. Simon Stephenson, ca 1994. So does this signature. me, 2012

Re: help

Sivaraman Dhamodharan8-Oct-12 20:44

Sivaraman Dhamodharan

8-Oct-12 20:44

Ha Ha. How can you reply like this Laugh | :laugh:

Programming Article

setup error

emanalshboul8-Oct-12 9:58

emanalshboul

8-Oct-12 9:58

i have installed c++ program ,and this program is failed
to install and error message showed{unable to load development environment dll}
note
i have installed dll-files
and i have tried ,the problem is founed.
please help me

Re: setup error

David Crow8-Oct-12 10:32

David Crow

8-Oct-12 10:32

emanalshboul wrote:
...the problem is founed.

You found the problem?

"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

"Show me a community that obeys the Ten Commandments and I'll show you a less crowded prison system." - Anonymous

Re: setup error

Richard MacCutchan8-Oct-12 22:16

Richard MacCutchan

8-Oct-12 22:16

You need to provide some more information on the problem. Is that the exact message that you see, and which DLL is it missing?

One of these days I'm going to think of a really clever signature.

Customize CFileDialog?

bosfan8-Oct-12 5:05

8-Oct-12 5:05

Hi all,
i make a nice CMyFileDialog derived from CFileDialog that works fine for my need, but i need just one function for my Dialog and this is when a user select some file in a list to catch this event, something like:
"OnItemChange", is this possible, something like CListCtrl function itemchange??? If yes can someone give me a suggestion how to do this?

Best Regards
bosfan

jeron18-Oct-12 5:52

jeron1

8-Oct-12 5:52

Maybe this[^]thread may help.

bosfan8-Oct-12 21:37

8-Oct-12 21:37

Hi,
thanks for answer, but in this example works only if you know the id of a list control, e.g. my own list controls on a dialog where i give the id to the ctrl? Right?

Best Regards
bosfan

David Crow8-Oct-12 6:48

David Crow

8-Oct-12 6:48

Check out the lpfnHook member.

bosfan9-Oct-12 3:05

9-Oct-12 3:05

Hello,
this is what i'm looking for, this works.
I found a example with OnNotify() and i use this as reference:
ImageImport Filedialog

Many thanks
Best wishes
Mirsad

modified 9-Oct-12 9:14am.

Sarath C8-Oct-12 22:35

Sarath C

8-Oct-12 22:35

This [^] might help. Also just search in Codeproject you can find many!

-Sarath.

My blog - Sharing My Thoughts

Rate the answers and close your posts if it's answered

bosfan9-Oct-12 3:06