Question

我正在尝试将密码加扰器从Javascript复制到C。它的作用是取字母的ASCII字符代码，将其置于底层，除以它，并从给定列表中获取随机字符。

Javascript版本：

function getScrambledPassword(pwd) {
    var cipher = ['k', 's', 'z', 'h', 'x', 'b', 'p', 'j', 'v', 'c', 'g', 'f', 'q', 'n', 't', 'm'];
    var result="";
    if (pwd == null)
        pwd = "";
    pwd = encodeURIComponent(pwd);
    //alert("encoded password: " + pwd);
    for(var i=0;i<pwd.length;i++) {
            var cc = pwd.charCodeAt(i);
        result += cipher[Math.floor(cc/16)] + cipher[cc%16];
    }
    //alert("scrambled password: " + result);
    return result;
}

正在运行的加扰器示例：https://jsfiddle.net/w5db66va/

到目前为止我做了什么：

#include <stdio.h>
#include <math.h>
#include <string.h>

static char *scramblePassword(char *pwd)
{
    char *cipher[] = {
        "k", "s", "z", "h",
        "x", "b", "p", "j",
        "v", "c", "g", "f",
        "q", "n", "t", "m"
    };

    char *result = "";
    for(int i=0; i < strlen(pwd); i++)
    {
        int cc = (int) pwd[i];
        printf("%d", cc);
        result + cipher[floor(cc/16)] + cipher[cc%16];
    }
    return *result;
}

int main(void)
{
    char *test[] = {"test", "testtwo", "testthree"};
    for (int i=0;i < sizeof(test); i++)
    {
        printf("Original: %s", test[i]);
        printf("Scrambled: %s", scramblePassword(test[i]));
    }
}

我遇到的问题是，当我运行c文件（编译后）时，它根本不会输出任何内容。我做错了什么，以至于我无法按照我的预期运行它？

Answer 1

发生了什么事你很幸运。您的程序在C中调用未定义的行为。

首先看这一行

result + cipher[floor(cc/16)] + cipher[cc%16];

首先，它没有做任何事情。这只是一个被抛弃的表达。你真正想要的是：

result += cipher[floor(cc/16)] + cipher[cc%16];

但它仍然无法发挥作用，因为C并不真正具有字符串的概念。字符串实际上只是由＆＃39; \ 0＆＃39;终止的字符序列。 result只是指向这样一个序列的指针，就像任何其他指针一样，当你向它添加一些东西时，你只需要增加指针所指向的位置。

此外，返回*result实际上取消引用指针并返回它指向的内容。

声明

result = "";

在某处使用以\0终止的空字节序列（即单个nul字节）分配一些内存。在堆栈上（或在寄存器中，取决于实现），分配结果并给出nul字节的地址。

当你返回*result时，你返回nul字节，但是调用者认为你正在返回一个指针，所以它会把那个nul字节解释为指针（我很惊讶你的代码没有给出实际编译时的错误）并且该指针可能是一个空指针。

在C中连接字符串是一个棘手的操作。您必须使用strcat或其中一个更安全的衍生产品。你必须确保为结果分配足够的空间，并且必须使用malloc动态地执行它，因为当你从函数返回时，本地分配的字符串会消失。

修改

另一件事......

C有多种数字数据类型。将一个整数除以另一个整数时，会得到整数结果。如果cc不能被16整除，则结果将为floor(cc/16)

Answer 2

继续发表评论，你的问题比你初步想象的要深一些。首先，您不希望cipher成为字符串数组，您只希望它是一个字符数组，例如：

    char cipher[] = "kszhxbpjvcgfqnm";

接下来，您无法返回在函数体内声明的数组。当result返回时，scramblePassword的内存将被销毁。您的选择是（1）在result中动态分配scramblePassword（并在main中释放它），或者（2）在result中为main声明存储空间将其作为参数传递给scramblePassword。 e.g：

#define MAX 32

static char *scramblePassword (char *pwd, char *result)
{
 ...
    return result;
}

int main(void)
{
    char result[MAX] = "";
    ...
        printf ("Scrambled: %s\n", scramblePassword (test[i], result));

最后，您的算法，如果要从cipher构建一个加扰字符数组，将导致选择超出cipher范围的索引，从而导致 未定义的行为 即可。如果意图只是为result[x]分配一个值而不管它是否是有效的可打印ASCII值，那么它可能没问题。但如果第一个是你的目标，那么算法的结果必须始终产生一个在cipher范围内的值，例如类似的东西：

         result[i] = cipher[((int)floor (cc / 16) + cc % 16) % sizeof cipher];

将所有这些部分放在一起，并回顾main是int类型，因此返回一个值，您可以执行以下操作：

#include <stdio.h>
#include <math.h>
#include <string.h>

#define MAX 32

static char *scramblePassword (char *pwd, char *result)
{
    char cipher[] = "kszhxbpjvcgfqnm";
    int i;

    for (i = 0; i < (int)strlen (pwd); i++)
    {
        int cc = (int) pwd[i];
        // result[i] = cipher[(int)floor (cc / 16)] + cipher[cc % 16];
        result[i] = cipher[((int)floor (cc / 16) + cc % 16) % sizeof cipher];
    }
    result[i] = 0;  /* you MUST nul-terminate to use as a string */

    return result;
}

int main(void)
{
    char *test[] = {"test", "testtwo", "testthree"};
    char result[MAX] = "";

    for (int i = 0; i < (int)(sizeof test/sizeof *test); i++)
    {
        printf ("\nOriginal : %s\n", test[i]);
        printf ("Scrambled: %s\n", scramblePassword (test[i], result));
    }

    return 0;
}

示例使用/输出

然后会产生以下可读输出：

$ ./bin/pwscramble

Original : test
Scrambled: ffgf

Original : testtwo
Scrambled: ffgffmb

Original : testthree
Scrambled: ffgffmcff

我将留给您研究该算法实际上应该做什么。如果您还有其他问题，请与我们联系。

Answer 3

C不是JS。

有许多微妙的问题，你的程序甚至都没有编译。

你可能想要这个：

#include <stdio.h>
#include <math.h>
#include <string.h>

static char *scramblePassword(char *pwd, char *result)
{
  char cipher[] = {    // you need an array of chars here, not an
                       // array of pointers to char
    'k', 's', 'z', 'h',
    'x', 'b', 'p', 'j',
    'v', 'c', 'g', 'f',
    'q', 'n', 't', 'm'
  };

  size_t i;
  for (i = 0; i < strlen(pwd); i++)
  {
    int cc = (int)pwd[i];
    //printf("%d", cc);
    result[i] = cipher[/*(int)floor*/(cc / 16)] + cipher[cc % 16];
                           // ^ actually you can drop the floor function,
                           // there is no floating point here, so integer
                           // division will do the job
  }

  result[i] = 0;  // this will NUL terminate the string
  return result;
}

int main(void)
{
  char *test[] = { "test", "testtwo", "testthree" };
  for (int i = 0; i < sizeof(test) / sizeof(test[0]); i++)
                    // ^ the number of elements is not sizeof(test)
                    // (that's the number of bytes the array takes in memory
                    // but sizeof(test) / sizeof(test[0])
  {
    char result[100];  // you cannot return arrays in C, you need to provide
                       // the array and pass the pointer to your function
    printf("Original: %s\n", test[i]);
    printf("Scrambled: %s\n", scramblePassword(test[i], result));
  }
}

创建密码扰码器

3 个答案: