Advertisement
If you have a new account but are having problems posting or verifying your account, please email us on hello@boards.ie for help. Thanks :)
Hello all! Please ensure that you are posting a new thread or question in the appropriate forum. The Feedback forum is overwhelmed with questions that are having to be moved elsewhere. If you need help to verify your account contact hello@boards.ie
Hi all! We have been experiencing an issue on site where threads have been missing the latest postings. The platform host Vanilla are working on this issue. A workaround that has been used by some is to navigate back from 1 to 10+ pages to re-sync the thread and this will then show the latest posts. Thanks, Mike.
Hi there,
There is an issue with role permissions that is being worked on at the moment.
If you are having trouble with access or permissions on regional forums please post here to get access: https://www.boards.ie/discussion/2058365403/you-do-not-have-permission-for-that#latest

Wrapping existing function in wrapper?

  • 22-04-2014 2:06am
    #1
    Closed Accounts Posts: 4,763 ✭✭✭


    Another style question. I resize images at several points in a script, and in each case I use an identical built-in function to shrink or grow the images. I chose to wrap this existing function in one of my own in case I needed to change it in future, which I haven't to this point.

    Is this a good or bad practice?


Comments

  • Registered Users, Registered Users 2 Posts: 2,029 ✭✭✭Colonel Panic


    "You ain't gonna need it". If you need to change the function later, change it later.


  • Registered Users, Registered Users 2 Posts: 1,931 ✭✭✭PrzemoF


    I'd get rid of it unless it makes calling the wrapped function easier/cleaner (i.e fixed parameters) or extends original features (i.e. fixed aspect ratio or scale width given height + aspect ratio). Keeping code just because you might need it in the future is wrong. Remove the code, give the commit a good description (you use a SCM, do you?) and if you'll ever need it it will be waiting for you without cluttering the current version of your code.


  • Closed Accounts Posts: 4,763 ✭✭✭Fenster


    Cheers!


Advertisement