访问冲突读取位置和错误读取字符串的字符

Posted 2023-02-21

【中文标题】访问冲突读取位置和错误读取字符串的字符

【英文标题】：Access violation reading location and error reading character of string

【发布时间】：2017-12-15 13:35:10

【问题描述】：

谁能解释为什么会抛出访问冲突读取位置错误以及为什么在 a[] 中我得到“错误读取字符串的字符”？我有两个字符串，必须从包含其他字符串的第一个字符串中删除所有单词。我做错了什么？

#include "stdafx.h"
#include<iostream>
#include<cstring>
using namespace std;
char s1[100] = ;
char s2[100] = ;
void Words(char s1[], char s2[]) 
  int k = 0;
  char*p1 = nullptr;
  char*np1 = nullptr;
  char*p2 = nullptr;
  char*np2 = nullptr;
  char *m[20];
  char *a[20];
  char s3[100] = ;
  for (int i = 0; i < 20; i++) 
    char n[50] = ;
    char l[50] = ;
    m[i] = n;
    a[i] = l;

  
  char delimeter[] = " ,.!?;:";
  p2 = strtok_s(s2, delimeter, &np2);
  while (p2 != nullptr) 
    strcpy(a[k], p2);
    k++;
    p2 = strtok_s(nullptr, delimeter, &np2);
  
  k = 0;
  p1 = strtok_s(s1, delimeter, &np1);
  while (p1 != nullptr) 
    strcpy(m[k], p1);
    k++;
    p1 = strtok_s(nullptr, delimeter, &np1);
  

  for (int i = 0; i < 20; i++) 
    for (int j = 0; j < 20; j++) 
      if (strcmp(m[i], a[j]) != 0 && m[i] != 0 && a[j] != 0) 
        strcat(s3, m[i]);
      
    
  
  puts(s3);
  for (int i = 0; i < 20; i++) 
    delete m[i];
    delete a[i];
  

主要功能：

int main()

  gets_s(s1);
  gets_s(s2);
  Words(s1, s2);
  return 0;

【问题讨论】：

为什么不直接使用std::string？毕竟这是可怕的 c++ 代码，更像是 c。

这是在浪费时间，而且可能弊大于利。你的老师应该感觉不好，你应该用一本好的 C++ 书来补充这门课程。

您选择一个字母的变量名称是不明智的，并且会使您的（已经很奇怪的）代码更难理解。

好的变量名不是为了编译器，也不是为了我们，而是为了你，这样你可以在调试时更容易地理解你自己的代码。 BTW Visual Studio 拥有世界一流且非常易于使用的调试器。了解如何使用它。

@ЮраБезлюдний 请指点你的老师here。

【参考方案1】：

这里至少有一个问题：

  for (int i = 0; i < 20; i++) 
    char n[50] = ;
    char l[50] = ;
    m[i] = n;
    a[i] = l;
  

在该循环之后，m 和 a 的所有元素都指向超出范围的变量。一旦离开for 循环的 之间的范围，变量n 和l 将不复存在。

您对指针有很多误解，您可能应该阅读一些有关 C 语言的好书（您编写的代码实际上是 C 而非 C++）。

肯定有更多错误。

【参考方案2】：

这里的邪恶太多了，我不知道从哪里开始。

您在静态分配的数组上调用 delete p1、p2、np1 和 np2 都未分配，您正在写信给他们

请使用string:

const regex re "([^ ,.!?;:]+)" ;
vector<string> s1Tokens sregex_token_iterator(cbegin(s1), cend(s1), re, 1), sregex_token_iterator() ;
vector<string> s2Tokens sregex_token_iterator(cbegin(s2), cend(s2), re, 1), sregex_token_iterator() ;

sort(begin(s1Tokens), end(s1Tokens));
sort(begin(s2Tokens), end(s2Tokens));

set_difference(cbegin(s1Tokens), cend(s1Tokens), cbegin(s2Tokens), cend(s2Tokens), ostream_iterator<string>(cout, "\n"));

Live Example

如果在理解了上面的例子之后你发现自己有兴趣进一步研究的话，这个例子可以很容易地进一步充实，我会从这里开始：https://***.com/a/38595708/2642059