#include <stdio.h>
void main()
{
double p,w,s,d,f;
      scanf("%f",&s);   
           if(s<250)
           {
               d=0;
           printf("总运费为f=p*w*s*(1-%f)",d);
           }
             else if(250<=s&&s<500)
             {
                 d=0.02;
           printf("总运费为f=p*w*s(1-%f)",d);
             }
           else if(500<=s&&s<1000)
           {
               d=0.05;
           printf("总运费为f=p*w*s*(1-%f)",d);
           }
           else if(1000<=s&&s<2000)
           {
               d=0.08;
           printf("总运费为f=p*w*s*%f",1-d);
           }
           else if(2000<=s&&s<3000)
           {
               d=0.1;
           printf("总运费为f=p*w*s(1-%f)",d);
           }
           else
           {
               d=0.15;
           printf("总运费为f=p*w*s*(1-%f)",d);

           }
}

#include <stdio.h>
int main()
{
    double p,w,s,d,f;
    scanf("%lf",&s);    
    if(s<250) {
        d=0;
    }
    else if(s<500) { 
        d=0.02;
    }
    else if(s<1000) { 
        d=0.05;
    }
    else if(s<2000) {
        d=0.08;
    }
    else if(s<3000) {
        d=0.1;
    }
    else {
        d=0.15;
    }
    printf("总运费为f=p*w*s*(1-%.2f)",d);
    return 0;
}

这么多if 不如用switch

两个错误点，一个不规范点，一个需要注意的点

错误：
1：变量声明后，需要赋值，未赋值变量真实数据为无法预测的，不可用作计算
2：对于输出函数printf()的使用方式不熟悉的话，应将运算表达式独立出来作为一个单独的语句，先学会走路，再跑，不要在概念未清晰前过度追求所谓的简洁，有效性和可靠性才是首要要求，printf()函数的使用中，在字符串符号""内的所有内容除了转义和引用，其他都会被当作字符输出处理，计算式不应置于""内，而是在 , 后的引用内容处使用计算式，最稳妥的是独立出来作为单独的语句

不规范：
main函数应当进行明确的声明，例如：
int main(void)//main函数处理并返回int类型数值(int是默认的形式)，main函数不接受任何外界参数(void)
{ }

需要注意：
不论是while还是if等等语句，位于括号内的条件判定每个判定都应当使用()括起来以保证逻辑判断按照编写者的想法正确运行，例如：
if((s>=250)&&(s<500))

为什么%f不行

scanf("%f",&s);这条语句应该改为scanf("%lf",&s),因为你的变量s定义的是double型，在C语言中double输入时的格式必须对应的是%lf，（具体是为什么一定要对应我就不知道了）我想如果不这样可能会造成数据丢失，因为float是单精度型（是4个字节），double是双精度型号（是8个字节），你可以试一下，把%f改为%lf就可以得到你想要的结果了。