写文件代码为什么出错

[tristan0429]

请大侠帮忙看看，这段代码存在着哪些潜在的问题。谢谢！

    class Program
    {
        static void Main(string[] args)
        {
            try
            {
                Thread t1 = new Thread(Logger.Write);
                t1.Name = "t1";
                Thread t2 = new Thread(Logger.Write);
                t2.Name = "t2";

                t1.Start();
                t2.Start();

            }
            catch (Exception ex)
            {
                Console.WriteLine(ex.Message);
            }
        }
    }


    class Logger
    {
        const int C = 10;

        public static void Write()
        {
            using (FileStream fs = new FileStream("c:\\a.txt", FileMode.OpenOrCreate, FileAccess.Write))
            {
                using (StreamWriter sw = new StreamWriter(fs))
                {
                    int i = 0;

                    while (i < C)
                    {
                       // 下行代码如果注释，可以运行，但不知道是否存在潜在的问题。 
                         //Thread.Sleep(100);
                        sw.WriteLine(Thread.CurrentThread.Name + "  --  " + DateTime.Now.ToLocalTime());
                        i++;
                    }
                }
            }
        }
    }

[tristan0429]

我没显示写关闭是因为用了 using block

[Quote=引用 12 楼 sunshine_anycall 的回复:]
用lock锁一下吧，
或者有个类可以实现多线程同时读，单线程写入的功能
[/Quote]

这个能具体点吗？谢谢！

[tweeger]

像这样：
static class Logger
{
private const int C = 10;
private static readonly object locker = new object();

public static void Write()
{
lock (locker)
{
using (FileStream fs = new FileStream("e:\\a.txt", FileMode.OpenOrCreate, FileAccess.Write))
{
using (StreamWriter sw = new StreamWriter(fs))
{
int i = 0;

while (i < C)
{
Thread.Sleep(100);
sw.WriteLine(Thread.CurrentThread.Name + " -- " + DateTime.Now.ToLocalTime());
i++;
}
}
}
}
}
}

[lethwei]

[Quote=引用 14 楼 jie3614 的回复:]
sw需要close()

不是线程的问题，线程可以多个，不过不能同时操作一个文件

你这儿sw没有close()
[/Quote]
不是 close() 的问题, using 中的语句自己最后会调用 dispose(), 不需要手动关闭
个人认为是线程同步的问题, 文件被一个线程以写方式打开, 另一个写方式打开的尝试应该会被拒绝, 故报错
简单点的方式是

public static void Write()
{
    static object l_obj = new object();
    lock (l_obj) {
        using (FileStream fs = new FileStream("c:\\a.txt", FileMode.OpenOrCreate, FileAccess.Write))
        {
            using (StreamWriter sw = new StreamWriter(fs))
            {
                int i = 0;

                while (i < C)
                {
                    // 下行代码如果注释，可以运行，但不知道是否存在潜在的问题。 
                    //Thread.Sleep(100);
                    sw.WriteLine(Thread.CurrentThread.Name + "  --  " + DateTime.Now.ToLocalTime());
                    i++;
                }
            }
        }
    }
}

[tristan0429]

试了一下，楼上的方法可以，一般做这种情况的操作是不是都这样实现的？还有什么实现方式？

[深海之蓝]

奇怪，我用多线程写，没用锁，也没出问题啊?

[lethwei]

写 log 的话, 我一般在程序初始化时就把 log 文件打开, 然后提供一个 WriteLog() 写该文件, WriteLog() 中用 lock 做简单的线程同步保证, 程序退出时关闭 log 文件

[优途科技]

up

[深海之蓝]

奇怪，刚还没有问题，现在试了试，竟然出现了问题你。。。。。。

[深海之蓝]

刚用 楼上的lock 貌似还有问题，在网上找到一个，经测试没有问题
private Mutex mWriter = null;
//你的方法
if (mWriter == null) mWriter = new Mutex();
mWriter.WaitOne();
try
{
//your code
}
catch
{
}
mWriter.ReleaseMutex();//

[tailor_ms]

Mark

[shijie_li]

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading;
using System.IO;
namespace ConsoleApplication7 {
class Program {
static void Main(string[] args) {
try {
Thread t1 = new Thread(Logger.Write);
t1.Name = "t1";
Thread t2 = new Thread(Logger.Write);
t2.Name = "t2";
Thread t3 = new Thread(Logger.Write);
t3.Name = "t3";

t1.Start();
t2.Start();
t3.Start();

}
catch (Exception ex) {
Console.WriteLine(ex.Message);
}
}
}
class LoggerText:IDisposable
{
static private LoggerText _instance;
static private object _lockObj;
static private object _lockInstanceObj = new object();
private FileStream _fs;
private StreamWriter _sw;
private LoggerText(){
_fs = new FileStream("c:\\a.txt", FileMode.OpenOrCreate, FileAccess.Write);
_sw = new StreamWriter(_fs);
_lockObj = new object();
}
public void wirteIn(string text){
lock(_lockObj)
{
_sw.WriteLine(text);
_sw.Flush();
}
}
static public LoggerText Instance(){
lock (_lockInstanceObj) {
if (_instance == null) {
_instance = new LoggerText();
}
}
return _instance;
}

#region IDisposable Members

void IDisposable.Dispose() {
_sw.Close();
_fs.Close();
}

#endregion
}
class Logger {
const int C = 1000;
public static void Write() {
int i = 0;
while (i < C) {
// 下行代码如果注释，可以运行，但不知道是否存在潜在的问题。
//Thread.Sleep(100);
LoggerText.Instance().wirteIn(Thread.CurrentThread.Name + " -- " + DateTime.Now.ToLocalTime() + " -- " + i.ToString());
i++;
}
}
}
}


注意 单例LoggerText在应用开始时初始化（Instance）在应用结束时干掉（Dispose()）

[shijie_li]

以上代码经测试 1000个线程并发运行，没有问题

mian 函数改为

static void Main(string[] args) {
try {
for (int i = 1; i < 100; i++) {
Thread t = new Thread(Logger.Write);
t.Name = "t"+i.ToString();
t.Start();
}
}
catch (Exception ex) {
Console.WriteLine(ex.Message);
}
}

[suners]

搬个板凳 坐着看看

[benbirdar]

楼主的问题跟sw没有调用Close()无关，
Using会自动释放资源关闭sw

看不处错误，友情帮顶！顺便Mark一会来看高手答案

[shijie_li]

楼主的问题，是同一文件，多个线程同时访问的问题啊

我贴的东西已经解决这个问题，等楼主放分

[kingtiy]

"c:\\a.txt"运行没问题，只是调试的时候，会出现多外线程同时处抢资源的情况

[Cherishny]

UP

[xianguang321]

注释掉可能出现问题,可能只有一个线程工作,另一个线程等待.因为CPU时间片轮询不过来.到时候你退出线程的时候就会有问题了!

[山茶树和葡萄树]

try
{
StreamReader TxtReader = new StreamReader(@"D:\Documents and Settings\Administrator\桌面\NewText1\1.txt", System.Text.Encoding.Default);
string strFileContent = TxtReader.ReadToEnd(); TxtReader.Close();

StreamWriter TxtWriter = new StreamWriter(@"D:\Documents and Settings\Administrator\桌面\NewText1\1.txt", true, System.Text.Encoding.Default);
TxtWriter.Write(strFileContent);//改写的对象
TxtWriter.Close();
MessageBox.Show("文件打开成功");
}
catch (Exception)
{

MessageBox.Show("文件打开失败");
}

[liusichen_0]

多线程使用的代码要注意资源的控制，像什么信号量啊，互斥量啊