[bt-devel] bug
Martin Gruner
bt-devel@crosswire.org
Tue, 26 Sep 2000 15:46:01 +0200
Hi Joachim,
how was your exam?
Yes, the backend needs some cleaning and IMO some simplification.
Let us talk about it when you're around.
My first thoughts: simplify the CModuleInfo Methods, maybe introduce a
CModuleInfoConfig object to separate the config info from the text related
functions.
Delete CKeyChain and ListCkeyChain, they are not really necessary.
Their place would be filled by 3 methods in CModuleInfo:
QStringList getBookList()
->chaches the result
->opening of lexicon presenters is speeded up
int getChapters(QString book)
->will compute the result when needed (no caching)
->links to a sword function
int getVerses(QString book, int chapter)
->see above
Martin
On Mon, 25 Sep 2000, you wrote:
> Hi Martin!
>
> This remebers me that we have to clean and restructure the backend.
> Sure, Torsten did it right, but we have to discuss which functions we need
> and we should realize it (at least parts) module independant.
>
> At the moment things are looked up in the module etc., sometimes it's
> possible to use only a VerseKey or SWKey object.
>
> But now I have to learn for Math, we have our exam tomorrow.
> I'll have a look at it tomorrow, OK?
>
> --Joachim
>
> > Hey,
> >
> > how can I set the current key of a module? I use now lookupText(), but
> > that is bad.
> >
> > I commented out 2 completely buggy functions in CSWORDBibleModuleInfo:
> > getNextKey and getPrevKey. Why should the bibles use different methods
> > than commentaries?
> > These functions in CSWORDCommentaryModuleInfo do work, only not for
> > books. The getprevkey() in the lexicon seems to be buggy, too (goes to
> > key 00000, although this key doesn't exist).
> > Joachim, could you help me?
> >
> > You can test it with the keychoosers in the Searchdialog (only for
> > testing).
> >
> >
> > Martin