I wrote some code that resizes an image to fit in a given box. I think that some parts of the code are not needed (for example if it's a box image). Is there a more efficient way to do this? Is there something wrong with my code (from the results it looks ok)? Is there a shorter version to do this?Bitmap original, resizedImage;try{    using (FileStream fs = new System.IO.FileStream(imageLabel.Text, System.IO.FileMode.Open))    {        original = new Bitmap(fs);    }    int rectHeight = BOXHEIGHT;    int rectWidth = BOXWIDTH;    //if the image is squared set it's height and width to the smallest of the desired dimensions (our box). In the current example rectHeight<rectWidth    if (original.Height == original.Width)    {        resizedImage = new Bitmap(original, rectHeight, rectHeight);    }    else    {        //calculate aspect ratio        float aspect = original.Width / (float)original.Height;        int newWidth, newHeight;        //calculate new dimensions based on aspect ratio        newWidth = (int)(rectWidth * aspect);        newHeight = (int)(newWidth / aspect);        //if one of the two dimensions exceed the box dimensions        if (newWidth > rectWidth || newHeight > rectHeight)        {            //depending on which of the two exceeds the box dimensions set it as the box dimension and calculate the other one based on the aspect ratio            if (newWidth > newHeight)            {                newWidth = rectWidth;                newHeight = (int)(newWidth / aspect);            }            else            {                newHeight = rectHeight;                newWidth = (int)(newHeight * aspect);            }        }        resizedImage = new Bitmap(original, newWidth, newHeight);    }}catch (Exception ex){    MessageBox.Show(ex.Message);}
Resizing image but keeping aspect ratio
c#;image
Your algorithm to calculate the final image dimensions could be improved by simplifying the logical process that you use to determine the final size.Fundamentally, there are two dimensions you are interested in, the height, and width of the final image. There are two possible outcomes for the scaling, one where the scaling produces the target height, and the other where it produces the target width. The scaling that is used is the one which produces the smallest image. The size of the final image is determined by the scaling factor, and there are two factors:float scaleHeight = (float)BOXHEIGHT / (float)original.Height;float scaleWidth = (float)BOXWIDTH / (float)original.Width;Now, we have two scaling ratios, which one produces the smaller image? The one that has the smallest scaling factor.float scale = Math.Min(scaleHeight, scaleWidth);Now you can simply do the adjustment:resizedImage = new Bitmap(original,         (int)(original.Width * scale), (int)(original.Height * scale));