tinytim-discuss

[tinyTiM-discuss] ITopicMapReader design

[Lars H. <he...@se...>]

Hi all,

The current (unused) ITopicMapReader interface provides three
different methods to deserialize topic maps.
<http://tinytim.svn.sourceforge.net/viewvc/tinytim/tinytim-mio/trunk/src/main/java/org/tinytim/mio/ITopicMapReader.java?view=markup>

I'd like to switch to minimised ITopicMapReader which provides just
a "void read() throws IOException" method. A nice side effect would be
that "read()" could do anything (read from a InputStream, read from a
database) etc.

The common TopicMapReader implementation would take an InputStream / a
File and a tmapi.core.TopicMap instance in the constructor or
alternatively provide some setters (even if I'd prefer immutable
implementations).

Even if the design change wouldn't break backwards compatibility
(since we never published the ITopicMapReader), I like to
hear your opinion.

What do you think? Any objections?

Best regards,
Lars
-- 
Semagia 
<http://www.semagia.com>

Re: [tinyTiM-discuss] ITopicMapReader design

[Markus U. <mar...@gm...>]

Lars Heuer schrieb:
> I'd like to switch to minimised ITopicMapReader which provides just
> a "void read() throws IOException" method. A nice side effect would be
> that "read()" could do anything (read from a InputStream, read from a
> database) etc.
>   
Makes the interface simpler and more generic... +1 :)

Ad astra, Markus

Re: [tinyTiM-discuss] ITopicMapReader design

[Lars H. <he...@se...>]

Hi Markus,

>> I'd like to switch to minimised ITopicMapReader which provides just
>> a "void read() throws IOException" method. A nice side effect would be
>> that "read()" could do anything (read from a InputStream, read from a
>> database) etc.
>>   
> Makes the interface simpler and more generic... +1 :)

Great! Someone reads my e-mails and replies to it. ;) Okay, I'll
change the ITopicMapReader and rename it to "TopicMapReader". I think
this name fits better in the TMAPI landscape. :)

Best regards,
Lars
-- 
Semagia 
<http://www.semagia.com>

Re: [tinyTiM-discuss] ITopicMapReader design

[Lars H. <he...@se...>]

[...]
> Great! Someone reads my e-mails and replies to it. ;) Okay, I'll
> change the ITopicMapReader and rename it to "TopicMapReader". I think
> this name fits better in the TMAPI landscape. :)

Fixed in Rev. 175.

Best regards,
Lars
-- 
Semagia 
<http://www.semagia.com>