Какие недостатки в этом коротком скрипте на jQuery?

$(this) плохо использовать потому что я не уверен в том, что this будет правильным, а именно - ссылкой.

Какие недостатки в этом коротком скрипте на jQuery?

On лучше использовать потому что вторым параметром можно передать селектор элементов - при использовании click у вас не будет такой возможности.