<br><br><div class="gmail_quote">On Fri, May 11, 2012 at 12:35 PM, Piotr Przybylski <span dir="ltr"><<a href="mailto:piotr.prz@gmail.com" target="_blank">piotr.prz@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


2012/5/11 Thilina Buddika Abeyrathna <<a href="mailto:thilinaabeyrathna@gmail.com" target="_blank">thilinaabeyrathna@gmail.com</a>>:<br>
<div><div>><br>
><br>
> On Mon, May 7, 2012 at 7:14 PM, Thilina Buddika Abeyrathna<br>
> <<a href="mailto:thilinaabeyrathna@gmail.com" target="_blank">thilinaabeyrathna@gmail.com</a>> wrote:<br>
>><br>
>><br>
>><br>
>> On Mon, May 7, 2012 at 5:34 PM, Marc Delisle <<a href="mailto:marc@infomarc.info" target="_blank">marc@infomarc.info</a>> wrote:<br>
>>><br>
>>> Hi,<br>
>>><br>
>>> We previously had a script called grab_globals.lib.php, which created<br>
>>> global variables from a few superglobals arrays.<br>
>>><br>
>>> When this script was removed a few weeks ago, we added at the beginning<br>
>>> of other scripts some code, for example in tbl_change.php:<br>
>>><br>
>>> if (isset($_REQUEST['where_clause'])) {<br>
>>>    $where_clause = $_REQUEST['where_clause'];<br>
>>> }<br>
>>><br>
>>> In the upcoming refactoring efforts, we should avoid this, because,<br>
>>> looking at the code further in the script, it's difficult to see the<br>
>>> origin of $where_clause. So, we should only refer to<br>
>>> $_REQUEST['where_clause'] (except in the situations where the global is<br>
>>> modified afterwards).<br>
>>><br>
>>> Another benefit is that this variable would not need to be passed as a<br>
>>> parameter to functions.<br>
>><br>
>><br>
>> Hi Marc,<br>
>> Thank you very much for pointing out.<br>
>><br>
> Hi Marc,<br>
> when I write functions for columns(function column, null column, value<br>
> column, ...) in insert form (line 408 in tbl_change.php) I have to pass lot<br>
> of parameters. But it is not a good practice for refactoring. And use<br>
> global variables also not a good solution. So is it ok to use a parameter<br>
> array. I need to pass around 10 parameters. I like to see your suggestions.<br>
><br>
<br>
</div></div>10 is not nice but I think it's more readable, if you have to pass<br>
these 10 arguments two or three times it shouldn't be too troublesome.<br>
If you don't like it, maybe create a special object type that you will<br>
pass around? But not a stdObject, a custom one so that parameters will<br>
be properly documented and you will get code assist in some smart<br>
editors (just a suggestion).<br>
<br>
--<br>
Regards,<br>
Piotr Przybylski<br></blockquote><div><br>Hi Piotr,<br>Thank you for your suggestion.  <br></div></div>-- <br><font face="georgia, serif">Regards.</font><br><br><font face="'trebuchet ms', sans-serif" size="4">Thilina Buddika Abeyrathna,</font><br>

<font face="georgia, serif">Department of Computer Engineering,</font><br>
<font face="georgia, serif">Faculty Of Engineering,</font><br><font face="georgia, serif">University of Peradeniya,</font><div><font face="georgia, serif">Sri Lanka.<br><br></font></div><br>