Question

我正在尝试在C中实现rot13-algorithm。但由于我对这种语言不太熟悉，我的代码就出现了问题。

基本上，我想将args []中的每个字母旋转到13个位置。但是这段代码看起来很缓慢：

#include <stdio.h>

char[] rotate(char c[]) {
  char single;
  int i;
  int alen = sizeof(c)/sizeof(c[0]);
  char out[alen];

  for(i=0;i<=alen;i+=1) {
    if(c[i]>='a' && (c[i]+13)<='z'){
      out[i] = c[i]+13;
    }
  }

  return out;
}

int main(int argc, char *argv[]) {
  printf("The given args will be rotated\n");
  int i;
  char rotated[sizeof(argv)/sizeof(argv[0])];

  rotated = rotate(argv);

  /* printing rotated[] later on */
  return 0;

}

我知道这里有很多漏洞 - 你能告诉我如何解决这个问题吗？

Answer 1

非常感谢，我用这段代码解决了这个问题

#include <stdio.h>

int rot13(int c){
  if('a' <= c && c <= 'z'){
    return rot13b(c,'a');
  } else if ('A' <= c && c <= 'Z') {
    return rot13b(c, 'A');
  } else {
    return c;
  }
}

int rot13b(int c, int basis){
  c = (((c-basis)+13)%26)+basis;
  return c;
}

int main() {
  printf("The given args will be rotated");
  int c;
  while((c = getchar()) != EOF){
    c = rot13(c);
    putchar(c);
  }
  return 0;
}

Answer 2

@Michael如何说这个char out[alen]不被编译器接受，因为你不能声明一个非常量值的数组大小。您的代码的另一个问题是for循环for( i = 0; i < = alen; i+=1 )数组从0开始，所以如果你执行for，直到lenght's位置，你将不在数组中。

关于代码：

您必须使用指向字符串开头的指针作为函数的参数，因为您不能在C中返回数组（但您可以返回指针）。
您的if( str[i] >= 'a' && (str[i]+13) <='z')不正确，因为您会将某些字母转换为符号来查看。

________ enter image description here -------------------------- ASCII CODE ！

    void rotate( char * str ) 
    {
        int i = 0;

        /* You do this until you find a '\0' */
        for( i = 0; str[ i ] != '\0' ; i++ ){

            /* Use the pointer notation if you passed a pointer. */
            /* If the letter is between a and m you can simply  sum it. */
            if( *( str + i ) >= 'a' && *( str + i ) < 'n')
                *( str + i ) += 13;       

            /* If the letter is between the n and z you have to do the opposite.*/
            else if( *( str + i ) >= 'n' && *( str + i ) <= 'z')
                *( str + i ) -= 13;
        }
    }

Answer 3

C中的数组大小必须在编译时设置，因此不能对数组大小使用非常量表达式。

考虑以下实施：

// in place rotate
void rotate(char *str) 
// str must be a zero-terminated string
{
  int i =0;
  // loop until str itself is not NULL and str[i] is not zero 
  for(i=0;str && str[i]; ++i) // ++i is a pre-increment
  {
    if(str[i] >= 'a' && (str[i]+13) <='z')
    {
      str[i] = str[i]+13;       // modifying str in place
    }
  }
}

然后您的main()可能如下所示：

int main(int argc, char *argv[]) 
{  
  printf("The given args will be rotated: %s\n", argv[1]);

  rotate(argv[1]);

  printf("Rotated: %s\n", argv[1]);
  return 0;
}

更新更高级版本的转换，负责处理str[i] + 13 > 'z'

时的情况

  for(i=0;str && str[i]; ++i) // ++i is a pre-increment
  {
      // ignore out of range chars
      if (str[i] < 'a' || str[i] > 'z') continue;
      // rotate 
      for (off = 13; off > ('z' - str[i]); ) 
      {
          off-= (1 + 'z' - str[i]);
          str[i] = 'a';
      }
      str[i]+=off;       
  }

Answer 4

此函数可以对rot13字符串进行编码/解码。它与VIM的g? rot13编码器兼容。

void rot13 (char *s) {
    if (s == NULL)
        return;

    int i;
    for (i = 0; s[i]; i++) {
        if (s[i] >= 'a' && s[i] <= 'm') { s[i] += 13; continue; }
        if (s[i] >= 'A' && s[i] <= 'M') { s[i] += 13; continue; }
        if (s[i] >= 'n' && s[i] <= 'z') { s[i] -= 13; continue; }
        if (s[i] >= 'N' && s[i] <= 'Z') { s[i] -= 13; continue; }
    }
}

关于Rot13-实施

4 个答案: