Quiero realizar la función de copiar y pegar un archivo de una carpeta a otra, y agregue un checkbox del datagriew para poder escoger varios registros y no solo uno por uno, por eso cree como un arreglo, bueno como se muestra en el código, y me cumple toda la función, todo ok, el problema es que sale a cada rato el
MessageBox.Show("Archivo Copiado Exitosamente en C:\FACTURAS.");
private void btnarchivos_Click(object sender, EventArgs e)
{
try
{
foreach (DataGridViewRow ite in dgfacturas.Rows)
{
DataGridViewCheckBoxCell chk = (DataGridViewCheckBoxCell)ite.Cells[0];
if (chk.Value == chk.TrueValue)
{
foreach (DataGridViewRow row in dgfacturas.Rows)
{
if (Convert.ToBoolean(row.Cells[0].Value))
{
string numser = row.Cells["Serie"].Value.ToString();
string numdoc = row.Cells["Documento"].Value.ToString();
string rutaInicial = @"\\hpserver\DBSQL\EF\20501820025\" + cmbanio.SelectedItem + @"\1SINCRONO\FACTURA\FILE\20501820025-01-" + numser + "-" + numdoc + ".pdf";
string rutaInicial2 = @"\\hpserver\DBSQL\EF\20501820025\" + cmbanio.SelectedItem + @"\1SINCRONO\FACTURA\FILE\20501820025-01-" + numser + "-" + numdoc + ".xml";
string rutaFinal = @"C:\FACTURAS\" + lblcliente.Text + @"\20501820025-01-" + numser + "-" + numdoc + ".pdf";
string rutaFinal2 = @"C:\FACTURAS\" + lblcliente.Text + @"\20501820025-01-" + numser + "-" + numdoc + ".xml";
if (!(Directory.Exists(@"C:\FACTURAS\" + lblcliente.Text)))
{
Directory.CreateDirectory(@"C:\FACTURAS\" + lblcliente.Text);
MessageBox.Show("Carpeta Creada!!");
}
if (File.Exists(rutaInicial) && File.Exists(rutaInicial2))
{
File.Copy(rutaInicial, rutaFinal, true);
File.Copy(rutaInicial2, rutaFinal2, true);
// MessageBox.Show("Archivo Copiado Exitosamente en C:\\FACTURAS.");
}
// MessageBox.Show("Archivo Copiado Exitosamente en C:\\FACTURAS.");
}
// MessageBox.Show("Archivo Copiado Exitosamente en C:\\FACTURAS.");
}
MessageBox.Show("Archivo Copiado Exitosamente en C:\\FACTURAS.");
}
else if (chk.Value == chk.FalseValue)
{
MessageBox.Show("Por favor escoja una fila.");
}
dgfacturas.EndEdit();
}
}
catch (Exception ex)
{
MessageBox.Show("Verificar los procesos."+ex);
}
}
foreach
cuando (a mi entender) uno solo es suficiente?. Lo que está en tu código puede leerse algo así: Por cada elemento que está marcado (primerforeach
) recorre todo el grid otra vez (segundoforeach
) y si el elemento está marcado, copialo. El resultado es que cada archivo se copia tantas veces como la cantidad de archivos marcados.