Chapter 19 - draw tool

[hkiame]

I find the drawPixel function and draw method in Picture class having extra code or probably I am missing something. The drawPixel function creates an object drawn which is later placed in an array passed as an argument to state.picture.draw method. Would it not be easy to just pass x, y and color as arguments and avoid creating the drawn object, placing it in an array and also avoiding the loop and object destructuring in state.picture.draw method. Below is what I suggest.

    function drawPixel({x, y}, state){
        dispatch({picture: state.picture.draw(x, y, color)});
    }

draw method in Picture class

    draw(x, y, color){
        let copy = this.pixel.slice();
        copy[x + y * this.width] = color;

        return new Picture(this.width, this.height, copy);
    }

[marijnh]

There's other code also calling draw, with arrays of changed pixels.