#include<iostream.h>
#include<math.h>
#define PI 3.1415926;
int main()
{
    float r,h,l,s1,s2,v1,v2;
    cin>>r>>h;
    l=2*PI*r;
    s1=PI*r*r;
    s2=4*PI*r*r;
    v1=4/3*PI*r*r*r;
    v2=PI*r*r*h;
    cout<<l<<endl;
    cout<<s1<<endl;
    cout<<s2<<endl;
    cout<<v1<<endl;
    cout<<v2<<endl;
    return 0;
}

PI的值是“ 3.1415926; ”注意那个分号，带入表达式相当于
l = 2*3.1415926;*r;
……
也就是r被当做指针了
把#define PI 3.1415926;中的分号去掉
还有浮点数的精度问题，改成double型吧

有两种那样的定义：
#define PI 3.1415926   //这种不加分号，常在C语言里用
const PI = 3.1515926;  //这种要加分号，写C++程序还是用这吧

在"cmath"头文件C++语言有"M_PI"常量.

比如 :

#include <iostream>
#include <cmath>
using namespace std;
int main()
{
    cout.precision(20);
    cout << M_PI << endl;
}

结果 :

3.141592653589793116

细节很头疼啊

好样

#define 不用分号 ps:其实也真麻烦,语句都以;结尾,在c标准里#define不算语句吧

太感谢各位了。
我问题解决了！！！！！！！
很高兴！！！！！！！！！！！

呃，第一，宏定义不是语句，它只是机械替换，第二，用const定义常量代替宏定义。第三，用内联函数代替宏定义。总之，对于宏，少用