Exam PDII Topic 11 Question 147 Discussion

The two changes that the developer should make in this trigger to adhere to best practices are to move the Region__c query to outside the loop and to remove the last line updating accountList as it is not needed. Moving the Region__c query to outside the loop avoids the SOQL query limit, as it reduces the number of queries from one per Account record to one for all Account records. The developer can use a Map to store the Region__c records by Name, and then use the Map to assign the OwnerId for each Account record. Removing the last line updating accountList as it is not needed avoids the DML statement limit, as it is redundant to update the records that are already in the trigger context. The developer can use the Trigger.new list to modify the Account records, and the changes will be saved automatically after the trigger execution. Using a Map to cache the results of the Region__c query by Id is not a valid answer, as it does not move the query outside the loop, and it does not match the Name field of the Account record. Using a Map accountMap instead of List accountList is not a valid answer, as it does not affect the query or the DML limits, and it does not change the logic of the trigger. Reference: [Apex Triggers], [Apex Best Practices], [Apex Developer Guide]