Re: commit of several changes

John Lamb <[email protected]>
Newsgroups gmane.comp.kde.kplato
Message-ID <[email protected]>
[email protected] wrote:
> The pert code is quite complicated. Is it documented anywhere? I hope it
> still
> works as indended. My tests indicated that it is still working, but perhaps
> someone who programmed that in the first place could have a look?)

There are some kdoc/doxygen style comments but no other documentation. 
What would be useful to you?

I'll try to have a look soon. I did a lot of the original pert/cpm 
stuff. As I recall, it depends almost entirely on nodes and relations 
and doesn't use the work breakdown structure at all.

> things to do:
> get the time calculation working. I think it does not work at the moment.
> And
> I would prefer to move it out of the models. It is spread over several
> classes 
> and IMHO it should be handled in project, or even better in a class of its
> own
> whith the sole responsability to perform these calculations. I think this
> is neccessary, as several algorithms for these calculations can be expected,
> and then we could have on class for each algorithm. It would be confusing to
> spread several calculation algorithms over several classes.)

Can you explain this one a little more? Do you mean 
KPTProject::calculate()?

I like the idea of doing this in a separate class.


-- 
JDL

Non enim propter gloriam, diuicias aut honores pugnamus set propter
libertatem solummodo quam Nemo bonus nisi simul cum vita amittit.
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.