Re: RE: [PEAR] [ANNOUNCEMENT] Numbers_Words-0.7.1 (beta) Released.

On May 25, 2004, at 3:48 PM, Antonio Carlos Venancio Junior wrote:

On May 25, 2004, at 3:20 PM, Antonio Carlos Venancio Junior wrote: snip

    So, we are missing a link to that path here:

---------- http://pear.php.net/package/Numbers_Roman

I don't control that, it's part of pearweb

----------

    I'll ask your permition to change some lines in your code to make it faster and "validate" the roman numbers to avoid things like "Numbers_Roman::toNumber('VX')" or "Numbers_Roman::toNumber('VIIII')" and so on.

????? Sorry but the package is very simple and I don't think that is slowed down by anything. To Number works fine for me so I don't understand what we are trying to avoid.

That are only two examples of _wrong_ roman numbers, because 'VX' doesn't exists and its computed as a 5 (five). And the correct form of a nine (9) is 'IX' and not 'VIIII'.

This is not a problem, if you convert VX that is not a valid numeral so it's your fault I assume. Same for VIIII. It will return you a rough guess, which cannot be accurate if you pass an invalid numeral. toNumber expects a valid Numeral as input. Of course if this is bad for your personal use, feel free to fork your own version and do it your way, by all means this is something you can do.

and BTW I don't think you can change lines on my code :D

Why do you think that? ;D~

Because last time I checked I was the lead in this package.

Regards,

David Costa, http://DotGeek.org http://pear.php.net/user/gurugeek