Solved

VB script help

Posted on 2008-09-29
4
172 Views
Last Modified: 2012-05-05
Hi,

I have a VB script that needs to be modified.

The script is supposed to get the sum of price and shipping from several sellers on Amazon and re-price my item against the smallest sum.

See below the portion that has to be modified.  

There is a lists the properties and methods available that I can provide .

Thanks,
akiles

' Find the lowest seller that is not me

    '

    LowSeller = ""

    For i = 0 To .sellerspassed - 1

        If .sellername(i) <> .mynickname Then

            LowSellerPrice = .sellerprice(i)

            LowSeller = .sellername(i)

            Exit For

        End If

    Next

    If LowSeller <> "" Then

         ' Deal with at least one seller found

        If InStr(1,Sellers,LowSeller & ";", vbTextCompare) <> 0 Then

            '

            ' Find the seller in the array

            '

            LowSellerArray = Split(Sellers, ";")

            LowShipCostArray = Split(ShipCosts, ";")

            ' Verify that arrays are correct size

            If UBound(LowSellerArray) <> UBound(LowShipCostArray) Then

                MSGBox "Array elements are different - " & UBound(LowSellerArray) & "\" & UBound(LowShipCostArray)

                MSGBox Sellers

                MSGBox ShipCosts

                Exit Function

            End If

            

            LowShipCost = -1

            For i = LBound(LowSellerArray) To UBound(LowSellerArray)

                If StrComp(LowSeller, LowSellerArray(i), vbTextCompare) = 0 Then

                    LowShipCost = LowShipCostArray(i)

                    Exit For

                End If

            Next

            If LowShipCost = -1 Then

                MSGBox "Unable to locate " & LowSeller & " seller in array"

                MSGBox Sellers

                MSGBox ShipCosts

                Exit Function

            End If

            

        Else

            ' Use the default ship cost

            LowShipCost = DefaultShipCost

        End If

        

        '

        ' Do the arithmetic to calculate price

        '

        .myprice = LowSellerPrice + LowShipCost - MyShipCost - 0.01

Open in new window

0
Comment
Question by:akiles99
  • 2
  • 2
4 Comments
 
LVL 1

Expert Comment

by:KUBIS
ID: 22605159
Your "problem" is in wrong if case. In your case your lowest price is the first price, that is different from your nick name. It should be like this:
    LowSeller = ""

    LowSellerPrice = .sellerprice(0)*5 ' for comapare ony, it should be big enough 

    For i = 0 To .sellerspassed - 1

        IF i=0 then 

        If .sellername(i) <> .mynickname Then

            if .sellerprice(i)<LowSellerPrice then 'lets compare

              LowSellerPrice = .sellerprice(i)

              LowSeller = .sellername(i)

              Exit For

            endif

        End If

    Next i

Open in new window

0
 
LVL 1

Accepted Solution

by:
KUBIS earned 500 total points
ID: 22605197
Ups . delete line 4
right way

hope it helps.



    LowSeller = ""

    LowSellerPrice = .sellerprice(0)*5 

    For i = 0 To .sellerspassed - 1

        If .sellername(i) <> .mynickname Then

            if .sellerprice(i)<LowSellerPrice then 'lets compare

              LowSellerPrice = .sellerprice(i)

              LowSeller = .sellername(i)

              Exit For

            endif

        End If

    Next i

Open in new window

0
 

Author Comment

by:akiles99
ID: 22605222
Hi sorry ,

I forget to delete this question.Anyway i will grade it.

The help i need is:
http://www.experts-exchange.com/Programming/Languages/Visual_Basic/Q_23774357.html

Same as this but the needs u can check there.

Please help me on this.

Thanks,
akiles
0
 

Author Closing Comment

by:akiles99
ID: 31501458
Thanks.
0

Featured Post

How your wiki can always stay up-to-date

Quip doubles as a “living” wiki and a project management tool that evolves with your organization. As you finish projects in Quip, the work remains, easily accessible to all team members, new and old.
- Increase transparency
- Onboard new hires faster
- Access from mobile/offline

Join & Write a Comment

Introduction I needed to skip over some file processing within a For...Next loop in some old production code and wished that VB (classic) had a statement that would drop down to the end of the current iteration, bypassing the statements that were c…
Background What I'm presenting in this article is the result of 2 conditions in my work area: We have a SQL Server production environment but no development or test environment; andWe have an MS Access front end using tables in SQL Server but we a…
As developers, we are not limited to the functions provided by the VBA language. In addition, we can call the functions that are part of the Windows operating system. These functions are part of the Windows API (Application Programming Interface). U…
This lesson covers basic error handling code in Microsoft Excel using VBA. This is the first lesson in a 3-part series that uses code to loop through an Excel spreadsheet in VBA and then fix errors, taking advantage of error handling code. This l…

757 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question

Need Help in Real-Time?

Connect with top rated Experts

17 Experts available now in Live!

Get 1:1 Help Now