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
      187
    • Comentarios
      97
    • Revisiones
      29

    Más información sobre "Un juego del Rabino en Excel"
    Última descarga
    Por pegones1

    2    1

  • Crear macros Excel

  • Mensajes

    • Estimados buenos días, Quisiera saber si me pueden brindar su soporte con lo siguiente. Tengo lo siguiente una data de FECHAS CON CANTIDADES y quisiera saber si hay alguna formula para poder contabilizar desde la última fecha cuando días son consecutivos, ejemplo si en una fila queda vacío porque no se repite y la fecha matriz es la ultima fecha quiere decir que no se repite y es 0.   DIAS CONSECUTIVOS.xlsx
    • Buenos días con todo, espero se encuentren bien de salud!. Favor quisiera ver si me pueden ayudar con lo siguiente. Tengo una data en excel con los siguiente criterios FECHAS DIFERENTES , CODIGO Y NOMBRE DEL PRODUCTO. Lo que quiero realizar es que si en la fecha 17-02  tienes cantidad x de códigos y si estos no se repite el día siguiente 18-02 que automáticamente se borre, esto con la finalidad de tener un control de a partir del 18 al 19  se repite 1 vez y no me considere 2 desde fecha 17-02  teniendo en cuenta que el producto en el 18-02 no aparece. Lo sombreado son los que se repiten .   TABLA ELIMINAR.xlsx
    • Vale mil gracias, en vdd se agradece todo el apoyo y comentarios
    • Hola nuevamente. Por eso la importancia de lo que mencionas tú como "ruta relativa". Tal cual comentas, y aún sin llegar a algunos detalles importantes para ayudarte, en realidad tal cual te comenté le día miércoles, pues sí podías hacer como comentabas, era cosa de obtener los Id de Windows (como tú los llamas) y la ruta de OneDrive en casa usuario y eso sí se puede obtener con VBA y luego pasarlo a PQ, pero medio que te cerraste en que "PQ no puede trabajar con rutas relativas", cosa cierta pero siempre hay formas. Si SAP puede o no guardar en OneDrive o SharePoint, pues si está mapeado en la PC ¡claro que se puede! Pero bueno, creo que si te es útil tu propia propuesta ¡avanza con eso!
    • Perdona @Abraham Valencia pero he estado liado estos días. En realidad todo el problema se reduce a estos dos problemas: Problema 1: El script que "fabrica" SAP y que luego "pego" en la macro, no es capaz de  guardar archivos en SharePoint. He estado buscando, y en realidad muchas personas tienen ese problema (no poder guardar un Excel en SAP a través de VBA). Eso muy probablemente sean problemas de permisos, que no puedo cambiar (no soy administrador de nada). Como no puedo solucionarlo así, paso al plan B, que es guardar en Excel que me genera SAP en el ordenador de cada usuario que ejecute la plantilla (y que sí está guardada en SharePoint), para después con PowerQuery llamar a ese Excel (el export). Para ello, pretendo guardar el export, en la ruta relativa "C:\..\..\..\OneDrive - NombreEmpresa\Documentos\SAP\SAP GUI" donde los \..\..\..\ saltan las rutas personales de cada usuario (tipo C:\users\IDusuario\). Eso lo hace bien, y el archivo se guarda en la ruta de cada usuario que lo usa, pero surge el problema 2 Problema 2: PowerQuery no trabaja con rutas relativas del tipo  "C:\..\..\..\OneDrive - NombreEmpresa\Documentos\SAP\SAP GUI" necesita que sea del tipo fija "C:\users\IDusuario\OneDrive - NombreEmpresa\Documentos\SAP\SAP GUI" pero claro, IDusuario es diferente para cada usuario.   Pero escribiendo todo esto, creo que he dado con una posible solución, no grabar el export en una ruta de usuario, sino en una en la raiz de C:, que siempre será igual para todos los usuarios, del tipo C:\Sap\export.xlsx que seria igual en todos los ordenadores. Asi sí podría decirle a PowerQuery que vaya siempre a la ruta C:\Sap\ que existirá en todos los ordenadores. Mañana intentaré hacer pruebas, aunque mi solución ideal seria que se pudiera guardar en el SharePoint. Saludos.
  • 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.