[Phpmyadmin-devel] Refactoring: Displaying query results

Chanaka Dharmarathna pe.chanaka.ck at gmail.com
Fri May 4 21:46:30 CEST 2012


On Thu, May 3, 2012 at 4:38 PM, Michal Čihař <michal at cihar.com> wrote:

> Hi
>
> Dne Tue, 1 May 2012 02:02:38 +0530
> Chanaka Dharmarathna <pe.chanaka.ck at gmail.com> napsal(a):
>
> > I'm identifying the refactoring points in 'display_tbl.lib.php' file. I
> am
> > trying to break the tasks on following areas.
> >
> > * As a library file this should not render the HTML codes in functions it
> > selves. First, all the HTML content in functions stored in string and
> > render at the bottom of the function.
> > Next, return the string having the HTML content, instead of rendering it
> > inside the function itself and, from the calling end render that content.
> > Probably method names will be modified with adding 'get' phrase.
> >
> > * Some naming conventions are used incorrectly. For method name in lib
> > file, naming convention is used generally used as 'PMA_methodName'. But
> > some are not following this.
> > While correcting them, suggested conventions in my proposal [0] is going
> to
> > applied.
> >
> > * There are several functions having more than 200 lines. I'm intending
> to
> > divide those functions to smaller ones. (PMA_displayTableNavigation(),
> > PMA_displayTableHeaders(), PMA_displayTableBody(),
> > PMA_displayVerticalTable(), PMA_displayTable())
>
> I think all this should be rather wrapped in object, so it would be
> PMA_Display::someFunction.
>
> --
>        Michal Čihař | http://cihar.com | http://blog.cihar.com
>
>
> ------------------------------------------------------------------------------
> 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 Michal,

Thanks for your reply. I got the point Michal. After the mentioned tasks
plus making this as a class, I can use 'PMA_Display::someFunction' where
needed.

Regards !
-- 
____________________________________

Chanaka Indrajith
Bsc.Computer Engineering Undergraduate
Faculty of Engineering
University of Peradeniya
Sri Lanka
____________________________________
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.phpmyadmin.net/pipermail/developers/attachments/20120505/66fb76b5/attachment.html>


More information about the Developers mailing list