I have a problem to solve summation in c++

[logieen]

Hi , Every one I have a problem to solve this summation in c++ ;
1/3 + 3/5 +5/7 .......

this the program I wrote

#include <stdio.h>
#include <iostream.h>
#include <conio.h>
#include <math.h>
 
main()
{
  int i=3.0,j=1.0,d,N;
  cout<<"Enter Number ";
  cin>>N;
  float sum;
     while(j<=N && i<=N){
  for (d=1; d <= N; d++){
    sum += j/i;
    printf("%d:  %2.2f\n", d, sum);
    i+=2;
    j+=2;
  }}
  getch();
}

and this is the run
http://www.el3mlak.net/up/images/6293428.png

is there a problem in my program >


thanks

[Savage]

Hi , Every one I have a problem to solve this summation in c++ ;
1/3 + 3/5 +5/7 .......

this the program I wrote

#include <stdio.h>
#include <iostream.h>
#include <conio.h>
#include <math.h>
 
main()
{
  int i=3.0,j=1.0,d,N;
  cout<<"Enter Number ";
  cin>>N;
  float sum;
     while(j<=N && i<=N){
  for (d=1; d <= N; d++){
    sum += j/i;
    printf("%d:  %2.2f\n", d, sum);
    i+=2;
    j+=2;
  }}
  getch();
}

and this is the run
http://www.el3mlak.net/up/images/6293428.png

is there a problem in my program >


thanks

A few problems:

1.u don't initialize sum to 0.0;

2.while loop is not needed here,a single for should solve it

3.please improve ur tabbing style(ur style makes code difficlut to read.

Savage

[weaknessforcats]

If you are coding in C++, you should be using the accumulates algorithm instead of re-coding it as you have done. There are no bugs in accumulates whereas your code needs work and when finally working will have to maintained thereby increasing the overall software costs.

[JosAH]

If you are coding in C++, you should be using the accumulates algorithm instead of re-coding it as you have done. There are no bugs in accumulates whereas your code needs work and when finally working will have to maintained thereby increasing the overall software costs.

This is all true but my guess is that the OP is beginning to study C++ and most
certainly, the STL hasn't been part of the subjects for study yet. Blame it on the
courseware instead if you like but in defense of the OP: they need to crawl first
before they can walk or run or even *appreciate* the power of the STL. And then
it's still anothert step to be able to *use* all that power.

kind regards,

Jos

[weaknessforcats]

Yes, it's a paradigm shift to start C++ with the STL and end it with low-level stuff like arrays and such. Your courseware was correct for the time in history that it was prepared.

[JosAH]

Yes, it's a paradigm shift to start C++ with the STL and end it with low-level stuff like arrays and such. Your courseware was correct for the time in history that it was prepared.

minor nitpick: that wasn't *my* courseware ;-)

kind regards,

Jos

[Savage]

minor nitpick: that wasn't *my* courseware ;-)

kind regards,

Jos

Jos,seriously,ur username should be:

Jos The Nitpicker

Savage