Hi Jukka,

My +1 for option A and B, I think it's better in the long run since 
configuration is an issue. All methods from option B should load default static 
configuration. I'd be happy to take the lead on JIRA'ing and implementing this 
in 0.5...

Cheers,
Chris



On 11/11/09 11:21 AM, "Jukka Zitting" <jukka.zitt...@gmail.com> wrote:

Hi,

At first I used static methods in the Tika facade class, but then I
switched to normal instance methods to make it easier to use custom
Tika configuration. However, now I'm thinking that the original design
was probably better.

Any opinions on this? Essentially, which one of the following do you
like better:

    a) new Tika().parse(...);

    b) Tika.parse(...);

The benefit of a) over b) is that you can say: new
Tika(config).parse(...), but most of the people who need custom
configuration will probably in any case be going beyond the Tika
facade.

BR,

Jukka Zitting


++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Chris Mattmann, Ph.D.
Senior Computer Scientist
NASA Jet Propulsion Laboratory Pasadena, CA 91109 USA
Office: 171-266B, Mailstop: 171-246
Email: chris.mattm...@jpl.nasa.gov
WWW:   http://sunset.usc.edu/~mattmann/
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Adjunct Assistant Professor, Computer Science Department
University of Southern California, Los Angeles, CA 90089 USA
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++


Reply via email to