Overview
Examples
Screenshots
Comparisons
Applications
Download
Documentation
Tutorials
Bazaar
Status & Roadmap
FAQ
Authors & License
Forums
Funding Ultimate++
Search on this site
Search in forums












SourceForge.net Logo
Home » U++ Library support » U++ Widgets - General questions or Mixed problems » DropList::Find() error?
DropList::Find() error? [message #9643] Wed, 23 May 2007 12:17 Go to next message
unodgs is currently offline  unodgs
Messages: 1366
Registered: November 2005
Location: Poland
Ultimate Contributor

Shouldn't DropList::Find(const Value& key) look like this:
int Find(const Value& k) const    { return list.Find(k); }

instead of
int Find(const Value& k) const    { return key.Find(k); }

Now FindKey is a copy of Find.
Re: DropList::Find() error? [message #9678 is a reply to message #9643] Thu, 24 May 2007 23:22 Go to previous messageGo to next message
mirek is currently offline  mirek
Messages: 13975
Registered: November 2005
Ultimate Member
unodgs wrote on Wed, 23 May 2007 06:17

Shouldn't DropList::Find(const Value& key) look like this:
int Find(const Value& k) const    { return list.Find(k); }

instead of
int Find(const Value& k) const    { return key.Find(k); }

Now FindKey is a copy of Find.


It's rather obsolete name. Naming there would be "FindValue" for what you suggest.

The "value" of DropList is always its key (little bit confusing if you write it down Smile, so unless there is "Value" in method name, it is about the key.

Mirek
Re: DropList::Find() error? [message #9681 is a reply to message #9678] Thu, 24 May 2007 23:45 Go to previous messageGo to next message
unodgs is currently offline  unodgs
Messages: 1366
Registered: November 2005
Location: Poland
Ultimate Contributor

Ok. May I add FindValue?
Re: DropList::Find() error? [message #9682 is a reply to message #9681] Thu, 24 May 2007 23:46 Go to previous message
mirek is currently offline  mirek
Messages: 13975
Registered: November 2005
Ultimate Member
Yes, a good idea. Just add Topic++ entry too Smile
Previous Topic: How to arrange widgets
Next Topic: Linux Suse 10.2 build problem
Goto Forum:
  


Current Time: Sun May 05 11:48:55 CEST 2024

Total time taken to generate the page: 0.02128 seconds