我正在尝试从终端编译并运行一个简单的C程序。这是我的代码:
#include <stdio.h>
//integers that are 3n, 3n + 1, and 3n + 2
int main(){
int n,i,a,b,c;
scanf("%d", &n);
for (;n>0;n--) {
scanf("%d", &i);
if(n%3==0){a+=1;}
if(n%3==1){b+=1;}
if(n%3==2){c+=1;}
}
printf("%d %d %d", &a,&b,&c);
return 0;
}
编译一次后会产生此警告:
warning: format specifies type 'int' but the argument has type 'int *' [-Wformat]
我不明白为什么它会生成该警告,因为我没有分配任何指针变量。它为什么产生它?
此外,在编译之后,我的c文件变成符号的次数增加了几倍。我不明白为什么会这样。 我的文件名是73.c,这就是我在终端上编译它的方式:
gcc 73.c -o 73.c
之后我的文件转换为这样的东西(这行和行):
œ˙Ì˛����Ä��������Ö� ��������H���__PAGEZERO��������������������������������������������������������(��__TEXT����������������������������������������������������__text����������__TEXT����������0�����˝�������0���������������Ä������������__stubs���������__TEXT����������.������������.��������������Ä�����������__stub_helper���__TEXT����������<�����$�������<���������������Ä������������__cstring�������__TEXT����������`������������`�����������������������������__unwind_info���__TEXT����������l�����H�������l������������������������������__eh_frame������__TEXT����������∏�����@�������∏��������������������������������Ë���__DATA��������������������������������������������������__nl_symbol_ptr�__DATA���������������������������������������������������__la_symbol_ptr�__DATA����������������������������������������������������H���__LINKEDIT������� ������������� ������`��������������������"��Ä0���� ����� ������������� �� ���@ ��0���������∏ ����� !��@���
我真的不知道发生了什么。我已经很久了,我知道它不应该那么难但我真的不知道出了什么问题。有人知道可能会发生什么吗?
答案 0 :(得分:4)
这里有两件大事是错的:
a
,b
和c
。你想打印整数,所以改变这个:
printf("%d %d %d", &a,&b,&c);
到此:
printf("%d %d %d", a, b, c);
我还添加了一些空格,但这只是为了美化代码,使其更具可读性,不会影响执行。
您不想打印地址!这就是你收到警告的原因。在此处阅读更多内容:What is the difference between the * and the & operators in c programming?
正如achelper所提到的,你不应该按照自己的意愿编译你的程序(但这不是代码的问题),但这是你应该习惯的东西。我将演示我为编译程序所做的工作(我也喜欢使用-Wall
标志,它会启用所有警告):
C02QT2UBFVH6-lm:~ gsamaras$ nano main.c
C02QT2UBFVH6-lm:~ gsamaras$ gcc main.c -Wall -o main
main.c:13:22: warning: format specifies type 'int' but the argument has type
'int *' [-Wformat]
printf("%d %d %d", &a,&b,&c);
~~ ^~
main.c:13:25: warning: format specifies type 'int' but the argument has type
'int *' [-Wformat]
printf("%d %d %d", &a,&b,&c);
~~ ^~
main.c:13:28: warning: format specifies type 'int' but the argument has type
'int *' [-Wformat]
printf("%d %d %d", &a,&b,&c);
~~ ^~
main.c:11:16: warning: variable 'c' is uninitialized when used here
[-Wuninitialized]
if(n%3==2){c+=1;}
^
main.c:5:16: note: initialize the variable 'c' to silence this warning
int n,i,a,b,c;
^
= 0
main.c:10:16: warning: variable 'b' is uninitialized when used here
[-Wuninitialized]
if(n%3==1){b+=1;}
^
main.c:5:14: note: initialize the variable 'b' to silence this warning
int n,i,a,b,c;
^
= 0
main.c:9:16: warning: variable 'a' is uninitialized when used here
[-Wuninitialized]
if(n%3==0){a+=1;}
^
main.c:5:12: note: initialize the variable 'a' to silence this warning
int n,i,a,b,c;
^
= 0
6 warnings generated.
C02QT2UBFVH6-lm:~ gsamaras$ ./main
2
1
2
1461300256 1461300252 1461300248C02QT2UBFVH6-lm:~ gsamaras$
通常,如果有疑问,将警告视为错误,那么让我们将这些代码一起调试,这将很有趣。专业提示:从第一个警告开始并解决它。在那之后,下一个警告可能是相关的,因此没有你真正阅读它们就消失了!
在应用我的建议后,与您在问题中描述的内容相关的所有警告都消失了。只留下未初始化的变量。
这很简单,只需将变量初始化为某个初始值,如-1,如果这样做有道理的话。
在您的情况下,您使用a
,b
和c
作为计数器!所以你必须将它们初始化为0,否则你开始将你的值添加到垃圾值,并调用未定义的行为!
所以改变这个:
int n,i,a,b,c;
到此:
int n, i, a = 0, b = 0, c = 0;
没有计数器的初始化,我得到了在我的机器上,现在
C02QT2UBFVH6-lm:~ gsamaras$ ./main
2
1
2
1505029184 1 1C02QT2UBFVH6-lm:~ gsamaras$
你看到printf()
给了我一些垃圾,而在我的计数器初始化为0后,我得到了:
C02QT2UBFVH6-lm:~ gsamaras$ ./main
2
1
2
0 1 1C02QT2UBFVH6-lm:~ gsamaras$
答案 1 :(得分:2)
gsamaras提到的要点是正确的,你应该使用:
printf("%d %d %d", a, b, c);
您使用的语法用于
scanf(传递需要填充数据的变量的地址。)
在定义时也初始化变量。
关于编译: 名称-o指定编译后生成的输出文件,因此它会覆盖您的源代码(即73.c)。 如果省略-o,则默认输出文件将生成为a.out。因此,对于编译,您可以使用以下命令并以./a.out:
运行该程序gcc 73.c
man gcc引用:
-o文件 将输出放在文件文件中。这适用于生成的任何类型的输出,无论是可执行文件,目标文件还是汇编程序 文件或预处理的C代码。
如果未指定-o,则默认为将可执行文件放在a.out中,source.so中的source.suffix对象文件,source.s中的汇编程序文件, source.suffix.gch中的预编译头文件,以及标准输出上的所有预处理C源。