ファイル名リストの置換処理

質問

以前、ファイルリストの取得について教えて頂きました。ありがとうございます。
取得したリストを文字列で入れ替えるにはどうすればよいでしょうか

例えば取得したリストの最初の位置の文字列dml[0]をEcoDataFileNameと置き換えたいです。

#include <Windows.h>
#include <map>
#include <vector>
#include <string>
#include <iostream>
#include "time.h"

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<string> files_;
public:
void search(const char* spec) {
typedef multimap<FILETIME,string> 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).c_str(); }
int size() const { return files_.size(); }
};

int main() {
DML_Backup dml;
char EcoDataFileName="MonJun131956122011.ecd";

dml.search("*.ecd");

｝

回答No.5

hitomura · ベストアンサー

なんか仕様が増えたーー！？

> 実はファイル自体存在させる個数が決まっており、ファイルのリストを一度作成した場合、後はリストに新規作成したファイルを追加するようにしたかったのです。
> ですから、１０個のリストになった場合にサイクリックに、今度は先頭から上書きして、古いファイルから消していくような仕様でした。
> その際実際存在する古いファイルも消すという内容です。
そういう仕様であるならば、なおさらファイル名の「上書き」メソッドは実装すべきではありません。そもそもファイル名の並びは最終更新日時の古い順のはずです。それを外部から書き換え可能にしてしまったら順番がばらばらになってしまいます。

実装すべきはファイルリストへの追加処理でしょう。とりあえず以下にファイル名のみ追加するものを書きましたが、クラスの役目を考慮して、この処理の中で実ファイルの作成・削除も行うというてもあります。
void DML_Backup::add(const string& filename)
{
　files_.push_back(filename);
　int erasing_files = files_.size() - 10;
　if (0 < erasing_files){
　　files_.erase(files_.begin(), files_.begin() + erasing_files);
　}
}

……でも、これだったらコンテナはvectorよりもlist（ただしoperator[]の書き換えが必要）やdequeのほうがいいような気がしますが。

回答No.4

hitomura

> これは例えば日付が古い順にソートした場合、リストの個数に上限があり、上限に達した場合、リストの先頭に戻り、順番に新しいファイルを上書きして行くという処理です。
何かおかしいと思ったら、そういうことですか。

このDML_Backupクラスの場合、外部からファイル名を書き換えるメソッドは不要です。というか、作ってはいけません。
なぜならば、このクラスで保持しているファイル名はsearchメソッドで検索したものであり、ディスク等にあるファイルという実体があります。外部からファイル名を書き換えると、その実体からクラス内のデータを乖離させることになります。
したがって、そのメソッドで元の名前のファイルを新しいファイル名に変更する処理を同時に行うのでない限り、外部からファイル名を書き換えるメソッドを作るべきではありません。

「リストの個数に上限があり」云々という制限はデータを表示する側の都合であり、データを表示する側で対処すべきです。たとえば以下（逐次処理を行って進捗をリストに表示する）のように処理を行うファイルのインデックスをリストの表示可能行数で割った余りでファイル名を書く行が求められます。

const int ListLines = 10; // リストの表示可能行数

int main() {
DML_Backup dml;

dml.search("*.ecd");

for (int i = 0; i < dml.size(); i++){
　int disp_line = i % ListLines;
　// リストの disp_line 行に dml[i] を書く
　// ファイル dml[i] に対して処理実行
}
｝

補足

ありがとうございます、実はファイル自体存在させる個数が決まっており、ファイルのリストを一度作成した場合、後はリストに新規作成したファイルを追加するようにしたかったのです。ですから、１０個のリストになった場合にサイクリックに、今度は先頭から上書きして、古いファイルから消していくような仕様でした。その際実際存在する古いファイルも消すという内容です。そのような理由で質問をさせて頂いておりました。

回答No.3

επιστημη（@episteme）

> string operator[](int inx) const { return files_.at(inx).c_str(); }

コレ↑を書き換え:

string operator[](int inx) const { return files_.at(inx); }
string& operator[](int inx) { return files_.at(inx); }

ってするだけでよくね?

# 教本買って「ちゃんと」勉強することを強くお勧めします。

回答No.2

hitomura

質問とは直接関係ありませんが、
　string operator[](int inx) const { return files_.at(inx).c_str(); }
は c_str() が余計で、
　string operator[](int inx) const { return files_.at(inx); }
で十分です。

回答No.1

gerugugu84

void DML_Backup::replace( const char* pstr )
{
　　vector<string>::iterator it = files_.begin();
　　if( files_.end() != it )
　　{
　　　　*it = pstr;
　　}
}

int main() {
　　DML_Backup dml;
　　char EcoDataFileName[]="MonJun131956122011.ecd";

　　// 先頭のstringを置き換える
　　dml.replace(EcoDataFileName);
｝

補足

ありがとうございます。では先頭から適宜、順番に任意の文字列に差し替えるにはどうすれば良いでしょうか？これは例えば日付が古い順にソートした場合、リストの個数に上限があり、上限に達した場合、リストの先頭に戻り、順番に新しいファイルを上書きして行くという処理です。わかりにくくてすみません。