Как упростить такую запись?

Question

[Alex10214]

Приветствую, может кто поможет упростить такую запись:

if (this.newAccount.model_settings !== "{}") {
      this.objForPost = `-n ${this.newAccount.account_name} -d ${this.newAccount.start_deposit} -c ${this.newAccount.client_name} -b ${this.newAccount.ib_account_number} -l ${this.newAccount.leverage} -e ${this.newAccount.description} -r ${this.newAccount.rebalance_model} -m "${this.newAccount.model_settings}"`;
    } else if (this.newAccount.target_weights !== "{}") {
      this.objForPost = `-n ${this.newAccount.account_name} -d ${this.newAccount.start_deposit} -c ${this.newAccount.client_name} -b ${this.newAccount.ib_account_number} -l ${this.newAccount.leverage} -e ${this.newAccount.description} -r ${this.newAccount.rebalance_model} -s ${this.newAccount.selected_systems} -w "${this.newAccount.target_weights}"`;
    } else if (this.newAccount.model_settings !== "{}" && this.newAccount.target_weights !== "{}") {
      this.objForPost = `-n ${this.newAccount.account_name} -d ${this.newAccount.start_deposit} -c ${this.newAccount.client_name} -b ${this.newAccount.ib_account_number} -l ${this.newAccount.leverage} -e ${this.newAccount.description} -r ${this.newAccount.rebalance_model} -s ${this.newAccount.selected_systems} -w "${this.newAccount.target_weights}" -m "${this.newAccount.model_settings}"`;
    }

Я только учусь, но мне кажется что выглядит не очень.

Comments

[Сергей Сергей]

выделить безусловно попадающее в результат,
по условиям прибавлять к результату соответствующие части строк,
выделить переменные для избавления от повторения написания свойств через точки.

Answer 1

[Сергей Соколов]

const map = {
  n: "account_name",
  d: "start_deposit",
  c: "client_name",
  b: "ib_account_number",
  l: "leverage",
  e: "description",
  r: "rebalance_model",
  m: "model_settings",
  s: "selected_systems",
  w: "target_weights",
}

let key = "ndcbler";

const hasSettings = this.newAccount.model_settings !== "{}";
const hasWeights = this.newAccount.target_weights !== "{}";

if (hasSettings && hasWeights) {
  key += "swm";
} else if (hasSettings) {
  key += "m";
} else if (hasWeights) {
  key += "sw";
// } else { // что делаем, если ни того ни того нет?
//   key = "";
}

this.objForPost = key.split("")
  .map((key) => `-${key} ${this.newAccount[map[key]]}`)
  .join(" ");

Comments

[Alex10214]

Спасибо, это тоже интересное решение. Щас буду все разбирать. Еще раз спасибо!

[Сергей Соколов]

Alex10214, чуть обновил - там и с логикой вопросы: до второго else-if не дошло бы никогда.

[Alex10214]

Сергей Соколов, спасибо. Я же сейчас пытаюсь сделать это так:

const t = Object.values(this.newAccount);


    for (const el in t) {
      if (t[el] !== "{}" && t[el] !== "[]" && t[el] !== []) {
        console.log(t[el]);
      }
    } ...

Answer 2

[Сергей delphinpro]

const с = `-n ${this.newAccount.account_name}`
       + ` -d ${this.newAccount.start_deposit}`
       + ` -c ${this.newAccount.client_name}`
       + ` -b ${this.newAccount.ib_account_number}`
       + ` -l ${this.newAccount.leverage}`
       + ` -e ${this.newAccount.description}`
       + ` -r ${this.newAccount.rebalance_model}`;

if (this.newAccount.model_settings !== "{}") {
  this.objForPost = с
      + ` -m "${this.newAccount.model_settings}"`;
} else if (this.newAccount.target_weights !== "{}") {
  this.objForPost = c
      + ` -s ${this.newAccount.selected_systems}`
      + ` -w "${this.newAccount.target_weights}"`;
} else if (this.newAccount.model_settings !== "{}"
        && this.newAccount.target_weights !== "{}"
) {
  this.objForPost = с
      + ` -s ${this.newAccount.selected_systems}`
      + ` -w "${this.newAccount.target_weights}"`
      + ` -m "${this.newAccount.model_settings}"`;
}

let {
  account_name,
  start_deposit,
  client_name,
  ib_account_number,
  leverage,
  description,
  rebalance_model,
  model_settings,
  selected_systems,
  target_weights,
} = this.newAccount;

const с = `-n ${account_name}`
       + ` -d ${start_deposit}`
       + ` -c ${client_name}`
       + ` -b ${ib_account_number}`
       + ` -l ${leverage}`
       + ` -e ${description}`
       + ` -r ${rebalance_model}`;

if (model_settings !== "{}") {
  this.objForPost = с + ` -m "${model_settings}"`;
} else if (target_weights !== "{}") {
  this.objForPost = c + ` -s ${selected_systems} -w "${target_weights}"`;
} else if (model_settings !== "{}" && target_weights !== "{}") {
  this.objForPost = с + ` -s ${selected_systems} -w "${target_weights}" -m "${model_settings}"`;
}

Comments

[Alex10214]

Спасибо, очень интересный вариант. Я же сейчас пытаюсь сделать так:

const t = Object.values(this.newAccount);


    for (const el in t) {
      if (t[el] !== "{}" && t[el] !== "[]" && t[el] !== []) {
        console.log(t[el]);
      }
    }

[Сергей Соколов]

если есть model_settings, уже точно не дойдёт до model_settings && target_weights

[Alex10214]

Сергей Соколов, Вы правы, еще раз спасибо!)