"Alvin Bruney [ASP.NET MVP]" <
www.lulu.com/owc> wrote in message
news:632818E3-8CD3-4CFA-BF41-57CC1DC4B9F2@microsoft.com...
>> Different developers have worked on it, that's why it has performance
>> issues in it.
> Not necessarily true by the way.
>
> This is what I do in my day job btw and I happen to work in the financial
> industry too. My starting point? I throw automated tools at it to get a
> sense of what I am dealing with - ugly code, good code gone bad, script
> kiddies turned programmers, hot shots who only care about their way etc.
> Tools like FxCop, Devpartner are a staple and good enough to give you a
> general idea.
>
> Next, I start with the code, line by line. If it's too big or I get
> discouraged, I target the important parts and code review those for
> performance issues. I have a list of issues that I look for in the code -
> common errors like leaky connections, performance intensive code that can
> be refactored, for loops and variable declarations inside loops, poorly
> structured logic, COM interop issues, XML manipulation etc. Automated
> tools can't really get after those issues, the eye and experience is what
> will get you there. If there's a particular routine that is slow, maybe I
> will throw Devpartner performance analyzer at it, maybe I'll take a look
> at the raw code, it depends on how I'm feeling about the code. I prefer to
> trust my eye against the code but that requires a lot of effort and time
> that I don't particularly have.
>
> If I grow suspicious at what I am looking at, I'll stop, step back and do
> a design review before continuing. Sometimes I start with a design review
> but these applications are typically built in concert with architects so I
> believe they are well intentioned to begin with. It falls apart when time
> constraints and lack of coding best practices mix in to the pot. From that
> point, I know enough to make a judgment call - this thing sucks, scrap it
> and start fresh or it's good it just needs some adjustments. Or, it's
> wrong from the design so everything else is a band-aid. When you make that
> call, know enough about best practices and proper coding to stand by your
> decision. You get better the more you do it.
>
> --
> --
> Regards,
> Alvin Bruney [MVP ASP.NET]
>
> [Shameless Author plug]
> The O.W.C. Black Book, 2nd Edition
> Exclusively on
www.lulu.com/owc $19.99
>
>
> "M Waseem" <mwaseem@gmail.com> wrote in message
> news:eMCMY4ngIHA.536@TK2MSFTNGP06.phx.gbl...
>> I have an application that has already been built for financial industry.
>> Different developers have worked on it, that's why it has performance
>> issues in it. I have to evaluate it and then bring up with my
>> suggestions. There are different ways to do it:
>>
>> - Execute profiling tools like ANTS, DevPArtner to identify problem areas
>> - Refactor the components to make the boundaries among them clarified
>> - Do a code review to identify issues related to algorithms and other
>> issues
>>
>> I just wanted to have suggestions from people around here that how should
>> I go about identifying the issues and proposing solutions to these
>> problems
>>
>> Regards,
>> Warfi
>>