使用ispunct的函数断言失败
Assertion Failure on function using ispunct
在构建"ReplacePuncsWithBlanks"函数后，我遇到断言失败。这与我在数组的开头和结尾插入空格有什么关系吗？
#include <iostream>
#include <ctype.h>
using namespace std;
const int SIZE = 100;
void GetString(char StringArray[]);
void CopyString(char StringArray[], char CopiedArray[]);
void MakeUpper(char CopiedArray[]);
void InsertBlanks(char CopiedArray[]);
void ReplacePuncsWithBlanks(char CopiedArray[]);
void DisplayString(char StringArray[], char CopiedArray[]);
int main()
{
char StringEntered[SIZE];
char CopiedString[SIZE];
GetString(StringEntered);
CopyString(StringEntered, CopiedString);
MakeUpper(CopiedString);
InsertBlanks(CopiedString);
ReplacePuncsWithBlanks(CopiedString);
DisplayString(StringEntered, CopiedString);
cout<<"Hit Enter.n";
cin.ignore();
}
/***************************************GetString********************************************
* Action: Prompts user to enter some text. *
* *
* Parameters: *
* IN: *
* *
* OUT: StringArray which is the array holding the original text inputted by *
* the user *
* *
* Returns: Nothing. *
* *
* Precondition: StringArray points to StringEntered array in Main. *
*********************************************************************************************/
void GetString(char StringArray[])
{
cout<<"Please enter a sentence: ";
cin.getline(StringArray, SIZE);
}
/***************************************CopyString********************************************
* Action: Copies data from one array (StringArray) to another (CopiedArray). *
* *
* Parameters: *
* IN: StringArray which has existing data. *
* *
* OUT: CopiedArray which will be a copy of StringArray. *
* *
* Returns: Nothing. *
* *
* Precondition: StringArray points to StringEntered array in Main. *
* CopiedArray points to CopiedString array in Main. *
*********************************************************************************************/
void CopyString(char StringArray[], char CopiedArray[])
{
for(int i=0; i<SIZE; ++i)
{
CopiedArray[i]=StringArray[i];
}
}
/**************************************MakeUpper*********************************************
* Action: Changes characters in the array to an uppercase letter. *
* *
* Parameters: *
* IN: CopiedArray which has mixed case letters. *
* *
* OUT: CopiedArray which will have uppercase letters. *
* *
* Returns: Nothing. *
* *
* Precondition: CopiedArray points to CopiedString array in Main. *
*********************************************************************************************/
void MakeUpper(char CopiedArray[])
{
char c;
for(int i=0; i<SIZE; ++i)
{
c=CopiedArray[i];
CopiedArray[i]=toupper(c);
}
}
/***********************************InsertBlanks*********************************************
* Action: Moves elements in the array up one spot and replaces the first element *
* with a space as well as the last element.
* *
* Parameters: *
* IN: CopiedArray prior to inserting the spaces. *
* *
* OUT: CopiedArray which will have a space in the first element and at the end.*
* *
* Returns: Nothing. *
* *
* Precondition: CopiedArray points to CopiedString array in Main. *
*********************************************************************************************/
void InsertBlanks(char CopiedArray[]){
int temp=0;
for (int i=SIZE-1; i>0; --i)
{
if(CopiedArray[i]==' ')
{
temp=i+1;
}
CopiedArray[i]=CopiedArray[i-1];
}
CopiedArray[0]=' ';
CopiedArray[temp]=' ';
CopiedArray[temp+1]=' ';
}
/********************************ReplacePuncsWithBlanks**************************************
* Action: Changes characters which are punctuation to whitepaces. *
* *
* Parameters: *
* IN: CopiedArray which may include punctuation. *
* *
* OUT: CopiedArray which will have have replaced punctuations with whitespaces.*
* *
* Returns: Nothing. *
* *
* Precondition: CopiedArray points to CopiedString array in Main. *
*********************************************************************************************/
void ReplacePuncsWithBlanks(char CopiedArray[])
{
for (int i=0; i<SIZE; ++i)
{
if (ispunct(CopiedArray[i]))
{
cout<<CopiedArray[i];
}
}
}
void DisplayString(char StringArray[], char CopiedArray[])
{
for(int i=0; i<SIZE; ++i)
{
if(StringArray[i]!=' ')
{
cout<<StringArray[i];
}
else
{
i=(SIZE-1);
}
}
cout<<endl;
for(int x=0; x<SIZE; ++x)
{
if(CopiedArray[x]!=' ')
{
cout<<CopiedArray[x];
}
else
{
x=(SIZE-1);
}
}
cout<<endl;
}
有什么想法吗？
在main函数的开头，StringEntered和CopiedString都包含垃圾。
GetString函数从输入中读取一行，并将StringEntered的开头替换为您输入的内容。然后跟随一个 字节，剩余的垃圾将保留下来。
稍后，CopyString将您的输入、 字节和垃圾复制到CopiedString。
然后，函数ReplacePuncsWithBlanks对该存储器块的每个字节进行操作。对于每个字节，您调用ispunct，这个函数非常特殊。它不希望它的参数是char类型(您可能认为是这样)，而是unsigned char类型。有关更多详细信息，请参阅文档。
问题是，在您的平台上，char的值可能在−128到127之间，并且字符串末尾的垃圾肯定包含一些负值。这些值不允许作为ispunct函数的参数。
你很幸运，你得到了一个断言错误。你的程序也可能会悄无声息地失败，或者让你的电脑爆炸。这被称为未定义行为。
要解决此问题，您需要将参数强制转换为正确的类型：
void ReplacePuncsWithBlanks(char CopiedArray[])
{
for (int i=0; i<SIZE; ++i)
{
if (ispunct((unsigned char)CopiedArray[i]))
{
cout<<CopiedArray[i];
}
}
}
更好的方法是在看到字符串字节的末尾时立即停止循环：
for (int i = 0; i < SIZE && CopiedArray[i] != ' '; i++) {
但即便如此，你也需要类型转换。
相关文章：
- 尝试使用 std：：vector<std：：thread时出现静态断言失败错误>
- uint_not_usable_without_attribute在业力规则中使用数字生成器时静态断言失败
- 从 exe 文件 (Visual Studio ） 启动时调试断言失败
- 在 CppUnit 中测试中止断言失败
- 使用扫描的调试断言失败
- 为什么我的Qt程序在断言失败后继续运行?
- 图片不显示，关闭时出错 --> 调试断言失败！表达式：is_block_type_valid(标头>_block_use)
- 访问提升:shared_ptr 主范围外崩溃，断言失败:px ！= 0.指针的正确用法是什么?
- C++ 调试断言失败 - 矢量下标超出视觉工作室的范围
- 错误:断言失败 (src.type(） == CV_8UC1） 在阈值中
- Boost Beast 异步服务器失败，断言失败:(id_ ！= T::id） 在多个 aync 调用中
- 使用imwrite OpenCV时断言失败
- 调试断言失败的缓冲区！=nullptr
- 使用 ofstream 写入文本文件时断言失败
- 提升继续恢复断言失败
- 获取有关调试断言失败的错误：表达式："(_Ptr_user &(_BIG_ALLOCATION_ALIGNMENT -1)) == 0" &&0
- OpenCV(3.4.1) 错误：断言失败(变暗 <= 2 && step[0] > 0) in cv：：Mat：：locateROI
- 在Win10上使用带有Qt5的OpenCV3.0，调试断言失败
- MFC GetDC() 断言失败 - "Not a window"
- OpenCV 错误:断言失败 - 矩阵减去
