Chinaunix首页 | 论坛 | 博客
  • 博客访问: 136358
  • 博文数量: 27
  • 博客积分: 0
  • 博客等级: 民兵
  • 技术积分: 136
  • 用 户 组: 普通用户
  • 注册时间: 2019-08-05 22:12
个人简介

纸上得来终觉浅,绝知此事要躬行。 生命不息,奋斗不止。

文章分类

全部博文(27)

文章存档

2021年(1)

2020年(26)

我的朋友

分类: C/C++

2020-05-16 17:49:44

这篇文章是本人在开发过程中遇到的一个问题,问题导致的原因至少有两个,一是对C语言认识不彻底,二是编程时的细心程度,对每一行代码的思考不足。其实内容真的没什么好些的,刚学C语言的人,都能看的很明白,写一下只是想把自己踩到到的坑暴露出来,来告诫自己以后不能再犯这样的错误。


起因:在一个函数中,有何查找XX的函数,需要用到memcmp()这个函数,这个函数本身没有问题,而在于在接返回值得时候出现了问题,例如 ret = memcmp(),之后一般都是需要判断返回值 ,问题就出现在ret 这个变量的类型上,现在你应该会说我,不就是个返回值吗?用int类型的变量去接住就好了,没错,我第一次就是定义的 int  ret ;然后用ret = memcmp(),就好了,是的,到这里完全没有问题,程序功能OK,但是后面出现了一件事。
   
这件事就是,我们项目组有规定,不能使用标椎的C语言的定义变量的格式,必须采用Linux内核的方式,采用变量typedef的形式,为了编程规范的需要,也为了后续移植的方便,类型定义如下:
#ifndef __BIT_TYPES_DEFINED__
#define __BIT_TYPES_DEFINED__
typedef __u8 u_int8_t;
typedef __s8 int8_t;
typedef __u16 u_int16_t;
typedef __s16 int16_t;
typedef __u32 u_int32_t;
typedef __s32 int32_t;

#endif /* !(__BIT_TYPES_DEFINED__) */

typedef __u8 uint8_t;
typedef __u16 uint16_t;
typedef __u32 uint32_t;
#if defined(__GNUC__)
typedef __u64 uint64_t;
typedef __u64 u_int64_t;
typedef __s64 int64_t;
#endif

/*
 * __xx is ok: it doesn't pollute the POSIX namespace. Use these in the
 * header files exported to user space
 */
typedef __signed__ char __s8;
typedef unsigned char __u8;

typedef __signed__ short __s16;
typedef unsigned short __u16;

typedef __signed__ int __s32;
typedef unsigned int __u32;

typedef __signed__ long __s64;
typedef unsigned long __u64;
#endif /* __ASSEMBLY__ */
这些都是在标椎内核的实现(截取了部分),分别在int_I64.htypes.h里面,有兴趣可以自己去看看源码,
于是为了统一风格,我将int类型换成了u_int32_t,好了,这下问题出现了,直接将int换成了unsigned int,说到这里大家应该都明白了,类型的范围发生了变化,直接导致ret的不可能小于0,在后面的判断中,if ret < 0,就永远成立不了了,问题就在这里了,一个简单的类型直接导致函数运行结果错误,由于这个问题,本来功能实现的代码,突然变得实现不了了,顿时诧异,(当时改了很多处类型),说到这里不得不承认C语言修为不够,但是也反映出问题,在编程时,对程序背后的原理和机制,是认识不足的,在使用C语言是,更应该站在操作系统的角度,站在内存的角度去思考问题,每一个变量在内存中的存放形式,范围,这样才能避免少出Bug
下面是验证的结果:
int main(int argc, const char *argv[])
{
 
 int ret = 0;
 
 char *str1 = "A This is test string A";
 char *str2 = "B This is test string B";
 ret = memcmp(str1, str2, strlen(str1));
 if (ret > 0) 
 {
 printf(" ret > 0 \n");
 }
 else 
 {
 printf("ret <= 0 \n");
 }
 return 0;
}
结果为:ret <= 0 ,没有问题,
改变之后,再次运行;
int main(int argc, const char *argv[])
{
 
 u_int32_t ret = 0;
 
 char *str1 = "A This is test string A";
 char *str2 = "B This is test string B";
 ret = memcmp(str1, str2, strlen(str1));
 if (ret > 0) 
 {
 printf(" ret > 0 \n");
 }
 else 
 {
 printf("ret <= 0 \n");
 }
 return 0;
}
结果为:ret > 0,就改动了一个类型,由此乐见,是多么大的问题。
给自己一个大嘴巴,不能再犯了。

本文没什么技术含量,旨在告诉自己在开发过程中一定要明确自己的自己在干什么,对自己写的每一行代码,都要去思考,甚至理解每一个变量背后的深层含义,这对于以后养成良好的编程习惯,去深入思考问题,帮助很大。

最后引用《程序员的自我修养》里的一句话,"真正了不起的程序员对自己程序的每一个字节都了如指掌"

阅读(2219) | 评论(0) | 转发(0) |
给主人留下些什么吧!~~