Как сделать рефакторинг кода?

Question

[hollanditkzn]

Я делаю тест и и все хранится в д mysql.
В базе хранятся таблица темы, вопросов и ответы
Я получил массивы вопросы по определенной теме, массив ответов и массив правильных ответов.

const Test = require('./model/test');
router.get('/testing/:id', async ctx => {
        let answearArr = []; //Массив ответов на вопросы
        let answerCorrectArr = []; // массив правильных ответов
        let test = await Test.test(ctx.params.id); // получаем все вопросы по данной теме
        for(i=0; i < test.length; i++){
            let answear = test[i].id;
            answearArr.push(await Test.answear(answear)); //перевожу все в массив по выборке
        }
        for(i=0; i < answearArr.length; i++){
            let arr = answearArr[i]; //Получаю все ответы на один вопрос
            for(n=0; n < arr.length; n++){
                let arr1 = arr[n]; //Получаю один ответ и дальше делаю проверку на правильность ответа
                if(arr1.answer === 1){
                    answerCorrectArr.push(arr1.id); //если правильный ответ, то помечаем в массив
                }
            }
        }
    })

Как сделать рефакторинг

Comments

[Сергей Попов]

Проясню один момент: рефакторинг - перепроектирование кода с полным сохранением функциональности и улучшением читаемости/производительности/потребления памяти и так далее. То есть равносильное преобразование с улучшениями.
Ты хочешь, чтобы тебе преобразовали код?

[hollanditkzn]

Сергей Попов, Ну если можно так сказать, то да. Просто слишком много for мне кажется и будет ли это на производительность сказываться, когда будет много данных или нет я пока что не знаю. Просто кажется что можно как-то его сократить, но пока что не понял как. Поэтому я хочу совет получить примерно как можно его упростить

Answer 1

[kulaeff]

Такое за бесплатно вам вряд ли кто-то сделает, но пару советов могу дать. Коль вы используете for совместно с array.push, то лучше заменить это все на map(). Типа:

let
    test = await Test.test(ctx.params.id), // получаем все вопросы по данной теме
    answerArr = test.map(item => await Test.answear(item.id));

А еще используйте eslint.

Comments

[Bhudh]

Поскольку он использует for не просто с array.push, но ещё и по условию, тут и filter не помешает.

[hollanditkzn]

Такой вопрос, а как вернуть значение в этот массив, потому что test.map(item => await Test.answear(item.id));
Не получается выводит ошибку, перевел на

let answearArr = await test.map(item => {
            Test.answear(item.id);
        });

Но у меня выходит пустой массив, точнее 3 underfined и второй момент если return Test.answear(item.id);
То я промись получаю. Как то не получается map вывестинормально

[kulaeff]

Пардон, правильнее будет сделать так:

const answerArr = test.map(item => Test.answear(item.id));

Promise.all(answerAll).then(callback);

Т.е. вы создаете массив промисов, затем "параллельно" их запускаете с помощью all, и после того как они выполнятся, в callback у вас будет массив ответов.