Prev: I couldnt find what is the wrong part of the function,could you helppp
Next: A couple of questions
From: Francis Glassborow on 16 Nov 2007 07:13
Ulrich Eckhardt wrote:
> emre esirik(hacettepe com. sci. and eng.) wrote:
>> thank you but still I couldnt find, po5 is a pointer and its getting
>> pol1 cofficient,
>
> Okay.

I agree with everything Ulrich has written. However there a re a couple
of extra points:
1) are you using C or C++?
2) if po5 is a pointer it must have been declared globaly, this is
generally a poor idea.
3) It seems that you are dealing with a bundle of arrays being passed as
arguments. Your resulting code is unmaintainable because it is riddled
with magic numbers (the sizes of the arrays.)
4) The complexity of the nesting makes it difficult for you to track
what is happening in your function. Think about refactoring so that the
inner loops are replaced by function calls.

>
>> code is
>> void div ( int pol1[] , int pol1p[] , int pol2[] , int pol2p[])
>
> Dude, what did I tell you about the use of 'div'? What did I tell you about
> things YOU need to provide for anyone to help? Please go and reread what I
> wrote!
>
>> is there anybody have a idea of why it isnt correct??????
>
> Nobody here even has an idea of what correct means in this context and how
> this incorrect looks like, because you didn't even provide a complete and
> minimal but compilable example!
>
> Uli
>
 | 
Pages: 1
Prev: I couldnt find what is the wrong part of the function,could you helppp
Next: A couple of questions