Pensando imediatamente, sua solução parece ser bem melhor. Porém, a longo prazo, se eu entendi bem, ela vai gerar problemas de manutenibilidade.

Você disse para colocarmos em um array os valores dos bônus. Isso implica na reprodução do mesmo efeito em várias armas diferentes, um +5 em cada. Porém, quando tivermos que balancear esse efeito para, por exemplo, +6, teremos que ir na mão em todos os objetos e alterar tal valor, já que ele não está centralizado em uma classe específica.

Além disso, vc diz para que um método em TArmaItem avalie os efeitos possíveis de TArmaItem. Isso também parece o mais simples a ser feito, porém torna-se complicado trabalhar com essa lógica em um jogo como o World of Warcraft (do meu exemplo) onde temos mais de 200 efeitos possíveis em um item. Teríamos um método gigantesco com um switch case que gastaria processamento e memória para descobrir, por exemplo, que somente 1 desses efeitos estava ativo no seu item.

Também há o cenário onde tal método fosse obrigado a testar vários bônus que não fossem específicos daquela arma.

O Decorator resolve esses problemas de uma forma prática, extremamente compacta e manutenível por dar a um objeto novas responsabilidades em tempo de execução e não em tempo de desenvolvimento. O código base nunca muda, independente de quantos efeitos uma arma venha a ter, levando também em conta dela não ter efeito algum.

Espero que tenha endendido a vantagem do padrão. Para mais informações, não deixe de ler também o artigo sobre ele na wikipédia: http://en.wikipedia.org/wiki/Decorator_pattern

Parabéns.