Variáveis locais vs globais e loop

Olá, fiz meu codigo de forma um pouco diferente e acabei ficando com algumas duvidas.

O resultado final é o mesmo, porém, gostaria de saber se o loop que utilizei está correto ou poderia encontrar algum problema caso o programa fosse ficar mais complicado.

Outra coisa que percebi comparando com o codigo do professor é que eu usei variáveis globais em vez de locais e gostaria se esta correto ou acabei dando sorte.

    var tela = document.querySelector("canvas");
    var pincel = tela.getContext("2d");

    function desenhaTexto (texto, x, y) {
        pincel.font = "20px Georgia";
        pincel.fillStyle = "black";
        pincel.fillText(texto, x, y);
    }

    desenhaTexto("Qual é a fração?", 0, 30);

    function desenhaQuadrado (x, y, tamanho, cor) {
        pincel.fillStyle = cor;
        pincel.fillRect(x, y, tamanho, tamanho);
        pincel.strokeStyle = "black";
        pincel.strokeRect(x, y, tamanho, tamanho);
    }

    function repeteQuadrado (x, y) {
        for (x = 0; x <= 150; x += 50) {
            desenhaQuadrado(x, y, 50, "green");

            if (x == 150) {
                desenhaQuadrado(x, y, 50, "white");
            }
        }
    }

    repeteQuadrado(0, 50);