我收到错误:“Test.exe中0x00d23737处的未处理异常:0xC0000005:访问冲突读取位置0x8a8c0344”执行以下代码时会发生这种情况:
int main(int argc, char* argv[])
{
string My_String_Array[30720];
Initialize_My_String_Array (My_String_Array); //i really doubt that there is something wrong in the definition of this function
for (int i=0;i<=30719;i++)
{
cout<<My_String_Array[i]<<endl;
}
system("pause");
return 0;
}
Initialize_My_String_Array()的代码如下:
void Initialize_My_String_Array (My_String_Array[30720])
{
string Initialization_Values[]={"A","B","C","D","E"};
int Index=0;
int i=0;
int j=0;
while (i<=30719)
{
My_String_Array[i]=Initialization_Values[Index];
i++;
j++;
if (j=6144)
{
j=0;
Index++;
}
}
任何想法都会受到赞赏。
答案 0 :(得分:5)
您的代码中存在许多问题,尤其是Initialize_My_String_Array
。
void Initialize_My_String_Array (My_String_Array[30720])
{
string Initialization_Values[]={"A","B","C","D","E"};
int Index=0;
int i=0;
int j=0;
while (i<=30719){
My_String_Array[i]=Initialization_Values[Index];
i++;
j++;
if (j=6144){
j=0;
Index++;
}
}
}
My_String_Array[30720]
,但您还没有给它一个类型。从上下文来看,我认为类型应该是std::string
。if
声明中,您已撰写if(j = 6144)
。 =
不是等于运算符,它是赋值运算符。 ==
是相等运算符。您的代码将6144分配给j
,然后将(非零)值提升为布尔值,这使得if语句始终执行,这意味着Index
总是递增。在循环的5次迭代中,Index
增加超过Initialization_Values
的大小,这会导致未定义的行为,并且在您的情况下会导致访问冲突。根据上下文,我认为更正的代码如下所示:
//Type is now correctly defined.
void Initialize_My_String_Array (string My_String_Array[30720])
{
string Initialization_Values[]={"A","B","C","D","E"};
int Index=0;
int i=0;
int j=0;
while (i<=30719){
My_String_Array[i]=Initialization_Values[Index];
i++;
j++;
if (j == 6144){
j=0;
Index++;
}
}
}
除此之外,您需要花一些时间重新评估您编写此代码的方式。 C风格的数组通常被认为是C ++中的不良实践,并且您的代码是一个完美的案例研究,为什么。 C ++ std::array
对象要优越得多,对于像这样的任何代码都应该是首选。最重要的是,你沉迷于一些涉及索引和数组边界的意大利面条代码。有更好的方法来处理它。我已经写了我认为是这个代码更好的版本:
void Initialize_My_String_Array (std::array<std::string, 30720> & My_String_Array)
{
const std::array<std::string, 5> Initialization_Values{"A","B","C","D","E"};
const size_factor = (My_String_Array.size() / Initialization_Values.size());
for(size_t index = 0; index < My_String_Array.size() && (index / size_factor) < Initialization_Values.size(); index++) {
My_String_Array[index] = Initialization_Values[index / size_factor];
}
}
int main(int argc, char* argv[]) {
std::array<std::string, 30720> My_String_Array;
Initialize_My_String_Array (My_String_Array);
for(std::string const& s : My_String_Array) {
std::cout << s << std::endl;
}
system("pause");
return 0;
}
还有一件事:在代码的某个地方,您已经编写了using namespace std;
之类的内容。摆脱它。 It's bad practice并且让您的代码更难以为其他用户解释(&#34; string
是自定义类型还是std::string
?&#34;)