Hola a todos;

-Adjunto macro contiene un "frm" con un "chek" y un botón.

Lo que hace la macro es crear una copia de seguridad. En función de lo que tenga activado en el check la hace de una manera u otra.

Activo: Guarda la copia de seguridad en un ruta predeterminada.

No Activo: Nos pide la ruta para guardar.

Pregunta: ¿Se puede depurar mejor el código o está bien así?

Saludos y buen día a todos

 

C.Seguridad.xlsm

Hola

Para evitar confusiones al usuario, es mejor que incluyas ambas opciones visualmente, de esta forma, se escoge una, y desde VBA lees la casilla que fue activada y corres el procedimiento correspondiente

Además intenta que desde un mismo procedimiento se pueda correr ambas copias, con un simple condicional If

 

Saludos 

Opino lo mismo que Gerson, sustituye el checkbox por 2 optionbutton.

Por ejemplo.

C.Seguridad.xlsm

Hace 15 horas, Gerson Pineda dijo:

Hola

Para evitar confusiones al usuario, es mejor que incluyas ambas opciones visualmente, de esta forma, se escoge una, y desde VBA lees la casilla que fue activada y corres el procedimiento correspondiente

Además intenta que desde un mismo procedimiento se pueda correr ambas copias, con un simple condicional If

 

Saludos 

Buenos días Gerson, gracias por el consejo. Tomo nota.

-En cuanto al código ¿está bien así?

Un saludo 

Hace 13 horas, Antoni dijo:

Opino lo mismo que Gerson, sustituye el checkbox por 2 optionbutton.

Por ejemplo.

C.Seguridad.xlsm 22.62 kB · 0 descargas

Bo día Mestre. 

Como apuntaba Gerson la idea de cambiar el "check" por botones me convence. Queda mejor visualmente.

Por cierto como le he comentado a Gerson el código esta bien así(?).

Moitas como sempre.

 

En 11/2/2021 at 1:00 , Benito Bartolomé dijo:

Buenos días Gerson, gracias por el consejo. Tomo nota.

-En cuanto al código ¿está bien así?

Pude detectar un fallo, que al elegir una carpeta y luego el usuario se arrepiente y cancela, siempre crea la copia, revisa e intenta resolver, aunque seguro con el aporte de Antoni, queda cubierto

 

Saludos

Muchas gracias por responder  Gerson, si me he dado cuenta de ese detalle y en principio lo he resuelto. 

Ahora no tengo conmigo la macro y no te lo puedo mostrar. 

Mañana si acaso te lo paso.

Un abrazo. ? 

 

Hace 14 horas, Gerson Pineda dijo:

Pude detectar un fallo, que al elegir una carpeta y luego el usuario se arrepiente y cancela, siempre crea la copia, revisa e intenta resolver, aunque seguro con el aporte de Antoni, queda cubierto

 

Saludos

Ahora mismo lo tengo así, a ver que te parece. Un saludo.

Sub CSGuardarComo()
Application.ScreenUpdating = False
Application.DisplayAlerts = False
If MsgBox("Seguro que quiere crear una copia de seguridad?", vbYesNo + vbQuestion, "Copia de Seguridad") = vbYes Then
    
    ruta = ThisWorkbook.path & "\"
    With Application.FileDialog(msoFileDialogSaveAs)
        .Title = "Guardar archivo como"
        .AllowMultiSelect = False
        .InitialFileName = ruta
        .FilterIndex = 2
        If .Show Then
            march = .SelectedItems(1)
            march = Replace(march, ".xlsm", "")
            ActiveWorkbook.SaveAs Filename:=march & ".xlsm", _
            FileFormat:=xlOpenXMLWorkbookMacroEnabled, CreateBackup:=False
            MsgBox "La copia se guardó en la siguiente ruta: " & ruta, vbInformation, "Aviso"
        End If
    End With
        Else
        MsgBox "Se ha cancelado", vbInformation, "Aviso"
        Exit Sub
End If
Application.ScreenUpdating = True
Application.DisplayAlerts = True
End Sub

 

Hace 7 horas, Benito Bartolomé dijo:

Ahora mismo lo tengo así, a ver que te parece. Un saludo.

Pero eso es para interactuar con el usuario, si desea o no, me refiero a que una vez, responde que si y luego piensa que mejor no, le daría cancelar, al cuadro de dialogo "guardar como" y en tu caso, sigue y guarda siempre

Para que eso no suceda debes hacer lo siguiente, observa la modificación

If .Show = -1 Then
    .Title = "Guardar archivo como"
    .AllowMultiSelect = False
    .InitialFileName = ruta
    .FilterIndex = 2
    If .Show Then
        march = .SelectedItems(1)
        march = Replace(march, ".xlsm", "")
        ActiveWorkbook.SaveAs Filename:=march & ".xlsm", _
        FileFormat:=xlOpenXMLWorkbookMacroEnabled, CreateBackup:=False
        MsgBox "La copia se guardó en la siguiente ruta: " & ruta, vbInformation, "Aviso"
    End If
Else
    VBA.MsgBox "No se guardo la copia", vbInformation, "Guardar"
End If

 

Saludos 

Hola Gerson de nuevo.  

Solo he notado dos cosas:

La primera es que la macro ha mejorado mucho.

Y la segunda que has tirado de galones?

Ignonte de mi. 

Un fuerte abrazo.  

Hace 9 horas, Benito Bartolomé dijo:

Y la segunda que has tirado de galones?

Pues aquí me pierdo, espero no sean galones de combustible jaja, con lo caro que esta! 

 

Saludos 

A guardar Gasolina, @Gerson Pineda!!!

Saludos

Hace 7 horas, Gerson Pineda dijo:

Pues aquí me pierdo, espero no sean galones de combustible jaja, con lo caro que esta! 

 

Saludos 

En España, Galones: Distintivos que indican la jerarquía militar, coloquialmente indica el nivel de conocimiento de un tema/situación.

? Sabía del término galones o tanquetas en Sudamérica, ahora no sé si desconocías el término "Galones" en España que seguramente sí.

  No me puedo imaginar un Coronel con seis galones en los hombros ?? porque si fuese así no pasaban de Sargento. 

Viva la diversidad cultural. 

Un saludo 

Como se nota que no has hecho la mili  ?

Hace 13 horas, Antoni dijo:

Como se nota que no has hecho la mili  ?

Como lo sabes. Desde entonces soy un hombre hecho y derecho??.

Por cierto los empleos como le gustaban a ellos llamar aún los recuerdo, infantería claro.

Si me dan al contador hasta los 18 años la vuelvo hacer.??

Hace 14 minutos , Benito Bartolomé dijo:

Como lo sabes. Desde entonces soy un hombre hecho y derecho??.

Por cierto los empleos como le gustaban a ellos llamar aún los recuerdo, infantería claro.

Si me dan al contador hasta los 18 años la vuelvo hacer.??

Incluso si el contador me lo ponen a 40 ????

Con toda es explicación clara de @Antoni entiendo mejor el termino de "galones", aunque actualmente esas chapas, por estos lares no tiene casi ningún valor

 

Saludos a todos!