<br><br><div class="gmail_quote">On Mon, May 7, 2012 at 7:14 PM, Thilina Buddika Abeyrathna <span dir="ltr"><<a href="mailto:thilinaabeyrathna@gmail.com" target="_blank">thilinaabeyrathna@gmail.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br><br><div class="gmail_quote"><div><div class="h5">On Mon, May 7, 2012 at 5:34 PM, Marc Delisle <span dir="ltr"><<a href="mailto:marc@infomarc.info" target="_blank">marc@infomarc.info</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
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.</blockquote><div><br></div></div></div><div>Hi Marc, </div><div>Thank you very much for pointing out.</div></div><span class="HOEnZb"><font color="#888888"><div><br></div></font></span></blockquote>

<div>Hi Marc,</div><div>when I write functions for columns(function column, null column, value column, ...) in insert form (line 408 in tbl_change.php) I have to pass lot of parameters. But it is not a good practice for refactoring. And use global variables also not a good solution. So is it ok to use a parameter array. I need to pass around 10 parameters. I like to see your suggestions.</div>

</div><br><br clear="all"><div><br></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>