I have no preference either way. a factory would be nice. peter
"BAZLEY, Sebastian" <[EMAIL PROTECTED]> wrote: The reason I suggested a factory might be useful is that some parsers might be shareable, and some might not. Using a constructor to acquire a parser makes it more difficult to share instances (I think). I had a look at using dynamic loading of the parser last night, and found that it was not trivial, as the parser classes have private constructors. [Adding a factory method would be one solution, I guess.] == I think it would be better to retrieve the parser instance in a separate class, not directly in HTTPSamplerFull, as this would make it a bit more flexible. Not sure if the (e.g.) getParser() method should always use the value of jmeter.html.parser, or whether it should accept another property name and/or a class name. Perhaps start with the fixed name, and create additional methods if they prove necessary later... == Another oops. I did not notice the html.parser package, so I put the extracte parser code in the html.sampler package. I think it would be better to move them, and they can then keep company with htmlparser.java. S. -----Original Message----- From: Jordi Salvat i Alabart [mailto:[EMAIL PROTECTED] Sent: 24 November 2003 18:45 To: JMeter Developers List Subject: Re: Parser refactoring; should Jmeter fetch more than images/appl ets? OK. I'm doing that. I will also try to devise some easy-to-reproduce test that we can use for comparison. En/na peter lin ha escrit: > > i like the idea of an iterator, since that is how we use it anyways :) > > peter > > [...] >>In the short-term, I suggest hard-coding the class names in HTTPSamplerFull, >>but it might be useful to use a factory in future. > > > Or simply grabbing jmeter.html.parser and instantiating the class from > the name? > > [...] --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] --------------------------------- Do you Yahoo!? Free Pop-Up Blocker - Get it now