エラーが出ます

Question

以下のコピーコンストラクタのサンプルコードをコンパイルすると、

'std::_Copy_impl': Function call with parameters that may be unsafe - this call relies on the caller to check that the passed values are correct. To disable this warning, use -D_SCL_SECURE_NO_WARNINGS. See documentation on how to use Visual C++ 'Checked Iterators'

という内容のエラーが出てきます。
なぜエラーが出るのでしょうか？
copy関数の部分が何かおかしいのでしょうか…
言語はVC++です。

IntArray.cpp

#include "IntArray.h"
#include <iostream>
#include <algorithm>					//copy 関数に必要
#include <cstdlib>					//exit 関数に必要
using namespace std;



//コピーコンストラクタ
IntArray::IntArray(const IntArray& other){
	m_array = new int[other.m_size];
	m_size = other.m_size;

	copy(other.m_array, other.m_array + m_size, m_array);


	cout << "コピーコンストラクタが呼ばれました。" << endl;
}


//コンストラクタ
IntArray::IntArray(int size){
	m_array = new int[size];
	m_size = size;

	cout << "コンストラクタが呼ばれました。" << "要素数は" << m_size << "です。" << endl;
}


//デストラクタ
IntArray::~IntArray(){
	delete[] m_array;

	cout << "デストラクタが呼ばれました。" << "要素数は" << m_size << "でした。" << endl;
}


//メンバへのアクセス関数
int IntArray::Get(int i){
	void CheckIndex(int i);				//インデックスのチェック
	return m_array[i];
}


void IntArray::Set(int i, int value){
	void CheckIndex(int i);				//インデックスのチェック
	m_array[i] = value;
}


//インデックスのチェック
void IntArray::CheckIndex(int i){
	if (0 <= i && i < m_size){
		//インデックスは正常です
	}
	else{
		cout << "インデックスが不正です!" << endl << "値 : " << i << endl;
		exit(EXIT_FAILURE);
	}
}


int IntArray::Size(){
	return m_size;
}

Copy.cpp

#include "IntArray.h"
#include <iostream>
using namespace std;



void Viss(int num){
	cout << "Viss : No." << num << endl;
}


void Show(IntArray array){

	Viss(2);

	for (int i = 0; i < array.Size(); i++){
		cout << array.Get(i) << ' ';
	}
	cout << endl;

	Viss(3);
}


int main(){
	IntArray array0to9(10);

	for (int i = 0; i < array0to9.Size(); i++){
		array0to9.Set(i, i);
	}

	Viss(1);
	Show(array0to9);
	Viss(4);
}

IntArray.h

class IntArray{
public:
	IntArray(const IntArray& other);	//コピーコンストラクタ
	IntArray(int size);					//コンストラクタ
	~IntArray();						//デストラクタ

	int Get(int i);						//メンバへのアクセス関数(取得)
	void Set(int i, int value);			//メンバへのアクセス関数(変更)
	void CheckIndex(int i);				//インデックスのチェック

	int Size();							//配列のサイズを返す


private:
	int* m_array;						//動的配列
	int m_size;							//配列の要素数
	
};

Answer 1

＜std::_Copy_impl': Function call with parameters that may be unsafe - this call relies on the caller to check that the passed values are correct. To disable this warning, use -D_SCL_SECURE_NO_WARNINGS. See documentation on how to use Visual C++ 'Checked Iterators'

プログラムを扱うなら英語は必須ですから意味がわかるようにしましょう。
この関数呼び出しは安全ではないです。
あとはこのメッセージが出なくなる方法と、文法書を読んでくださいとなっていますね。

Comments

[takesita8]

いや、その方法自体を教えてほしかったんですが…^^;

Answer 2

こんにちは。

このQAと同じ問題と思います。
人によってはありがたく感じる機能かも知れませんが、私はマイクロソフトの小さな親切大きなお世話と感じます。

標準ライブラリを使うだけで出るのであれば、上記リンク先のように#pragma warning(disable : 4996)が妥当なように思います。

他の対策は、該当関数を使わないか、警告を「気にしない」しかないです。
前者はマルチプラット・フォーム対応する場合には障害になりそうです。
後者はリスクが高いので採用すべきではありません。（他の警告が埋もれてしまい、見落とす。）

Comments

[takesita8]

ありがとうございます！
おかげで解決できました