[Pharo-project] [ANN] Continue working in SimpleLogger

Mariano Martinez Peck marianopeck at gmail.com
Thu Oct 21 03:48:28 EDT 2010


On Thu, Oct 21, 2010 at 3:22 AM, Yanni Chiu <yanni at rogers.com> wrote:

> On 20/10/10 9:36 AM, Mariano Martinez Peck wrote:
>
>> What I did basically is to define SimpleLogger like the public API. But
>> then, this class delegates to a LoggerStrategy. I created the following
>> implementations: TranscriptLoggerStrategy, NullLoggerStrategy,
>> FileLoggerStrategy and FixesFileNameLoggerStrategy.  In addition, I
>> created a superclass for them, marking the API with SimpleLogger. This
>> class is LoggerStrategy.
>>
>> So, now you can:
>>
>> - Enable and disable easily the logging (thanks to NullLoggerStrategy).
>> - Change on runtime WHERE or HOW to log (just change the strategy).
>> - Be able to write into a File, a Transcript, a FixedFileName, or nothing.
>> - Extend to you own strategy (FileAppendLoggerStrategy??
>> SocketLoggerStrategy?  etc...)
>>
>
> My 2cents. I've not used or looked at SimpleLogger, but recall that it was
> a single class, plus a metacello config class.
>
> The new functionality sounds very useful, but is it still simple. It sounds
> like it was simple to add extra features - maybe because SimpleLogger was
> simple. Will the next person who comes along find a similarly simple
> implementation that can be easily adapted to specific needs.
>
> A hierarchy of strategy classes no longer sounds simple. What if I don't
> need all the different strategies loaded, will the configuration then allow
> multiple build targets for various individual or groups of strategies.
> Starting to sound complicated.
>

Yes, that's true. However, take a look at the code, and I promisde that in
no more than 10 minutes you get the idea how to use it. There are 5 classes,
with an average of 5 methods each.


>
> IMHO, it should be a fork: SomewhatSimpleOnItsWayToComplicatedLogging :)
>
>
heheheh


> --
> Yanni
>
>
> _______________________________________________
> Pharo-project mailing list
> Pharo-project at lists.gforge.inria.fr
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.pharo.org/pipermail/pharo-dev_lists.pharo.org/attachments/20101021/fe11fc45/attachment-0001.html>


More information about the Pharo-dev mailing list