c-在双指针数组中将字符串改为大写

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

并且这些单词需要打印在CAPS中。除了访问双指针数组以使用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;
}

*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]); 
}

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

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

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

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

这也解释了"toupper"不能像您预期的那样工作——您不是向它传递单个字符，而是向以null结尾的字符串传递指针。

从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
  }

考虑这个例子：

#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("%sn", strs[i]);
    }
    return 0;
}

编辑：

此外，您还可以决定是否使用argv[0]（可执行文件的名称），并在需要时更改代码。此外，还可以增加对malloc结果的检查，并进行其他改进。。。如果您需要：-）

使用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中，字符串只是一个char的数组，因此需要解引用两次才能获得第二级数组（单个字母）中的数据。每次添加*时，都可以将其视为删除一层指针。你可以把*算子看作是&算子的逆。

这条线是你的问题线

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++;
}

相关内容

最新更新

热门标签：