遍历对象列表 C++
Iterating over a list of objects c++
基本上我正在为 class 项目创建一个基本的 shell 程序。我一直在迭代目录中的 files/folders。
我的错误在函数 DisplayDirectoryContents()
class Directory
{
private:
map<string, pair<list<Folder>, list<File> > > directoryContents;
string directoryPath;
public:
list<Folder> getFolders() {return directoryContents[directoryPath].first;}
list<File> getFiles() {return directoryContents[directoryPath].second;}
string getDirectoryPath() {return directoryPath;}
Directory()
{
directoryPath = "root/";
File *file = new File("Test");
directoryContents[directoryPath].second.push_back(*file);
}
void DisplayDirectoryContents()
{
// Get files and folders from directory
list<File> files = this->getFiles();
for(int i = 0; i < files.size(); i++)
{
cout << files->getFileName(); << Error here
}
}
};
我试过几种不同的方法来设置这个功能,但我似乎无法获得正确的代码来让它工作。
我认为这会奏效,但每次我尝试弄乱 DisplayDirectoryContents() 函数时都会抛出错误。有没有人有任何提示可以帮助我解决这个问题?谢谢
class File
{
string fileName;
string fileTime;
public:
string getFileTime(){return fileTime;}
string getFileName(){return fileName;}
void setFileTime(){time_t now = time(NULL);
fileTime = ctime(&now);}
void setFileName(string newFileName){fileName = newFileName;}
File(){}
File(string fName)
{
fileName = fName;
time_t now = time(NULL);
fileTime = ctime(&now);
}
void MakeFile(string fName);
void RemoveFile(string fName);
};
如果您能够使用 C++11 或更高版本,则可以使用范围 for
循环来迭代列表的内容。
void DisplayDirectoryContents()
{
// Get files and folders from directory
list<File> files = this->getFiles();
// Iterate over each item in the list.
// Use a reference to avoid copying of the items.
for ( File& file : files )
{
cout << file.getFileName();
}
}
扩展到 R Sahu's :
您不能使用原始循环进行迭代的原因是 std::list
不提供索引运算符(您需要改用 std::vector
)。
基于范围的 for 循环的替代方法(不过,我更喜欢这种方法,除非你有特定的理由不这样做)——或者如果你没有可用的 C++11 的方法——正在使用迭代器:
for(std::list<File>::iterator i = files.begin(); i != files.end(); ++i)
{
std::cout << i->getFileName();
}
使用 C++11,您可以 for(auto i = files.begin(); ...
.
即使在 C++11 中使用迭代器循环的用例也可以将项目与其后继项进行比较:
// check for empty list first, as std::prev would fail on!)
for(auto i = files.begin(); i != std::prev(files.end()); ++i)
{
if(someCondition(*i, *std::next(i))
{
// do something else
}
}
已经有 std::remove_if
for and you should prefer it (together with a lambda; but don't forget to apply erase
afterwards, see erase-remove-idiom!),所以只是为了说明;从更简单的方法开始:
for(auto i = files.begin(); i != files.end(); ) // no ++i (!)
{
if(condition)
{
i = files.erase(i);
// fine for std::list; with std::vector, if removing more
// than one single element, we'd move or even copy subsequent
// elements multiple times, which is quite inefficient
}
}
改进的变体(std::remove_if
也是如此):
auto pos = files.begin();
for(auto i = files.begin(); i != files.end(); ++i)
{
if(!condition)
// ^ (!)
{
*pos++ = std::move(*i);
}
}
// here, std::remove_if already stops, but returns pos...
files.erase(pos, files.end());
移动或复制赋值运算符可能仍然存在未正确处理自赋值的最小问题,if(!condition && i != pos)
将涵盖该问题。
基本上我正在为 class 项目创建一个基本的 shell 程序。我一直在迭代目录中的 files/folders。
我的错误在函数 DisplayDirectoryContents()
class Directory
{
private:
map<string, pair<list<Folder>, list<File> > > directoryContents;
string directoryPath;
public:
list<Folder> getFolders() {return directoryContents[directoryPath].first;}
list<File> getFiles() {return directoryContents[directoryPath].second;}
string getDirectoryPath() {return directoryPath;}
Directory()
{
directoryPath = "root/";
File *file = new File("Test");
directoryContents[directoryPath].second.push_back(*file);
}
void DisplayDirectoryContents()
{
// Get files and folders from directory
list<File> files = this->getFiles();
for(int i = 0; i < files.size(); i++)
{
cout << files->getFileName(); << Error here
}
}
};
我试过几种不同的方法来设置这个功能,但我似乎无法获得正确的代码来让它工作。
我认为这会奏效,但每次我尝试弄乱 DisplayDirectoryContents() 函数时都会抛出错误。有没有人有任何提示可以帮助我解决这个问题?谢谢
class File
{
string fileName;
string fileTime;
public:
string getFileTime(){return fileTime;}
string getFileName(){return fileName;}
void setFileTime(){time_t now = time(NULL);
fileTime = ctime(&now);}
void setFileName(string newFileName){fileName = newFileName;}
File(){}
File(string fName)
{
fileName = fName;
time_t now = time(NULL);
fileTime = ctime(&now);
}
void MakeFile(string fName);
void RemoveFile(string fName);
};
如果您能够使用 C++11 或更高版本,则可以使用范围 for
循环来迭代列表的内容。
void DisplayDirectoryContents()
{
// Get files and folders from directory
list<File> files = this->getFiles();
// Iterate over each item in the list.
// Use a reference to avoid copying of the items.
for ( File& file : files )
{
cout << file.getFileName();
}
}
扩展到 R Sahu's
您不能使用原始循环进行迭代的原因是 std::list
不提供索引运算符(您需要改用 std::vector
)。
基于范围的 for 循环的替代方法(不过,我更喜欢这种方法,除非你有特定的理由不这样做)——或者如果你没有可用的 C++11 的方法——正在使用迭代器:
for(std::list<File>::iterator i = files.begin(); i != files.end(); ++i)
{
std::cout << i->getFileName();
}
使用 C++11,您可以 for(auto i = files.begin(); ...
.
即使在 C++11 中使用迭代器循环的用例也可以将项目与其后继项进行比较:
// check for empty list first, as std::prev would fail on!)
for(auto i = files.begin(); i != std::prev(files.end()); ++i)
{
if(someCondition(*i, *std::next(i))
{
// do something else
}
}
已经有 std::remove_if
for and you should prefer it (together with a lambda; but don't forget to apply erase
afterwards, see erase-remove-idiom!),所以只是为了说明;从更简单的方法开始:
for(auto i = files.begin(); i != files.end(); ) // no ++i (!)
{
if(condition)
{
i = files.erase(i);
// fine for std::list; with std::vector, if removing more
// than one single element, we'd move or even copy subsequent
// elements multiple times, which is quite inefficient
}
}
改进的变体(std::remove_if
也是如此):
auto pos = files.begin();
for(auto i = files.begin(); i != files.end(); ++i)
{
if(!condition)
// ^ (!)
{
*pos++ = std::move(*i);
}
}
// here, std::remove_if already stops, but returns pos...
files.erase(pos, files.end());
移动或复制赋值运算符可能仍然存在未正确处理自赋值的最小问题,if(!condition && i != pos)
将涵盖该问题。