Реагировать - при сортировке элементы не на месте - PullRequest
1 голос
/ 16 июня 2020

Я учусь реагировать и решил попробовать создать визуализатор сортировки. Я начал с пузырьковой сортировки и в значительной степени преуспел в создании визуализатора basi c. Я добавил возможность изменять скорость сортировки и длину сортируемого массива. Я тестировал его несколько раз и обнаружил, что иногда при отображении отсортированного массива некоторые элементы не на своих местах. Это можно увидеть, если у вас высокий элемент не на том месте, где он должен быть (сам массив за сценой отсортирован правильно). Итак, что-то происходит с элементами дисплея, и я не знаю что. После завершения сортировки и если некоторые элементы не на своих местах, если я внезапно изменю скорость, элементы вернутся туда, где они должны быть. Я предполагаю, что это потому, что speed является частью состояния, и происходит повторное изменение. Что мне делать, чтобы это исправить?

Вот мой код:

import React, { useContext, useState, useEffect } from 'react';

const NUMBER_OF_ELEMENTS = 10;

const DEFAULT_COLOR = 'white';
const COMPARE_COLOR = 'darkred';
const DONE_COLOR = 'green';

const SPEED = 4;

const SPEEDS = [1, 5, 10, 25, 50, 100, 150, 200, 250, 300];

const randomIntFromInterval = (min, max) => {
    return Math.floor(Math.random() * (max - min + 1) + min);
}

const Dummy = () => {
    const [arr, setArr] = useState([]);
    const [numberOfElements, setNumberOfElements] = useState(NUMBER_OF_ELEMENTS);
    const [speed, setSpeed] = useState(SPEED);

    const timeout_id = [];

    useEffect(() => {
        generateArray();
    }, [numberOfElements]);

    const reset = () => {
        resetColors();
        generateArray();
    }

    const generateArray = () => {
        const arr1 = [];
        for(let i = 0; i < numberOfElements; i++)
        {
            arr1[i] = randomIntFromInterval(5, 100);
        }
        console.log(arr1);
        setArr(arr1);
    }

    const resetColors = () => {
        const arrayBars = document.getElementsByClassName('array-bar');
        for(let i = 0; i < arrayBars.length; i++) {
            arrayBars[i].style.backgroundColor = DEFAULT_COLOR;
        }
    }

    const bubbleSort = (arr, n) => {
        let i, j, temp, swapped, delay = 1;
        for(i = 0; i < n - 1; i++) 
        {
            swapped = false;
            for(j = 0; j < n - i - 1; j++) 
            {
                createColor([j, j + 1], COMPARE_COLOR,  delay++);
                if(arr[j] > arr[j + 1]) 
                {
                    // swap arr[j] and arr[j+1] 
                    temp = arr[j];
                    arr[j] = arr[j + 1];
                    arr[j + 1] = temp;
                    swapped = true;
                    createAnimation(j, j + 1, delay++);
                }
                createColor([j, j + 1], DEFAULT_COLOR, delay++);
            }
            createColor([n - i - 1], DONE_COLOR, delay++);

            // If no two elements were  
            // swapped by inner loop, then break 
            if(swapped === false) break;
        }

        const leftovers = [];
        for(let k = 0; k < n - i - 1; k++) {
            leftovers.push(k);
        }

        createColor(leftovers, DONE_COLOR, delay++);
    }

    const createAnimation = (one, two, delay) => {
        const arrayBars = document.getElementsByClassName('array-bar');
        const id = setTimeout(() => {
            const barOneHeight = arrayBars[one].style.height;
            const barTwoHeight = arrayBars[two].style.height;
            arrayBars[two].style.height = `${barOneHeight}`;
            arrayBars[one].style.height = `${barTwoHeight}`;
        }, SPEEDS[speed - 1] * delay);
        timeout_id.push(id);
    }

    const createColor = (indexes, color, delay) => {
        const arrayBars = document.getElementsByClassName('array-bar');
        const id = setTimeout(() => {
            for(let i = 0; i < indexes.length; i++) {
                arrayBars[indexes[i]].style.backgroundColor = color;
            }
        }, SPEEDS[speed - 1] * delay);
        timeout_id.push(id);
    }

    const handleSort = (arr) => {
        bubbleSort(arr, arr.length);
    }

    const handlerRange = (e) => {
        setNumberOfElements(e.target.value);
    }

    const stopTimeOuts =() => {
        for(let i = 0; i < timeout_id.length; i++) {
            clearTimeout(timeout_id[i]);
        }
    }

    const handleSpeed = (e) => {
        setSpeed(e.target.value);
    }

    const maxVal = Math.max(...arr);

    return (
        <div>
            <div className="array-container" style={{height: '50%', backgroundColor: 'black'}}>
                {arr.map((value, idx) => (
                    <div className="array-bar"
                         key={idx}
                         style={{
                            backgroundColor: DEFAULT_COLOR,
                            height: `${(value * 100 / maxVal).toFixed()}%`,
                            width: `${85 / arr.length}%`,
                            display: 'inline-block',
                            margin: '0 1px'
                         }}>
                    </div>
                ))}
            </div>

            <div className="buttons-container">
                <button onClick={() => handleSort(arr)}>Sort!</button>
                <button onClick={() => reset()}>Reset</button>
                <button onClick={() => stopTimeOuts()}>Stop!</button>
            </div>

            number of elements: {numberOfElements}
            <div className="slider-container">
                1
                <input type="range" 
                       min="1" 
                       max="100" 
                       onChange={(e) => handlerRange(e)} 
                       className="slider" 
                       id="myRange" 
                />
                100
            </div>

            speed: {speed}
            <div className="slider-container">
                1
                <input type="range" 
                       min="1" 
                       max="10" 
                       onChange={(e) => handleSpeed(e)} 
                       className="slider" 
                       id="myRange" 
                />
                10
            </div>

        </div>
    );
}

export default Dummy;

1 Ответ

1 голос
/ 16 июня 2020

Установите ключ внутри div как уникальный идентификатор вместо idx - в этом случае вы можете использовать value:

/* -- snip -- */
<div className="array-bar"
     key={value}
/* -- snip -- */

Это прекратит реагировать на переработку каждого div с их соответствующими индексы и вместо этого повторно отрисовывают и меняют порядок на основе нового массива.

Причина, по которой вы видите неожиданное поведение, заключается в том, что response использует ключи для идентификации каждого элемента. Итак, если вы отсортируете свой массив, а затем примените индекс в качестве идентификатора при рендеринге, react получит элемент, который был сначала визуализирован с этим индексом в качестве ключа, и поместит его в это место. Путем изменения ключа на a уникальный идентификатор, response не смешивает элементы (поскольку уникальный идентификатор никогда не изменяется по отношению к его элементу) и теперь может точно отображать каждый элемент в том порядке, который вы намереваетесь.

Пример: Мы отображаем список:

<div id=1>Foo</div> // Id "1" is now bound to this element
<div id=2>Bar</div>

Итак, если вы затем переупорядочите список следующим образом (обратите внимание на изменения идентификатора):

<div id=1>Bar</div> // this will be transformed to <div id=1>Foo</div>
<div id=2>Foo</div> // this will be transformed to <div id=2>Bar</div>

Тогда реакция преобразует элементы в любой идентификатор, которому они были назначены первым - Это Вот почему так важно иметь уникальных идентификаторов . Вы можете сгенерировать уникальный идентификатор с помощью библиотеки, которая генерирует uuids или другие достаточно случайные строки.

Вы можете прочитать больше здесь: https://medium.com/@robinpokorny / index-as-a-key-is-an-anti -pattern-e0349aece318

Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...