Quel est le moyen le plus propre de coder la logique pour la recherche, créer si non trouvé, rechercher à nouveau

Question

J'ai écrit beaucoup de code comme le suivant assez récemment.

Je n'aime pas le code dupliqué dans le bloc else.

Y at-il une chose évidente qui me manque? J'ai réfléchi à 'goto' mais je l'ai abandonné quand j'ai vu une possibilité de boucle infinie.

Je sais que la chose la plus évidente à faire est de créer une fonction séparée. La raison pour laquelle j'hésite, c'est parce que, comme je l'ai dit, j'ai souvent été confronté à ce scénario, ce qui fait qu'il y a pas mal de fonctions. Cela semble trop complexe pour ce que je reçois en échange (c'est-à-dire pas de duplication de code).

 Logger.Log("Finding parent.", 
      System.Diagnostics.TraceEventType.Start); 

     query = string.Format(
      "Select Id, a " + 
      "From Parent__c " + 
      "Where a ='{1}' limit 1", childId); 

     queryResult = DoSOQLQuery(queryResult, query); 

     string parentId; 

     if (queryResult != null && queryResult.size > 0) 
     { 
      parentId = ((Parent__c)queryResult.records[0]).Id; 

      Logger.Log(string.Format("Parent__c.Id={0}", 
       parentId), 
       System.Diagnostics.TraceEventType.Verbose); 
     } 
     else 
     { 
      Logger.Log("Parent not found.", 
       System.Diagnostics.TraceEventType.Error); 

      Logger.Log("Creating parent.", 
       System.Diagnostics.TraceEventType.Start); 

      string apexToExecute = string.Format(
       "Utility.CreateParent('{0}');", 
       childId); 

      this.webServices.execute(apexToExecute); 

      queryResult = DoSOQLQuery(queryResult, query); 

      if (queryResult != null && queryResult.size > 0) 
      { 
       parentId = ((Parent__c)queryResult.records[0]).Id; 

       Logger.Log(string.Format("Parent__c.Id={0}", 
        parentId), 
        System.Diagnostics.TraceEventType.Verbose); 
      } 
     } 

     Logger.Log("Done finding parent", 
      System.Diagnostics.TraceEventType.Stop); 

Answer 1

Donc, si je devais résumer votre code comme

SEARCH 

IF (FOUND) THEN 
    RETRIEVE 
ELSE 
    CREATE 
    RETRIEVE 

pourriez-vous pas faire comme

SEARCH 

IF (NOT FOUND) THEN 
    CREATE 

RETRIEVE