这是在 C 中执行此操作的正确方法吗？

Question

我应该更改书面代码中的某些内容吗？编译器说一切正常——没有错误或警告。

• 您正在建造一个新家，并且您已经准确计算了地基所需的水泥量。
• 理想情况下，您想购买确切数量的水泥，但商店只出售 120 磅袋装的水泥。
• 这些包每个售价 45 美元。

请编写一个 C 程序，计算您为建造地基而必须购买的水泥成本。

• 您的程序应首先读取一个十进制数字，该数字表示新家地基所需的水泥量（以磅为单位）。
• 然后，您的程序应显示您必须购买的水泥袋的总成本，以获得足够的水泥来建造地基。
• 为了使您的程序更简单，您需要保证所需的水泥量永远不会是 120 的倍数。

到目前为止我的代码：

#include <stdio.h>
#include <math.h>
int main(void) {
    int price=45, totalPrice, OneBag=120;
    float needed;
    do
    {
        printf("Enter the amount of cement you need for your foundation that is not dividable by 120: ");
        scanf("%f", &needed);
    } while (fmodf(needed, OneBag)==0);
    
    totalPrice = ((int)needed/OneBag+1)*(price);
    printf("Total cost of cement you will need for your foundation is %d", totalPrice);
    

    return 0;
}

Answer 1

说实话，我没有看到您的代码中有任何真正的错误，但在我看来，还有改进的余地：

每袋的价格和袋子的大小都是常量，您实际上可以在代码中明确说明，这样更具可读性，并且可以让编译器更好地优化您的代码。

您实际上也不必检查输入是否是 120 的倍数，因为它不是。

还有一个叫做ceil 的函数（或者ceilf，如果使用浮点数），它实际上接受任何数字并向上增加到最接近的整数。这对这个作业非常有用。

最后一件事仅仅因为编译器说没问题，并不意味着它实际上是。

代码：

#include <stdio.h>
#include <math.h>

const float price_per_bag = 45;
const float bag_size = 120;

int main(void) {
    float needed;
    printf("Enter the amount of cement needed:\n");
    scanf("%f", &needed);

    float price = ceilf(needed / bag_size) * price_per_bag;
    printf("total price: %i\n", (int)price);
}

Answer 2

#include <stdio.h>

int main(void){
      
   float totalC;
   int count=0, totalPrice, i ;
      
   scanf("%f", &totalC);

   for (i=0; i<totalC; i+=120)
        {
           count = count+1;   
         }
     

         totalPrice = count *45;
         printf("%d", totalPrice);
         return 0;

      }