Want to protect your cyber security and still get fast solutions? Ask a secure question today.Go Premium

x
?
Solved

Need help with a component trying to use Lookup

Posted on 2006-07-12
2
Medium Priority
?
215 Views
Last Modified: 2010-04-05
Ok,

I'm using Absolute DB (componentace.com) with a table that is setup in the following way:

------------------------------------------------------|
BLink | OLink | Option                       | PBCode|
------------------------------------------------------|
201   | 1001  | 'Integral Dust Drawer' | 'IDD'    |
------------------------------------------------------|

So, this is a price book table, with BLink(book link), OLink(option link) and a PBCode which is a string value, and option text.  

I have a combo box component that I'm trying to automatically fill with option text based on an array that is passed to it.  Here's the code that I build the combo box with:

Var
  modelArray: Array [0..2] of String;
Begin

    modelArray[0] := 'GSMBASE';
    modelArray[1] := 'HOPPER';
    modelArray[2] := 'IDD';

    //_Configuration is the combo box I'm trying to build....
    _Configuration.LoadItemsFromCodes(modelArray);

//Here's the code for the combo box.  It has properties of BLink and OLink, which are integer values.  So in this case the BLink is set to 201:

Function TPBxComboBox.LoadItemsFromCodes(codeArray: Array Of String): Integer;
Var
    i,daOLink: Integer;
     SearchList, SearchList2: Variant;
Begin
    Result := -1; // error code for NIL value for datasets
    ErrMsg := 'Nil Datasets';
    If Not Assigned(dsOptions) Then
        Exit;
    If Not Assigned(PItem) Then
        Exit;

    xOLinkTrack := False;

    For i := 0 To High(codeArray) Do
    Begin
        With dsOptions As TABSTable Do
        Begin
           
            If State = dsInactive Then
                Open;

            Self.Items.Clear;
            xOLinkList.Clear; //this is a string list that contains an olink for each item in the combo box

            //lookup the OLink for the PBCode that was passed via the codeArray[] array:
            daOLink := dsOptions.Lookup('BLink;PBCode', VarArrayOf([BLink,codeArray[i]]), 'OLink');

            Self.xOLinkList.Add(IntToStr(daOLink));
           
            //use the Olink we just found to lookup the Option text for placing in the combo box
            Items.Add(dsOptions.Lookup('BLink;OLink', VarArrayOf([BLink, daOLink]), 'Option'));

            Self.ItemIndex := 0;
            Result := Items.Count;

            //other various code....
            If Items.Count > 0 Then
            Begin
                xOLinkTrack := True;
                ziHold      := ihOptions;
                Self.Change;
            End;
        End;
    End;
End;

As you can see, this should find an olink for the each of the values that I pass to the LoadItemsFromCodes function, then use that olink to find the option text and load that text into the combo box.  However, it only puts the 'IDD' or Integral Dust Drawer option in.  The two other array values are simply not present in the combo box, even though they are present in the dsOptions table and should be in the combo box.  I cannot figure out why.  This is my first time using the Lookup method, so I may be missing something easy here.

Thanks!
0
Comment
Question by:light_bulb
2 Comments
 
LVL 3

Author Comment

by:light_bulb
ID: 17091334
Oh shoot, I found the problem.  I should have put the following lines before the loop instead of inside it:

Self.Items.Clear;
xOLinkList.Clear;

I was clearing the lists every time through the loop.  Oh well, not the first time I've made that mistake.
0
 
LVL 17

Accepted Solution

by:
Wim ten Brink earned 2000 total points
ID: 17106729
Damn! I was about to say that too. :-)

You might want to put the line "If State = dsInactive Then Open;" also a bit higher, outside the loop. No need to check if it's inactive for every item in the array, is it? This will probably speed up your code with about 5 clock cycles, which on a 1 GHz computer will save you... Well, it's not much, I know...

Go ask for a refund and have this question deleted before someone else sees your mistake... ;-)

0

Featured Post

Get your problem seen by more experts

Be seen. Boost your question’s priority for more expert views and faster solutions

Question has a verified solution.

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

Introduction The parallel port is a very commonly known port, it was widely used to connect a printer to the PC, if you look at the back of your computer, for those who don't have newer computers, there will be a port with 25 pins and a small print…
Introduction I have seen many questions in this Delphi topic area where queries in threads are needed or suggested. I know bumped into a similar need. This article will address some of the concepts when dealing with a multithreaded delphi database…
Integration Management Part 2
How can you see what you are working on when you want to see it while you to save a copy? Add a "Save As" icon to the Quick Access Toolbar, or QAT. That way, when you save a copy of a query, form, report, or other object you are modifying, you…
Suggested Courses

581 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