You're completely right. I'll leave the wrong title, though.
>This isn't "refactoring", it's "enhancement".
>
>There's a HUGE difference.
>
>cheers
>
>>Hi everybody,
>>
>>I have a custom function (thanks to Cetin & Borislav) that produces an Excel from DBF and applies some additional formatting. This function is called from several modules and it uses an array of field names.
>>
>>Now I have a need to add a certain color to some cells. For instance, we're sending the Excel file to third party and we want it to receive it with certain fields filled in. I want to add empty columns myself and I want them in the different color, so the third party would know, where new fields start.
>>
>>There could be several approaches:
>>
>>1. Leave the original function in tact, create a new function to apply additional formatting and call both functions.
>>
>>2. Have some optional parameters in the original function.
>>
>>I think, the second approach is better. I'm thinking, what kind of parameters would I need to add to the routine to make this generic enough?
>>
>>What do you think?
>>
>>Thanks in advance.
If it's not broken, fix it until it is.
My Blog