# Help with c++ program for series

• C/++/#
urgent help with c++ program for series

i wanna form program 2 sum this
pie=4[1-1/3 +1/5 -1/7+..... 1/n]

{
int n,z,i;
float x,y,pie;
cin>>n;
i=0;
while (i<=n)
{
y=1/((2*i)+1);
i=i+2;
}

for(z=1;z<=n;z=z+2)
{
x=1/((2*z)+1);

}

pie=4*(y-x);
cout<<pie;
}

cant understand wat to do...this is wat i did but its giving 0

Related Programming and Computer Science News on Phys.org
finchie_88
Isma said:
y=1/((2*i)+1);
x=1/((2*z)+1);
I don't know much about programming, but shouldn't the two lines above be:
Code:
 y = y + 1/((2*i)+1);
x = x + 1/((2*z)+1);
otherwise, you won't get the sum of the terms, and so y = x, and so you get pie to be 0.

Last edited by a moderator:
so will i initialize x nd y first?

Yeah, initialize them to zero and make finchie's changes, and it should do what you describe.

#include <iostream.h>
void main()
{
int n,z,i;
float x,y,pie;
cin>>n;
i=0;
y=0;
while (i<=n)
{
y=y+1/((2*i)+1);
i=i+2;

}
x=0;
for(z=1;z<=n;z=z+2)
{
x=x+1/((2*z)+1);

}

pie=4*(y-x);
cout<<pie;
}

it still wont work

dduardo
Staff Emeritus
1) You can write the program using 1 for loop. You don't need to separate the task of getting the negative part and the positive parts of the series.
2) You have typecasting issues. While doing your calculations you need to promote your ints to floats.

This is pretty much as tight as you can get it:

Code:
#include <iostream>

using namespace std ;

int main( void ) {
int n; double pi=0 ;
cin >> n ;
for( int i=1, j=0 ; i<n+1; i+=2, j+=1 ) {
j%2 ? pi-=1.0/i : pi+=1.0/i ;
}
cout << 4*pi << endl ;
return 0 ;
}
If you don't understand something in the above code, don't hesitate to ask.

enigma
Staff Emeritus
Gold Member
You are having problems with 'int' to 'float' conversions.

These lines are the problem:

y=y+1/((2*i)+1);
x=x+1/((2*z)+1);

y is a float, but i and z are integers. Because they are integers, the whole section 1/((2*i)+1) is treated like an integer, and equal to 0 for any value of i larger than 0.

There are a few ways to get around this. Either create a dummy float 'foo' and then do

foo = i;
y=y+1/((2*foo)+1);

or more cleanly look up what the static_cast command does and change the lines to

y=y+(1/((2*static_cast<float>(i))+1));

EDIT: looks like dduardo beat me to it.

thanks a lot...i do get it now nd i ll try to form less loops now
thanx a lot for the tip

EDIT: plz can u tell wat this statement means
j%2 ? pi-=1.0/i : pi+=1.0/i ;
nd y are we doing it?

Last edited:
dduardo
Staff Emeritus
j%2 ? pi-=1.0/i : pi+=1.0/i ;

is just a shorthand for:

Code:
if(j%2){
pi-=1.0/i ;
} else {
pi+=1.0/i ;
}
The % operator is called the modulus operator. It gives you the remainder. In this case j%2 gives you the remainder of j divided by 2. If the reminder is 0, then you know the number is even, else it is odd.

Also this:

1.0/i

is identical to this:

1/(double)i

This basically promotes the integer to double

Last edited:

1) You can write the program using 1 for loop. You don't need to separate the task of getting the negative part and the positive parts of the series.
2) You have typecasting issues. While doing your calculations you need to promote your ints to floats.

This is pretty much as tight as you can get it:

Code:
#include <iostream>

using namespace std ;

int main( void ) {
int n; double pi=0 ;
cin >> n ;
for( int i=1, j=0 ; i<n+1; i+=2, j+=1 ) {
j%2 ? pi-=1.0/i : pi+=1.0/i ;
}
cout << 4*pi << endl ;
return 0 ;
}
If you don't understand something in the above code, don't hesitate to ask.
Hi i am new user of c++ language and i didnt get how this loop is working.

mgb_phys
Homework Helper

Hi i am new user of c++ language and i didnt get how this loop is working.
for( int i=1, j=0 ; i<n+1; i+=2, j+=1 ) {
The for statement is split into three parts with ";"
The first part is only executed once before the start of the loop, this creates variables i and j and sets them to 0.
The second part "i<n+1" is tested each time through the loop, the loop exits when this is false, so in this case when i is greater or equal to n
The third part is executed each time through the loop, it increments i and j for each iteration

# include <iostream>
using namespace std;
int main(){
int num;
cout <<"enter the desired num";
cin >>num ;
cout <<"*"<<endl;

for (int j=1;j<=num;j++)

{
for (int i =1;i <=j;i++)
if (i%2==0)
cout <<" ";

else
cout <<i<<"*"<<" ";
cout<<endl;

}

return 0;
}
this is the program and giving the result :
*
1*
1*3*
1*3* 5*

But the desired result is :
*
1*
1*3*
1*3*
1*3*7*
1*3*7*11*

Defennder
Homework Helper

You mean to say you want to produce a series of integers which increments by 2,4,4 then after that it how does it increment? 2,4,4,6,6,6,8,8,8,8...?

1*3*
1*3*
There is a repeated line here, is that intentional?

Last edited:

You mean to say you want to produce a series of integers which increments by 2,4,4 then after that it how does it increment? 2,4,4,6,6,6,8,8,8,8...?

There is a repeated line here, is that intentional?
no i want series should be produce in 2,4,4,4,4,4,4,4, increment...............
and result should be,,,,,,,,,,,1,3,7,11,15and so on,,,,,,,,,,,,,

Defennder
Homework Helper

Well then that isn't too hard. You need an encompassing for loop for loop to print out all the sequences and within it, you need a cout statement to print out the initial +2 increment and then another for loop to print out the rest, incrementing the variable by +4 each time.

Hi tomorrow i have a test and i need your help........
please helo me in calling a two dimensional array
I want two store two values that is height and weight of 10 person in the system by using array array and using switch.As this is a long ques i need help only in this,Please reply me with correct program as soon as possible.

# include <iostream>
using namespace std;
void Array ( int height[],int weight[],int size){
for (int i =0;i<10;i++)
{
cout <<"enter the height";
cin >> height ;
cout <<"enter the weight";
cin>>weight;

}
}
double abc(int choice)
{
switch (choice)
{
case 1:
Array(int height[],int weight[],10);
}
}

int main(){
int choice;
cin>>choice;
cout <<abc(choice);
cout <<endl;
return 0;