RE: [PEAR-DEV] Re: Package proposal Science_Weather
| From: | LIMBOURG Arnaud | Date: | Fri, 22 Aug 2003 07:37:07 +0000 |
| Subject: | RE: [PEAR-DEV] Re: Package proposal Science_Weather | ||
| Groups: | php.pear.dev | ||
| Request: | Send a blank email to pear-dev+get-20367@lists.php.net to get a copy of this message | ||
Ok, i prefer the way without the $var_name, i just wanted confirmation it
was a feature and not an uplanned behaviour :)
Arnaud.
> Hi Arnaud,
>
> This is a personal choice. I have started using @param string blah
> because of the risk of typos. PhpDocumentor 1.x does not return a
> warning if a @params parameter name does not match an actual
> parameter.
> Why?
>
> /**
> * @param string the format string
> * @param string $args... an unlimited number of strings
> */
> function sprintf($format)
> {
> }
>
> Variable arguments can't be documented if we are strict. Version 2.0
> will fix this (once error handling is fully fixed in PEAR, so that we
> can raise notices) by raising a notice if a parameter name
> does not match.
>
> I've found that when parameters change names, it is more
> convenient to
> avoid the $varname in the parameter, because it will shunt to a new
> parameter definition, but that is a personal choice. If you are
> meticulous, the way listed below works just fine.
>
> Greg
>
> LIMBOURG Arnaud wrote:
>
> > > /**
> > > * Sets the neccessary account-information for weather.com
> > > *
> > > * @param string $partnerID
> > > * @param string $licenseKey
> > > * @access public
> > > */
> > > function setAccountData($partnerID, $licenseKey)
> >
> > Which brings me to a question greg.
> >
> > I notice you write the parameter variable like $partnerID in the
> > phpdoc. However, when it's not there but the @param tags are in the
> > same order as they appear in the function() definition,
> phpdocumentor
> > parses this intelligently.
> >
> > So what is the ways to go ?
> >
> > @param string $var blah
> >
> > or just
> >
> > @param string blah
> >
> > I switched to the last one since it works very well.
> >
> > Arnaud.
> >
>