Solving a Problem with Bisection Method (C Program)

AI Thread Summary
The discussion revolves around a C program implementing the bisection method to find roots of a quadratic equation. The initial code provided leads to infinite output due to incorrect variable assignments and logic errors. Key points include the need to properly assign values to x1 and x2 based on function evaluations rather than directly using x1 and x2 in the condition checks. Suggestions are made to use a new variable for the function evaluation at x1, ensuring that the correct conditions are checked to update x1 and x2. Additionally, it's emphasized that the initial values for x1 and x2 must be chosen such that the function values at these points have opposite signs, indicating a root exists between them. The importance of formatting code for clarity and the need for constructive feedback in programming discussions are also highlighted.
swty todd
Messages
69
Reaction score
0
here is the c program for arriving at a solution using bisection method.
The ouputi s going infinte..pls help ...i think it has more of syntax errors than logical errors...

Code:
main()
{
	float a,b,c,x1,x2,x,series;
	double d;

	printf("enter a,b,c and x1(pos) & x2(neg)");
	scanf("%f%f%f%f%f", &a, &b, &c, &x1, &x2);

	read:

	x = (x1 + x2) / 2;
	series = a * x * x + b * x + c;
	d = fabs(series);

	if (d > 0.0001)
	{
		if (x * x1 < 0)
			x = x2;
		else
			x = x1;

		goto read;
	}
	else
	{
		printf("ans=%f", x);
	}

	return 0;
}
 
Last edited:
Technology news on Phys.org
Code:
goto read;

Arf!
 
You will probably get some help faster my learning to format code so someone can read it.



Code:
main()
{
	float a,b,c,x1,x2,x,series;
	double d;

	printf("enter a,b,c and x1(pos) & x2(neg)");
	scanf("%f%f%f%f%f", &a, &b, &c, &x1, &x2);

	read:

	x = (x1 + x2) / 2;
	series = a * x * x + b * x + c;
	d = fabs(series);

	if (d > 0.0001)
	{
		if (x * x1 < 0)
			x = x2;
		else
			x = x1;

		goto read;
	}
	else
	{
		printf("ans=%f", x);
	}

	return 0;
}
 
okkk
sry but thnx...
 
swty todd said:
read:
x=(x1+x2)/2;
...
if (x*x1<0) x=x2 ; else x=x1;
goto read;
Note that you reassign x each time at "read:" so the if statement assignements have no effect. I'm not famliar with the algorithm, but you probably should be assigning x1 or x2 in your if statement:

read:
x=(x1+x2)/2;
...
if (x*x1<0) x2=x ; else x1=x;
goto read;

Would have been nice if the previous posts offered some constructive help.
 
thnx...i will try my best...
 
Jeff Reid said:
Would have been nice if the previous posts offered some constructive help.

It would be even nicer if you decided to keep your opinions on earlier posts to yourself. :wink:
 
shoehorn said:
It would be even nicer if you decided to keep your opinions on earlier posts to yourself. :wink:

yeah...dude and guess what ...it would be much more nicer if u followed what u preach :wink: :wink:
 
i edited my code..bt still isn't workin..wld b glad 2 recive ur help

Code:
#include<stdio.h>
#include<math.h>
main()
{float a,b,c,x1,x2,x,series;
double d;



printf("enter a,b,c and x1(pos) & x2(neg)");
scanf("%f%f%f%f%f",&a,&b,&c,&x1,&x2);


x=0;
d=1;

 while(d>0.0001)
    {
      x=(x1+x2)/2;
      

     series=a*x*x+b*x+c;
      

     d=fabs(series);
      

         if(x*x1 < 0)
             x2=x;
         else
             x1=x;
    }

     

printf("ans=%f",x);



return 0;
}
 
  • #10
The if statement should multiply the function(x1) * function(x), not x1 and x themselves. Using a new variable called series1:

series = a*x*x + b*x + c;
series1 = a*x1*x1 + b*x1 + c;
...
if (series * series1) < 0) x2 = x; else x1 = x;
...

Also note that the initial values for x1 and x2 need to be chosen so that

a*x1*x1 + b*x1 + c > 0
a*x2*x2 + b*x2 + c < 0

A parabola would have to cross the x-axis twice for this method to work, and there would be two possible answers. The initial values for x1 and x2 would have to chosen so that only one of the answers would occur in the range x1 .. x2.
 
Last edited:
  • #11
u r rite..series and series1 get multiplied and the main equation shld better not b quadratic..thnx...
 

Similar threads

Back
Top