Como a abordagem não parece ser boa, resolvi fazer um exemplo compilável que resolvesse o problema de uma forma genérica.
using System;
using System.IO;
using Util.IO;
public class MergeFiles {
public static void Main(string[] args) {
int bufferSize;
FileUtil.MergeTextFiles(args[0], args[1], args[2], (int.TryParse(args[3], out bufferSize) ? bufferSize : 0));
}
}
namespace Util.IO {
public static class FileUtil {
public static void MergeTextFiles(string targetFileName, string sourcePath, string searchPattern = "*.*", int bufferSize = 0) {
if (string.IsNullOrEmpty(sourcePath)) {
sourcePath = Directory.GetCurrentDirectory();
}
if (targetFileName.IndexOfAny(System.IO.Path.GetInvalidPathChars()) != -1) {
throw new ArgumentException("Diretório fonte especificado contém caracteres inválidos", "sourcePath");
}
if (string.IsNullOrEmpty(targetFileName)) {
throw new ArgumentException("Nome do arquivo destino precisa ser especificado", "targetFileName");
}
if (string.IsNullOrEmpty(targetFileName)) {
throw new ArgumentException("Nome do arquivo destino precisa ser especificado", "targetFileName");
}
if (targetFileName.IndexOfAny(System.IO.Path.GetInvalidFileNameChars()) != -1) {
throw new ArgumentException("Nome do arquivo destino contém caracteres inválidos", "targetFileName");
}
var targetFullFileName = Path.Combine(sourcePath, targetFileName);
if (bufferSize == 0) {
File.Delete(targetFullFileName);
foreach (var file in Directory.GetFiles(sourcePath, searchPattern)) {
if (file != targetFullFileName) {
File.AppendAllText(targetFullFileName, File.ReadAllText(file));
}
}
} else {
using (var targetFile = File.Create(targetFullFileName, bufferSize)) {
foreach (var file in Directory.GetFiles(sourcePath, searchPattern)) {
if (file != targetFullFileName) {
using (var sourceFile = File.OpenRead(file)) {
var buffer = new byte[bufferSize];
int bytesRead;
while ((bytesRead = sourceFile.Read(buffer, 0, buffer.Length)) > 0) {
targetFile.Write(buffer, 0, bytesRead);
}
}
}
}
}
}
}
}
}
Coloquei no GitHub para referência futura.
Em versões mais novas dá para reduzir esse código.
O método Main()
está aí só para facilitar um teste rápido, não está em condições de produção. O método MergeTextFiles()
está bem razoável para uso. Não está 100%, não fiz uma unidade de testes para ele, não documentei, não pensei em todas as situações possíveis, mas já está bem encaminhado.
Você pode escolher um tamanho de buffer se quiser controlar melhor a forma de cópia. Se achar que nunca vai precisar disto, pode tirar esta parte do método. Mas não faz mal deixar, já que o padrão é fazer a cópia integral dos arquivos dentro dos critérios do da implementação corrente do .NET.
Melhorias possíveis
Algumas melhorias podem ser feitas para deixar mais genérico ainda ou adicionar funcionalidades. Você poderia, por exemplo, colocar um último parâmetro parâmetro extraNewLineOptions extraNewLineOption = extraNewLineOptions.NoExtraNewLine
e uma enumeração enum extraNewLineOptions { NoExtraNewLine, SelectiveExtraNewLine, AlwaysExtraNewLine }
.
Para permitir que uma quebra de linha extra possa ser colocada no final de cada arquivo para garantir que não vá encavalar textos. Isto pode ser útil mas na maioria dos casos não é necessário, por isso ele ficaria desabilitado por default. Deixo para a criatividade de cada um a implementação disso, principalmente pelo SelectiveExtraNewLine()
que só colocaria uma quebra de linha se não existe no final do arquivo, não é tão trivial implementar. É possível criar um overload para melhorar o uso dos parâmetros.
Outra melhoria é permitir que a cópia seja feita de forma assíncrona. Muito útil se tiver grandes volumes de arquivos.
E o método poderia ser quebrando em partes também.
Dependendo da versão do .NET
Usei recursos para poder rodar em praticamente qualquer versão do .NET. Se for garantido que será usado em versões mais recentes, é possível trocar as verificações de parâmetros por Contract.Requires()
. Ou até é possível retirar tudo isso já que a verificação de todos esses problemas também são feitas nos métodos chamados. Claro que você perderia a localidade da informação de onde exatamente originou o erro.
Infelizmente não há um método público para verificar a validade do coringa antecipadamente. Mas se for necessário é possível verificar como é implementado nos fontes do .NET (e possivelmente nos fontes do Mono também (no .NET Core).
Se você tiver o C# 6 (através do Roslyn), algumas melhorias poderão ser feitas.
Poderia usar um using Util.IO.FileUtil;
e depois chamar o método diretamente: MergeTextFiles("combo.txt", ".", "*.txt")
.
Além disso as declarações int bufferSize;
no método Main()
e int bytesRead;
poderiam ser feitas inline durante a sua utilização durante o TryParse()
e o while
respectivamente: int.TryParse(args[3], out var bufferSize
e while ((int bytesRead = sourceFile.Read(buffer, 0, buffer.Length)) > 0) {
.
Veja o exemplo em C# 6 no ideone. E no .NET Fiddle. Também coloquei no GitHub para referência futura.