Problem with writing to files

Hello,
I am trying to make a function where I can input a filename and a string for insertion into a file.

Code:

---

void write(string filename, string text)
{
    ofstream file;
    file.open(filename);
    file << text;
    file.close();
    return;
}

---

Can anyone tell me what is wrong with this? It gives me an error on the 3rd and 4th lines when I try to compile.
Thanks,

Maybe you forgot to include this line?

Code:

---

#include <fstream>

---

Post what the errors say, help you will be alot easier then.

No, I have fstream

Code:

---

#include <iostream>
#include <cstdlib>
#include <ctime>
#include <fstream>

using std::endl;
using std::string;

//***WRITES*FILES*********
void write(string filename, string text)
{
    ofstream file(filename);
    file << text;
    file.close();
    return;
}

---

Dev C++ gives these error messages
line 12 - '
line 12 - [Each
line 12 - parse
line 12 - `file'

std::ostream, not ostream.

Ok, now for line 12 its giving me error no
Its also giving me some new errors in iosfwd and fstream when I used std:: ostream.

Post your updated code with the exact error messages.

Code:

---

#include <iostream>
#include <fstream>
#include <cstdlib>
#include <ctime>

using std::endl;
using std::string;

//***WRITES*FILES*********
void write(string filename, string text)
{
    std::ostream file(filename);
    file << text;
    file.close();
    return;
}

---

line 12 - no
(iosfwd) line 61 - candidates
are - std::basic_ostream<char, std::char_traits<char> >::basic_ostream(const
line 14 - no

Ah. That's because the constructor that you want for an output stream take a null terminated C-style string as an argument. As such, you should write:

Code:

---

void write(const std::string& filename, const std::string& text)
{
    std::ostream file(filename.c_str());
    file << text;
    file.close();
    return;
}

---

The file.close() and return are actually unnecessary.

Its still giving me the same error messages.

Oh, because now you wrote std::ostream instead of std::ofstream. My mistake from an earlier post, actually.

Haha, it works!
Thanks for all the help and sorry for being such a noob.