[SalesForce] Using List/Set in If Statement

Pretty basic question (I think), but after searching for a few hours and trying different things I can't seem to find a solution. Pretty new to Apex, but I do have experience in SQL, Java/C#, and Python so nothing is too foreign to me.

In my trigger on Opportunity I'm doing a check on whether or not the Opportunity Owner belongs to a few specific UserRoleIds. This was my original try:

// Step 1. Create list of Users where the UserRole Name has AE or CM in it
// I originally did this by querying the UserRole object and then the User role but realized that was redundant

List<User> listUsers = [SELECT Id FROM User WHERE UserRole.Name like '%AE%' OR UserRole.Name like '%CM%'];

// other code

for (Opportunity opp : trigger.New) {
// other stuff

if (/*I want to see if opp.OwnerId is in that list/set/SOQL query that I created*/)

My understanding is that you can't really do that type of search on a list in that scenario, so I did something where I converted my list to a set. Problem there is I couldn't figure out how to convert List<User> or Set<User> to Set<Id> and then use set.contains(opp.OwnerId).

My two questions

Can I search if opp.OwnerId is found in a list (either a list of Users or a list of Ids)?
If not, how do I convert a SOQL query/list into the proper Set type; i.e. a Set of Ids or Strings, so that I could then use the contains method?

I hope that was clear, but I'm trying to understand this at a higher/conceptual level rather than just getting the right code for this specific example.

I suppose another thing to ask is if it's possible to save a SOQL query in a list of Ids or Strings rather than the object you're querying?

Thanks so much!

Dylan

Best Answer

I think perhaps the best way is to search if the opp.OwnerId is in a Map<User> that you can create from the result of your query.

Map<Id,User> userMap = new Map<Id,User>([SELECT Id FROM User WHERE UserRole.Name like '%AE%' OR UserRole.Name like '%CM%']);

Now you can iterate the opportunities and use containsKey() on the map to see if the User is the owner.

for (Opportunity opp : trigger.New) { 
    if (userMap.containsKey(opp.OwnerId)){
        //opp owner is one of your users with the specific role
    }
}

There are a number of cool things you can do converting to and from Lists, Sets and Maps. You can use the addAll() method on a List or a Set, Map.containsKey(), Map.values() and Map.keySet() among other things.

Have a look at the official documentation for Maps, Lists and Sets

Related Solutions

[SalesForce] SOQL query with Dataloader using List Literal in WHERE Clause

If you cannot get an answer or the list in lengthy, What I have done in the past is:

Create boolean Flag named ExtporData__c on the object
Updated all accounts via data loader using CSV with ID and ExportData__c field that should be exported

then in the data loader for export I simply used EmportData__c = true in the where clause

[SalesForce] How to search for a string in a list of strings

The easiest way to implement that kind of lookup in code is with a Map.

Your code could look like this:

public class ChainReactionUserClass {
    @future public static void ChainReactionStart(set<ID> recordIDs){
        //Grab all required fields from user.  Essetial to test class running.
        list<user> users = [SELECT ID, Firstname, Lastname, email, name, username, street, city, state, postalcode, country,
                    UserRoleId, CommunityNickname, TimeZoneSidKey, LocaleSidKey, EmailEncodingKey, ProfileId,
                    LanguageLocaleKey 
                    FROM user
                    WHERE ID in :recordIDs];

        Map <String,String> regions = new Map <String,String> ('Boston' => 'Nort East', 'San Francisco' => 'South West'); // etc.

        list<contact> contactInsert = new list<contact>();

        for (User u: users){

            String region = regions.Get (u.City);

            Contact c = new Contact(
            FirstName = u.FirstName,
            LastName = u.LastName,
            MailingStreet = u.street,
            MailingCity = u.city,
            MailingState = u.state,
            MailingPostalCode = u.PostalCode,
            email = u.email,
            Region__c = region);
            contactInsert.add(c);
        }
        insert contactInsert;    
    }
}

The issue with this approach is that you're hard-coding your city-to-region relationships in your code. A better approach would be to create an object model that represents your city to region mapping, and query it inside your Apex. This way, your users can expand your city to region mapping outside code (just by point and click).

Edit: A few notes on your original code:

You were inserting contactInsert inside the for loop. It has to be inserted outside the loop, once you've populated the whole list; otherwise, you're using your DML limits very inefficiently.
You were checking for an empty cities list inside the for loop. The check needs to be done outside (although, depending on what your code does, it might not be necessary... the for will simply not iterate at all).

Best Answer

Related Solutions

[SalesForce] SOQL query with Dataloader using List Literal in WHERE Clause

[SalesForce] How to search for a string in a list of strings

Related Topic