使用c#进行文件拆分的问题

时间:2015-12-29 18:24:39

标签: c# multithreading backgroundworker

我一直在尝试制作一个程序,将较大的文本文件拆分成较小的部分,以便更容易使用。

我目前有两个问题,无法弄清楚发生了什么。

问题1:后台工作人员有时会多次开火。我似乎无法弄清楚它决定运行的原因或次数。它将运行拆分并在最终文件上,它似乎循环回到do work的开头并再次运行它。它还会启动多个完成工作的任务。更复杂的是,如果我将文件数量设置为不同的数字,我可以获得后台工作人员似乎触发的不同次数,但它并不直接与文件数量相关。有时,相同数量的文件会导致后台工作程序仅触发一次,有时会多次触发。

问题2:有时拆分不会创建所有文件。对于一些文件,如果我运行它将创建前两个文件,然后删除其余文件。似乎只有在我将数字设置为3个要分割的文件时才会发生。如果我采用行计数并将其加起来它应该正确相等。所以我不确定那里发生了什么。

致电主题

private void StartSplit()
    {
        if (int.TryParse(NumberOfFilesTB.Text, out _numberOfFiles))
        {
            if (bg.IsBusy)
            {
                ((MainWindow)Application.Current.MainWindow).SetStatus("Warning",
                    "Please only run one split process at a time.");
                return;
            }

            ((MainWindow)Application.Current.MainWindow).DisplayAlert(
                "Split is running, you will receive an alert when it has finished. You may use other tools while the split is running.");

            var args = new List<string> { _filepath, _includeHeaders.ToString(), _numberOfFiles.ToString() };
            bg.DoWork += bg_DoWork;
            bg.WorkerReportsProgress = true;
            bg.ProgressChanged += ProgressChanged;
            bg.RunWorkerCompleted += bg_RunWorkerCompleted;
            bg.WorkerSupportsCancellation = true;
            bg.RunWorkerAsync(args);
            ProcessText.Text = "Running split process";
        }
        else
        {
            ((MainWindow)Application.Current.MainWindow).SetStatus("Warning", "Please enter a number for number of files");
        }
    }

后台主题

  private void bg_DoWork(object sender, DoWorkEventArgs e)
    {
        var args = e.Argument as List<string>;
        string filepath = args[0];
        string includeHeaders = args[1];
        int numberOfFiles = Convert.ToInt32(args[2]);
        int numberOfRows = _lineCount / numberOfFiles;
        _tempath = Path.GetDirectoryName(_filepath);
        Directory.CreateDirectory(_tempath+"\\split");

        if (includeHeaders == "True")
        {
            using (var reader = new StreamReader(File.OpenRead(filepath)))
            {
                _lines.Clear();
                _header = reader.ReadLine();
                _lines.Add(_header);

                for (int i = 0; i < _lineCount; i++)
                {

                    if (bg.CancellationPending)
                    {
                        e.Cancel = true;
                        break;
                    }

                    int percentage = (i + 1) * 100 / _lineCount;

                    bg.ReportProgress(percentage);

                    _lines.Add(reader.ReadLine());

                    if (i % numberOfRows == 0)
                    {
                        _counter++;
                        Debug.WriteLine(i);

                        if (i == 0)
                        {
                            //skip first iteration 
                            _counter = 0;
                            continue;

                        }
                        _output = _tempath + "\\" + "split\\" + _fileNoExt + "_split-" + _counter + _fileExt;
                        _filesMade.Add(_output);
                        File.WriteAllLines(_output, _lines.ConvertAll(Convert.ToString));
                        _lines.Clear();
                        _lines.Add(_header);
                    }
                }
            }
        }
        else
        {
            using (var reader = new StreamReader(File.OpenRead(filepath)))
            {
                _lines.Clear();
                _header = reader.ReadLine();
                _lines.Add(_header);
                for (int i = 0; i < _lineCount; i++)
                {
                    if (bg.CancellationPending)
                    {
                        e.Cancel = true;
                        break;
                    }

                    int percentage = (i + 1) * 100 / _lineCount;
                    bg.ReportProgress(percentage);
                    _lines.Add(reader.ReadLine());

                    if (i % numberOfRows == 0)
                    {
                        _counter++;

                        if (i == 0)
                        {
                            //skip first iteration
                            _counter = 0;
                            continue;
                        }
                        string output = _tempath + "\\" + "split\\" + _fileNoExt + "_split-" + _counter + _fileExt;
                        _filesMade.Add(_output);
                        File.WriteAllLines(output, _lines.ConvertAll(Convert.ToString));
                        _lines.Clear();

                    }
                }
            }

        }
    }

运行工作人员已完成

private void bg_RunWorkerCompleted(object sender, RunWorkerCompletedEventArgs e)
    {
        if (e.Cancelled)
        {
            StopSplit();
            _filesMade.Clear();
            ProcessText.Text = "Split cancelled";
            return;
        }
        _filesMade.Clear();
        ProcessText.Text = "Split has completed, click here to open the directory";
    }

1 个答案:

答案 0 :(得分:3)

我打赌你的BgW是你班上的一员...
在Startsplit()中,每次执行此函数时都会添加一个新的回调    这就是它多次运行的原因。

晚餐后的其他答案。

晚餐...... 你的计算方法有多种错误:
1)如果你丢失文件,我敢打赌它是最后一个。例如。 30行,3档:
{= 1}}在i = 0,10,20处为零,但我没有达到30 2)你缺少线条,例如31行4档:
文件保存在i = 7,14,21,28。缺少29-31行。

我建议您使用嵌套for循环,外部用于文件,内部用于行,并改进计算。并将所有列表和计数器放在函数内!
我希望你能理解我的回答。我讨厌在平板电脑上打字。但也不想为此启动我的电脑...; - )