Solved

VB script help

Posted on 2008-09-29
4
175 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

Free Tool: Port Scanner

Check which ports are open to the outside world. Helps make sure that your firewall rules are working as intended.

One of a set of tools we are providing to everyone as a way of saying thank you for being a part of the community.

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

Enums (shorthand for ‘enumerations’) are not often used by programmers but they can be quite valuable when they are.  What are they? An Enum is just a type of variable like a string or an Integer, but in this case one that you create that contains…
This article describes some techniques which will make your VBA or Visual Basic Classic code easier to understand and maintain, whether by you, your replacement, or another Experts-Exchange expert.
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…
Get people started with the process of using Access VBA to control Outlook using automation, Microsoft Access can control other applications. An example is the ability to programmatically talk to Microsoft Outlook. Using automation, an Access applic…

856 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