[Phpmyadmin-devel] NOT A PROGRAMMER

Maria Medina mvolley13 at yahoo.com
Mon Jun 11 22:26:01 CEST 2012


I NEED TO REDIRECT MY WEBSITE.... HOW AND WHERE CAN I DO THAT????


From: Chanaka Dharmarathna <pe.chanaka.ck at gmail.com>
To: phpmyadmin-devel at lists.sourceforge.net 
Sent: Monday, June 11, 2012 4:02 PM
Subject: Re: [Phpmyadmin-devel] Refactoring: Displaying query results





On Sun, Jun 10, 2012 at 11:21 PM, Dieter Adriaenssens <dieter.adriaenssens at gmail.com> wrote:

2012/6/9 Chanaka Dharmarathna <pe.chanaka.ck at gmail.com>:
>
>>
>>
>> On Fri, Jun 8, 2012 at 10:58 AM, Chanaka Dharmarathna
>> <pe.chanaka.ck at gmail.com> wrote:
>>>
>>> Hi All,
>>>
>>> I have a doubt of using @see tag in Docblock comments for functions.
>>> In the comments of functions PMA_getTable() and PMA_getTableBody() in
>>> display_tbl.lib.php file, @see tag is used for different purposes.
>>> As I feel PMA_getTableBody() used that corrrectly.
>>> What should be the actual behavior ?
>>>
>>> Regards !
>>> --
>>> ____________________________________
>>>
>>> Chanaka Indrajith
>>> Bsc.Computer Engineering Undergraduate
>>> Faculty of Engineering
>>> University of Peradeniya
>>> Sri Lanka
>>> ____________________________________
>>>
>>
>> Hi All,
>>
>> I am fixing some code violations in my code now.
>> For remove code violation of large comparison in a if condition, what would
>> be the better approach ?
>>
>> Ex:-
>> if ((PMA_strlen($wkbval) > $GLOBALS['cfg']['LimitChars'])
>>     && ($_SESSION['tmp_user_values']['display_text'] ==
>> self::DISPLAY_PARTIAL_TEXT)
>> ) {
>> // code
>> }
>>
>> Is that good to use tempory variable insted of using
>> $_SESSION['tmp_user_values']['display_text'] ?
>
>It depends :
>
>If the value of $_SESSION['tmp_user_values']['display_text'] is used
>later on in the code, you could assign it to a shorter local
>(temporary) variable, and use that variable with a shorter name, in
>the if statement and further on in the code.
>
>If it is used only oncet you might consider assigning the result of
>
>$_SESSION['tmp_user_values']['display_text'] == self::DISPLAY_PARTIAL_TEXT
>
>to a local (temporary) variable and use that variable in the if statement.
>
>Hope this helps!
>
>--
>Kind regards,
>
>Dieter Adriaenssens
>
>
>------------------------------------------------------------------------------
>Live Security Virtual Conference
>Exclusive live event will cover all the ways today's security and
>threat landscape has changed and how IT managers can respond. Discussions
>will include endpoint security, mobile security and the latest in malware
>threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
>_______________________________________________
>Phpmyadmin-devel mailing list
>Phpmyadmin-devel at lists.sourceforge.net
>https://lists.sourceforge.net/lists/listinfo/phpmyadmin-devel
>
Hi Dieter,

Thanks for your reply.
I agreed your first point.
But I feels little odd in the second point because only for one place we use an additional variable.
Its better if we can put that kind of conditions to two lines. Currently it gives errors by code sniffer.

Regards !
-- 
____________________________________

Chanaka Indrajith
Bsc.Computer Engineering Undergraduate
Faculty of Engineering
University of Peradeniya
Sri Lanka
____________________________________


------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and 
threat landscape has changed and how IT managers can respond. Discussions 
will include endpoint security, mobile security and the latest in malware 
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
_______________________________________________
Phpmyadmin-devel mailing list
Phpmyadmin-devel at lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/phpmyadmin-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.phpmyadmin.net/pipermail/developers/attachments/20120611/0f2f75b9/attachment.html>


More information about the Developers mailing list