2

我需要转换命令行中给出的参数，例如： $ myprogram hello world

并且单词需要用大写字母打印。除了访问双指针数组以使用 toupper() 进行更改之外，我可以做任何事情

static char **duplicateArgs(int argc, char **argv)
{
    char **copy = malloc(argc * sizeof (*argv));
    if(copy == NULL){
        perror("malloc returned NULL");
        exit(1);
    }

    int i;
    for(i = 0; i<argc; i++){
         copy[i] = argv[i];   
    }
    char **temp;
    temp = &copy[1];
    *temp = toupper(copy[1]); 

    return copy;
}
4

5 回答 5

3 
*temp = toupper(copy[1]);

toupper如果要转换整个字符串，则转换单个字符：

char *temp = copy[1]; /* You don't need a double pointer */
size_t len = strlen(temp);

for (size_t i = 0; i < len; i++) {
    temp[i] = toupper(temp[i]); 
}
于 2015-02-18T06:29:55.380 回答
1

从函数原型的手册页是toupper()

int toupper(int c);

在您的代码中，参数copy[1]不是int值。

相反，您想要的是检查每个元素，如果它们是小写，则将它们转换为大写。伪代码看起来像

for(i = 0; i<argc; i++){
         copy[i] = malloc(strlen(argv[i])+ 1);  //allocate memory 

for (j = 1; j < argc; j++)
  for (i = 0; i < strlen(argv[j]); i++)
  {
    if (islower(argv[j][i]))   //check if it is lower case
         copy[j-1][i] = toupper(argv[j][i]);
    else
         copy[j-1][i] = argv[j][i];          //do not convert
  }
于 2015-02-18T06:23:37.570 回答
1

我假设传递给您的函数的参数char **argv是直接从 main 传递的，因此它表示一个指向指向每个命令行参数的指针数组开头的指针。

argc表示命令行参数的数量。

在您的函数内部，您创建一个新缓冲区，然后将 argv 的内容复制到其中。因此，您正在创建指向命令行参数的指针数组的副本，而不是命令行参数字符串本身。

我猜您打算复制字符串，而不是指向字符串的指针（这有什么意义？）。我建议您查看函数 strdup 和/或 strncpy 来复制实际的字符串。

这也解释了“toupper”不能按预期工作 - 不是将单个字符传递给它，而是传递一个指向以空字符结尾的字符串的指针。

于 2015-02-18T06:33:12.907 回答
0

您在使用该toupper()函数时遇到错误，因为您试图传入一个字符串而不是单个字母。这是描述该功能的手册页的摘录：

DESCRIPTION
     The toupper() function converts a lower-case letter to the corresponding
     upper-case letter.  The argument must be representable as an unsigned
     char or the value of EOF.

您有一个指向指针的指针，您可以将其可视化为这样的东西。在 C 中，字符串只是chars 的数组，因此您需要取消引用两次才能获取第二级数组（单个字母）中的数据。每次添加 an 时，*您都可以将其视为删除一层指针。您可以将*运算符视为运算符的逆&运算。

这条线是你的问题线

temp = &copy[1];

试试这个

//This is a pointer to an individual string
char *temp = copy[1];
//Keep going while there are letters in the string
while(*temp != NULL) {
    //convert the letter
    toupper(*temp);
    //Advance the pointer a letter
    temp++;
}
于 2015-02-18T06:58:52.173 回答
0

考虑这个例子：

#include <stdlib.h>
#include <stdio.h>
#include <ctype.h>
#include <string.h>

static char **duplicateArgs(int argc, char **argv)
{
    char **copy = NULL;
    // allocate memry for pointers to new lines
    copy = (char **)malloc(sizeof(char *) * argc);
    int line, chr;
    for(line = 0; line < argc; line++)
    {
        // allocate memory for new line
        copy[line] = (char *)malloc(sizeof(char) * (strlen(argv[line]) + 1));
        // copy with changes
        for(chr = 0; chr <= strlen(argv[line]); chr++)
        {
            copy[line][chr] = toupper(argv[line][chr]);
        }
    }
    return copy;
}

int main(int argc, char * argv[])
{
    char ** strs;
    int i;
    strs = duplicateArgs(argc, argv);
    for(i = 0; i < argc; i++)
    {
        printf("%s\n", strs[i]);
    }
    return 0;
}

编辑：

您也可以决定使用 argv[0]（可执行文件的名称）并根据需要更改代码。malloc还可以添加结果检查，以及其他改进...如果您需要:-)

于 2015-02-18T07:05:48.683 回答