0

所以我有一个函数，它需要一个指向“字符串”数组的指针（我将字符串理解为只是后面跟着“\0”的内存块）。由于字符串已经是指向字符串第一个字节的指针，所以我的指针实际上是一个** doublePointer。但是，我对 Ying Yang 进行了分段故障，老实说，我不知道在低级别视图中发生了什么。下面是我的代码，它的功能是读取字符并将第一个单词的第一个字母（在字符串中）和句号后大写。

    void autocaps(char ** words)
    {

    /* Add code here */
    //Period Boolean
    bool next=false;
    //First Word Boolean
    bool fcap=true;
    //Counter Variable
    int i=0;
    int j=0;
    //Second Pointer
    char** wordx = words;

    //LowerCase Bomb & Period Flagging
    while(wordx[i][j]!='\0'){
      while(wordx[i][j]!='\0'){
       //A-Z Filter
       if((wordx[i][j]>='A')&&(wordx[i][j]<='Z')){
      wordx[i][j]+=32;
       }
       if(wordx[i][j]=='.'){
      next=true;
       }
      j++;
      }
    i++;
    }

 i=0;
 j=0;
 //Cap First Word & Cap Post Period
 while(words[i]!='\0'){
   while(words[i][j]!='\0'){
    //a-z Filter
    if((words[i][j]>=97)&&(words[i][j]<=122)){
  if(fcap){
    words[i][j]-=32;
    fcap=false;
  }
  if(next){
    words[i][j]-=32;
  }
    }
    j++;
  }
 i++;
}
return;

}

当我打印通过参数传递的原始指针时，我出现了段错误。如果有人可以向我解释这个的低级概念，因为我很困惑，我到处扔三颗星和四颗星，我什至不知道它是否让我离调试代码更近或更远。

谢谢你！！

4

2 回答 2

0

您说的是“字符串数组”，但根据您的代码和您的意图（大写第一个字符和句点之后），我将假设您打算将其用于传递给函数的单个字符串。

字符串只是一个字符数组，后跟一个 0 值。你想要做的是遍历字符数组。要从字符指针获取要取消引用指针的字符。这就是为什么您在下面我们实际检查字符串中的字符的任何地方都看到“*words”的原因。

双指针、三指针和四指针使您远离解决方案。

这是一个重新设计的示例：

void autocaps(char* words)
{
  bool fcap=true; /* true so first letter is made cap */

  while (*words != '\0') 
  {
    if (fcap)
    {
        /* capitalize */
        if ((*words >= 'a') && (*words <= 'z'))
        {
            *words -= 32;
        }

        fcap = false;
    }
    else
    {
        /* not cap */
        if ((*words >= 'A') && (*words <= 'Z'))
        {
            *words += 32;
        }
    }

    /* period - cap next letter */
    if (*words == '.')
    {
        fcap = true;
    }

    /* step to next character in array */
    words++;
  }

  return;
}
于 2013-02-08T07:58:46.157 回答
0

我正在将字符串数组初始化为连续字符串，例如：

char *words[2] = {"hello", "world"};

那么您将在第一次到达时遇到分段错误

**words+=32;

因为您正在尝试修改只读位置。

您应该像这样初始化字符串：

char word1[] = {'h', 'e', 'l', 'l', 'o', 0};
char word2[] = {'w', 'o', 'r', 'l', 'd', 0};
char *words[] = {word1, word2};

另一个问题确实是

*words++;

您不想这样做有两个原因：

  1. 正如其他海报提到的那样， ++ 优先于取消引用运算符，因此您实际上是将单词数组从一个字符串增加到下一个，而不是一个字符到下一个。
  2. 解引用运算符则没有任何影响。

现在，如果你想前进到下一个字符，你不想这样做：

(*words)++

因为现在您实际上是在更改 words 数组中的指针，而您不想这样做。

于 2013-02-08T20:23:34.487 回答