EOFException error in I/O. Are threads the problem?

Hi,

I have been using the following code for years now with now problem. Suddenly now that I am trying
to use it with 2 threads I get an EOFException. I thought that threads were supposed to be mutually
exclusive (in other words one cannot run until another one has finished). Both of my threads use
this code. Can anyone suggest a way that I could make sure only one thread at a time is trying
to I/O? I am new to threads.

Code:

---

public static void writeObj(Vector v, String s){

        String dataloc2 = s;

        Vector ObjectVector2 = new Vector();     
        ObjectVector2 = v;     
   
        try
        {
            FileOutputStream fos5 = new FileOutputStream(dataloc2);
            ObjectOutputStream oos5 = new ObjectOutputStream(fos5);
            oos5.writeObject(ObjectVector2);
            oos5.flush();
            oos5.close();
            System.out.println("wrote "+dataloc2);
 
        }
        catch(Exception exception)
        {
            System.out.println("Exception during serialization: " + exception);
            System.exit(0);
        }

 } // end of method

---

Thanks in advance for your help!

If you made that method synchronized, only one thread at a time could call it or any other synchronized static method of the same class. Note that this would not prevent another thread from opening the same file by some other means.

Ok thanks for the advice. I put replaced public with synchronized and it still compiled, so will try to run it.

PS: not sure what you mean by "Get in the habit of using standard Java naming conventions! Except #7 and maybe #8. Those are kind of dumb."

What are number 7 and number 8?

Thanks again for the advice.

That's kjkrum's signature line, not part of the response to your question.

Quote:

---

What are number 7 and number 8?

---

Click the link if you really want to know. I prefer Code Conventions for the Java(TM) Programming Language: Contents myself.

db

Yeah sorry, I figured that out after I sent the post.....

You don't need to replace public with synchronized. A method can be both.