>>>>>I'm with Viv ... the delegate approach sounds good to me too.
>>>>>
>>>>>~~Bonnie
>>>>>
>>>>>
>>>>>
>>>>
>>>>Thank you for your vote. This is going to be a long ugly refactor, but it is one of those things that has been needed for a long time.
>>>>
>>>>Here is how one of the current methods ends after 1800 lines of code:
>>>>
>>>>

>>>>                                                       End If
>>>>                                                    End If
>>>>                                                End If
>>>>                                            End If
>>>>                                        Next
>>>>                                    End If
>>>>                                Next
>>>>                            End If
>>>>                        Next
>>>>                    End If
>>>>                End If
>>>>
>>>>

>>>
>>>That certainly brings back some ugly memories of inherited code. I worked on one VFP app that had a method with 3000 lines of code.
>>>
>>>Not putting that on VFP. You can write elegant code and ugly code in about any language. It's just that there are pro technicians and then there are shoemakers ;-)
>>
>>I think I have learned more from maintaining poorly written code than in any formal training. I didn't understand how evil globals were until I had to maintain an app where EVERY variable was declared PUBLIC. I learned alot about side effects keeping that app running.
>
>In a perfect world I will never have to do maintenance work on an ugly app again. And no one will ever have to work on any of mine ;-)

Unfortunately, I have to maintain MY code. I cringe whenever I have to revisit code I wrote a couple of years ago! OTOH, it has worked well as ugly as it is.