Finance | Project Operations, Human Resources, ...

Set class

(3) Share

Report

Posted on by ..

1,965

Hi,

Is there a difference in performance? both give me the same result
so is is better to use set.in() or setEnumerator?

Set  set = new Set(Types::String);
SetEnumerator   setEnumerator;
while select...
{
    str x = this.get(...);
    if (x)
    {
       set.add(x);
    }

}

if (!set.empty())
{
   setEnumerator = set.getEnumerator();
   setEnumerator.reset();
   while (setEnumerator.moveNext())
   {
       if(this.existInTable1(setEnumerator.current())
       {
          contractClass = new ContractClass;
          contractClass.parmValue(setEnumerator.current());
          List1.addEnd(contractClass);
       }
   }
}

Set  set = new Set(Types::String);
while select ....
{
    str x= this.get(..);
    if (x && !set.in(x)
    {
        set.add(x);
        if(this.existInTable1(x)
        {
          contractClass = new ContractClass;
          contractClass.parmValue(x);
          List1.addEnd(contractClass);
        }
    }

}

Categories:

Dynamics 365 Finance

I have the same question (0)

All responses (2)

Answers (0)

Anton Venter 20,345 Super User 2025 Season 2 on at

Like (0)

Report
Hi,

Not sure, the getEnumerator is the preferred method in my opinion. You can time the two methods using the .NET Stopwatch class.

//stopwatch System.Diagnostics.Stopwatch sw = new System.Diagnostics.Stopwatch(); sw.Start(); //add code to time here sw.Stop(); info(strFmt("Done. Duration %1s.", sw.ElapsedMilliseconds/1000));

Was this reply helpful? Yes No
Suggested answer

Martin Dráb 237,880 Most Valuable Professional on at

Like (1)

Report

Iterating the collection takes some non-zero time, therefore not doing so is more efficient. But it won't make much difference if your collection isn't huge. I would rather suspect that get() and existInTable1() may be a performance problem; check out if you can't improve them (e.g. by replacing get() with SQL code in a view and use a noexists join with Table1). But that's just wild guessing without more information about these methods.

You can also improve the latter approach by calling add() and checking it's return value, instead of calling in() and optionally add().

Was this reply helpful? Yes No

Join the conversation

Community site session details

Set class

Helpful resources

Quick Links

Responsible AI policies

Neeraj Kumar – Community Spotlight

Congratulations to the November Top 10 Community Leaders!

Leaderboard > Finance | Project Operations, Human Resources, AX, GP, SL

Featured topics

Product updates

Community site session details

Set class

Helpful resources

Quick Links

Responsible AI policies

Neeraj Kumar – Community Spotlight

Congratulations to the November Top 10 Community Leaders!

Subscribe to this forum!

Select categories

Leaderboard > Finance | Project Operations, Human Resources, AX, GP, SL

Featured topics

Product updates