efficacité du code play framework scala et problèmes futurs

Question

Salut je le bloc de code suivant

processSteps.map { 
      step => 
      val prerequisiteFuture = processStepPrerequisitesDTO.getProcessStepPrerequisiteProcessTemplateIds(step.id.get) 
      prerequisiteFuture.map(prereqTemplates => { 
       processTemplateDTO.getProcessTemplates(prereqTemplates).map(pres => { 
       step.stepPrerequisites = Some(pres) 
       // Fetches the ProcessStep Prerequisites 
       processStepPrerequisitesDTO.getProcessStepPrerequisitesByProcessTemplateId(step.id.get).map(processStepPrerequisites => { 
        processStepPrerequisites.map(processStepPrerequisite => { // Eintzelne Vorbedingung 
        // Feteches The Reference to the Process Step Prerequisite Template 
        processStepPrerequisitesDTO.getProcessStepsPrerequisiteProcessTemplate(step.id.get).map(preReqs => { 
         preReqs.foreach(preReqRelation => { 
         processStepPrerequisite.processTemplate_id = preReqRelation.processtemplate 
         processStepPrerequisite 
         }) 
        }) 
        processStepPrerequisites 
        }) 
        step.prerequisites = Some(processStepPrerequisites) 
       }) 
       step 
       }) 
      }) 

Je veux chercher des données de base de données et je dois reorganisate les quelques données.

Il fonctionne ... mais pas bien

problèmes sont:

1) Code n'est pas efficace 2) Souvent, il retourne l'étape au et sans la partie middple avec le processStepPrerequisite est terminé

Quelle serait une bonne façon intelligente de résoudre ce problème? grâce

Answer 1

Vous pouvez utiliser une pour la compréhension pour faire vos affaires de la manière suivante:

val preReqFuture = for{ 
    step <- processSteps 
    prereqTemplates <- processStepPrerequisitesDTO.getProcessStepPrerequisiteProcessTemplateIds(step.id.get) 
    pres <- processTemplateDTO.getProcessTemplates(prereqTemplates) 
    processStepPrerequisites <-processStepPrerequisitesDTO.getProcessStepPrerequisitesByProcessTemplateId(step.id.get) 
    preReqs <- processStepPrerequisitesDTO.getProcessStepsPrerequisiteProcessTemplate(step.id.get) 
} yield (step, pres, processStepPrerequisites, preReqs) 

preReqFuture.map{ (step, pres, processStepPrerequisites, preReqs) => 
    processStepPrerequisites.map(processStepPrerequisite => { 
     preReqs.foreach(preReqRelation => { 
      processStepPrerequisite.processTemplate_id = preReqRelation.processtemplate 
      processStepPrerequisite 
     }) 
    }) 
    step.stepPrerequisites = Some(pres) 
    step.prerequisites = Some(processStepPrerequisites) 
    step 
}.recover { 
    case error: Throwable => //Deal with error 
} 

Note: Je suppose que processSteps est aussi un avenir.

Answer 2

Si vous voulez, step à retourner lorsque toutes les actions sont terminées, il suffit de retourner dans le dernier bloc map. Pour l'efficacité de votre code, le problème est probablement pas dans l'extrait que vous avez fourni mais dans les méthodes processStepPrerequisitesDTO.