Thread: How to write back to a temp file keeping the orginal or modifed org file format?

Code:

/*
Write a program that reads an input text file, 
identifying and auto-correcting the following common typing errors: 
‘Space before comma or a full-stop ( period ) ’, ‘Multiple spaces as opposed to 
a single space’, and ‘Missing spaces’. 

* */
#include <iostream>
#include <fstream>
#include <cstring>
#include <stdlib.h>
 
 using std::cout;
 using std::endl;


int main (int argc, const char **argv)
{
     
    
    if( argc < 2)
    {
        printf("error: no file [ %s ]\n", argv[argc]);
        exit(1);
    }

     
    std::ifstream infile(argv[1]);
    std::ofstream temp;
    temp.open("temp.txt");
    std::string str;
    
    if (infile.is_open()) {
     
    char c;
    int position = 0;
       // get length of file:
    infile.seekg (0, infile.end);
    int length = infile.tellg();
    cout<< "length is = "<<length<<endl;
    infile.seekg (0, infile.beg);
    
    while (infile.get(c)) 
    {
        
        if ((int)c == 32)
        {
            // get cur pos
            position = infile.tellg();
            //move back 2 to get c again to see
            // what was before (curr c) it
            infile.seekg(position-2, std::ios::beg);
            infile.get(c);
            if ( (int)c == 32 )
            {
                cout<<"Got extra space"<<endl;
                //put it back where it was before hand
                infile.seekg(position,std::ios::beg);
            }
            else // if no extra space, still needs to be put back
                infile.seekg(position, std::ios::beg);
        }
        if ((int)c == 44)
        {
             
            position = infile.tellg();
            
            infile.seekg(position-2, std::ios::beg);
            infile.get(c);
            if ( (int)c == 32 )
            {
                cout<<"Got extra space before comma"<<endl;
                infile.seekg(position,std::ios::beg);
            }
            else
                infile.seekg(position, std::ios::beg);
        }
        if ((int)c == 46)
        {
        
            position = infile.tellg();
            
            infile.seekg(position-2, std::ios::beg);
            infile.get(c);
            if ( (int)c == 32 )
            {
                cout<<"Got extra space before period"<<endl;
                infile.seekg(position,std::ios::beg);
            }
            else
                infile.seekg(position, std::ios::beg);
        }
         // and it does not work yet. 
        if ( (int)c ==  10) {
        //    cout<<"END LINE ";
            temp << '\n'; }
        else if ( (int)c == 32 )
            temp << " ";
        else
            temp << c; 
    } // end while
    } // end if
    infile.close();
    temp.close();
    //remove(argv[1]);
    //rename("temp.txt", argv[1]);
   //std::ifstream tempfile(argv[1]);
    std::ifstream tempfile("temp.txt");

    
    if (tempfile.is_open()) {
     
    while (true)
    {
        tempfile >> str;
        
        if (tempfile.eof()) break;
            cout << str ;
    }
}
  tempfile.close();

return 0;    
}

I am lacking in knowing how to get it back in another file the way it came out, and it is getting extra letters put into that temp file, and as I have discovered this method removes white space and end line. and trying to put it back like I am does not work.

Code:

$ ./auto_correct auto_correct_test_file
length is = 53
Got extra space before comma
Got extra space
Got extra space before period
Got extra space
Got extra space
Got extra space
Got extra space
Thiss,wassmyytesttfileeanddhereeweegooagainn

test file

Code:

This , was my  test file .
and here    we  go again.

The >> << operators eat whitespace, so that's probably our problem. You could use the get and put methods on your streams as an option.

Btw, you don't need to cast chars to make comparisons, you can just do:

Code:

if (c == ' ') // space
if (c == ',') // comma
// etc.

As for your algorithm, have you considered just seeking forward and keeping track of the last few (relevant) characters? That might be another approach to your goal.

If I had to do this myself, I would first put it all into a std::string and write the final output to file in a single swoop. But I've never been comfortable with stream operations.

Originally Posted by Hodor

I might look again once it is formatted in a readable manner

there is not ANSI standard to formatting code, it is just fine like it is. if you cannot read it then get some glasses. as well that just shows how limited you really are to coding. so don't bother and I am not playing your childish games go away.

Originally Posted by Guest

The >> << operators eat whitespace, so that's probably our problem.

Btw, you don't need to cast chars to make comparisons, you can just do:

Code:

if (c == ' ') // space
if (c == ',') // comma
// etc.

As for your algorithm, have you considered just seeking forward and keeping track of the last few (relevant) characters? That might be another approach to your goal.

If I had to do this myself, I would first put it all into a std::string and write the final output to file in a single swoop. But I've never been comfortable with stream operations.

i've never played with this myself, I gave that peek function a try and got no where, and then did it like this, it works as far as finding the what it is looking for, I casted them only because I wanted to use the numbers.

Originally Posted by Guest

You could use the get and put methods on your streams as an option.

(edit)
I guess back to C in C++

Originally Posted by Hodor

And why did you want to use the numbers? To highlight even further your stupidity?

you have no idea how much in an A$$ you are making of yourself right now. do you?

I'm sorry your mommy and daddy didn't love you like they should have so you have this superiority complex and feel that need to cut down anyone you can find to make yourself feel better about yourself.

Originally Posted by Hodor

I've rewritten your program to how I'd write it:

Code:

#include <stdio.h>

int is_odd(unsigned n); int is_even(unsigned n); int is_even(unsigned n)
{    return n == (unsigned)
0 ? (unsigned) 1 : (int)
is_odd
(n-1);
}

int is_odd(unsigned n) {
return n
==
0 ? 0
: (n == 1 ? 1
: is_even(n-1)); } int

main(void) {
    unsigned num = 12;

    printf("%u is "
    "%s\n", num, is_even(num) ? (const char *)"even" : "odd");

return (0);
    }

that is good Johnny what a good boy you are, you have shown what is call the dexterity of coding. showing others its flexibility now go show your mommy and tell her what you did, and maybe she will give you a lolly pop and pat you on the head. then you can go and try to make others feel lesser then you so you can feel Superior to them. which is only short lived then you just go out and find someone else to pick on so you can try to feed your poorly structured ego. to make yourself feel better about yourself. ( Superior to others ) .

superiority complex due to feelings of inferiority

PS your formatting sucks btw

I think you're wasting your time Hodor with this shrub - though it is entertaining

He continues to post rubbish, but never listens to any critisism.

> infile.seekg(position-2, std::ios::beg);
I'd like to see what happens with this "program" when presented with a file where the first character is a comma.