[crapmyster]

Hello,

I'm having a absolute nightmare at the moment. I'm trying to write a char array to a text file. This works fine but I want to be able to format so that each element is on a different line. At the moment it is all on one line in the text file. I have tried different things such as putting:

file << parent[i] << endl; 

But all this does is print out random symbols or whatever they are which are not supposed to be. The working code is just printing the correct data in one line, which is below:

void writeToFile()
{
        ofstream file;
        file.open("test.txt", ios::app);
        file << "\n" << endl;
        for(int i = 0; i < arraySize; i++)
        {
                file << parent[i];
        }
        file.flush();
        file.close(); 
}

Thanks,
Mark

[ishkabible]

show us the actual code from what you tried.

[crapmyster]

ishkabible, on 02 November 2012 - 08:54 AM, said:

show us the actual code from what you tried.

void writeToFile()
{
        ofstream file;
        file.open("test.txt", ios::app);
        for(int i = 0; i < arraySize; i++)
        {
                file << parent[i] << endl;
        }
        file.flush();
        file.close(); 
}

This outputs ਊਠਠਠਠਠਠਠਠਠਠਠਠਠਠਠਪਠਠਠਠਠਠਠਠਠਠਠਠਠਠਠਊ

This post has been edited by crapmyster: 02 November 2012 - 09:01 AM

[JackOfAllTrades]

What is the content of arraySize and parent?

[Rahul69]

@crapmyster Maybe the file is written in binary mode, have to tried to read from the file and print on console?

[quim]

The concept is good. Your problem might be with the arraySize variable. you either did not initialize it or assigned a value that is more than the string character, so this is why you get weird/random value when you output to the string.
you must declare/define arraySize and parent, unless it is a global variable in which you should initialize it.

try this:

void writeToFile()
{

        int arraySize = 11;
        char parent[arraySize] = "Hello World";
        ofstream file;
        file.open("test.txt", ios::app); /*notice that ios::app is the append mode, so when you write to the file it will append*/
        /*file << "\n" << endl; // take this out it must be inside the loop but you probably only one one line*/
        for(int i = 0; i < arraySize; i++)
        {
                file << parent[i] << "/n"; // add newline between each character
        }
        file.flush();
        file.close(); 
}

[crapmyster]

This is defined as global:

const int arraySize = 31;
char parent[arraySize];

[jimblumberg]

And where did you ever fill in your array? Show us the smallest complete program that illustrates your problem.

Jim

[Oler1s]

Let me simplify. As a beginner, don't use globals. That's a ham-fisted principle, but as a beginner, you are unlikely to use globals correctly.

[jjl]

This sounds like a combination of you not having any data in your character array, as well as trying to output a non-null terminated string.

[crapmyster]

Just to clear up there is characters in the array. An combination of '*' and ' ' (spaces). The array is being outputted to a text file, however in a straight line. I'm trying to output it in the text file so that each element is on a new line.

#include <iostream>
#include <fstream>
using namespace std;
const int arraySize = 31;
char parent[arraySize];
char child[arraySize];

void initilise()
{
        for(int i = 0; i < arraySize; i++)
        {
                parent[i] = ' ';
                child[i] = ' ';
        }
        parent[15] = '*';
        
}

void writeToFile()
{
        ofstream file;
        file.open("test.txt", ios::app);
        for(int i = 0; i < arraySize; i++)
        {
                file << parent[i];
        }
        file.flush();
        file.close(); 
}

Above is the code that working with.

[JackOfAllTrades]

This is like pulling teeth.

Show us ALL of your code. If you don't actually call initilise(sic), your arrays won't be initialized.

[crapmyster]

sorry, forgot to include:

#include <iostream>
#include <fstream>
using namespace std;
const int arraySize = 31;
char parent[arraySize];
char child[arraySize];

void initilise()
{
        for(int i = 0; i < arraySize; i++)
        {
                parent[i] = ' ';
                child[i] = ' ';
        }
        parent[15] = '*';
        
}

void writeToFile()
{
        ofstream file;
        file.open("test.txt", ios::app);
        for(int i = 0; i < arraySize; i++)
        {
                file << parent[i];
        }
        file.flush();
        file.close(); 
}

void menu()
{
        while(true)
        {
                cout << "" << endl;
                cout << "Press 's' to Start" << endl;
                cout << "Press 'e' to Exit" << endl;
                cout << "Enter a choice:" << endl;
                char input;
                cin >> input;
		switch(input)
		{
			case 's': initilise();
                                  writeToFile();
				  break;
                        case 'e': 
                                  break;
			default:
				  cout << "Error: Wrong selection, choose again" << endl;
				  break;
		}
        }
}

int main()
{
    menu();
    return 0;
}

[JackOfAllTrades]

Works fine for me (except exiting your menu doesn't work).

What compiler are you using?

[crapmyster]

JackOfAllTrades, on 03 November 2012 - 04:43 AM, said:

Works fine for me (except exiting your menu doesn't work).

What compiler are you using?

Hey, I've been using the NetBeans IDE with some compiler which I think is from Cygwin. I booted in Linux and used the gcc compiler and I get the right output to the file. So the netbeans compiler is being gay. If someone can explain why it is not working how it should in netbeans and it is in Linux would be much appreciated.
Also, note exiting the menu won't work as I have not fully completed that.
Thanks, everyone.