none
Como agrego un metodo a otro metodo c# RRS feed

  • Pregunta

  •     private void Eliminar(string Imagen)
            {             
                string path = Path.Combine(Server.MapPath("../Documentos/"),Imagen);
               
                if (System.IO.File.Exists(path))
                {
                    try
                    {
                        System.IO.File.Delete(path);
                      
                    }
                    catch (System.IO.IOException ev)
                    {
                        Console.WriteLine(ev.Message);
                        return;
                    }
                }
            }
            protected void GridDocumentos_RowDataBound(object sender, GridViewRowEventArgs e)
            {
                e.Row.FindControl("Imagen").ToString();
    
    
            }
            #endregion
    
            private void EliminarTabla(int Id_Documento)
            {
                ws.BorrarDocumento(Id_Documento);
            }
    Quiero meter  el metodo  EliminarTabla    Al metodo Eliminar en la parte de  
      System.IO.File.Delete(path);   Aqui para eliminar tambien el archivo de la tabla de sql.
    miércoles, 6 de mayo de 2015 21:18

Respuestas

  • hola

    pero porque quieres hacer eso ? no es correcto, estas rompiendo con las repsonsabilidades de cada funcionalidad

    por un lado define el metodo que elimina el archivo, por otro el metodo que elimina el registro y si quieres coordinar ambos crea un tercer metodo

    private void Eliminar(int Id_Documento, string Imagen)
    {
    	EliminarTabla(Id_Documento);
    	EliminarArchivo(Imagen);
    }
    
    private void EliminarArchivo(string Imagen)
    {             
    	string path = Path.Combine(Server.MapPath("../Documentos/"),Imagen);
       
    	if (System.IO.File.Exists(path))
    	{
    		try
    		{
    			System.IO.File.Delete(path);
    		  
    		}
    		catch (System.IO.IOException ev)
    		{
    			Console.WriteLine(ev.Message);
    			return;
    		}
    	}
    }
    
    
    private void EliminarTabla(int Id_Documento)
    {
    	ws.BorrarDocumento(Id_Documento);
    }

    saludos


    Leandro Tuttini

    Blog
    MVP Profile
    Buenos Aires
    Argentina

    jueves, 7 de mayo de 2015 0:17

Todas las respuestas

  • Y porque no lo pones justo enseguida?:

     private void Eliminar(string Imagen, int Id_Documento)
            {             
                string path = Path.Combine(Server.MapPath("../Documentos/"),Imagen);
               
                if (System.IO.File.Exists(path))
                {
                    try
                    {
                        System.IO.File.Delete(path);
                        EliminarTabla(Id_Documento);
                    }
                    catch (System.IO.IOException ev)
                    {
                        Console.WriteLine(ev.Message);
                        return;
                    }
                }
            }
    Simplemente sobrecarga el método.
    Saludos.
    miércoles, 6 de mayo de 2015 21:45
  • hola

    pero porque quieres hacer eso ? no es correcto, estas rompiendo con las repsonsabilidades de cada funcionalidad

    por un lado define el metodo que elimina el archivo, por otro el metodo que elimina el registro y si quieres coordinar ambos crea un tercer metodo

    private void Eliminar(int Id_Documento, string Imagen)
    {
    	EliminarTabla(Id_Documento);
    	EliminarArchivo(Imagen);
    }
    
    private void EliminarArchivo(string Imagen)
    {             
    	string path = Path.Combine(Server.MapPath("../Documentos/"),Imagen);
       
    	if (System.IO.File.Exists(path))
    	{
    		try
    		{
    			System.IO.File.Delete(path);
    		  
    		}
    		catch (System.IO.IOException ev)
    		{
    			Console.WriteLine(ev.Message);
    			return;
    		}
    	}
    }
    
    
    private void EliminarTabla(int Id_Documento)
    {
    	ws.BorrarDocumento(Id_Documento);
    }

    saludos


    Leandro Tuttini

    Blog
    MVP Profile
    Buenos Aires
    Argentina

    jueves, 7 de mayo de 2015 0:17