(无符号数>=0)引起的警告

2012-08-16 10:10:39 2k 阅读 AI 写同款 · GEO 优化AI 选题数据监测一键分发 ›

ARM Compiler 警告深度解析：从 3 个高频警告（#177, #186, #223）看代码质量提升本文深入解析ARM Compiler三个高频警告（#177, #186, #223），揭示其背后潜藏的代码质量问题，并提供可落地的解决方案。这些警告涉及未引用变量、无符号数比较和隐式函数声明，可能引发跨平台兼容性和内存安全等严重问题。文章还对比了ARM与x86架构的差异表现，并给出系统化的防御策略和编译器配置建议，帮助开发者提升嵌入式代码质量。阅读详情

头文件
#define FOO 0

c文件
void f(unsigned int n)
{
if (n >= FOO)
{
dosomething();
}

按照程序的逻辑，不管FOO的定义是0还是1还是多少，都是要做这个比较的。
编译器给了个警告，认为无符号数和0比较没有任何意义。
老板不允许有任何警告，但是我又不想把这个比较去掉，因为万一哪天有人改了头文件，比如#define FOO 10，那就又需要加这个比较了，但是很可能就想不起来。
有没有两全其美的好办法?

我用的编译器是ARM的realview

谢谢了。

...全文

Keil常见错误警告 1、 warning: #767-D: conversion from pointer to smaller integer 解释：将指针转换为较小的整数影响：可能造成的影响：容易引起数据截断，造成不必要的数据丢失。如果出现bug，很难调试。改正：尽量避免这种转换，避免不了要确定转换的数据不会引起数据丢失。 2、 warning: #177-D:variable "i 阅读详情

bluejays

C语言 70,039 成员 243,237 内容

+ 关注 + 加入社区

切换为时间正序

bluejays 2012-08-19

打赏
举报

谢谢同学们。
好办法很多阿，而且好像也不太难。不过我的思路有点不太灵活，碰到类似这种有点像脑筋急转弯的问题就转不过来，嘿嘿

yimingxinshou 2012-08-16

打赏
举报

感觉10楼说的挺好。。。

lin51616780 2012-08-16

打赏
举报

#if foo == 0
dosomething();
#else
if (n >= FOO)
{
dosomething();
}
#endif

也许这样可以大概把试试

hitcser01 2012-08-16

打赏
举报

《C专家编程》说过，除非确实有必要，否则不要用unsigned，尤其是该变量理论上不应该存在负数时（例如年龄）不要用unsigned，中文版P24

lin51616780 2012-08-16

打赏
举报

无符号数恒大于或等于0 无符号数最小是0
这个比较没有意义你仔细想想你在做什么

AnYidan 2012-08-16

打赏
举报

关闭这个警告

AnYidan 2012-08-16

打赏
举报

关闭这个警告

Code_Talk 2012-08-16

打赏
举报

[Quote=引用 6 楼的回复:]
#define FOO 0
改成
const unsigned int F00 = 0;
试试
[/Quote]
这样可以啊。

冻结 2012-08-16

打赏
举报

#define FOO 0
改成
const unsigned int F00 = 0;
试试

hello_world000 2012-08-16

打赏
举报

[Quote=引用 3 楼的回复:]
最好还是修改void f(unsigned int n)的原型为void f(int n)
因为在#define FOO 0 时，FOO的类型默认是int的
[/Quote]
我想说的是，万一FOO定义为一个负数，判断的时候就会出问题的

hello_world000 2012-08-16

打赏
举报



void f(unsigned int n)

{

    if (n > FOO || n == FOO)

    {

        dosomething();

    }

}

改成这样也好傻啊

hello_world000 2012-08-16

打赏
举报

最好还是修改void f(unsigned int n)的原型为void f(int n)
因为在#define FOO 0 时，FOO的类型默认是int的

nice_cxf 2012-08-16

打赏
举报

[Quote=引用 1 楼的回复:]

利用短路
if (!FOO || n >= FOO)
{
dosomething();
}
[/Quote]
大概没用，还会警告
最简单的就是加个临时变量:
void f(unsigned int n)
{
unsigned int tmp=0;
if (n >= (FOO+tmp))
{
dosomething();
}
不过代码看起来很傻

aozhi 2012-08-16

打赏
举报

利用短路
if (!FOO || n >= FOO)
{
dosomething();
}

mymtom 2012-08-16

打赏
举报

条件编译嘛



#define FOO     0



void f(unsigned int n)

{

#if (FOO > 0)

        if (n >= FOO)

#endif

        {

                n++;

        }

}

Joseph-Growth 2012-08-16

打赏
举报

(n-FOO>=0)

赵4老师 2012-08-16

打赏
举报

if ((signed int)n >= FOO)

proorck6 2012-08-16

打赏
举报

你们老板有问题。

(无符号数>=0)引起的警告

相关推荐