IProjectController

[Matt Rogers]

On Jul 2, 2007, at 3:47 AM, Jens Herden wrote:

> On Sunday 01 July 2007, Matt Rogers wrote:
>> On Jul 1, 2007, at 11:10 AM, Andreas Pakulat wrote:
>>> On 01.07.07 16:21:14, Jens Herden wrote:
>>>> we have:
>>>>
>>>>     virtual IProject* projectAt( int ) const = 0;
>>>>     virtual int projectCount() const = 0;
>>>>     virtual QList<IProject*> projects() const = 0;
>>>>
>>>>
>>>> Do we really need all of them? I think it should be OK to drop
>>>> projectAt() and
>>>> projectCount() .
>>>
>>> Well, according to our wiki we should drop QList<IProject*>  
>>> projects()
>>> const;
>>> http://www.kdevelop.org/mediawiki/index.php/Platform_API_TODO
>>>
>>> I added the two count/at methods specifically because of this note.
>>>
>>> Andreas
>>
>> We need to keep all three of them. The first two that don't return a
>> list are for scripting purposes (think usage from dbus commandline)
>> and the one that returns the list is for C++ usage. IMHO, requiring
>> index iteration from C++ is just plain stupid when we can use  
>> iterators.
>
> I understand what you mean. However I am not sure about the plain  
> stupid
> because the list of open project will not be very long.
>
> Jens
>

It doesn't matter how long the list is. But IMO, the use of a non  
object-oriented construct (in this case, index-based iteration) in  
object oriented code is really quite silly.
--
Matt