Help with my notepad.

I'm making a little notepad, and what I got is a single line input. Now I want it to be multiline. Basically, when the user presses the enter key, it should create a new line on the output too. It may sound vague, here is the source code:

Code:

---

#include <iostream>
#include <fstream>
using namespace std;

string text;
string e;

int write() {
cin >> text;
ofstream file ("text.txt", ios::app);
file << text << " ";
}

int main() {
      cout << "Text: ";
      write();
while (text != e) {
      write();
}
cin.get ();
return 0;
}

---

Can I let it recognise an input of the "enter" key?

Code:

---

string e;

...

while (text != e)

---

Where is e initialized?

Quote:

---

Originally Posted by thelimpkid

Can I let it recognise an input of the "enter" key?

---

You'll have to use a different method of getting input. cin >> will strip any whitespace (tabs/spaces/newlines) when attempting to read a string. Maybe you could try getline(cin,text). If string is zero length, then user entered a newline.

Here's some changes/things I noticed.

Code:

---

#include <iostream>
#include <fstream>
using namespace std;

string text;
string e; //<-- 'e' is never given a value

int write() {
cin >> text;
ofstream file ("text.txt", ios::app);
file << text << " ";
}

int main() {
      cout << "Text: ";
      write();
while (text != e) { //<-- since 'e' is never given a value it's almost like saying "while not equal
      write();            // to nothing"
}
cin.get ();
return 0;
}

---

Using what hk_mp5 said try this...

Code:

---

#include <iostream>
#include <fstream>
using namespace std;

char text[255];
string e;

int write() {
cin.getline(text,255);
ofstream file ("text.txt", ios::app);
file << text << " ";
}

int main() {
      cout << "Text: ";
      write();
while (text != e) { //i dont know what you're trying to do with e here so i left this
      write();
}
cin.get ();
return 0;
}

---

>> Using what hk_mp5 said try this...
No reason to change to a C style string. There is a version of getline for C++ strings.

Code:

---

getline(cin, text);

---

Of course, now istead of outputting a space after the text you would output the newline character.

wow, thanks alot! I got it all wrong I see. Well I wanted to create a loop that would never end. That's what I tried with e.

Thanks alot for the help!

Quote:

---

Well I wanted to create a loop that would never end.

---

Code:

---

while (true) {
    ...
    if (...) break;
}

---

or

Code:

---

for(;;)

---

-- just as an alternative - they are pretty much the same thing... Some people seem to think that whiel(1)/while(true) is easier to read/understand - but if you spend enough time programming and reading other peoples code, it's just a matter of time until you find a for-statement with "nothing" in the parenthesis but semicolons - and that makes a loop with no end condition.

--
Mats

Ideally there would be a "forever" keyword, as in

Code:

---

forever {
  // code
}

---

with no "arguments" (is that the right word?), but since there isn't, while (true), which has just one, is conceptually the closest thing available, so I prefer that, though the for version has fewer characters.

Here's what I've got now:

Code:

---

#include <iostream>
#include <fstream>
using namespace std;

string text;

int write() {
getline(cin, text);
if (text == "newline") {
ofstream file ("text.txt", ios::app);
file << endl;
}
else {
ofstream file ("text.txt", ios::app);
file << text << " ";
}
}
int main() {
      cout << "Text: ";
      write();
while (true) {
      write();
}
cin.get();
return 0;
}

---

Can I replace "newline" with the "[enter key]"?

Go back to your original write() function at the top of this thread.

Change cin >> text to getline (cin, text). That will change your code from reading in a single word to reading in a whole line.

Then, change the file << text << " "; line to output text followed by the newline character instead of the space character. You're using " " there, but you don't want to do that, you want to use '\n' because '\n' is the newline character. The only reason you need this is because getline discards the newline character from the user input. The reason you had " " before was because cin >> discards spaces from the user input.

Those two things are all you needed to change in your write() function.

Hmm, I got that right now:

Code:

---

#include <iostream>
#include <fstream>
using namespace std;

string text;

int write() {
cin >> text;
ofstream file ("text.txt", ios::app);
file << text << endl;
}

int main() {
      cout << "Text: ";
      write();
while (true) {
      write();
}
cin.get ();
return 0;
}

---

I want to write lines, and I want to break the line when I press the enter key. I don't want to break the line after every word. I'm sorry if I wasn't clear enough about it. I just want a basic notepad so I can develop it to a news updating app.

You still need to use getline instead of cin >>. The other part is correct.

woops, yea, so this:

Code:

---

int write() {
getline (cin, text);
ofstream file ("text.txt", ios::app);
file << text;
}

---

Now it's still single line. How can I make it multiline?

Why did you change the second (output) part? It was correct.

Code:

---

int write() {
getline (cin, text);
ofstream file ("text.txt", ios::app);
file << text << endl;
}

---

If you mean this, it's not what I want to do with it. I don't want it to have each word on a new line. I want to create a newline when I press the "enter" key.

That doesn't put each word on a new line. It does put each line on a new line. Did you try it? I think you tried if before you switched to getline.

There are two parts to this. Input and output.

Using getline for input means that the entire line that you type will be read into text.

Using file << text << endl for output means that the entire string stored in text is output the file on its own line.

Combining the two means that the entire line you type into the console is added on its own line in the file. That's what you want.

Yes, thanks alot for your time and help! It works fine now.