c - 使用指针手动复制字符串时出错

Question

我正在创建这个程序作为大学作业的一部分。目标是将 char* slogan = "Comp10120 is my favorite module"; 复制到一个新字符串，同时删除辅音并将所有字母大写。这是我的代码:

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

void printStrings();

char *slogan = "Comp10120 is my favourite module";
char *p = slogan;
char *slogan_copy = NULL;

int main ()
{   
    //Get size of original string
    int slogan_size = 0;
    while (*p++ != '\0')
        slogan_size++;

    // Dynamically allocate memory to copy of string
    slogan_copy = (char*) malloc ((slogan_size+1) * sizeof(char));
    //Place string terminator at end of copy string
    slogan_copy[slogan_size] = '\0';

    //Reset p pointer to start of string
    p = slogan;
    int offset = 0;

    while (*p != '\0')
    {
        //If the current element in the string is a consonant,
        //or as defined in the if statement,
        //if p is not a vowel and is between a and z or A and Z:
        if ((!(*p == 'a' || *p == 'e' || *p == 'i' || *p == 'o' || *p == 'u')) 
            && (((*p > 'a') && (*p < 'z')) || ((*p > 'A') && (*p < 'Z'))))
            p++;
        else
            //Copy element to slogan_copy and capitalise
            slogan_copy[offset++] = *p++;
            slogan_copy[offset] = toupper(slogan_copy[offset]);
    }

    //Place string terminator after last element copied.
    slogan_copy[offset] = '\0';

    printStrings();

    return 0;
}

void printStrings ()
{
    printf("Origianl String: %s\n",*slogan);
    printf("Modified String: %s",*slogan_copy);
}

当我尝试执行时，出现错误

initializer element is not constant
 char *p = slogan;
           ^~~~~~

我假设这是因为我试图对 slogan 执行操作，就好像它只是一个常规字符数组，而不是指向字符串的指针。但是，我不知道如何修复此错误。

除此之外，我尝试将 char*slogan = "Comp10120 is my favorite module"; 更改为 char slogan[] = "Comp10120 is my favorite module"; 出于好奇，看看它是否可行。它符合要求，但在执行时崩溃。关于如何修改我的代码以使其工作的任何想法？

Answer 1

你的程序有很多错误。请考虑您对全局变量的使用，并考虑在需要的地方使用 const，但这是一个非常好的起点，所以我测试了您的程序，它似乎可以通过 4 个简单的更正工作:

1.移除全局环境中的p初始化

8: //char *p = slogan;
9: char *p;

1. 在 main block 中设置 p

   主要内容(){ p = 口号； ...

2. 从 printf 语句中的 slogan 中删除 astrix 它已经是一个指向 char 数组的指针

   printf("原始字符串: %s\n",slogan);printf("修改后的字符串:%s",slogan_copy);

希望对你有帮助