Почему это плохая идея переопределить собственные методы?

Question

Я задал вопрос, где я был переопределяющим push, используя Object.defineProperty. Оригинальный вопрос: here. Код, над которым я работал, находится на my codepen.

Пользователь оставил следующий комментарий:

Это плохая идея, чтобы перезаписать нативный метод, как на прототипе или на экземпляры объектов

Почему?

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

Неправильно ли делать то, что я сделал, и если да, то в чем альтернатива?

Answer 1

Одна из проблем, о которых я могу думать, заключается в том, что теперь у вас есть метод push, который очень специфичен для вашей проблемы, и что если вам понадобится обычный метод push. Вы действительно не знаете, будет ли это так или нет, но это может привести к необходимости реорганизации вашего кода позже, чтобы обеспечить как «нормальный толчок», так и «измененный толчок».

Еще одна проблема, о которой я могу думать, заключается в том, что если вы делаете что-то подобное, и вы не единственный, кто работает над кодом, кто-то другой может не знать, что push переопределен и больше не соответствует стандартным поведением.

Это может вызвать потенциальные проблемы. Возможно, вы не сильно изменили его, но в случае, если внесенные вами изменения вызывают ошибку, вы можете оставить кого-то, кто почесывает голову, задаваясь вопросом, почему (по его мнению, обычный) толчок вызывает странное поведение.