18 Aug
2008
18 Aug
'08
4:45 p.m.
We have had this same discussion a while ago - please search the
archives. Making apps notice-free in PHP is a pain and not worth the
time -the developers should focus on new features instead of esoteric
fixes that don't add a benefit. Except that the code blows up and
becomes unreadable.
So don't waste your time "fixing" this and focus on outstanding
features. PHP does not require Java style coding, so don't enforce it.
lg, Mike
--
Michael Baierl
http://mbaierl.com/
Am 18.08.2008 um 18:13 schrieb Eduard Krieger <eduard.krieger@googlemail.com
>:
> Thomas Bruederli schrieb:
>> Making RoundCube notice-free would require us to change hundreds of
>> line of >> code including 3rd party libs. PHP will grumble on every access to a >> uninitialized var and when testing for an undefined array index. >> >> To reduce programming code (and effort!) to a minimum, it's better
>> to mute >> notices. Real problems are still reported as warning or error. > I know why there is an option to control the logging output :) . But > still this is a bad coding style to ignore even just notices. And
> like i > wrote before in rare cases a notice is a result of an (semantic) > programming-error before. Since in most cases one must just add a > "isset($X)" i think this is not so big problem. If this is too much
> (too > annoying, useless) for you main coder to do i can do this. > _______________________________________________ > List info: http://lists.roundcube.net/dev/ _______________________________________________ List info: http://lists.roundcube.net/dev/
>> line of >> code including 3rd party libs. PHP will grumble on every access to a >> uninitialized var and when testing for an undefined array index. >> >> To reduce programming code (and effort!) to a minimum, it's better
>> to mute >> notices. Real problems are still reported as warning or error. > I know why there is an option to control the logging output :) . But > still this is a bad coding style to ignore even just notices. And
> like i > wrote before in rare cases a notice is a result of an (semantic) > programming-error before. Since in most cases one must just add a > "isset($X)" i think this is not so big problem. If this is too much
> (too > annoying, useless) for you main coder to do i can do this. > _______________________________________________ > List info: http://lists.roundcube.net/dev/ _______________________________________________ List info: http://lists.roundcube.net/dev/