replace a string using Text i/o

[PureAwesomeness]

This program is to replace all the occurrences of a specific string from a text file.
let say to remove John from a text file.

on i/o this topic i just started learning it and im not really sure what are my logical errors. So can somebody please give me some advice on this program. Thanks!


in command prompt i passed in
java removeText.java John john.txt

import java.io.*;
import java.util.*;

	public class removeText
	{
		public static void main (String[] args) throws Exception
		{
			// Check command line parameter usage
			if(args.length !=2)
			{
				System.out.println("Usage: java Exercise8.21 John FileName");
				  System.exit(0);
			}

			//Check FileName if exists
			File SourceFile = new File(args[1]);
			if(!SourceFile.exists())
			{
				System.out.println("Source file "+args[1]+" does not exist");
				System.exit(0);
			}

			//Create input and output files
			Scanner input= new Scanner(SourceFile);
			PrintWriter output = new PrintWriter(SourceFile);

			while(input.hasNext())
			{
				String s1 = input.nextLine();
				String s2 = s1.replaceAll(args[0], "");
                                                   	output.println(s2);
			}
			 input.close();
			 output.close();
		}//end of main
	}//end of removeText

[Fubarable]

You can't input and output to the same source file. Instead how about writing to a dummy file, then after complete, rename it to the source file's name, thereby overwriting the source file.

[PureAwesomeness]

the question that i have requires me to pass in only one file.

if possible can you give me an example of using a dummy file?

[Fubarable]

You only need to pass in a single file parameter. Simply create a new file with a name of your choosing, and then write to it. If you don't know how to do this, then have a peek at the IO tutorial on the Sun tutorial site, and you'll be able to learn all you need to know.

[PureAwesomeness]

can I create a dummy file(a copy of sourceFile) and then use one of them as scanner and the other one as printwriter?

[Singing Boyo]

@ Fubarable it is possible to read/write to the same file, as long as all reading is complete before the OutputStream to the file is opened...

so some psuedocode for you... (I'm avoiding the dummy file idea, which may be better)

test file validity and create Scanner.
Read all the lines in the file and store them in an array/ArrayList //probably ArrayList...
Create PrintWriter.
for(all lines in array/ArrayList){
Remove String that needs to be removed
Write line to file
}
Close Scanner and PrintWriter

Of course some of this could be moved around, but it's the general idea.

[Fubarable]

@ Fubarable it is possible to read/write to the same file, as long as all reading is complete before the OutputStream to the file is opened...

Yep, thanks for the clarification, ... but this will fall flat if the files are large. Streaming in place with a dummy file doesn't have this limitation and at least in my opinion is a better idea.

[PureAwesomeness]

This is Singing Boyo's suggestion but the codes have errors

import java.io.*;
import java.util.*;

	public class test
	{
		public static void main (String[] args) throws Exception
		{
			// Check command line parameter usage
			if(args.length !=2)
			{
				System.out.println("Usage: java test John FileName");
      			System.exit(0);
			}

			//Check FileName if exists
			File SourceFile = new File(args[1]);
			if(!SourceFile.exists())
			{
				System.out.println("Source file "+args[1]+" does not exist");
				System.exit(0);
			}

			//Create input and output files
			Scanner input= new Scanner(SourceFile);

			char[] chars;
			while(input.hasNext())
			{
				String s1 = input.nextLine();
			    chars = s1.toCharArray(s1.split(" "));
			}

			for(int i=0; i<chars.length; i++)
			{
				if(chars[i] == "John")
				chars[i].replaceAll(chars[i],"");
			}
			PrintWriter output = new PrintWriter(SourceFile);

			for(int i=0; i<chars.length; i++)
			{
				output.print(chars[i]+ " ");
			}
			 input.close();
			 output.close();
		}//end of main
	}//end of exercise8.21

This is Fubarable suggestion of creating a dummy file but it also has some errors in it

import java.io.*;
import java.util.*;

	public class test
	{
		public static void main (String[] args) throws Exception
		{
			// Check command line parameter usage
			if(args.length !=2)
			{
				System.out.println("Usage: java test John FileName");
      			System.exit(0);
			}

			//Check FileName if exists
			File SourceFile = new File(args[1]);
			if(!SourceFile.exists())
			{
				System.out.println("Source file "+args[1]+" does not exist");
				System.exit(0);
			}

			//char[] char = SourceFile.toCharArray()

			//Create a dummy file

			File DummyFile = new File("TempFile.txt");
			boolean renameFile = DummyFile.renameTo(SourceFile);

			//Create a target file

			File TargetFile = new File(args[1]);


			//Create input and output files
			Scanner input= new Scanner(DummyFile);
			PrintWriter output = new PrintWriter(TargetFile);

			while(input.hasNext())
			{
				String s1 = input.next();
				String s2 = s1.replaceAll(args[0], "");
				output.println(s2);

			}
			DummyFile.delete();
			 input.close();
			 output.close();
		}//end of main
	}//end of test

can someone please help me with some of those errors?

[angryboy]

			while(input.hasNext())
			{
				String s1 = input.nextLine();
      [B]  // WHAT? what are you trying to do?
			    chars = s1.toCharArray(s1.split(" "));[/B]
			}

			for(int i=0; i<chars.length; i++)
			{
   [B]     // err... no you can't compare a char to a string.
				if(chars[i] == "John")
        // no such method for chars
				chars[i].replaceAll(chars[i],"");
[/B]			}

I'd suggest that you read a file, then write to a dummy file to see if they are exactly the same before you try to edit the text. This way, you know there's no coding error in reading/writing.

btw, there is a third option which was not suggested for good reasons.

[COLOR=NAVY]import[/COLOR] java[COLOR=BLACK].[/COLOR]io[COLOR=BLACK].[/COLOR][COLOR=BLACK]*[/COLOR][COLOR=BLACK];[/COLOR]
[COLOR=PURPLE]public[/COLOR] [COLOR=PURPLE]class[/COLOR] RandomAccessFileDemo [COLOR=BLACK]{[/COLOR]
  [COLOR=PURPLE]public[/COLOR] [COLOR=PURPLE]static[/COLOR] [COLOR=PURPLE]final[/COLOR] String NEWLINE [COLOR=BLACK]=[/COLOR] 
    System[COLOR=BLACK].[/COLOR]getProperty[COLOR=BLACK]([/COLOR][COLOR=SILVER]"line.separator"[/COLOR][COLOR=BLACK])[/COLOR][COLOR=BLACK];[/COLOR]
  
  [COLOR=PURPLE]public[/COLOR] [COLOR=PURPLE]static[/COLOR] [COLOR=PURPLE]void[/COLOR] main[COLOR=BLACK]([/COLOR]String[COLOR=BLACK][[/COLOR][COLOR=BLACK]][/COLOR] args[COLOR=BLACK])[/COLOR][COLOR=BLACK]{[/COLOR]
    [COLOR=NAVY]try[/COLOR][COLOR=BLACK]{[/COLOR]
      RandomAccessFile raf [COLOR=BLACK]=[/COLOR] [COLOR=NAVY]new[/COLOR] RandomAccessFile
          [COLOR=BLACK]([/COLOR][COLOR=SILVER]"hey.txt"[/COLOR][COLOR=GREEN]/*file-name*/[/COLOR][COLOR=BLACK],[/COLOR][COLOR=SILVER]"rw"[/COLOR][COLOR=GREEN]/*read-write mode*/[/COLOR][COLOR=BLACK])[/COLOR][COLOR=BLACK];[/COLOR]
      
      [COLOR=PURPLE]long[/COLOR] length [COLOR=BLACK]=[/COLOR] raf[COLOR=BLACK].[/COLOR]length[COLOR=BLACK]([/COLOR][COLOR=BLACK])[/COLOR][COLOR=BLACK];[/COLOR]
      String line[COLOR=BLACK];[/COLOR]
      [COLOR=NAVY]for[/COLOR][COLOR=BLACK]([/COLOR][COLOR=PURPLE]long[/COLOR] rPos[COLOR=BLACK]=[/COLOR][COLOR=ORANGE][B]0L[/B][/COLOR][COLOR=BLACK],[/COLOR] wPos[COLOR=BLACK]=[/COLOR][COLOR=ORANGE][B]0L[/B][/COLOR][COLOR=BLACK];[/COLOR] rPos[COLOR=BLACK]<[/COLOR]length[COLOR=BLACK];[/COLOR] raf[COLOR=BLACK].[/COLOR]seek[COLOR=BLACK]([/COLOR]rPos[COLOR=BLACK])[/COLOR][COLOR=BLACK])[/COLOR][COLOR=BLACK]{[/COLOR]
        [COLOR=GREEN][I]// READ[/I][/COLOR]
        line [COLOR=BLACK]=[/COLOR] raf[COLOR=BLACK].[/COLOR]readLine[COLOR=BLACK]([/COLOR][COLOR=BLACK])[/COLOR][COLOR=BLACK];[/COLOR]
        rPos [COLOR=BLACK]=[/COLOR] raf[COLOR=BLACK].[/COLOR]getFilePointer[COLOR=BLACK]([/COLOR][COLOR=BLACK])[/COLOR][COLOR=BLACK];[/COLOR]

        [COLOR=GREEN][I]// EDIT TEXT[/I][/COLOR]
        line [COLOR=BLACK]=[/COLOR] line[COLOR=BLACK].[/COLOR]replaceAll[COLOR=BLACK]([/COLOR][COLOR=SILVER]"\\D"[/COLOR][COLOR=BLACK],[/COLOR][COLOR=SILVER]"~"[/COLOR][COLOR=BLACK])[/COLOR][COLOR=BLACK];[/COLOR]
        [COLOR=GREEN][B]/**
          Make sure new line.length() 
          does not exceed old line.length()
         */[/B][/COLOR]
        
        [COLOR=GREEN][I]// WRITE[/I][/COLOR]
        raf[COLOR=BLACK].[/COLOR]seek[COLOR=BLACK]([/COLOR]wPos[COLOR=BLACK])[/COLOR][COLOR=BLACK];[/COLOR]
        raf[COLOR=BLACK].[/COLOR]writeBytes[COLOR=BLACK]([/COLOR]line [COLOR=BLACK]+[/COLOR] NEWLINE[COLOR=BLACK])[/COLOR][COLOR=BLACK];[/COLOR]
        wPos [COLOR=BLACK]=[/COLOR] raf[COLOR=BLACK].[/COLOR]getFilePointer[COLOR=BLACK]([/COLOR][COLOR=BLACK])[/COLOR][COLOR=BLACK];[/COLOR]
      [COLOR=BLACK]}[/COLOR]
      raf[COLOR=BLACK].[/COLOR]setLength[COLOR=BLACK]([/COLOR]raf[COLOR=BLACK].[/COLOR]getFilePointer[COLOR=BLACK]([/COLOR][COLOR=BLACK])[/COLOR][COLOR=BLACK])[/COLOR][COLOR=BLACK];[/COLOR] [COLOR=GREEN][I]// TRUNCATE[/I][/COLOR]
      raf[COLOR=BLACK].[/COLOR]close[COLOR=BLACK]([/COLOR][COLOR=BLACK])[/COLOR][COLOR=BLACK];[/COLOR] [COLOR=GREEN][I]// duh...[/I][/COLOR]
    [COLOR=BLACK]}[/COLOR][COLOR=NAVY]catch[/COLOR][COLOR=BLACK]([/COLOR]IOException x[COLOR=BLACK])[/COLOR][COLOR=BLACK]{[/COLOR]
      System[COLOR=BLACK].[/COLOR]err[COLOR=BLACK].[/COLOR]println[COLOR=BLACK]([/COLOR][COLOR=SILVER]"OMG! Now your file is corrupted!"[/COLOR][COLOR=BLACK])[/COLOR][COLOR=BLACK];[/COLOR]
    [COLOR=BLACK]}[/COLOR]
  [COLOR=BLACK]}[/COLOR]
[COLOR=BLACK]}[/COLOR]

[PureAwesomeness]

		while(input.hasNext())
			{
				String s1 = input.nextLine();
        // WHAT? what are you trying to do?
			    chars = s1.toCharArray(s1.split(" "));
			}

			for(int i=0; i<chars.length; i++)
			{
        // err... no you can't compare a char to a string.
				if(chars[i] == "John")
        // no such method for chars
				chars[i].replaceAll(chars[i],"");
			}

WHAT? what are you trying to do?
im trying to put each word in an array, i guess that is not a correct way.

// err... no you can't compare a char to a string.
If i successfully puting each word in an array i would comparing a string to string. i guess that failed too.

// no such method for chars
thanks! now i know =]