将旧 cstring 的标记连接到新的 c 字符串

Question

我们的教授给了我们一个回文作业，在这个作业中我们需要编写一个函数来删除所有标点符号，空格，并将 c 字符串中的大写字母转换为小写字母。我遇到的问题是当我 debug/run 它时，在我输入函数的 cstring 之后，它给出了 "Debug Assertion failed" 错误，并且只给出了 c 字符串输入的小写字母版本的输出.有没有人建议我如何修复或改进这段代码？

更新：我通过像 geeksforgeeks 那样标记字符串来修复我的错误。但现在我遇到的问题是，当将 s cstring 的标记连接到 new_s c-string 时，它只连接第一个s 到 new_s 的令牌。这是我的代码：

#define _CRT_SECURE_NO_WARNINGS //I added this because my IDE kept giving me error saying strtok is unsafe and I should use strtok_s. 
#include <iostream>
#include <iomanip>
#include <cstring>
using namespace  std;

/*This method removes all spaces and punctuation marks from its c-string as well as change any uppercase letters to lowercase. **/
void removePuncNspace(char s[])
{
    char new_s[50], *tokenptr;

    //convert from uppercase to lowercase
    for (int i = 0; i < strlen(s); i++) (char)tolower(s[i]);

    //use a cstring function and tokenize s into token pointer and eliminate spaces and punctuation marks
    tokenptr = strtok(s, " ,.?!:;");

    //concatenate the first token into a c-string.
    strcpy_s(new_s,tokenptr);

    while (tokenptr != NULL)
    {
        tokenptr = strtok('[=10=]', " ,.?!:;"); //tokenize rest of the string
    }

    while (tokenptr != NULL)
    {
        // concat rest of the tokens to a new cstring. include the [=10=] NULL as you use a cstrig function to concatenate the tokens into a c-string.
        strcat_s(new_s, tokenptr);
    }

    //copy back into the original c - string for the pass by reference.
    strcpy(s, new_s);
}

我的输出是：

输入一行：
汉娜看到蜜蜂了吗？汉娜做到了！
是回文

Answer 1

首先，正如@M.M所说，当你想继续标记同一个字符串时，你应该调用strk(NULL, "..")，而不是'[=12=]'。

其次，你的程序逻辑没有多大意义。您将字符串 s 拆分为子字符串，但实际上从未将它们连接到 new_s。当你到达第二个 while 时，tokenptr 肯定是 NULL，所以你永远不会进入循环。

为了修复您的代码，我将两个 while 合并为一个，并添加了一个 if 以在 tokenptr 为 NULL 时不调用 strcat(new_s, tokenptr)。

void removePuncNspace(char s[])
{
    char new_s[50], *tokenptr;

    //convert from uppercase to lowercase
    for (int i = 0; i < strlen(s); i++) (char)tolower(s[i]);

    //use a cstring function and tokenize s into token pointer and eliminate spaces and punctuation marks
    tokenptr = strtok(s, " ,.?!:;");

    //concatenate the first token into a c-string.
    strcpy(new_s,tokenptr);

    while (tokenptr != NULL)
    {
        tokenptr = strtok(nullptr, " ,.?!:;"); //tokenize rest of the string
        if (tokenptr != NULL)
            strcat(new_s, tokenptr);
    }

    //copy back into the original c - string for the pass by reference.
    strcpy(s, new_s);
}

P.S：我使用了非安全版本的 cstring 函数，因为出于某种原因，我的编译器不喜欢安全版本。

将旧 cstring 的标记连接到新的 c 字符串

concatenating tokens of an old cstring into a new c-string

c++

runtime-error

c-strings

palindrome