If-lause

Terve,

Mikähän tässä on vikana? Hinnastosta pitäisi kopioida hinnat tuotetaulukkoon. Nyt makro sujahtaa läpi, mutta ei tee mitään. Ilman tuota If-lausetta toimii, eli iffittelyssä jokin vika?


Sub hinnoita()

Dim laskuri As Long
Dim apuri As Long
laskuri = 1
apuri = 2
Do While laskuri < 42520
laskuri = laskuri 1
If Worksheets("Hinnat").Cells(laskuri, 1).Value = Worksheets("Tuotteet").Cells(apuri, 2).Value Then
Worksheets("Tuotteet").Cells(apuri, 10).Value = Worksheets("Hinnat").Cells(laskuri, 8).Value
Worksheets("Tuotteet").Cells(apuri, 11).Value = Worksheets("Hinnat").Cells(laskuri, 9).Value
apuri = apuri 1
End If

Loop

End Sub
1469
Äänestä

Muut lukivat myös

Vastaukset 1

Vanhimmat

Sun tapauksessasi kannattaisi varmaan käyttää Tuotteet-taulukossa PHAKU-funktiota. Sen kun tekisi juuri tuon vaaditun homman.

Mutta jos haluat tuolla tapaa makrolla tehdä, niin sitten tarvitset _periaatteessa_ kaksi sisäkkäistä silmukkaa, joista ulompi kasvattaa laskuri-muuttujaa ja sisempi apuri-muuttujaa. Sitten kun vastaavuus löytyy niin tiedot kopioidaan, apuri-muuttuja nollataan ja sisemmästä silmukasta poistutaan.

Jos vastaavuutta ei löydy, niin sisemmän silmukan muuttuja nollataan myös. Sun koodissasi voi käydä esim niin, että jos Tuotteet-taulukon toinen rivi ei täsmää minkään Hinnat-taulukon rivin kanssa, niin makro sujahtaa läpi yrittämättä edes etsiä vastaavuutta Tuotteet-taulukon kolmannelle riville.


- Mika

Äänestä

Kommentoi aloitusta


hinnoittaja kirjoitti:

Terve,

Mikähän tässä on vikana? Hinnastosta pitäisi kopioida hinnat tuotetaulukkoon. Nyt makro sujahtaa läpi, mutta ei tee mitään. Ilman tuota If-lausetta toimii, eli iffittelyssä jokin vika?


Sub hinnoita()

Dim laskuri As Long
Dim apuri As Long
laskuri = 1
apuri = 2
Do While laskuri < 42520
laskuri = laskuri 1
If Worksheets("Hinnat").Cells(laskuri, 1).Value = Worksheets("Tuotteet").Cells(apuri, 2).Value Then
Worksheets("Tuotteet").Cells(apuri, 10).Value = Worksheets("Hinnat").Cells(laskuri, 8).Value
Worksheets("Tuotteet").Cells(apuri, 11).Value = Worksheets("Hinnat").Cells(laskuri, 9).Value
apuri = apuri 1
End If

Loop

End Sub

Anonyymi
5000

Ilmoita asiaton sisältö

  • Sisältää materiaalia, joka on loukkaavaa, herjaavaa, rasistista, uhkailevaa tai ahdistelevaa.

  • Materiaali sisältää nimiä, yhteystietoja tai muita henkilökohtaisia tietoja. Julkisuuden henkilöistä, julkisissa viroissa toimivista ihmisistä sekä yritysten vastuuhenkilöistä saa keskustella työhön liittyen. Myös yksityiselämään liittyvistä asioista voi keskustella siltä osin, kuin niistä on julkisesti kerrottu.

  • Sisältää lapsille haitallista tai heiltä kiellettyä materiaalia. Seksuaalinen sisältö on sallittua aiheeseen tarkoitetuilla palstoilla, joiden ikäraja on 18 vuotta.

  • Lainvastaista sisältöä voi olla mm. yksityiselämää loukkaavan tiedon levitys, kiihottaminen kansanryhmää vastaan, laiton uhkaaminen, alaikäisiin liittyvä seksuaalinen sisältö/grooming, petokset, identiteettivarkaudet, kunnianloukkaukset ja tekijänoikeusrikkomukset.

  • Sisältö ei liity palstan tai keskusteluketjun aiheeseen.

  • Viesti ei ole suomea tai ruotsia (pl. International Forums), se sisältää mainontaa, se sisältää tekijänoikeuksin suojattua materiaalia, se on massapostitus tai jokin muu syy.

Jaa keskustelu