Saltar al contenido

ayuda para depurar código


Recommended Posts

publicado

Pues resulta que hace tiempo hice un pequeño programa para hacer facturas con excel y vba, pero he descubierto que cuando escribo en la factura unas cantidades con decimales pues al guardarlas en el registro de vba no hace caso a dichos decimales, considerando que no son tales y me agrega dos decimales.

Me explico , si yo pongo en cantidad 2, precio unitario 15,15, me da un total de 30,30, al que luego se le calcula su iva y el total, pues al grabar estos datos en el registro me guarda en Cantidad 2, en precio unitario 1.515,00 y en subtotal 3.030,00.

 

El codigo con el que recogo estos datos es el siguiente:

Call Abrir_Tb_Detalle_Facturas

ActiveSheet.Range("A24").Activate
With RsDetalleFacturas

    Do While ActiveCell.Value <> ""
    
        If .EOF Or .BOF Then
        'si la base de datos de detalles esta vacia
                'agregamos
                
                .AddNew
                .Fields(1) = pesado 'UCase(ActiveCell.Offset(-15, 2).Value) 'no consigo grabar este dato con celda
                .Fields(2) = UCase(ActiveCell.Value) 'descripcion
                .Fields(3) = UCase(ActiveCell.Offset(0, 1).Value) 'cantidad
                .Fields(4) = UCase(ActiveCell.Offset(0, 2).Value) 'precio unitario
                .Fields(5) = UCase(ActiveCell.Offset(0, 3).Value) 'importe bruto

                .Fields(6) = numfac 'UCase(ActiveCell.Offset(-15, 3).Value) 'numero de la factura escrita
                .Fields(7) = nif 'nif del cliente
                .Update
        Else 'y si no esta vacia la base de datos de detalles, pues buscamos ese detalle
        
            .Requery
                'agregamos
                .AddNew
                .Fields(1) = pesado 'UCase(ActiveCell.Offset(-15, 2).Value) 'no consigo grabar este dato con celda
                .Fields(2) = UCase(ActiveCell.Value) 'descripcion
                .Fields(3) = UCase(ActiveCell.Offset(0, 1).Value) 'cantidad
                .Fields(4) = UCase(ActiveCell.Offset(0, 2).Value) 'precio unitario
                .Fields(5) = UCase(ActiveCell.Offset(0, 3).Value) 'importe bruto

                .Fields(6) = numfac 'UCase(ActiveCell.Offset(-15, 3).Value) 'pesado 'numero de la factura escrita
                .Fields(7) = nif 'nif del cliente
                .Update
        End If
    ActiveCell.Offset(1, 0).Activate
    Loop
    Call Cerrar_Tb_Detalle_Facturas
End With
    Call Desconectar_Base
    
    ActiveSheet.Range("C17").Activate
    ActiveWorkbook.Save
    MsgBox "la factura ha sido agregada correctamente a la base de datos", vbInformation
Call Finalizar_Macro
End If
End With

Call Crear_Pdf
MsgBox "El fichero pdf se ha grabado correctamente", vbInformation

End Sub

 

y en el registro lo tengo configurado de la siguiente manera

Cantidad = Numero

Precio_Unitario = Moneda, con las siguientes especificaciones:

formato euro

2 decimales

requerido si

indexado no

 

Precio_Bruto = Moneda, con las mismas especificaciones que Precio_Unitario

 

No se si lo que tengo es que cambiar la formula con la que cogo el numero de la celda o si tengo que cambiar la configuración del registro.

 

Siento haberme extendido tanto, pero el programa subirlo es muy grande

publicado

Sin la base de datos y el archivo es difícil ayudarte, en cualquier caso, no se porqué utilizas la función Ucase() en un dato numérico, sería mas lógico utilizar una función de conversión de tipos, CDbl() por ejemplo.

Supongo que debes tener la coma como separador decimal en Excel, si es así, hay que sustituir la coma por el punto en el dato antes de pasar a la BD.

Prueba así, a ver si hay suerte:

.Fields(4) = Replace(CDbl(ActiveCell.Offset(0, 2).Value), ",", ".")  'precio unitario

Revisa el tipo de dato en el diseño de la tabla y en la hoja.

publicado

Voy a probarlo enseguida. El tema es que yo no tengo ni idea de programar, solo voy sacando comandos y acoplando lo que leo / veo en un curso que hay en youtube, pero este escollo no soy capaz de resolver.

Lo dicho, voy ha probarlo y os comento el resultado.

 

Muchas gracias

publicado

funciona perfecto, genial, investigare el coimando que me has facilitado para poder comprenderlo mejor

 

Muchas gracias

 

por mi parte podeis darlo por concluido y resuelto

publicado

Lo mas probable es que así también funcione:

 

.Fields(4) = Replace(ActiveCell.Offset(0, 2).Value, ",", ".")  'precio unitario

 

.

  • Silvia bloqueó este tema

Archivado

Este tema está ahora archivado y está cerrado a más respuestas.

  • 109 ¿Te parecen útiles los tips de las funciones? (ver tema completo)

    1. 1. ¿Te parecen útiles los tips de las funciones?


      • No
      • Ni me he fijado en ellos

  • Ayúdanos a mejorar la comunidad

    • Donaciones recibidas este mes: 0.00 EUR
      Objetivo: 130.00 EUR
  • Archivos

  • Estadísticas de descargas

    • Archivos
      188
    • Comentarios
      98
    • Revisiones
      29

    Más información sobre "Cambios en el Control Horario"
    Última descarga
    Por pegones1

    3    1

  • Crear macros Excel

  • Mensajes

    • Saludos amigos espero estén bien  En la actualidad utilizo la siguiente macro para generar un código alfanumérico continuo que al guardar los datos se copia en la Celdas “B5” de las Hojas 58 (MATRIZ1) y Hojas59 (MATRIZ2) Private Sub CommandButton3_Click()     If Hoja58.Range("B5").Value < 9999 Then Hoja58.Range("B5").Value Hoja58.Range("B6").Value + 1     Else        Hoja58.Range("B5").Value = 1     End If If Hoja59.Range("B5").Value < 9999 Then Hoja59.Range("B5").Value = Hoja59.Range("B6").Value + 1 Else        Hoja59.Range("B5").Value = 1     End If End Sub El código se genera normalmente sin embargo no se copia inmediatamente en las Celdas “B5” sino que aparece en las Celdas “B6” mientras que las Celdas “B5” siempre permanecen en blanco, actualmente el código consta de tres letras y cuatro dígitos (RCI-0000) pero me interesa poderle concatenar el mes y año en formato mm-yy para que el código finalmente quede expresado de la siguiente forma RCI-0000-00-00 Mucho les sabre agradecer la ayuda que me puedan dar RCI PRUEBA..xlsm
    • Saludos amigos espero estén bien  En la actualidad  los datos provenientes de los TextBox 5, 6 y 7 se guardan en las Columnas H, I y O de las Hojas 58 (MATRIZ1) y Hojas59 (MATRIZ2) sin embargo cuando el texto es abundante no se visualiza completamente en las celdas correspondientes de allí que necesito que cada fila se ajuste al tamaño de la celda que contenga mayor cantidad de texto para lo cual he probado con la siguiente macro: Private Sub Worksheet_SelectionChange(ByVal Target As Range) Cells.EntireRow.AutoFit End Sub Sin embargo, dicha macro coloca todas las filas del tamaño del texto que contengan y lo que en realidad necesito es que dicha macro solo se aplique desde la Fila 5 en adelante. Mucho les sabre agradecer la ayuda que me puedan dar RCI PRUEBA..xlsm
    • Saludos amigos espero estén bien  En la actualidad utilizo la siguiente macro para que la fecha aparezca en el Textbox2: Private Sub UserForm_activate() Me.TextBox2.Value = VBA.Date End Sub Sin embargo, una vez guardo los datos la fecha cambia de formato y se guarda en formato mm/dd/yyyy en vez de dd/mm/yyyy y este cambio de fecha me distorsiona los resultados de las fórmulas en las Columnas J en las Hojas 58 (MATRIZ1) y Hojas59 (MATRIZ2) Mucho les sabre agradecer la ayuda que me puedan dar RCI PRUEBA..xlsm
    • ¡Hola! Después de un buen tiempo ausente...  @Gabriela hg ya que veo que tienes Excel 365, te dejo mi solución para piezas y gramos.   Extraer piezas - gramos 365_GP.xlsx
  • Visualizado recientemente

    • No hay usuarios registrado para ver esta página.
×
×
  • Crear nuevo...

Información importante

Echa un vistazo a nuestra política de cookies para ayudarte a tener una mejor experiencia de navegación. Puedes ajustar aquí la configuración. Pulsa el botón Aceptar, si estás de acuerdo.