1

这个问题我已经有一段时间了,我已经搜索过这种类型的错误,我相信它与内存泄漏或指向任何内容的指针有关。

我一遍又一遍地检查了我的代码,但由于我不知道如何调试它,我无法准确找到此问题发生的位置。即使我尝试在代码的第一行设置断点,它也会崩溃。

它正在从文件中读取一堆 ISBN 并检查它们是否有效。

虽然看起来很多,但逻辑很简单。

这是我的代码:

#include <iostream>
#include <fstream>
#include <iomanip>
#include <list>
#include <ctype.h>
#include <string.h>
#include <stdlib.h>
#include <vector>

using namespace std;

class Isbn
{
   private:
   string isbnCode;

   public:

   Isbn()
   {            
   }

   Isbn(string isbn): isbnCode(isbn)
   {               
   }

   ~Isbn()
   {              
   }

   string getIsbn()
   {
      return isbnCode;
   }

   void setIsbn(string input)
   {
      isbnCode = input;
   }
};

void setListOfIsbn(const string filename, list<Isbn> &listOfIsbn);
void validateIsbns(const list<Isbn> listOfIsbn, list<bool> &validations);
void printValidations(const list<Isbn> listOfIsbn, 
     const list<bool> validations);
string bToS(const bool bValue);

int main(int argc, char *argv[])
{
   list<Isbn> listOfIsbn;
   list<bool> validations;

   string inputFile = argv[1];

   setListOfIsbn(inputFile, listOfIsbn);
   validateIsbns(listOfIsbn, validations);
   printValidations(listOfIsbn, validations);

   return 0;
}

void setListOfIsbn(const string filename, list<Isbn> &listOfIsbn)
{
   list<Isbn>::const_iterator iterator;

   ifstream fin; 

   fin.open(filename.c_str()); 
   char ch;
   string isbnCode;

   while (!fin.eof()) 
   { 
      fin.get(ch);

      if (ch == '\n')
      {
         Isbn isbn;

         (isbn).setIsbn(isbnCode);
         listOfIsbn.push_back(isbn);
         isbnCode = "";         
      }
      else
      {
         isbnCode.append(reinterpret_cast<const char*>(ch));
      }
   } 
}

void validateIsbns(const list<Isbn> listOfIsbn, list<bool> &validations)
{
   list<Isbn>::const_iterator itr;

   for (itr = listOfIsbn.begin(); itr != listOfIsbn.end(); itr++)
   {
      Isbn isbn = *itr;
      string isbnCode = isbn.getIsbn();
      string isbnCodeReform = "";
      vector<int> products;
      int sumOfProducts = 0;
      unsigned int i;

      for (i = 0; i < isbnCode.length(); itr++)
      {
         if(isalnum(isbnCode[i]))
         {
            isbnCodeReform[i] = isbnCode[i];
         }
      }

      for (i = 0; i < (isbnCodeReform.length() - 1); itr++)
      {
         if(isbnCodeReform[i] == 'X')
         {
            isbnCodeReform[i] = 10;
         }         

         products[i] = isbnCodeReform[i] * (10 - i);      
      }

      vector<int>::const_iterator itr;

      for (itr = products.begin(); itr != products.end(); itr++)
      {
         sumOfProducts += products[*itr];
      }

      if ((sumOfProducts % 11) == 0)
      {
         validations.push_back(true);
      }
      else
      {
         validations.push_back(false);
      }
   }
}

void printValidations(const list<Isbn> listOfIsbn, 
     const list<bool> validations)
{
   list<Isbn>::const_iterator itr;
   list<bool>::const_iterator itr2 = validations.begin(); 

   for(itr = listOfIsbn.begin(); itr != listOfIsbn.end(); itr++)
   {
      string validate = bToS(*itr2);
      Isbn isbn = *itr;
      cout << isbn.getIsbn() + ": " + validate + "\n";
      itr2++;
   }
}

string bToS(const bool bValue)
{
   if(bValue == 0)
   {
      return "False";
   }
   else
   {
      return "True";
   }
}

任何帮助将非常感激!

4

2 回答 2

5

我试图在下面详细说明尽可能多的明显错误。可能还有更多。有了这么多错误,很明显您采用了错误的编程方法。首先你应该清楚地考虑你正在编写的代码,仅仅写一些看起来大致正确的东西是不够的,你必须通过编程来得到它正确的。其次,您正在编写太多低质量的代码。你应该写几行代码,在你再写之前让它们工作。您在这里错误地堆积了错误,因此很明显您一直在进行零测试。即使是专业人士也无法工作,更不用说初学者了。工作意味着工作,而不仅仅是编译。正如这段代码所示,编写大量可编译的代码很容易。最后学会使用调试器,它会帮助加载。显然,您设置了某种复杂的客户端/服务器。忘记这一点,在您自己的机器上安装编译器和调试器。

isbnCode.append(reinterpret_cast<const char*>(ch));

是错的

isbnCode += ch;

是你想要的。除非您真的知道自己在做什么,否则不要使用 reinterpret_cast。

while (!fin.eof()) 
{ 
    fin.get(ch);

是检查文件结尾的错误方法

while (fin.get(ch)) 
{ 

是你想要的。

事实上,整体void setListOfIsbn(const string filename, list<Isbn> &listOfIsbn)太复杂了。这完全一样,但代码少了三倍

void setListOfIsbn(const string filename, list<Isbn> &listOfIsbn)
{
    ifstream fin(filename.c_str());
    string isbnCode;
    while (getline(fin, isbnCode))
        listOfIsbn.push_back(isbnCode);
}

此代码不正确

  string isbnCodeReform = "";
  for (i = 0; i < isbnCode.length(); itr++)
  {
     if(isalnum(isbnCode[i]))
     {
        isbnCodeReform[i] = isbnCode[i];
     }
  }

因为 isbnCodeReform 是一个零长度的字符串,所以isbnCodeReform[i]会失败。可能你的意思是这个

  string isbnCodeReform = "";
  for (i = 0; i < isbnCode.length(); itr++)
  {
     if(isalnum(isbnCode[i]))
     {
        isbnCodeReform.push_back(isbnCode[i]);
     }
  }

这是相同的错误

vector<int> products;
for (i = 0; i < (isbnCodeReform.length() - 1); itr++)
{
     if(isbnCodeReform[i] == 'X')
     {
        isbnCodeReform[i] = 10;
     }         

     products[i] = isbnCodeReform[i] * (10 - i);      
}

同样 products 是一个零长度数组,所以products[i]会失败。同样,您可能的意思是

     products.push_back(isbnCodeReform[i] * (10 - i));

这是关于迭代器如何工作的困惑

  for (itr = products.begin(); itr != products.end(); itr++)
  {
     sumOfProducts += products[*itr];
  }

你可以写

  for (itr = products.begin(); itr != products.end(); itr++)
  {
     sumOfProducts += *itr;
  }

或者你可以写

  for (i = 0; i < products.size(); i++)
  {
     sumOfProducts += products[i];
  }

你所拥有的是两者的混合,这是行不通的。

于 2013-05-04T15:13:17.217 回答
0

错误发生在这里:

     isbnCode.append(reinterpret_cast<const char*>(ch));

你将一个字符(很可能是一个数字)转换为const char *- 所以我们有一个字符的值,用作指针。这几乎肯定会在任何有任何内存访问检查的系统上失败。

您应该将字符添加到您的字符串中。最简单的方法是:

     isbnCode += ch; 
于 2013-05-04T15:16:33.470 回答