From kmail-devel Tue Jun 03 07:31:46 2003 From: Cornelius Schumacher Date: Tue, 03 Jun 2003 07:31:46 +0000 To: kmail-devel Subject: Re: [PATCH] UI abstraction proposal (partially only) X-MARC-Message: https://marc.info/?l=kmail-devel&m=105462549126570 On Tuesday 03 June 2003 00:00, Andreas Gungl wrote: > > attached you find a diff containing a first idea on how to separate > the UI code from the processing code. It covers only the address book > access in KMail. > I formed an abstract class for the UI callbacks, then I inherited a > class covering the current Gui interactions. While this in principle makes sense (we have similar classes in libkabc and ksync), I think it would be better to store the callback object in a member variable instead of passing it as parameter to all functions using it. That makes the API much clearer. > I would like to have some comments on the patch, especially on naming > conventions and related topics. E.g. the classes are named > *GuiCallback, IMO it's better readable than *UiCallback, although > text UIs are of course possible. I think "Ui" is better than "Gui" because it's more acurate. In addition I don't think that it's necessary to include "Callback" in the class name as it doesn't provide any information what the class is meant to do. My suggestion would be to name the class "AddressBookUi". -- Cornelius Schumacher _______________________________________________ KMail Developers mailing list kmail@mail.kde.org http://mail.kde.org/mailman/listinfo/kmail