Sorry if we've duplicated work. I thought I had said I would be doing this.
-- Salut,
Jordi.
En/na BAZLEY, Sebastian ha escrit:
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]
--------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
