- html - 出于某种原因,IE8 对我的 Sass 文件中继承的 html5 CSS 不友好?
- JMeter 在响应断言中使用 span 标签的问题
- html - 在 :hover and :active? 上具有不同效果的 CSS 动画
- html - 相对于居中的 html 内容固定的 CSS 重复背景?
所以基本上我想读取一个文本文件,其中的数字代表拍卖、正在出售的元素数量、有多少报价以及实际报价。我要做的就是输出最高报价作为售价,但由于某种原因,我的程序只打印出最后的价格,而不是最高的价格。
#include <stdio.h>
//main function
int main() {
//Declaring variables and arrays
float numberBids[15], max, sum = 0 ,numberAuc;
float bids[10];
int i, y, j= 0, x, z = 0;
char filename[100]= "";
//User puts in filename
printf("Please enter the name of the file.\n");
scanf("%s",&filename);
//Opens file
FILE * finp=fopen(filename,"r");
//Scans info
fscanf(finp, "%f" , &numberAuc);
for(i=0; i < numberAuc; i++){
fscanf(finp, "%f",&numberBids[i]);
for (x = 0; x < numberBids[i];x++)
{
fscanf(finp, "%f", &bids[i]);
max = bids[i];
}
//Replaces old max with newer one if larger
for(j; j<numberBids; j++)
if (bids[i]>max)
max = bids[i];
}
//Sum
sum += bids[i];
//Print out to the output
for ( y = 0; y < numberAuc; y++ )
{
y = y+ 1;
printf("Auction %d was sold for $%.2f\n", y, bids[z]);
z++;
y = y- 1;
}
//Close
fclose(finp);
return 0;
}
这是文本文件的内容
5
4
100 500 250 300
1
700
3
300 150 175
2
920 680
8
20 10 15 25 50 30 19 23
最佳答案
您没有重新分配实际的出价
金额。代码中有很多未使用和无意义的语句(我会用更多内容更新),但主要部分是你并没有真正用你的 max 做任何事情
变量。
您的代码(带有一些添加的注释):
for (x = 0; x < numberBids[i];x++){
/*Why are you using the `i` variable in a loop of `x`?*/
fscanf(finp, "%f", &bids[i]);
/*Setting the max to each as it's read in - doesn't do anything except waste cycles*/
max = bids[i];
}
/*
* You are comparing j to a pointer, numberBids here.
* You're saying: "while j is less than some memory address"
*/
for(j; j<numberBids; j++){
/*
* You aren't using the j variable anyways, so you're comparing the same
* two numbers here every iteration (max and bid[i], which doesn't change
* until the next auction since you are using `i`)
*/
if (bids[i]>max){
max = bids[i];
}
}
据我所知,您的代码只需要执行以下操作:
for each auction:
get number of bids
get bid amounts
get the maximum of these bids
所以代码应该如下所示:
/*for each auction*/
for(int i=0; i < numberAuc; i++) {
/*get number of bids*/
fscanf(finp, "%f", &numberBids[i]);
float max = 0;
for (int x = 0; x < numberBids[i]; x++){
/*get bid amounts*/
fscanf(finp, "%f", &bids[x]);
/*find the maximum*/
max = bids[x] > max ? bids[x] : max;
}
printf("Auction %d was sold for $%.2f\n", i, max);
}
与大括号保持一致。其中一些位于下一行,而其他则与需要确定作用域的语句(for 循环等)位于同一行。另外,它可能看起来更干净,但我个人建议在一行语句周围使用大括号,例如 for
/if
。如果您稍后需要扩展它,它们就在那里,而且如果您正确对齐,它还可以提高可读性...
可以在您使用变量的范围内声明变量,而不是在顶部全部声明。如今的编译器已经足够智能,可以优化初始化,因此您无需担心在循环中创建 100500 个 int ,并且预先声明预留空间现在也不必太担心。这是需要保持一致的其他事情 - 因此,如果您坚持将所有范围的声明放在函数的顶部,请对所有内容都这样做。
特别是因为您是初学者 - 将编译器警告视为错误(事实上,编译器有自动执行此操作的标志,以强制您修复它们)。它可以避免代码中的一些问题(例如将迭代整数与指针进行比较)。
获取文件名的方法被认为是不安全的 (scanf
)。
这样做:
char filename[100];
fgets(filename, sizeof(filename), stdin);
使用 fgets
从 stdin 获取用户输入有一个警告 - 它包括按 ENTER 后的 \n
换行符。只需在该符号处终止字符串即可:
unsigned len = strlen(filename)-1;
filename[len] = '\0';
打开文件时,执行某种错误处理,以防 fopen
返回 NULL。如果它没有正确打开,则该程序中不会发生任何其他事情,因为它全部依赖于文件中的内容,而且您还会抛出一个空指针。
最后,使用int
来存储整数。 float
可以工作,但是 float 可能......变化无常。 numberBids[15]
、bids[10]
和 numberAuc
都可以而且应该是整数。
关于c - 我的输出不稳定,我们在Stack Overflow上找到一个类似的问题: https://stackoverflow.com/questions/40066213/
我正在使用 OUTFILE 命令,但由于权限问题和安全风险,我想将 shell 的输出转储到文件中,但出现了一些错误。我试过的 #This is a simple shell to connect t
我刚刚开始学习 Java,我想克服在尝试为这个“问题”创建 Java 程序时出现的障碍。这是我必须创建一个程序来解决的问题: Tandy 喜欢分发糖果,但只有 n 颗糖果。对于她给第 i 个糖果的人,
你好,我想知道我是否可以得到一些帮助来解决我在 C++ 中打印出 vector 内容的问题 我试图以特定顺序在一个或两个函数调用中输出一个类的所有变量。但是我在遍历 vector 时收到一个奇怪的错误
我正在将 intellij (2019.1.1) 用于 java gradle (5.4.1) 项目,并使用 lombok (1.18.6) 来自动生成代码。 Intellij 将生成的源放在 out
编辑:在与 guest271314 交流后,我意识到问题的措辞(在我的问题正文中)可能具有误导性。我保留了旧版本并更好地改写了新版本 背景: 从远程服务器获取 JSON 时,响应 header 包含一
我的问题可能有点令人困惑。我遇到的问题是我正在使用来自 Java 的 StoredProcedureCall 调用过程,例如: StoredProcedureCall call = new Store
在我使用的一些IDL中,我注意到在方法中标记返回值有2个约定-[in, out]和[out, retval]。 当存在多个返回值时,似乎使用了[in, out],例如: HRESULT MyMetho
当我查看 gar -h 的帮助输出时,它告诉我: [...] gar: supported targets: elf64-x86-64 elf32-i386 a.out-i386-linux [...
我想循环遍历一个列表,并以 HTML 格式打印其中的一部分,以代码格式打印其中的一部分。所以更准确地说:我想产生与这相同的输出 1 is a great number 2 is a great
我有下面的tekton管道,并尝试在Google Cloud上运行。集群角色绑定。集群角色。该服务帐户具有以下权限。。例外。不确定需要为服务帐户设置什么权限。
当尝试从 make 过滤非常长的输出以获取特定警告或错误消息时,第一个想法是这样的: $ make | grep -i 'warning: someone set up us the bomb' 然而
我正在创建一个抽象工具类,该类对另一组外部类(不受我控制)进行操作。外部类在某些接口(interface)点概念上相似,但访问它们相似属性的语法不同。它们还具有不同的语法来应用工具操作的结果。我创建了
这个问题已经有答案了: What do numbers starting with 0 mean in python? (9 个回答) 已关闭 7 年前。 在我的代码中使用按位与运算符 (&) 时,我
我写了这段代码来解析输入文件中的行输入格式:电影 ID 可以有多个条目,所以我们应该计算平均值输出:**没有重复(这是问题所在) import re f = open("ratings2.txt",
我需要处理超过 1000 万个光谱数据集。数据结构如下:大约有 1000 个 .fits(.fits 是某种数据存储格式)文件,每个文件包含大约 600-1000 个光谱,其中每个光谱中有大约 450
我编写了一个简单的 C 程序,它读取一个文件并生成一个包含每个单词及其出现频率的表格。 该程序有效,我已经能够在 Linux 上运行的终端中获得显示的输出,但是,我不确定如何获得生成的显示以生成包含词
很难说出这里要问什么。这个问题模棱两可、含糊不清、不完整、过于宽泛或夸夸其谈,无法以目前的形式得到合理的回答。如需帮助澄清此问题以便重新打开,visit the help center . 关闭 1
1.普通的输出: print(str)#str是任意一个字符串,数字··· 2.格式化输出: ?
我无法让 logstash 正常工作。 Basic logstash Example作品。但后来我与 Advanced Pipeline Example 作斗争.也许这也可能是 Elasticsear
这是我想要做的: 我想让用户给我的程序一些声音数据(通过麦克风输入),然后保持 250 毫秒,然后通过扬声器输出。 我已经使用 Java Sound API 做到了这一点。问题是它有点慢。从发出声音到
我是一名优秀的程序员,十分优秀!