Tem muitas maneiras de fazer isso. Não acho a sua errada, é bem clara e didática. Mas tem maneiras mais curtas, como a do Mr Felix. Convertendo pra array dá para fazer uns truques também, por exemplo:

    var numbers = '1111121111';
    numbers.split('').every(function(num, i, arr) { return num == arr[0] }); // false

Ou ainda mais curto em ES-2015:

    const numbers = '1111121111';
    [...numbers].every( (num, i, arr) => num == arr[0] ); // false