Como o JavaScript é uma linguagem em que não é preciso especificar o tipo do dado que você espera receber no argumento passado para a função (nesse caso você chamou o argumento de callback
) o código que usa esse argumento precisa fazer essa checagem ou então pode ocorrer um erro quando ele estiver utilizando esse argumento.
No seu código, você espera que o argumento callback
seja um variável do tipo função e essa função receba 3 argumentos: o elemento do array, o índice do array e o próprio array. O código define a assinatura dessa forma porque provavelmente ele foi baseado no código do método forEach
.
Por que está fazendo dessa forma?
Porque você espera que o usuário dessa função forEach2
a utilize da seguinte forma: umArrayQualquer.forEach2(function (v, i, arr) { ... });
Assim, quando for utilizado, a função será passada no argumento callback
e internamente vai fazer a chamada para a função corretamente.
O JavaScript é esperto o suficiente para que o usuário da sua função possa ter uma assinatura mais simples como umArrayQualquer.forEach2(function (v) { ... });
e funcionar sem erro o seu código que faz a chamada, mas imagine que o usuário da sua função não utilize dessa forma. Se ele fizer a chamada assim: umArrayQualquer.forEach2(123);
. Ao invés de passar uma função que espera 3 argumentos, passou um número. Isso vai causar um erro no seu código.
Array.prototype.forEach2 = function (callback) {
for (let i = 0; i < this.length; i++) {
callback(this[i], i, this)
}
}
var umArrayQualquer = [1, 3, 4];
console.log("com função completa");
umArrayQualquer.forEach2(function (v, i, arr) {
console.log(v);
});
console.log("com função simplificada");
umArrayQualquer.forEach2(function (v) {
console.log(v);
});
console.log("com argumento errado");
umArrayQualquer.forEach2(123);
Por esse motivo, sempre que você está estendendo uma classe é importante fazer checagens dentro do código para garantir que o usuário da sua função não provoque um erro imprevisto.
Array.prototype.forEach2 = function (callback) {
if (typeof callback === 'function')
for (let i = 0; i < this.length; i++) {
callback(this[i], i, this)
}
}
var umArrayQualquer = [1, 3, 4];
console.log("com função simplificada");
umArrayQualquer.forEach2(function (v) {
console.log(v);
});
console.log("com argumento errado");
umArrayQualquer.forEach2(123);
É claro que eu coloquei um exemplo bem extremo e como provavelmente você mesmo faria uso da sua extensão, não cometeria esse tipo de erro ao usar, mas coloquei aqui apenas para ilustrar que é importante você fazer checagens para evitar erros por conta dessa falta de tipagem do JavaScript.
Você precisaria fazer sempre dessa forma, passando os mesmos valores que o forEach
recebe? Não. Como você está estendendo a classe Array você pode definir os argumentos da forma que precisa para atender a sua necessidade. Se você estiver estendendo porque seu código vai ser usado por outras pessoas, então é interessante manter o padrão porque o usuário do código vai esperar que uma versão diferente de um forEach
tenha características semelhantes de uso.
O nome que você dá para a sua função também é importante. Se você chamar de forEach2
é uma boa dica para quem for usar entender que é semelhante a um forEach
, e se você implementa com algo completamente diferente estará atrapalhando quem vai usar a função. Então uma dica é sempre que for estender uma classe existente procure estudar a classe primeiro, quais métodos ela tem e como funcionam, principalmente para você não reinventar a roda.