Хорошо ли собирать массив объектов путём конкатинации строк и дальнейшего применения eval()?

Question

[Влад]

У меня база – backend, поэтому и интересует данный вопрос.

const arrayLabels = ['India', 'Indonesia', 'Russia', 'China', 'Bangladesh', 'Nepal', 'Bhutan', 'Myanmar', 'Laos', 'Cambodia', 'Vietnam', 'Malaysia'];
    const arrayData = [2467, 560, 540, 20, 114, 121, 103, 85, 17, 150, 20, 295];
    const arrayBackgroundColor = ['#55E6C1', '#3B3B98', '#F97F51', '#25CCF7', '#EAB543', '#CAD3C8', '#FEA47F', '#1B9CFC', '#F8EFBA', '#58B19F', '#2C3A47', '#B33771'];

    function dataSort(arrayLabels, arrayData, ...params) {
      console.log(`number of arguments: ${params.length}`);
      let arrayOfObjStr = '[';
      arrayLabels.forEach((label, i) => {
        arrayOfObjStr += `{labels: '${label}',\n` +
                          `data: '${arrayData[i] || 0}',\n`;
        params.forEach(function(param, a) {
          arrayOfObjStr += `arr${a}: '${param[i] || 0}',\n`;
        });
        arrayOfObjStr += '},\n';
      });
      arrayOfObjStr += ']';

      let arrayOfObj = eval(arrayOfObjStr).sort((a, b) => {
        return parseInt(b.data) > parseInt(a.data);
      });


    /* ... */
  }

Как-то не по джавоскриптовски этот код выглядит ))

Comments

[Simkav]

Можете показать что вы хотите получить на выходе? Если честно очень страшное что-то у вас тут

Answer 1

[Rsa97]

А что мешает сразу формировать объект без лишних преобразований строку и обратно?

function dataSort(arrayLabels, arrayData, ...params) {
  console.log(`number of arguments: ${params.length}`);
  const arrayOfObj = [];
  arrayLabels.forEach((label, i) => {
    const el = { labels: label, data: arrayData[i] || 0 };
    params.forEach(function(param, a) {
      el[`arr${a}`] = param[i] || 0;
    });
    arrayOfObj.push(el);
  });
  arrayOfObj = arrayOfObj.sort((a, b) => parseInt(b.data) > parseInt(a.data));
}

Comments

[Влад]

Спасибо, разобрался и всё понял – формируешь заранее и пушишь.
Единственное, у вас тут:

arrayOfObj = arrayOfObj.sort((a, b) => parseInt(b.data) > parseInt(a.data));

где `arrayOfObj = ` не нужно, так как выше он const :)

[WbICHA]

Спасибо, разобрался и всё понял – формируешь заранее и пушишь.

Влад, нет, это хрень, тут надо мап использовать.

И не вот это доисторическое произведение: function(param, a) {, а нормальную стрелочную функцию.

[Влад]

WbICHA, да это нюансы, важна идея. Ну пусть будет map, хотя это здесь отнюдь не принципиально IMHO.

Мне вообще нужно было эти дополнительные аргуметы (...args) сложить во вложенный объект, а не добавлять к свойствам базового объекта arrayOfObj.

Вот попрактиковался и вот что у меня получилось:

const arrayLabels = ['India', 'Indonesia', 'Russia', 'China', 'Bangladesh', 'Nepal', 'Bhutan', 'Myanmar', 'Laos', 'Cambodia', 'Vietnam', 'Malaysia'];
const arrayData = [2467, 560, 540, 20, 114, 121, 103, 85, 17, 150, 20, 295];
const arrayBackgroundColor = ['#55E6C1', '#3B3B98', '#F97F51', '#25CCF7', '#EAB543', '#CAD3C8', '#FEA47F', '#1B9CFC', '#F8EFBA', '#58B19F', '#2C3A47', '#B33771'];
const arrayOneMore = ['zzz', 'zzz', 'zzz', 'zzz', 'zzz', 'zzz', 'zzz', 'zzz', 'zzz', 'zzz', 'zzz', 'zzz'];
// etc.

function chartDataSort(arrayLabels, arrayData, ...args) {
      console.log(`number of argents: ${args.length}`);

      const arrayOfObj = [];
      arrayLabels.map((label, i) => {
        const el = { labels: label, data: arrayData[i] || 0 };
        let argsObj = {};
        args.map((arg, a) => {
          argsObj[a] = arg[i] || 0;
        });
        el.args = argsObj;
        arrayOfObj.push(el);
      });
      arrayOfObj.sort((a, b) => parseInt(b.data) > parseInt(a.data));
/* ... */

Не хватает практики расширенной работы с объектами/массивами, долго эти вещи рожаю.
Теперь буду вымучивать формирование отдельных массивов и их return из функции ))

[WbICHA]

Ну пусть будет map, хотя это здесь отнюдь не принципиально IMHO.

Влад, исходя из того, какой код у тебя получился, с ес6 у тебя всё плохо... Почитай как работает мап, потому что если ты подобное извращение где-нибудь продемонстрируешь в публичном месте, в лучшем случае тебя линчуют.)

Answer 2

[WbICHA]

Эвал... Зачем? То, что ты написал, нечитабельно, а уж ошибки исправлять вообще боль.

function dataSort(arrayLabels, arrayData, ...rawParams) {
    console.log(`number of arguments: ${params.length}`);
    
    const getParams = (pIndex) => rawParams.reduce((acc, p, i) => {
      acc[`arr${i}`] = p[pIndex] || '';
      return acc;
    }, {});
    
    const arrayOfObj = arrayLabels
      .map((label, i) => ({
        labels: label,
        data: arrayData[i],
        ...getParams(i),
      }))
      .sort((a, b) => b.data - a.data);
    
    /* ... */
  }

Comments

[WbICHA]

Нет, ошибся, не заметил 1 момент. Щас поправлю.

[WbICHA]

Поправил.