Multiplication Interpreted as Pointer

[jalfor]

Not much to say really...this isn't working and I don't have a clue how to fix it without putting them all in separate statements which I assume you wouldn't have to do. Anyway, here it is:

angle = 2 * PI * i / circle_points;

If it matters, angle is a double, PI is a #defined double, i is an integer and circle_points is an integer.

Thanks for any help

[Moschops]

This code below does not interpret integers as pointers. What are you doing differently?

#include <iostream>
#define PI 3.14159

int main()
{
double angle;
int i = 10;
int circle_points = 50;
angle = 2 * PI * i / circle_points; 

std::cout << angle;

return 0;
}

[jalfor]

hmm...well, here is the full code:

#include <SFML\Window.hpp>
#define PI 3.141592653589793238462643383279503;
#define X .525731112119133606 
#define Z .850650808352039932
using namespace std;

sf::Window App(sf::VideoMode(800, 600, 32), "OpenGL Thing");

int main()
{
	while (App.IsOpened())
	{
		GLint circle_points = 100; 
		glBegin(GL_LINE_LOOP); 
		for (int i = 0; i < circle_points; i++) 
		{    
			double angle = 2 * PI * i / circle_points; 
			glVertex2f(cos(angle), sin(angle)); 
		} 
		glEnd();
		App.Display();
	}
}

The error is on the 'i', it says Operand of '*' must be a pointer.

[1vRz3TCk]

#define PI 3.141592653589793238462643383279503; delete the ;

[jalfor]

Thanks, that fixed it

Why does the semi-colon break it? Wouldn't it just create an empty statement?

[1vRz3TCk]

When you compile the preprocessor replaces PI in the code with 3.141592653589793238462643383279503;, so the line with the error becomes:
double angle = 2 * 3.141592653589793238462643383279503; * i / circle_points;

Edit:
It would be better to use:
const double PI = 3.1etc.; instead of #define...