C 程序显示多个打印

Question

我写了一个 c 代码来计算电费 它给出了我期望的输出，但它重复输出太多我试图调试它但似乎不知道我在哪里弄错了有人可以查看我的代码吗？ 我只希望第一个输出可见，而不是所有这些结果

#include <stdio.h>
#include <stdlib.h>

int main(){
    GetInputs();
    CalConsumCharge();
    CalExtraCharges();
    DispBill();
}

void GetInputs(){
    int unittot;
    int unit1;
    int Prev_Cons;
    printf("Enter Previous units consumption: ");
    scanf("%d",&Prev_Cons);
    printf("Enter the units consumed: ");
    scanf("%d",&unittot);
    unit1 = unittot - Prev_Cons;
    CalConsumCharge(unit1);
}

void CalConsumCharge(int unit){
    int total = 0;
    if(unit<=50){
        total = (unit*22);
    }
    else if(unit<=100){
        total = ((50*22)+(unit-50)*30);
    }
    else if(unit<=200){
        total = ((50*22)+(50*30)+(unit-100)*36);
    }
    else if(unit<=350){
        total = ((50*22)+(50*30)+(100*36)+(unit-200)*70);
    }
    else if(unit<=650){
        total = ((50*22)+(50*30)+(100*36)+(150*70)+(unit-350)*90);
    }
    else if(unit<=1000){
        total = ((50*22)+(50*30)+(100*36)+(150*70)+(300*90)+((unit-650)*135));
    }
    else{
        total = ((unit*145));
    }

    CalExtraCharges(total);
}

void CalExtraCharges(int total){
    int total1;
    total = total/100;
    if(total>=2000){
        total1 = total + 10 +20;
    }
    else{
        total1 = total +10;
    }
    DispBill(total1);
}

void DispBill(int total1){
    if(total1>=2000){
        printf("A penalty Charge was Placed Because your bill is over 2000 L.E");
        printf("Bill amount is: %d L.E",total1);
    }
    else if(total1<2000) {
        printf("your bill is : %d L.E",total1);
    }
}

Answer 1

你应该编译你的代码，然后看看会发生什么（错误、警告）。

首先，您忘记声明函数。在执行 main 函数之前添加以下声明。

void GetInputs();
void CalConsumCharge(int unit);
void CalExtraCharges(int total);
void DispBill(int total1);

其次，因为你在GetInputs中调用CalConsumCharge，在CalConsumCharge中调用CalExtraCharges等等，所以你只要在main函数中调用GetInputs就够了。

int main(){
    GetInputs();
}

如果你想在主函数中调用许多函数作为你的代码，你应该在每个函数的末尾返回值，如下所示：

所有函数的声明：

int GetInputs();
int CalConsumCharge(int unit);
int CalExtraCharges(int total);
void DispBill(int total1);

对每个函数进行一点修正

int GetInputs() {
   ...
   unit1 = unittot - Prev_Cons;
   return unit1;
}

int CalConsumCharge(int unit) {
    ...
    else{
        total = ((unit*145));
    }
    return total;
}

int CalExtraCharges(int total) {
   ...
   else{
        total1 = total +10;
    }
    return total1;
}

然后，在主函数中：

int main () {
   int unit1 = GetInputs();
   int total = CalConsumCharge(unit1);
   int total1 = CalExtraCharges(total);
   DisBill(total1);

   return 0;
}

Answer 2

好的，这段代码需要很多的爱和关注。我很惊讶你可以编译并运行它。您的主函数只需要调用 GetInputs()，因为它会调用所有其他函数。

我重构了主要代码和其余代码以保持整洁。我没有改变任何逻辑，它运行良好。看看：

#include <stdlib.h>
#include <stdio.h>

void DispBill(int total1)
{

    if(total1>=2000)
    {
        printf("A penalty Charge was Placed Because your bill is over 2000 L.E");
        printf("Bill amount is: %d L.E",total1);
    }
    else if(total1<2000) 
    {
        printf("your bill is : %d L.E",total1);
    }
}

void CalExtraCharges(int total)
{
    int total1;
    total = total/100;

    if(total>=2000)
    {
        total1 = total + 10 +20;
    }
    else
    {
        total1 = total +10;
    }

    DispBill(total1);
}

void CalConsumCharge(int unit)
{
    int total = 0;

    if(unit<=50)
    {
        total = (unit*22);
    }
    else if(unit<=100)
    {
        total = ((50*22)+(unit-50)*30);
    }
    else if(unit<=200)
    {
        total = ((50*22)+(50*30)+(unit-100)*36);
    }
    else if(unit<=350)
    {
        total = ((50*22)+(50*30)+(100*36)+(unit-200)*70);
    }
    else if(unit<=650)
    {
        total = ((50*22)+(50*30)+(100*36)+(150*70)+(unit-350)*90);
    }
    else if(unit<=1000)
    {
        total = ((50*22)+(50*30)+(100*36)+(150*70)+(300*90)+((unit-650)*135));
    }
    else
    {
    total = ((unit*145));
    }

    CalExtraCharges(total);
}

void GetInputs(void)
{
    int unitTotal;
    int unitOne;
    int Prev_Cons;

    printf("Enter Previous units consumption: ");
    scanf("%d",&Prev_Cons);

    printf("Enter the units consumed: ");
    scanf("%d",&unitTotal);

    unitOne = unitTotal - Prev_Cons;

    CalConsumCharge(unitOne);
}

int main()
{
    GetInputs();
}

这是我的输出：