Как исправить System.NullPointerException: попытка разыменовать нулевой объект в классе Apex, используя Map<parentId, child>?

Question

Получение ошибки в строке " taskObj.OwnerId = ConMap.get(ben.contact__c).contact__r.OwnerId;" потому что поле ownerid находится в контакте.

Контакт является родителем льгот, здесь я получаю все льготы в стартовом методе. Я хочу добавить contactid только один раз, если у него более одного дочернего элемента, для которого я использовал SET. Я хочу использовать карты, так как мне нужно получить контактное поле OwnerId из контактного объекта, который я получаю в запросе в методе start. Как получить доступ к полю contact.ownerId с помощью карты? ниже код.

 global Database.QueryLocator start(Database.BatchableContext bc) {
    Query='select contact__r.ownerId, contact__c, Task_creation_date__c, Status__c, task_created__c, type__c from Benefit__c Where Task_creation_date__c= TODAY AND Status__c IN (\'Active\',\'Pending\') AND Task_created__c =FALSE AND Type__c!=\'Short Term Medical\'';
    return Database.getQueryLocator(query);

}

global void execute(Database.BatchableContext bc, List<Benefit__c> scope){
    // process each batch of records

    List<Contact> contacts = new List<Contact>();
    List<Task> TaskList = new List<Task>();
    set<id> conset = New set<id>();
    Map<id,benefit__c> ConMap = new map<id,Benefit__c>();
    for (Benefit__c b : scope) {
            conset.add(b.contact__c);
            ConMap.put(b.contact__c, b);
            b.task_created__c = TRUE;
     }
      system.debug('contact and its benefit------'+ConMap);
     recordsProcessed = conset.size();
     //List<Contact> tempList = new List<Contact>();
    // tempList = [Select Id,OwnerId, firstname from Contact where Id IN:(conset)];
     if(!ConMap.isEmpty())
      {
       for(Benefit__c ben : ConMap.values())
       {
        Task taskObj = new Task();
                        taskObj.OwnerId = ConMap.get(ben.contact__c).contact__r.OwnerId;

Я хочу заполнить contact ownerid как идентификатор владельца, но как мне получить к нему доступ с карты и сохранить уникальный идентификатор контакта на карте?

Answer 1

Я переписал код для решения этой проблемы и оптимизировал код, как показано ниже.

public with sharing class MyBatch implements Database.Batchable<AggregateResult>{
public Iterable<AggregateResult> start(Database.BatchableContext context)
{
    return [
        SELECT Contact__c, Contact__r.OwnerId owner FROM Benefit__c
        WHERE Contact__c != null AND ...
        GROUP BY Contact__c, Contact__r.OwnerId
    ];
}
public void execute(Database.BatchableContext context, List<AggregateResult> scope)
{
    Set<Id> contactIds = new Set<Id>();
    List<Task> tasks = new List<Task>();
    for (AggregateResult aggregate : scope)
    {
        Id ownerId = (Id)aggregate.get('owner');
        Id contactId = (Id)aggregate.get('Contact__c');

        contactIds.add(contactId);
        tasks.add(new Task(OwnerId=ownerId, /*other fields*/));
    }
    insert tasks;

    List<Benefit__c> benefits = [
        SELECT Id FROM Benefit__c WHERE Contact__c IN :contactIds
    ];
    for (Benefit__c benefit : benefits)
        benefit.Task_Created__c = true;
    update benefits;
}
public void finish(Database.BatchableContext context) { /*additional logic*/ }}

Answer 2

Я вижу, что пакетный запрос не имеет условия фильтрации 'Contact__c!= Null'. Таким образом, возможно, что в одной из записей пособий отсутствует значение в поле "Contact__c", и вы не найдете его на карте. Вы можете решить это двумя способами:

1. Добавьте 'Contact__c!= Null' в запрос селектора, если вам не нужны эти записи.

(Или же)

2. Проверьте значение 'null' в цикле for, как показано ниже:

   if(!ConMap.isEmpty())
   {
       for(Benefit__c ben : ConMap.values())
       {
           if(String.isBlank(ben.Contact__c)){
               /* continue;
                  or
                  throw exception()
               */
           }
   
           Task taskObj = new Task();
           taskObj.OwnerId = ConMap.get(ben.contact__c).contact__r.OwnerId;