Angular - 如何最小化相同代碼的使用？

Question

我不確定如何更好地處理和更改以下實現。

我有兩種訂閱方法，可以重新分配如下值。

public patientLastStatus = "";

this.treatmentStageOne.subscribe(resp => {
      if (resp) {
        this.patientLastStatus = "Passed";
      }
    });
this.treatmentStageTwo.subscribe(resp => {
      if (resp) {
        this.patientLastStatus = "Still in Observation";
      }
    });

我明白了，以上是完全不同的兩種訂閱方式。 在這里， this.patientLastStatus的值只是重新賦值。 但我不明白在什么基礎上 Sonarqube 向我展示了更新或重構這個 function 以便它的實現不會重復第 88 行上的那個。

如何以更好的方式改變它？

非常感謝

感謝大家。

Answer 1

您可以將此邏輯提取到更高階的 function 中，它接受新的分配值：

const conditionalChange = newValue => 
  resp => {
    if (resp) {
      this.patientLastStatus = newValue;
    }
  }

現在您可以在兩個訂閱中重復使用它而無需復制代碼：

this.treatmentStageOne.subscribe(conditionalChange("Passed"));
this.treatmentStageTwo.subscribe(conditionalChange("Still in Observation"));

Answer 2

Sonarqube 正在拋出警告，因為它看到重復。嘗試通過方法更新this.patientLastStatus 。

public patientLastStatus = "";

public updatePateintStatus = (status)=>{
   this.patientLastStatus = status;
}
this.treatmentStageOne.subscribe(resp => {
      if (resp) {
        this.updatePateintStatus("Passed");
      }
    });
this.treatmentStageTwo.subscribe(resp => {
      if (resp) {
        this.updatePateintStatus("Still in observations...");
      }
    });