This isn't a design question, really, though it may seem like it. (Well, okay, it's kind of a design question). What I'm wondering is why the C++ std::fstream classes don't take a std::string in their constructor or open methods. Everyone loves code examples so:
#include <iostream>
#include <fstream>
#include <string>
int main()
{
std::string filename = "testfile";
std::ifstream fin;
fin.open(filename.c_str()); // Works just fine.
fin.close();
//fin.open(filename); // Error: no such method.
//fin.close();
}
This gets me all the time when working with files. Surely the C++ library would use std::string wherever possible?
-
Is there any class in STL that takes a string... I dont think so (couldnt find any in my quick search). So it's probably some design decision, that no class in STL should be dependent on any other STL class (that is not directly needed for functionality).
From Magnus Westin -
There are several places where the C++ standard committee did not really optimize the interaction between facilities in the standard library.
std::stringand its use in the library is one of these.One other example is
std::swap. Many containers have a swap member function, but no overload of std::swap is supplied. The same goes forstd::sort.I hope all these small things will be fixed in the upcoming standard.
From Christopher -
I believe that this has been thought about and was done to avoid the dependency; i.e. #include <fstream> should not force one to #include <string>.
To be honest, this seems like quite an inconsequential issue. A better question would be, why is std::string's interface so large?
From DrPizza -
@DrPizza:
It is inconsequential, that is true. What do you mean bystd::string's interface being large? What does large mean, in this context - lots of method calls? I'm not being facetious, I am actually interested.From Bernard -
It is inconsequential, that is true. What do you mean by std::string's interface being large? What does large mean, in this context - lots of method calls? I'm not being facetious, I am actually interested.
It has more methods than it really needs, and its behaviour of using integral offsets rather than iterators is a bit iffy (as it's contrary to the way the rest of the library works).
The real issue I think is that the C++ library has three parts; it has the old C library, it has the STL, and it has strings-and-iostreams. Though some efforts were made to bridge the different parts (e.g. the addition of overloads to the C library, because C++ supports overloading; the addition of iterators to basic_string; the addition of the iostream iterator adaptors), there are a lot of inconsistencies when you look at the detail.
For example, basic_string includes methods that are unnecessary duplicates of standard algorithms; the various find methods, could probably be safely removed. Another example: locales use raw pointers instead of iterators.
From DrPizza -
By taking a C string, the fstream class reduces dependency on the
std::stringclass. Now, you may wonder then why isn't there a transparent conversion from astd:stringto a C string, so a class that expects a C string could still take astd::stringjust like a class that expects astd::stringcan take a C string.The reason is that this would cause a conversion cycle, which in turn may lead to problems. For example, suppose
std::stringwould be convertible to a C string so that you could usestd::strings withfstreams. Suppose also that C string are convertible tostd::strings as is the state in the current standard. Now, consider the following:void f(char* cstr, std::string str); void f(std::string str, char* cstr); void f(char* cstr1, char* cstr2); void g() { char* cstr = "abc"; std::string str = "def"; f(cstr, str); // ERROR: ambiguous }Because you can convert either way between a
std::stringand a C string the call tof()could resolve to any of the threef()alternatives, and is thus ambiguous. The solution is to break the conversion cycle by making one conversion direction explicit, which is what the STL chose to do withc_str().From wilhelmtell -
@ Bernard:
Monoliths "Unstrung." "All for one, and one for all" may work for Musketeers, but it doesn't work nearly as well for class designers. Here's an example that is not altogether exemplary, and it illustrates just how badly you can go wrong when design turns into overdesign. The example is, unfortunately, taken from a standard library near you... ~ http://www.gotw.ca/gotw/084.htmFrom swmc -
Maybe it's a consolation: all fstream's have gotten an open(string const &, ...) next to the open(char const *, ...) in the working draft of the C++0x standard. (see e.g. 27.8.1.6 for the basic_ifstream declaration)
So when it gets finalised and implemented, it won't get you anymore :)
From Pieter -
The stream IO library has been added to the standard C++ library before the STL. In order to not break backward compatibility, it has been decided to avoid modifying the IO library when the STL was added, even if that meant some issues like the one you raise.
From Drealmer
0 comments:
Post a Comment