12个滑稽的C语言面试问答——《12个有趣的C语言问答》评析(4)

前文链接:http://www.cnblogs.com/pmer/p/3324063.html

8,Making changes in Code segment
Q:以下代码运行时一定会崩溃,你能说出原因吗? 

#include<stdio.h>

int main(void)
{
    char *ptr = "Linux";
    *ptr = 'T';

    printf("
 [%s] 
", ptr);

    return 0;
}

A:这是因为,通过 *ptr = 'T',此行代码尝试更改只读内存存储的字符串'Linux'。此操作当然行不通所以才会造成崩溃。

Answer: This is because, through *ptr = ‘T’, the code is trying to change the first byte of the string ‘Linux’ kept in the code (or the read-only) segment in the memory. This operation is invalid and hence causes a seg-fault or a crash.

评:

  很难说这段代码一定崩溃(crash)或发生段错误(seg-fault)。
  在C标准中,修改String literal(The program attempts to modify a string literal)是一种UB(undefined behavior),即未定义行为。用通俗的话来讲,就是C语言没规定这样写有意义。这种情况下,什么事情都可能发生,无论发生什么,都是代码的错误,是程序员的责任。

  事实上,修改String literal可能导致各种结果,有些情况下也可能执行时并不产生崩溃或段错误。

  K&R认为修改String literal应该规定为unspecified behavior(不同于undefined behavior,unspecified behavior可能有多种结果,但代码本身不存在语义错误,而undefined behavior是没有语义),但标准委员会并没有采纳K&R的建议。

  对照一下原文,不难发现译文漏掉了in the code segment。

  但原文说string literal存于代码段或只读段,也是不正确的。这种说法没有依据。C语言并没有规定实现应该把string literal存放在什么地方,这是由实现自己安排的事情。

9,Process that changes its own name Q:你能否写一个程序在它运行时修改它的名称? (Can you write a program that changes its own name when run?)
A:以下代码可以完成 :
Answer: Following piece of code tries to do the required : 

#include<stdio.h>

int main(int argc, char *argv[])
{
    int i = 0;
    char buff[100];

    memset(buff,0,sizeof(buff));

    strncpy(buff, argv[0], sizeof(buff));
    memset(argv[0],0,strlen(buff));

    strncpy(argv[0], "NewName", 7);

    // Simulate a wait. Check the process
    // name at this point.
    for(;i<0xffffffff;i++);

    return 0;
}

评:

  这个问答比较雷人。

  首先标题是要改变Process的名字,问题是Process有名字吗?

  但在Question中又说要在运行时改Program的名字。如果说Program有名字,那么就只可能是对应可执行文件的名字。可执行文件的名字和Process的名字是无法划等号的。

  Process通常是通过PCB(Process Control Block)管理,其标识的方法通常是id号,当然,PCB中也有所谓外部标识符。如果把PCB的外部标识符理解为Process的名字,虽然极为牵强,但舍此似乎也无法再有任何其他解释了。

  查了一下,某些语言中似乎有Process Name这个概念,也有相关的函数。但是C语言中没有这个概念。

  再看代码,无非是修改了一下argv[0]所指向的字符串而已。尽管argv[0]指向的字符串确实是程序的名称,但这种改变没有实际意义——下次启动程序还得用原来的名字,因为外存中可执行文件的名字并没有改变。

  代码中的注释部分又提到了Process Name,就这样一会Process Name,一会又是Program Name,题目的设计者自己明显概念不清。

  注释说在这里模拟等待,搞不清究竟要在这里等什么。怎么看都觉得是在装模作样。
  再来看代码风格。 

 int i = 0;
  //……
 for(;i<0xffffffff;i++);

  把for语句分为两部分来写,风格乖张。

  把循环体部分的空语句写在行尾,也是一种恶习。对比一下下面的写法就知道了: 

for ( i = 0 ; i < 0xffffffff ; i ++ )
   ;

  这条for语句本身的语义在一定条件下也成问题。假如int类型的最大值为0x7fffffff,当i值达到int类型最大值之后,i再自加就溢出了。而溢出,是一种未定义行为(UB)。

  也就是说,这条语句说的很可能并不是让i值从0逐次加1一直变化到0xffffffff-1。代码作者心里想的究竟是什么,也许他自己都不知道。顺便说一下,int类型的最大值为0x7fffffff时,0xffffffff不是int类型,而是unsigned int类型。

    char buff[100];

    memset(buff,0,sizeof(buff));

    strncpy(buff, argv[0], sizeof(buff));
    memset(argv[0],0,strlen(buff));

       strncpy(argv[0], "NewName", 7);

  这段代码,依然是很蠢。首先莫名其妙地把buff清0: 

  memset(buff,0,sizeof(buff));

  实际上去掉这句没有任何问题。因为后面的 

  strncpy(buff, argv[0], sizeof(buff));

  还要再次填充。举个例子,下面代码段 

   char s[4];
   strncpy( s , "1" , 4);
   printf("%d %d %d %d
",s[0],s[1],s[2],s[3]);

的输出,一定是 

  49 0 0 0

  这就说明,原来代码中memset()填充的0毫无意义,因为紧接着还要再填充一次。我不是说memset()没用,但到现在为止我看到的大多数memset()都是稀里糊涂地在做无用功,是在滥用memset()。再次说明了很多使用memset()的人根本不知道自己的代码究竟是在做什么。所谓对“大型商业程序,这是良好代码风格的习惯”云云,纯粹是胡扯。

  后面一句

memset(argv[0],0,strlen(buff));

不但没用,逻辑上也有点荒唐。应该

memset(argv[0],0,strlen(argv[0]));

才合理。尽管我们知道,这里strlen(argv[0])和strlen(buff)的值是一样的。 

strncpy(argv[0], "NewName", 7);

  这一句属于不懂装懂、煞有介事。作者使用strncpy()希望不至于拷贝越界,但这里的7是"NewName"的长度,而不是argv[0]所指向字符串的长度。如果strlen(argv[0])小于7,显然还会发生越界。 

  综上所述,这段代码不但不可能改变Program Name,连正确地修改命令行参数都谈不上,更不要说改变Process Name了。  

  所以,这个问答的“问”,是一个哗众取宠的伪问题。“答”则是生拉硬扯、胡编滥造的解答。

原文地址:https://www.cnblogs.com/pmer/p/3327262.html