A better way to say this would be

```python
shirt_desc = dict(SHIRT_OPTS)[self.shirt]
shirt_color_desc = dict(SHIRT_COLOR_OPTS)[self.shirt_color]
```

which is better because it doesn't rely on the implementation detail that the shirt options are integers starting with 0, which isn't generally true for other option fields.

Although an even better way of saying this would be to use the built-in Django ``get_FIELD_display`` method, e.g.

```python
shirt_desc = self.get_shirt_display()
shirt_color_desc = self.get_shirt_color_display()
```

which will do the same thing.