Vba probleema

Kääpiöäly

Vien excelin kautta tietoja toiminnanohj.järjestelmäämme.

Nyt mulla on seuraava koodi:

Set xclapp = GetObject("Excel.Application")
Set xclwbk = xclapp.Workbooks.Open("c:\documents and settings\kukkuu.xls")
Set xclsht = xclwbk.Sheets("Sheet1")

For i = 2 To xclapp.ActiveCell.SpecialCells(11).Row
For j = 1 To xclapp.ActiveCell.SpecialCells(11).Column
If j = 1 Then infot = xclsht.Cells(i, j).Value
Next

Kun aloitan tämän makron, avaa excel tämän saman tiedoston kahteen kertaan? Miten tuota exceliä pitäisi kutsua, että se tajuisi ottaa tiedot kyseisestä työkirjasta? Miten siis määritetään avoinna oleva työkirja objectiksi?

10

520

    Vastaukset

    Anonyymi (Kirjaudu / Rekisteröidy)
    5000
    • untokuurna

      Ja missäköhän tämä koodi on?

      Jos koodi on jo samassa työkirjassa niin silloin muutetaan viittaus (Set xclsht = xclwbk.Sheets("Sheet1")) avoinna olevaan työkirjaan. Tupla-avautumista ei pitäisi tapahtua jos koodi on jossain toisessa työkirjassa.

      Kunde osannee auttaa tuossa ensimmäisessä kohdassa.

      • viittaus referensseissä Excel kirjastoon!

        Sub SiirräExceliin()
        Dim Excel As Object
        Dim ExcelSheet As Object

        On Error Resume Next
        Set Excel = GetObject(, "Excel.Application")
        If Err 0 Then
        Err.Clear
        Set Excel = CreateObject("Excel.Application")
        If Err 0 Then
        MsgBox "Exceliä ei voitu aukaista ", vbExclamation
        End
        End If
        End If
        On Error GoTo 0
        Excel.Visible = True
        Excel.Workbooks.Open ("c:\documents and settings\kukkuu.xls")
        ' oma koodi alkaa...
        Sheets(1).Activate
        infot = Range("J9")
        MsgBox infot
        ' oma koodi loppuu...
        Excel.Quit
        Set Excel = Nothing
        End Sub


      • kääpiöäly
        kunde kirjoitti:

        viittaus referensseissä Excel kirjastoon!

        Sub SiirräExceliin()
        Dim Excel As Object
        Dim ExcelSheet As Object

        On Error Resume Next
        Set Excel = GetObject(, "Excel.Application")
        If Err 0 Then
        Err.Clear
        Set Excel = CreateObject("Excel.Application")
        If Err 0 Then
        MsgBox "Exceliä ei voitu aukaista ", vbExclamation
        End
        End If
        End If
        On Error GoTo 0
        Excel.Visible = True
        Excel.Workbooks.Open ("c:\documents and settings\kukkuu.xls")
        ' oma koodi alkaa...
        Sheets(1).Activate
        infot = Range("J9")
        MsgBox infot
        ' oma koodi loppuu...
        Excel.Quit
        Set Excel = Nothing
        End Sub

        Sorry, oon selittänyt taas huonosti. Tuo koodi on siis samaisessa excelissä moduuli1:ssä.

        Eli olen tehnyt makron kyseiseen exceliin, joka poimii tiedot sheetiltä1 ja vie ne järjestelmään. koodi toimii muuten, mutta tällä koodilla tiedosto "kukkuu" aukeaa uudestaan, vaikka se on jo auki.
        Jos vaihdan tuon CreateObjectin GetObjectiksi, saan virheilmoituksen automation error, invalid syntax.
        Tässä koko koodi, lukuunottamatta kirjautumista järjestelmään. Miten tuohon alkuun määritellään, että kyse on juurikin tästä työkirjasta?:

        Set xclapp = CreateObject("Excel.Application")
        Set xclwbk = xclapp.Workbooks.Open("c:\documents and settings\kukkuu.xls")
        Set xclsht = xclwbk.Sheets("Sheet1")
        For i = 2 To xclapp.ActiveCell.SpecialCells(11).Row
        For j = 1 To xclapp.ActiveCell.SpecialCells(11).Column
        If j = 1 Then infot = xclsht.Cells(i, j).Value
        Next
        session.findById("wnd[0]").maximize
        session.findById("wnd[0]/mbar/menu[1]/menu[3]").Select
        session.findById("wnd[0]/usr/ctxt[0]").Text = excel.Cells(1, 10).Value
        session.findById("wnd[0]/usr/ctxt[4]").Text = infot
        session.findById("wnd[0]/usr/ctxt[4]").SetFocus
        session.findById("wnd[0]/usr/ctxt[4]").caretPosition = 7
        session.findById("wnd[0]").sendVKey 0
        Next
        MsgBox "ALL" & CStr(xclapp.ActiveCell.SpecialCells(11).Row - 1) & "Excel rows processed"

        End Sub


      • kääpiöäly kirjoitti:

        Sorry, oon selittänyt taas huonosti. Tuo koodi on siis samaisessa excelissä moduuli1:ssä.

        Eli olen tehnyt makron kyseiseen exceliin, joka poimii tiedot sheetiltä1 ja vie ne järjestelmään. koodi toimii muuten, mutta tällä koodilla tiedosto "kukkuu" aukeaa uudestaan, vaikka se on jo auki.
        Jos vaihdan tuon CreateObjectin GetObjectiksi, saan virheilmoituksen automation error, invalid syntax.
        Tässä koko koodi, lukuunottamatta kirjautumista järjestelmään. Miten tuohon alkuun määritellään, että kyse on juurikin tästä työkirjasta?:

        Set xclapp = CreateObject("Excel.Application")
        Set xclwbk = xclapp.Workbooks.Open("c:\documents and settings\kukkuu.xls")
        Set xclsht = xclwbk.Sheets("Sheet1")
        For i = 2 To xclapp.ActiveCell.SpecialCells(11).Row
        For j = 1 To xclapp.ActiveCell.SpecialCells(11).Column
        If j = 1 Then infot = xclsht.Cells(i, j).Value
        Next
        session.findById("wnd[0]").maximize
        session.findById("wnd[0]/mbar/menu[1]/menu[3]").Select
        session.findById("wnd[0]/usr/ctxt[0]").Text = excel.Cells(1, 10).Value
        session.findById("wnd[0]/usr/ctxt[4]").Text = infot
        session.findById("wnd[0]/usr/ctxt[4]").SetFocus
        session.findById("wnd[0]/usr/ctxt[4]").caretPosition = 7
        session.findById("wnd[0]").sendVKey 0
        Next
        MsgBox "ALL" & CStr(xclapp.ActiveCell.SpecialCells(11).Row - 1) & "Excel rows processed"

        End Sub

        Siis kukkuu on avoinna ja haluat siitä siirtää Sheet1 A sarakkeen tiedot alueelta A2:Axx vai?
        Jos näin on niin excelin objekti on turha

        parempi toteuttaa näin

        Sub Siirrä()
        Dim vika As Long
        vika = Range("A65536").End(xlUp).Row
        Sheets("Sheet1").Activate
        For Each solu In Range("A2:A" & vika)
        session.findById("wnd[0]").maximize
        session.findById("wnd[0]/mbar/menu[1]/menu[3]").Select
        session.findById("wnd[0]/usr/ctxt[0]").Text = Cells(1, 10).Value
        session.findById("wnd[0]/usr/ctxt[4]").Text = infot
        session.findById("wnd[0]/usr/ctxt[4]").SetFocus
        session.findById("wnd[0]/usr/ctxt[4]").caretPosition = 7
        session.findById("wnd[0]").sendVKey 0
        Next
        End Sub


      • Kääpiöäly
        kunde kirjoitti:

        Siis kukkuu on avoinna ja haluat siitä siirtää Sheet1 A sarakkeen tiedot alueelta A2:Axx vai?
        Jos näin on niin excelin objekti on turha

        parempi toteuttaa näin

        Sub Siirrä()
        Dim vika As Long
        vika = Range("A65536").End(xlUp).Row
        Sheets("Sheet1").Activate
        For Each solu In Range("A2:A" & vika)
        session.findById("wnd[0]").maximize
        session.findById("wnd[0]/mbar/menu[1]/menu[3]").Select
        session.findById("wnd[0]/usr/ctxt[0]").Text = Cells(1, 10).Value
        session.findById("wnd[0]/usr/ctxt[4]").Text = infot
        session.findById("wnd[0]/usr/ctxt[4]").SetFocus
        session.findById("wnd[0]/usr/ctxt[4]").caretPosition = 7
        session.findById("wnd[0]").sendVKey 0
        Next
        End Sub

        Joo siis tämän tiedoston nimi on kukkuu, eli yhdestä tiedostosta kyse. Tuo on muuten hyvä, mutta nyt siirtyy kaikki sarakkeen A rivit yhtenä könttänä, kun mulla tuossa koodissa oli tarkoitus siirtää rivi kerrallaan.

        Eli mulla on kaksi paikkaa toim.ohj.järjestelmässä, johon dataa siirrän. Toinen on tuo session.findById("wnd[0]/usr/ctxt[0]").Text = Cells(1, 10).Value, jossa tuo Cells(1, 10) on aina vakio.
        ja toinen:
        session.findById("wnd[0]/usr/ctxt[4]").Text = infot, johon pitäisi siis tulla aina arvo A-sarakkeesta. Ensin A1, sitten A2 jne.,
        Jokaisen A-sarakkeesta syötetyn solun jälkeen järjestelmä painaa automaattisesti tallennusta: session.findById("wnd[0]").sendVKey 0 Ja valitsee seuraavan solun A-sarakkeesta, sekä ilmoittaa, kun rivit on käsitelty.

        Tuo mun aiemmin esittämä koodi siis toimii kun se laukaistaan jostain toisesta paikkaa, nyt kuitenkin haluasin, että kaikki olisi tuossa yhdessää kukkuu tiedostossa. Eli Sheetillä 1 arvot ja makro tietojen vientiin

        Helkkari näitä on vaikea selittää, kun olen vielä aika keltanokka varsinkin excelin kanssa.


      • Kääpiöäly kirjoitti:

        Joo siis tämän tiedoston nimi on kukkuu, eli yhdestä tiedostosta kyse. Tuo on muuten hyvä, mutta nyt siirtyy kaikki sarakkeen A rivit yhtenä könttänä, kun mulla tuossa koodissa oli tarkoitus siirtää rivi kerrallaan.

        Eli mulla on kaksi paikkaa toim.ohj.järjestelmässä, johon dataa siirrän. Toinen on tuo session.findById("wnd[0]/usr/ctxt[0]").Text = Cells(1, 10).Value, jossa tuo Cells(1, 10) on aina vakio.
        ja toinen:
        session.findById("wnd[0]/usr/ctxt[4]").Text = infot, johon pitäisi siis tulla aina arvo A-sarakkeesta. Ensin A1, sitten A2 jne.,
        Jokaisen A-sarakkeesta syötetyn solun jälkeen järjestelmä painaa automaattisesti tallennusta: session.findById("wnd[0]").sendVKey 0 Ja valitsee seuraavan solun A-sarakkeesta, sekä ilmoittaa, kun rivit on käsitelty.

        Tuo mun aiemmin esittämä koodi siis toimii kun se laukaistaan jostain toisesta paikkaa, nyt kuitenkin haluasin, että kaikki olisi tuossa yhdessää kukkuu tiedostossa. Eli Sheetillä 1 arvot ja makro tietojen vientiin

        Helkkari näitä on vaikea selittää, kun olen vielä aika keltanokka varsinkin excelin kanssa.

        Sub Siirrä()
        Dim vika As Long
        vika = Range("A65536").End(xlUp).Row
        Sheets("Sheet1").Activate
        For Each solu In Range("A2:A" & vika)
        session.findById("wnd[0]").maximize
        session.findById("wnd[0]/mbar/menu[1]/menu[3]").Select
        session.findById("wnd[0]/usr/ctxt[0]").Text = Cells(1, 10).Value
        session.findById("wnd[0]/usr/ctxt[4]").Text = solu
        session.findById("wnd[0]/usr/ctxt[4]").SetFocus
        session.findById("wnd[0]/usr/ctxt[4]").caretPosition = 7
        session.findById("wnd[0]").sendVKey 0
        Next
        MsgBox "ALL " & vika - 1 & " Excel rows processed"
        End Sub


      • Kääpiöäly
        kunde kirjoitti:

        Sub Siirrä()
        Dim vika As Long
        vika = Range("A65536").End(xlUp).Row
        Sheets("Sheet1").Activate
        For Each solu In Range("A2:A" & vika)
        session.findById("wnd[0]").maximize
        session.findById("wnd[0]/mbar/menu[1]/menu[3]").Select
        session.findById("wnd[0]/usr/ctxt[0]").Text = Cells(1, 10).Value
        session.findById("wnd[0]/usr/ctxt[4]").Text = solu
        session.findById("wnd[0]/usr/ctxt[4]").SetFocus
        session.findById("wnd[0]/usr/ctxt[4]").caretPosition = 7
        session.findById("wnd[0]").sendVKey 0
        Next
        MsgBox "ALL " & vika - 1 & " Excel rows processed"
        End Sub

        Nyt toimii hyvin! Ja erittäin suuri kiitos jälleen!


      • kääpiöäly
        kunde kirjoitti:

        Sub Siirrä()
        Dim vika As Long
        vika = Range("A65536").End(xlUp).Row
        Sheets("Sheet1").Activate
        For Each solu In Range("A2:A" & vika)
        session.findById("wnd[0]").maximize
        session.findById("wnd[0]/mbar/menu[1]/menu[3]").Select
        session.findById("wnd[0]/usr/ctxt[0]").Text = Cells(1, 10).Value
        session.findById("wnd[0]/usr/ctxt[4]").Text = solu
        session.findById("wnd[0]/usr/ctxt[4]").SetFocus
        session.findById("wnd[0]/usr/ctxt[4]").caretPosition = 7
        session.findById("wnd[0]").sendVKey 0
        Next
        MsgBox "ALL " & vika - 1 & " Excel rows processed"
        End Sub

        Miten koodia on muutettava, jos halutaan sarakkeen B arvot kohtaan: session.findById("wnd[0]/usr/ctxt[0]").Text = Cells(1, 10).Value, per rivi?

        Yritin näin mutta ei toimi:

        Dim vika As Long
        Dim vika2 As Long
        vika = Range("A65536").End(xlUp).Row
        vika2 = Range("B65536").End(xlUp).Row
        Sheets("Sheet1").Activate
        For Each solu In Range("A2:A" & vika)
        For Each solu2 In Range("B2:B" & vika2)
        session.findById("wnd[0]/usr/ctxt[0]").Text = solu2
        session.findById("wnd[0]/usr/ctxt[4]").Text = solu
        session.findById("wnd[0]/usr/ctxt[4]").SetFocus
        session.findById("wnd[0]/usr/ctxt[4]").caretPosition = 7
        session.findById("wnd[0]").sendVKey 0
        Next
        Next
        MsgBox "ALL " & vika - 1 & " Excel rows processed"
        End Sub


      • kääpiöäly kirjoitti:

        Miten koodia on muutettava, jos halutaan sarakkeen B arvot kohtaan: session.findById("wnd[0]/usr/ctxt[0]").Text = Cells(1, 10).Value, per rivi?

        Yritin näin mutta ei toimi:

        Dim vika As Long
        Dim vika2 As Long
        vika = Range("A65536").End(xlUp).Row
        vika2 = Range("B65536").End(xlUp).Row
        Sheets("Sheet1").Activate
        For Each solu In Range("A2:A" & vika)
        For Each solu2 In Range("B2:B" & vika2)
        session.findById("wnd[0]/usr/ctxt[0]").Text = solu2
        session.findById("wnd[0]/usr/ctxt[4]").Text = solu
        session.findById("wnd[0]/usr/ctxt[4]").SetFocus
        session.findById("wnd[0]/usr/ctxt[4]").caretPosition = 7
        session.findById("wnd[0]").sendVKey 0
        Next
        Next
        MsgBox "ALL " & vika - 1 & " Excel rows processed"
        End Sub

        Sub Siirrä()
        Dim vika As Long
        vika = Range("A65536").End(xlUp).Row
        Sheets("Sheet1").Activate
        For Each solu In Range("A2:A" & vika)
        session.findById("wnd[0]").maximize
        session.findById("wnd[0]/mbar/menu[1]/menu[3]").Select
        session.findById("wnd[0]/usr/ctxt[0]").Text = solu.Offset(0, 1)
        session.findById("wnd[0]/usr/ctxt[4]").Text = solu
        session.findById("wnd[0]/usr/ctxt[4]").SetFocus
        session.findById("wnd[0]/usr/ctxt[4]").caretPosition = 7
        session.findById("wnd[0]").sendVKey 0
        Next
        MsgBox "ALL " & vika - 1 & " Excel rows processed"
        End Sub

        Keep EXCELing
        @Kunde


      • kääpiöäly
        kunde kirjoitti:

        Sub Siirrä()
        Dim vika As Long
        vika = Range("A65536").End(xlUp).Row
        Sheets("Sheet1").Activate
        For Each solu In Range("A2:A" & vika)
        session.findById("wnd[0]").maximize
        session.findById("wnd[0]/mbar/menu[1]/menu[3]").Select
        session.findById("wnd[0]/usr/ctxt[0]").Text = solu.Offset(0, 1)
        session.findById("wnd[0]/usr/ctxt[4]").Text = solu
        session.findById("wnd[0]/usr/ctxt[4]").SetFocus
        session.findById("wnd[0]/usr/ctxt[4]").caretPosition = 7
        session.findById("wnd[0]").sendVKey 0
        Next
        MsgBox "ALL " & vika - 1 & " Excel rows processed"
        End Sub

        Keep EXCELing
        @Kunde

        Ohhoijjaa, kun tunnen taas itseni tyhmäksi.... Kiitos!


    Ketjusta on poistettu 1 sääntöjenvastaista viestiä.

    Luetuimmat keskustelut

    1. Useita puukotettu Tampereella

      Mikäs homma tämä nyt taas on? "Useaa henkilöä on puukotettu Tampereen keskustassa kauppakeskus Ratinan lähistöllä." ht
      Tampere
      232
      4421
    2. Kuka rääkkää eläimiä Puolangalla?

      Poliisi ampui toistakymmentä nälkiintynyttä eläintä Puolangalla Tilalta oli ollut karkuteillä lähes viisikymmentä nälkii
      Puolanka
      74
      2862
    3. Leipivaaran päällä on kuoleman hiljaista.

      Suru vai suuri helpotus...
      Puolanka
      47
      2403
    4. Meneeköhän sulla

      oikeasti pinnan alla yhtä huonosti kuin mulla? Tai yhtä huonosti mutta jollain eri tyylillä? Ei olisi pitänyt jättää sua
      Ikävä
      43
      1687
    5. Koska näit kaivattusi viimeksi

      Milloin tapasit rakkaasi? Ja etenikö suhde yhtään?
      Ikävä
      78
      1379
    6. Lähetä terveisesi kaipaamallesi henkilölle

      Vauva-palstalta tuttua kaipaamista uudessa ympäristössä. Kaipuu jatkukoon 💘
      Ikävä
      69
      1080
    7. PS uusimman gallupin rakettimainen nousija

      https://yle.fi/a/74-20170641 Aivan ylivoimaisesti suurin kannatuksen nousu PS:lle. Nousu on alkanut ja jatkuu 2 vuoden
      Maailman menoa
      137
      919
    8. Tekiskö nainen mieli tavata...

      Viikonloppuna ja...?
      Ikävä
      64
      860
    9. Sellainen tunne sydämessä

      Että nainen olet kaivannut minua. Tai sanonko että oikeastaan koet sitä samaa nostalgiaa, kaipuuta ja mukavia muistoja,
      Ikävä
      86
      854
    10. Annan meille mahdollisuuden

      Olen avoimin mielin ja katson miten asiat etenevät. Mutta tällä kertaa sun on tehtävä eka siirto.Sen jälkeen olen täysil
      Ikävä
      53
      832
    Aihe