Using public static object for locking thread shared resources

Question

Lets Consider the example,
I have two classes
1.  Main_Reader -- Read from file
public  class Main_Reader
     public static object tloc=new object();
       public void Readfile(object mydocpath1)
           lock (tloc)
               string mydocpath = (string)mydocpath1;
               StringBuilder sb = new StringBuilder();
               using (StreamReader sr = new StreamReader(mydocpath))
                   String line;
                   // Read and display lines from the file until the end of 
                   // the file is reached.
                   while ((line = sr.ReadLine()) != null)
                       sb.AppendLine(line);
               string allines = sb.ToString();
2. MainWriter -- Write the file
public  class MainWriter
      public void Writefile(object mydocpath1)
          lock (Main_Reader.tloc)
              string mydocpath = (string)mydocpath1;
              // Compose a string that consists of three lines.
              string lines = "First line.\r\nSecond line.\r\nThird line.";
              // Write the string to a file.
              System.IO.StreamWriter file = new System.IO.StreamWriter(mydocpath);
              file.WriteLine(lines);
              file.Close();
              Thread.Sleep(10000);
In main have instatiated two function with two threads.
 public string mydocpath = "E:\\testlist.txt";  //Here mydocpath is shared resorces
         MainWriter mwr=new MainWriter();
         Writefile wrt=new Writefile();
           private void button1_Click(object sender, EventArgs e)
            Thread t2 = new Thread(new ParameterizedThreadStart(wrt.Writefile));
            t2.Start(mydocpath);
            Thread t1 = new Thread(new ParameterizedThreadStart(mrw.Readfile));
            t1.Start(mydocpath);
            MessageBox.Show("Read kick off----------");
For making this shared resource thread safe, i am using  a public static field,
  public static object tloc=new object();   in class Main_Reader
My Question is ,is it a good approach.
Because i read in one of msdn forums, "avoid locking on a public type"
Is any other approach for making this thread safe.

Answer 1

Hi ,
Since they are easily accessed by all threads, thus you need to apply any kind of synchronization (via locks, signals, mutex, etc).
Mutex:https://msdn.microsoft.com/en-us/library/system.threading.mutex(v=vs.110).aspx
Thread signaling basics:http://stackoverflow.com/questions/2696052/thread-signaling-basics
Best regards,
Kristin
We are trying to better understand customer views on social support experience, so your participation in this interview project would be greatly appreciated if you have time. Thanks for helping make community forums a great place.
Click
HERE to participate the survey.