AskOverflow.Dev

AskOverflow.Dev Logo AskOverflow.Dev Logo

AskOverflow.Dev Navigation

  • Início
  • system&network
  • Ubuntu
  • Unix
  • DBA
  • Computer
  • Coding
  • LangChain

Mobile menu

Close
  • Início
  • system&network
    • Recentes
    • Highest score
    • tags
  • Ubuntu
    • Recentes
    • Highest score
    • tags
  • Unix
    • Recentes
    • tags
  • DBA
    • Recentes
    • tags
  • Computer
    • Recentes
    • tags
  • Coding
    • Recentes
    • tags
Início / coding / Perguntas / 77482341
Accepted
bnthsrikanth
bnthsrikanth
Asked: 2023-11-15 00:36:47 +0800 CST2023-11-15 00:36:47 +0800 CST 2023-11-15 00:36:47 +0800 CST

Existe algum escopo para melhorar o desempenho do meu código?

  • 772

Tenho um serviço Salesforce Apex REST (LeadService) que processa leads recebidos no formato JSON. Além disso, estou procurando maneiras de aprimorar o tratamento de erros e tornar o código mais fácil de manter. estrutura geral do código

@RestResource(urlMapping='/api/lead')
global class Service {
@HttpPost
global static string createdata(){
   
    RestResponse res = Restcontext.response;
    String requestBody = RestContext.request.requestBody.toString();
    try {   
        // Deserialize JSON data into a list of LeadDataDeserializer objects
        List<LeadDataDeserializer> externalLeads = (List<LeadDataDeserializer>)
            JSON.deserializeStrict(requestBody, List<LeadDataDeserializer>.class);            
        // Transform LeadDataDeserializer objects into Lead data
        List<Lead> students = new List<Lead>();
        for(LeadDataDeserializer info : externalLeads) {
            Lead t_leads = setLeadFields(info);
            t_leads.company='Test Company';
            students.add(t_leads);
        }
        if(students.isEmpty()){
            res.statusCode=400;
            return 'Empty list';
            
           
        }
        else{
            List<Response> responseretn=new List<Response>();
            Database.UpsertResult[] srList = Database.upsert(students, Lead.External_Id__c, false);
            // Process upsert results if needed
            Integer i=0;
            for(Database.UpsertResult upResult:srList){
                if(upResult.isSuccess()){
                    responseretn.add(new Response(upResult.getId(),true,students[i].MobilePhone));
                }
                else{
                    System.debug(upResult.getErrors());
                    responseretn.add(new Response(upResult.getId(),false,students[i].MobilePhone));
                }
                i+=1;
            }
        
            String jsonReqBody=JSON.serialize(responseretn);
            res.statusCode = 201;
            return jsonReqBody;
            
           
                            
        }
        
    } catch(Exception e) {
        // Handle exceptions
        res.Statuscode = 500;
        return 'Internal Server Error';

        
    }
}
//Response wrapper to return
public class Response{
    public string leadId{get;set;}
    public boolean isSuccess{get;set;}
    public string mobilePhone{get;set;}
    
    public Response(String leadId,Boolean isSuccess,String mobilePhone){
        this.leadId=leadId;
        this.isSuccess=isSuccess;
        this.mobilePhone=mobilePhone;
    }
}

public static Lead setLeadFields(LeadDataDeserializer info){
    Lead extLead=new Lead();
    extLead.LastName=info.Name;
    extLead.CountryCode__c=info.countryCode; 
    extLead.MobilePhone=info.phoneNumber;
    extLead.mx_WhatsApp_Number__c=extLead.MobilePhone;
    extLead.mx_IP_Address__c=info.ipAddress;
    extLead.External_Id__c=extLead.MobilePhone;
    if(info.leadStage!=null){
        extLead.Status=info.leadStage;
    }
    if(info.campaignName!=null){
        extLead.mx_Campaign_Name__c=info.campaignName;
    }
    if(info.campaignSource!=null){
        extLead.SourceCampaign__c=info.campaignSource;
    }

    //20 more if conditions with null check like above (serializeddata.field!=null)
   
    return extLead;
}}

Estou usando várias instruções if para tratar apenas para incluir campos que possuem valor e não para substituir o valor do registro anterior se o valor proveniente da integração estiver vazio

Como isso pode ser tratado de forma eficiente?

salesforce
  • 1 1 respostas
  • 13 Views

1 respostas

  • Voted
  1. Best Answer
    eyescream
    2023-11-15T08:30:47+08:002023-11-15T08:30:47+08:00
    if(students.isEmpty()){
       res.statusCode=400;
       return 'Empty list';
    }
    

    isso é um pouco tarde. Não mudará muito no desempenho, mas logicamente seria melhor verificar a lista desserializada mais cedo e retornar então, não depois de extrair dela.

    Eu não acho que você precise retornar uma string. Você poderia facilmente retornar List<Response>e SF irá serializá-lo para você.

    Como você deseja lidar com os problemas? Salve o que puder? Você pode inserir o sObject auxiliar para qualquer problema e executar um relatório sobre isso. Ou use eventos de plataforma com "publicar imediatamente" para que algum sistema de monitoramento ou até mesmo um gatilho especial possa processá-los.

    Talvez eu incluísse um contador de erros, então, se a taxa de falha for 100%, retorne algo diferente de 201

    E se você fizer isso se ele enviar> 10 mil linhas (ou os efeitos colaterais resultarão em> 10 mil dmls) ... Se for uma preocupação legítima, eu provavelmente a reescreveria para iniciar um trabalho em lote (os lotes podem assumir o escopo e iterar isso, eles nem sempre precisam começar com uma consulta). Pontos de bônus que você poderia implements Database.RaisesPlatformEventse trabalho quase concluído, SF fará muitos tratamentos de erros para você sem a necessidade de "tarefa savepoint-try-catch-rollback-insert manual ou qualquer outra coisa"

    Quanto ao código de mapeamento real... não é ótimo, mas também não é muito ruim. Você pode ser um pouco ingênuo com verificações de nulos - pois algumas variáveis String.isNotBlank​​serão melhores, dependendo de qual JSON é produzido pela fonte.

    Eu manteria o mapeamento do campo origem-destino em outro lugar (configuração personalizada? Metadados personalizados?) Para que você não precise recompilá-lo, implantar etc. toda vez que adicionar um novo campo.

    Se você realmente quiser, pode ler sobre JSON.serialize com o parâmetro para pular nulos (então, por exemplo, você pode desserializar a entrada, serializá-la novamente com os nulos ignorados, desserializar na segunda vez, limpar). Ou defina cegamente todos os campos com base na entrada, então sObject.getPopulatedFieldsAsMap e itere através deles verificando o que é nulo ... mas parece um pouco inteligente demais para o que é necessário. Às vezes, o simples é mais eficaz.


    Precisa mesmo ser uma API REST personalizada? Se você tiver controle sobre o formato de origem, poderá considerar compositea API padrão, com menos trabalho. Dê uma olhada em meu https://salesforce.stackexchange.com/a/274696/799 (incluindo o cabeçalho "allOrNone"). O upsert normal não lida bem com vários registros, é verdade - mas isso é próximo o suficiente!

    E bem... Código nem sempre é a resposta. Se você tem certeza absoluta de que a API padrão não funcionará - você sabia que pode chamar fluxos pela API REST ?

    • 0

relate perguntas

  • Página do Visualforce gerando erro "Não é possível acessar a página" no Salesforce

Sidebar

Stats

  • Perguntas 205573
  • respostas 270741
  • best respostas 135370
  • utilizador 68524
  • Highest score
  • respostas
  • Marko Smith

    destaque o código em HTML usando <font color="#xxx">

    • 2 respostas
  • Marko Smith

    Por que a resolução de sobrecarga prefere std::nullptr_t a uma classe ao passar {}?

    • 1 respostas
  • Marko Smith

    Você pode usar uma lista de inicialização com chaves como argumento de modelo (padrão)?

    • 2 respostas
  • Marko Smith

    Por que as compreensões de lista criam uma função internamente?

    • 1 respostas
  • Marko Smith

    Estou tentando fazer o jogo pacman usando apenas o módulo Turtle Random e Math

    • 1 respostas
  • Marko Smith

    java.lang.NoSuchMethodError: 'void org.openqa.selenium.remote.http.ClientConfig.<init>(java.net.URI, java.time.Duration, java.time.Duratio

    • 3 respostas
  • Marko Smith

    Por que 'char -> int' é promoção, mas 'char -> short' é conversão (mas não promoção)?

    • 4 respostas
  • Marko Smith

    Por que o construtor de uma variável global não é chamado em uma biblioteca?

    • 1 respostas
  • Marko Smith

    Comportamento inconsistente de std::common_reference_with em tuplas. Qual é correto?

    • 1 respostas
  • Marko Smith

    Somente operações bit a bit para std::byte em C++ 17?

    • 1 respostas
  • Martin Hope
    fbrereto Por que a resolução de sobrecarga prefere std::nullptr_t a uma classe ao passar {}? 2023-12-21 00:31:04 +0800 CST
  • Martin Hope
    比尔盖子 Você pode usar uma lista de inicialização com chaves como argumento de modelo (padrão)? 2023-12-17 10:02:06 +0800 CST
  • Martin Hope
    Amir reza Riahi Por que as compreensões de lista criam uma função internamente? 2023-11-16 20:53:19 +0800 CST
  • Martin Hope
    Michael A formato fmt %H:%M:%S sem decimais 2023-11-11 01:13:05 +0800 CST
  • Martin Hope
    God I Hate Python std::views::filter do C++20 não filtrando a visualização corretamente 2023-08-27 18:40:35 +0800 CST
  • Martin Hope
    LiDa Cute Por que 'char -> int' é promoção, mas 'char -> short' é conversão (mas não promoção)? 2023-08-24 20:46:59 +0800 CST
  • Martin Hope
    jabaa Por que o construtor de uma variável global não é chamado em uma biblioteca? 2023-08-18 07:15:20 +0800 CST
  • Martin Hope
    Panagiotis Syskakis Comportamento inconsistente de std::common_reference_with em tuplas. Qual é correto? 2023-08-17 21:24:06 +0800 CST
  • Martin Hope
    Alex Guteniev Por que os compiladores perdem a vetorização aqui? 2023-08-17 18:58:07 +0800 CST
  • Martin Hope
    wimalopaan Somente operações bit a bit para std::byte em C++ 17? 2023-08-17 17:13:58 +0800 CST

Hot tag

python javascript c++ c# java typescript sql reactjs html

Explore

  • Início
  • Perguntas
    • Recentes
    • Highest score
  • tag
  • help

Footer

AskOverflow.Dev

About Us

  • About Us
  • Contact Us

Legal Stuff

  • Privacy Policy

Language

  • Pt
  • Server
  • Unix

© 2023 AskOverflow.DEV All Rights Reserve