Rectangle.Offset() doesn't work with foreach() (bug?)

Graphics
1

Hello guys,
I have a problem (or I found a bug). I have this property:

public Rectangle BoundingBox
{
get
{
Rectangle spriteBounds = {
new Rectangle((int)(8* canScale), (int)(3* canScale), (int)(47* canScale), (int)(15* canScale)),
new Rectangle((int)(2* canScale), (int)(18* canScale), (int)(58* canScale), (int)(44* canScale)),
new Rectangle((int)(12* canScale), (int)(60* canScale), (int)(40* canScale), (int)(12* canScale)),
new Rectangle((int)(28* canScale), (int)(70* canScale), (int)(10* canScale), (int)(26* canScale)),
new Rectangle((int)(8* canScale), (int)(95* canScale), (int)(49* canScale), (int)(76* canScale))};
foreach (Rectangle rect in spriteBounds)
** {**
** rect.Offset(canPosition - canOrigin * canScale);**
** }**
return spriteBounds;
}
and the Offset() operation doesn’t work. But, if I try this:

public Rectangle BoundingBox
{
get
{
Rectangle spriteBounds = {
new Rectangle((int)(8* canScale), (int)(3* canScale), (int)(47* canScale), (int)(15* canScale)),
new Rectangle((int)(2* canScale), (int)(18* canScale), (int)(58* canScale), (int)(44* canScale)),
new Rectangle((int)(12* canScale), (int)(60* canScale), (int)(40* canScale), (int)(12* canScale)),
new Rectangle((int)(28* canScale), (int)(70* canScale), (int)(10* canScale), (int)(26* canScale)),
new Rectangle((int)(8* canScale), (int)(95* canScale), (int)(49* canScale), (int)(76* canScale))};
spriteBounds[0].Offset(canPosition - canOrigin * canScale);
** spriteBounds[1].Offset(canPosition - canOrigin * canScale);**
** spriteBounds[2].Offset(canPosition - canOrigin * canScale);**
** spriteBounds[3].Offset(canPosition - canOrigin * canScale);**
** spriteBounds[4].Offset(canPosition - canOrigin * canScale);**
return spriteBounds;
}
}
the Offset() operation works fine.

2

This is a simple misunderstanding of C# variable scope. Rectangle is a struct and therefore passes by value (not by reference, like a class). So, your foreach rect only exists within the scope of the foreach statement itself. Solution: foreach (ref Rectangle rect in spriteBounds)

3

Thank a lot.