メンバ変数の取得

Question

二次元の配列をメンバ変数の値を取得したいのですが、上手くいきません。

どなたか教えて頂けないでしょうか。

class DML_Backup
{
　char ecoFileList[256][32];
public:
  char GetsearchEcoDataFileName(){return **ecoFileList;}
}

void main()
{
DML_Backup m_ecoData;
char ecoFileList[256][32];

**ecoFileList = m_ecoData.GetsearchEcoDataFileName();

}

episteme · Accepted Answer

> 自宅の環境の2005では通りました。ただ会社の環境だと下記のエラーが出たと思います。

「会社の環境」とやらがわかん。知らんよそんなもん。

episteme · Answer

これではどうでしょう。2005で通るか自信はないのだけれども。 #include #include #include #include #include using namespace std; bool operator<(const FILETIME& x, const FILETIME& y) { if ( x.dwHighDateTime < y.dwHighDateTime ) return true; if ( y.dwHighDateTime < x.dwHighDateTime ) return false; return x.dwLowDateTime < y.dwLowDateTime; } class DML_Backup { vector files_; public: void search(const char* spec) { typedef multimap map_type; map_type files; WIN32_FIND_DATAA find_data; HANDLE handle = FindFirstFileA(spec, &find_data); if ( handle != INVALID_HANDLE_VALUE) { do { files.insert(map_type::value_type(find_data.ftLastWriteTime, find_data.cFileName)); } while ( FindNextFileA( handle, &find_data) ); FindClose(handle); } files_.clear(); for ( map_type::iterator iter = files.begin(); iter != files.end(); ++iter ) { files_.push_back(iter->second); } } string operator[](int inx) const { return files_.at(inx); } int size() const { return files_.size(); } }; int main() { DML_Backup dml; dml.search("d:\work\*.cpp"); int n = dml.size(); for ( int i = 0; i < n; ++i) { cout << dml[i] << endl; } }

episteme · Answer

> ただこちらの環境ではエラーになります。
 
コンパイラが旧いようです。
僕とこではVC++10(VS2010)でコンパイルしました。

コンパイラをupgradeするか、コードを適宜書き換える必要があります。

episteme · Answer

せっかくのC++がもったいない。なんでわざわざcharの二次元配列使わんならんのだろ。 #include #include #include #include #include using namespace std; bool operator<(const FILETIME& x, const FILETIME& y) { if ( x.dwHighDateTime < y.dwHighDateTime ) return true; if ( y.dwHighDateTime < x.dwHighDateTime ) return false; return x.dwLowDateTime < y.dwLowDateTime; } class DML_Backup { vector files_; public: void search(const char* spec) { typedef multimap map_type; map_type files; WIN32_FIND_DATAA find_data; HANDLE handle = FindFirstFileA(spec, &find_data); if ( handle != INVALID_HANDLE_VALUE) { do { files.insert(map_type::value_type(find_data.ftLastWriteTime, find_data.cFileName)); } while ( FindNextFileA( handle, &find_data) ); FindClose(handle); } files_.clear(); transform(files.begin(), files.end(), back_inserter(files_), [](const map_type::value_type& item) { return item.second;}); } string operator[](int inx) const { return files_.at(inx); } int size() const { return files_.size(); } }; int main() { DML_Backup dml; dml.search("d:\work\*.cpp"); int n = dml.size(); for ( int i = 0; i < n; ++i) { cout << dml[i] << endl; } }

hitomura · Answer

質問の処理をこなすだけなら

class DML_Backup
{
　char ecoFileList[256][32];
public:
char** GetsearchEcoDataFileName(){return ecoFileList;}
}

void main()
{
DML_Backup m_ecoData;
char ecoFileList[256][32];

memcpy(ecoFileList, m_ecoData.GetsearchEcoDataFileName(), sizeof(char) * 256 * 32);
}
で可能です。

ただし、このGetsearchEcoDataFileName()の実装では ecoFileList を private にした意味がありません。
　m_ecoData.GetsearchEcoDataFileName()[0][0] = '\0';
というようなコードでクラス外から好き勝手に値を変更できるからです。

というか、このデータの持ち方を見ると前の質問
http://oshiete.goo.ne.jp/qa/6798323.html
で私が心配した「呼び出し元で想定した個数以上ファイルが存在した場合」と「ファイル名が極端に長い場合」に問題があるようです。
これでは .eco ファイルが 300 個あった場合や、 filenameistoolongtostockinchar32field.eco というようなファイル名の .eco ファイルがあったときに問題が発生します。

処理全体が見えないから全体から見て適切かどうかわかりませんが、
class DML_Backup
{
　std::vector<std::string> ecoFileList;
public:
　std::vector<std::string>::size_type GetsearchEcoDataSize() const {return ecoFileList.size();}
　std::string GetsearchEcoDataFileName(int index) const {return ecoFileList[index];}
// あるいは
//　const std::string& GetsearchEcoDataFileName(int index) const {return ecoFileList[index];}
}

void main()
{
DML_Backup m_ecoData;
// char ecoFileList[256][32]; ：不要

for(int i = 0; i < m_ecoData.GetsearchEcoDataSize(); i++){
　// 各ファイルに対する処理を行う
　// ファイル名は GetsearchEcoDataFileName(i) で取得可能、char* でほしいなら c_str() を使えばでできる
}
}

とするか、ファイルリストを表すクラスを導入するかすべきではないでしょうか。

onosuke · Answer

プログラムの記述方法が明らかに間違っています。

＞二次元の配列をメンバ変数の値を取得したい
とありますが、C言語では、以下2種類の解釈があります。
　解釈ア．メンバ変数から、ポインタへ、二次元配列のポインタ値を取得したい
　解釈イ．メンバ変数から、別の二次元配列へ、配列の値全て（256×32)をコピーしたい

解釈アの場合、「ポインタの宣言」と「ポインタの参照」を混同していますよ。
解釈イの場合、配列の値のコピーはmemcpy関数等でコピー操作を行う必要があります。代入では配列をコピーできません。

解釈アの場合：
class DML_Backup {
　　char ecoFileList[256][32];
　public:
　　char GetsearchEcoDataFileName(){return ecoFileList;}
}
void main(){
　DML_Backup m_ecoData;
　char (*ecoFileList)[32];
　ecoFileList = m_ecoData.GetsearchEcoDataFileName();
}

解釈イの場合：
class DML_Backup {
　　char ecoFileList[256][32];
　public:
　　void GetsearchEcoDataFileName(char p[256][32]){
　　　memcpy(p,ecoFileList,sizeof(ecoFileList));
　　　return;
　　}
}
void main(){
　DML_Backup m_ecoData;
　char ecoFileList[256][32];
　m_ecoData.GetsearchEcoDataFileName(ecoFileList);
}

kmee · Answer

大きな間違いは2つ。

> char GetsearchEcoDataFileName(){return **ecoFileList;}
とあるように、 GetsearchEcoDataFileNameが返すのは 「char型が一つ」 です。
char型は、よくある処理系では8ビットの整数です。
配列は返しません。

実際に、 **echoFileList と、charを返しています。
**echoFileList → *(echoFileList[0]) → echoFIleList[0][0] です。
これはmain関数での
> **ecoFileList = m_ecoData.GetsearchEcoDataFileName();
でも同じです。echoFIleList[0][0] に受け取ってます。

もう一つは、
ecoFileList = m_ecoData.GetsearchEcoDataFileName();
として = で配列の中身をコピーしたいのであれば、普通はできません。配列を=でつないでも、配列の内容をコピーするようには作られてないからです。

メンバ変数の取得

> 自宅の環境の2005では通りました。

これではどうでしょう。

この回答への補足

> ただこちらの環境ではエラーになります。

この回答への補足

せっかくのC++がもったいない。

この回答への補足

質問の処理をこなすだけなら

プログラムの記述方法が明らかに間違っています。

この回答への補足

大きな間違いは2つ。

似たような質問が見つかりました

関連するカテゴリからQ&Aを探す

デイリーランキングこのカテゴリの人気デイリーQ&Aランキング

マンスリーランキングこのカテゴリの人気マンスリーQ&Aランキング