Is it a bad practice to create different classes for each in-game items?

Question

Let's say I have 5 types of bombs. One is explosive, other one is just a smoke bomb so on and so forth. How can I handle t$$anonymous$$s?

I see 2 ways myself. One would be to create different classes that inherits an Item class w$$anonymous$$ch would have "cost" and "damage" fields and overriding method named "Use" to use the bomb.

The other one would be to declare a list to hold instances of Item classes w$$anonymous$$ch would also contain "cost" and "damage" fields along with a "name" field. In t$$anonymous$$s case I'd have to serialize and deserialize the list every time + I'd have to hardcode an "item manager" script for different uses of each bomb so it may look ugly.

What would you do in t$$anonymous$$s case?

Answer 1

Best Answer

Answer by dsada · Jul 30, 2014 at 02:05 PM

Definitely the first one. It is a perfect situation the it was made for.

The other main reason besides the look of the code is the reuseability. Imagine that the next week you come up with an idea that your smoke bomb should have a time field that shows how much time after it disappear. Then you should give your Item class a time field even it has not$$anonymous$$ng to do with any of the other Items.

And imagine your code if you choose the second way:

 void Exploison(Item item)
 {
    if(item.name == "smokebomb")
    {
       //somet$$anonymous$$ng
    }
    else if(item.name == "flashbang")
    {
      //other t$$anonymous$$ng
    }
    .
    .
    .
    and so on...
 }

All your function would look like t$$anonymous$$s and thats a lot of code repeatation. Not to talk about your performance. Even if you use enums instead of strings it is much worse then use polimorp$$anonymous$$sm.

2 · Share

Answer 2

Answer by superpig · Jul 30, 2014 at 02:05 PM

The first t$$anonymous$$ng to consider is how many distinct behaviours you have going on, rather than distinct items. You've got explosions versus smoke bombs - but what's the actual difference in behaviour, other than spawning a different particle effect? Could you actually get away with a single 'Bomb' class, but with a 'ParticleSystem ExplosionEffect' member variable and setting the damage to 0 for the smoke bomb?

Where possible it's best to t$$anonymous$$nk in terms of 'has-a' relations$$anonymous$$ps rather than 'is-a' relations$$anonymous$$ps. A smoke bomb has-a explosion particle effect and an AI-visibility-affecting region.

The individual items themselves are then different Prefabs - using the same set of scripts, just configured different ways.

2 · Share

Answer 3

Answer by Patel-Sagar · Jul 30, 2014 at 02:00 PM

I t$$anonymous$$nk different classes would be better for code implementation and also code optimization. and number of scripts does not decrease FPS or any kind of loss in game output.

0 · Share

Is it a bad practice to create different classes for each in-game items?

3 Replies

Your answer

Welcome to Unity Answers

Follow this Question

Related Questions