在Windows上我没有在Mac上得到strcpy错误？

Question

- 所有修改后的代码仍然拒绝运行，请帮助 -

当我在Windows中编译我的代码时，出现内存错误。但是，在我最初编码此代码的Mac中，它工作正常。我需要在Windows上正常工作。

这与我使用strcpy处理char字符串的方式有关，Mac似乎很好（我想这与gcc和微软的做事方式有关）。

---

下面是抱怨者的代码： 的main.cpp

#include "Cust.h" 
using namespace std; 

int main (int argc, char * const argv[]) { 
    Cust customers[500]; 
    char tmpString[70] = " "; 
    char * pch = new char[255]; 
    string tmpAcctFN = " "; 
    string tmpAcctLN = " "; 
    ifstream input("P3_custData.txt"); 
    for (int idx = 0; idx < 130; idx++){ 
     input.getline(tmpString, 70, '\n'); 
     strcpy(pch,strtok(tmpString," "),255); 
     customers[idx].setAcctNum(pch); 
     cout << pch << endl; 
     strcpy(pch, strtok(NULL," "));; 
     customers[idx].setAcctFN(pch); 
     cout << pch << endl; 
     strcpy(pch, strtok(NULL," "));; 
     customers[idx].setAcctLN(pch); 
     cout << pch << endl; 
     strcpy(pch, strtok(NULL," "));; 
     customers[idx].setCurrBalance(atol(pch)); 
     cout << pch << endl; 
     strcpy(pch, strtok(NULL," "));; 
     customers[idx].setPIN(atoi(pch)); 
     cout << pch << endl; 
    } 
    input.close(); 
    return 0; 
} 

Cust.h

/* 
* Cust.h 
* Project 3 
* 
* Created by Anthony Glyadchenko on 11/17/09. 
* Copyright 2009 __MyCompanyName__. All rights reserved. 
* 
*/ 
#include <iostream> 
#include <string> 

using namespace std; 

#ifndef CUST_H 
#define CUST_H 

class Cust{ 
public: 
    char * getAcctNum(); 
    void setAcctNum(char num[]); 
    double getCurrBalance(); 
    void setCurrBalance(double balance); 
    void addToCurrBalance(double amount); 
    void subFromCurrBalance(double amount); 
    void setAcctFN(char firstName[]); 
    void setAcctLN(char lastName[]); 
    char * getAcctFN(); 
    char * getAcctLN(); 
    void setPIN(int pin); 
    int getPIN(); 

private: 
    char acctNum[255]; 
    char acctFN[255]; 
    char acctLN[255]; 
    double currBalance; 
    int pin; 
    char fileName[255]; 
}; 
#endif 

Cust.cpp

/* 
* Cust.cpp 
* Project 3 
* 
* Created by Anthony Glyadchenko on 11/17/09. 
* Copyright 2009 __MyCompanyName__. All rights reserved. 
* 
*/ 
#include <fstream> 
#include <string> 
#include <sstream> 
#include "Cust.h" 

using namespace std; 

char * Cust::getAcctNum(){ 
    return acctNum; 
} 

void Cust::setAcctNum(char num[]){ 
    strcpy(acctNum,num); 
} 

double Cust::getCurrBalance(){ 
    return currBalance; 
} 

void Cust::setCurrBalance(double balance){ 
    currBalance = balance; 
} 

void Cust::addToCurrBalance(double amount){ 
    currBalance += amount; 
} 

void Cust::subFromCurrBalance(double amount){ 
    currBalance -= amount; 
} 

void Cust::setAcctFN(char firstName[]){ 
    strcpy(acctFN,firstName); 
} 

void Cust::setAcctLN(char lastName[]){ 
    strcpy(acctLN,lastName); 

} 

char * Cust::getAcctFN(){ 
    return acctFN; 
} 

char * Cust::getAcctLN(){ 
    return acctLN; 
} 

void Cust::setPIN(int pin){ 
    Cust::pin = pin; 
} 

int Cust::getPIN(){ 
    return pin; 
} 

这里是我的堆栈跟踪：

Index Function 
-------------------------------------------------------------------------------- 
1  msvcr90d.dll!68d7f693() 
2  [Frames below may be incorrect and/or missing, no symbols loaded for msvcr90d.dll] 
*3  P3.exe!main(int argc=0, char * const * argv=0x0036fcd0) 
4  P3.exe!_FreeLibrary@4() 
5  P3.exe!@ILT+170(__except_handler4)() 
6  kernel32.dll!75eb3677() 
7  ntdll.dll!77b29d72() 
8  ntdll.dll!77b29d45() 

Answer 1

有几件事情要检查（抱歉不会下载代码）：

1. 没有G ++的* .c有警告？如果这样修复它们。
2. 确实g ++ -W有警告？如果这样修复它们。
3. 确实g ++ -W -Wall有警告？如果这样修复它们。
4. 确实g ++ -W -Wall -Wextra有警告吗？如果这样修复它们。
5. 确实g ++ -W -Wall -Wextra -ansi有警告？如果这样修复它们。
6. 确实g ++ -W -Wall -Wextra -ansi -pedantic有警告吗？如果这样修复它们。

在微软上，尝试在命令行中添加/ W4来打开警告，再次修复任何问题。

很可能你正在做一些“愚蠢的事情”，而且编译器可能会帮助你理解它的含义。

编辑：

从与标志以上，你会看到编译代码：

Cust.h：33：错误：ISO C++禁止零大小的数组 'ACCTNUM' Cust.h：34 ：错误：ISO C++禁止零大小数组''acctFN' Cust.h：35：错误：ISO C++禁止零大小数组''acctLN' Cust.h：38：错误：ISO C++禁止零大小的数组'fileName ' Cust.h：33：错误：ISO C++禁止零大小数组'acctNum' Cust.h：34：错误：ISO C++禁止零大小的数组''acctFN' Cust.h：35：错误：ISO C++禁止零大小数组'acctLN' Cust。h：38：错误：ISO C++禁止零大小数组'fileName'

所以你的代码是无效的C++。您正在将一个名称复制到一个太小的数组中 - 该数组有0个元素。你真正需要做的是在声明它们或将它们声明为指针时给数组一个大小，然后使用“new”分配适量的memroy。

Answer 2

很可能是您的strcpy函数的实现，它可能在如何在Mac上编码以及如何在Windows上编码之间存在差异。

Answer 3

将无效的缓冲区，过小的缓冲区等传递给strcpy会导致未定义的行为 - 几乎可能发生任何事情。在Mac上，问题发生但不明显，而在Windows上则会导致崩溃。

Answer 4

char acctNum[]; 
char acctFN[]; 
char acctLN[]; 

有你的问题就在这里。你似乎从来没有为这些字符串的任何地方分配任何空间。 setAcctNum（）中的strcpy（）溢出未定义数组的边界，并覆盖别的东西。实际上，这真是太棒了。

您可能应该使用std :: string，而不是 - 至少会使内存管理更容易。