Apex Triggers - 47 (Accenture Interview Scenario) 

Thanks brother

Sure bhai if Mohit sir said to implement it i will create a video for sure but it will take time to create a video on it

@@sfdcninjas Thank you sir! I will wait for it.

@@abhijit14820 try this trigger ShareAccountWithSecondaryOwner on Account (after insert, after update) { List sharesToInsert = new List(); List sharesToDelete = new List(); // Map to hold the secondary owner Id for each account Map accountIdToSecondaryOwnerId = new Map(); // Collect secondary owner Ids for newly inserted or updated accounts for (Account acc : Trigger.new) { // Check if the secondary owner lookup field has changed if (Trigger.isInsert || acc.Secondary_Owner__c != Trigger.oldMap.get(acc.Id).Secondary_Owner__c) { accountIdToSecondaryOwnerId.put(acc.Id, acc.Secondary_Owner__c); } } // Query existing account shares for accounts that are being updated List existingShares = [SELECT Id, AccountId FROM AccountShare WHERE AccountId IN :accountIdToSecondaryOwnerId.keySet()]; // Delete existing shares for updated accounts for (AccountShare share : existingShares) { sharesToDelete.add(new AccountShare(Id = share.Id)); } // Insert new shares for updated accounts for (Id accountId : accountIdToSecondaryOwnerId.keySet()) { sharesToInsert.add(new AccountShare( AccountId = accountId, UserOrGroupId = accountIdToSecondaryOwnerId.get(accountId), AccountAccessLevel = 'Read' )); } // Perform DML operations if (!sharesToDelete.isEmpty()) { delete sharesToDelete; } if (!sharesToInsert.isEmpty()) { insert sharesToInsert; } }

@@abhijit14820 try this. source: chat GPT trigger ShareAccountWithSecondaryOwner on Account (after insert, after update) { List sharesToInsert = new List(); List sharesToDelete = new List(); // Map to hold the secondary owner Id for each account Map accountIdToSecondaryOwnerId = new Map(); // Collect secondary owner Ids for newly inserted or updated accounts for (Account acc : Trigger.new) { // Check if the secondary owner lookup field has changed if (Trigger.isInsert || acc.Secondary_Owner__c != Trigger.oldMap.get(acc.Id).Secondary_Owner__c) { accountIdToSecondaryOwnerId.put(acc.Id, acc.Secondary_Owner__c); } } // Query existing account shares for accounts that are being updated List existingShares = [SELECT Id, AccountId FROM AccountShare WHERE AccountId IN :accountIdToSecondaryOwnerId.keySet()]; // Delete existing shares for updated accounts for (AccountShare share : existingShares) { sharesToDelete.add(new AccountShare(Id = share.Id)); } // Insert new shares for updated accounts for (Id accountId : accountIdToSecondaryOwnerId.keySet()) { sharesToInsert.add(new AccountShare( AccountId = accountId, UserOrGroupId = accountIdToSecondaryOwnerId.get(accountId), AccountAccessLevel = 'Read' )); } // Perform DML operations if (!sharesToDelete.isEmpty()) { delete sharesToDelete; } if (!sharesToInsert.isEmpty()) { insert sharesToInsert; } }

Maybe to avoid that null we can use oppMap.KeySet() instead of accIds, which there will be only accs with opps And also probably we will need to check on update account trigger for accs with no opps where we can just make blank the description field Also probably it will be better to check null for newOpp .AccountId before adding it to accIds set

Maybe to avoid that null we can use oppMap.KeySet() instead of accIds, which there will be only accs with opps And also probably we will need to check on update account trigger for accs with no opps where we can just make blank the description field Also probably it will be better to check null for newOpp .AccountId before adding it to accIds set

Yes just add NULLS LAST in your Soql and It will work. list Opplist = [SELECT Id,Name,Amount,AccountId from Opportunity where AccountId IN :AccIds ORDER By Amount DESC NULLS LAST Limit 1];

@@Tushar.S.Kitchen You cannot use LIMIT 1 here.. if you change the accountId on an opp, your query will return opp record for a single account and the opp related to other acc will be left out.

Sure you will get a video on it this week

@@sfdcninjas Thank you sir I am practising the scenarios and the explanation was good

keep watching and supporting bro

Hi sure , in which company’s interview it was asked?

Astrait IT services

Sure i will create a video on it but you have to wait for. is that ok with you?

@@ShwetaBanne why would an unknown company ask such a difficult trigger question ? Is it to show potential candidate down ?

bro here is your solution, thanks for sharing a scenario: handler class: public with sharing class conAccOppController { //S:2- whenever the contact is created if contact has any related account then if account doesn't have any related opportunity it will create new opp for that account , if account has any related opportunity then the sum of amount of opportunity stored in related account custom field - asked in interview public static void afterInsert(list newCons){ set accIds=new set(); for(contact con:newCons){ if(con.AccountId!=null){ accIds.add(con.AccountId); } } //account with no opportunities: list accWithNoOppor = [select id,name from account where id in :accIds and id not in (select accountId from opportunity)]; List newOpps=new List(); for(Account acc:accWithNoOppor){ newOpps.add(new Opportunity(name='Opp created from '+acc.name+' account',accountId=acc.id,CloseDate=date.today()+10,stageName='Prospecting')); } if(newOpps.size()!=0){ insert newOpps; } aggregateResult[] agg=[select accountId,sum(amount) sumAmounts from opportunity group by accountId]; list accAmUpdate=new List(); for(aggregateResult agg2:agg){ string ids=(String) agg2.get('accountId'); decimal totalAmounts=(decimal) agg2.get('sumAmounts'); account acc=new account(id=ids,Total_Opportunity_Amount__c=totalAmounts); accAmUpdate.add(acc); } if(accAmUpdate.size()!=0){ update accAmUpdate; } } } trigger: trigger accConOpp on Contact (after insert) { conAccOppController.afterInsert(Trigger.new); }

Hi vipul thanks for providing scenario i will create a video on it for sure but please wait it might take time

Ok sure. Thanks!😊

Create video on this scenario too sir

You will get it tomorrow bro

Hi buddy , using subquery is not a good practice in apex

@madhumohan2952 - the problem lies in your for loop where you are iterating the opplist1... when this for loop finishes its iteration, the map will be having the lower amount opp name set for the respective account because your key will be overriding for every for loop iteration.. you can have this inside a if-block and only populate your map when its not having the particular key as an account ID.