Android code to rotate an image based on exif information

ImageRotator.java

// imports
public class ImageRotator {

        public static Bitmap rotateImage(String path) throws IOException {
            Bitmap bitmap = BitmapFactory.decodeFile(path);
            return rotateImage(bitmap);
        }

        public static Bitmap rotateImage(Bitmap bitmap) throws IOException {
            int rotate = 0;
            ExifInterface exif;
            exif = new ExifInterface(path);
            int orientation = exif.getAttributeInt(ExifInterface.TAG_ORIENTATION,
                    ExifInterface.ORIENTATION_NORMAL);
            switch (orientation) {
                case ExifInterface.ORIENTATION_ROTATE_270:
                    rotate = 270;
                    break;
                case ExifInterface.ORIENTATION_ROTATE_180:
                    rotate = 180;
                    break;
                case ExifInterface.ORIENTATION_ROTATE_90:
                    rotate = 90;
                    break;
            }
            Matrix matrix = new Matrix();
            matrix.postRotate(rotate);
            return Bitmap.createBitmap(bitmap, 0, 0, bitmap.getWidth(),
                    bitmap.getHeight(), matrix, true);
        }
}

what is path in new ExifInterface(path);

@pikaboo that is the String of the file's path

public static Bitmap rotateImage(String path) throws IOException {
        Bitmap bitmap = BitmapFactory.decodeFile(path);
        return rotateImage(bitmap, path);
    }

 public static Bitmap rotateImage(Bitmap bitmap, String path) throws IOException {
        int rotate = 0;
        ExifInterface exif;
        exif = new ExifInterface(path);
............

thanks

Wouldn't it be simpler to to get rid of the first rotateImage() and check if the bitmap object in the second rotateImage() is valid like so?

/* 🦀 rotateImage(String path) is dead 🦀 */

public static Bitmap rotateImage(Bitmap bitmap, String path) throws IOException {

        if(bitmap == null){ //if bitmap in param is null, use the path and put something in it!
            bitmap = BitmapFactory.decodeFile(path);
        }

        int rotate = 0;
        [...]
}

That would work I suppose. The intention of having both functions was for the code to be explicit about the input the function is taking (whether it's bitmap or path - we cannot consume both). This separation of functions makes it more readable. Combining into one function works, but you lose that intuitive ability of a consumer to understand that we need either bitmap or path but not both