3

我正在尝试在 C 中实现 rot13 算法。但由于我对该语言不是很熟悉，所以我的代码在这里遇到了一些问题。

基本上，我想将 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;

}

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

4

4 回答 4

7

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

#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;
}
于 2012-11-24T11:44:34.360 回答
2

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

关于代码：

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

________ 在此处输入图像描述 -------------- ASCII码！

    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;
        }
    }
于 2012-11-22T21:50:30.593 回答
1

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;       
  }
于 2012-11-22T21:40:52.730 回答
1

此函数可以对 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; }
    }
}
于 2013-01-31T22:24:52.063 回答