Fresco内部诟病引起的初次从网络加载PNG图片失败

如题描述，这个问题在项目中存在已久，今天由于自己的功能在首页，初次启动的体验极其糟糕，所以硬下头皮把这个问题解决了。



先来描述一下怎么样一个差的体验吧。就是当我第一次加载网络PNG（当时并不知道是由于这个原因引起的）的时候，就一直显示的是默认的图片，本来按照Fresco的机制是，先显示默认图片，然后去网络加载图片，当图片下载下来就会替换掉原来的默认图片。

感谢https://github.com/facebook/fresco/issues/1803这篇文章让我有了思路。

其实在此之前我已经定位到错误了，我在加载图片的回调里面打了日志，发现没加载出来的图片都走了onFailure方法，抛出了一个异常

IllegalArgumentException Problem decoding into existing bitmap

这个异常是在BitmapFactory.java里面抛出来的，有四个地方抛出，分别是不同的decodeResource，decodeByteArray，decodeStream，decodeFileDescriptor这几个方法抛出的，我们看看其中一个decodeStream吧

public static Bitmap decodeStream(InputStream is, Rect outPadding, Options opts) {
// we don't throw in this case, thus allowing the caller to only check
// the cache, and not force the image to be decoded.
if (is == null) {
return null;
}

Bitmap bm = null;

Trace.traceBegin(Trace.TRACE_TAG_GRAPHICS, "decodeBitmap");
try {
if (is instanceof AssetManager.AssetInputStream) {
final long asset = ((AssetManager.AssetInputStream) is).getNativeAsset();
bm = nativeDecodeAsset(asset, outPadding, opts);
} else {
bm = decodeStreamInternal(is, outPadding, opts);
}

if (bm == null && opts != null && opts.inBitmap != null) {
throw new IllegalArgumentException("Problem decoding into existing bitmap");
}

setDensityFromOptions(bm, opts);
} finally {
Trace.traceEnd(Trace.TRACE_TAG_GRAPHICS);
}

return bm;
}

可以看到是在转换bitmap的时候抛出来的。

结合那篇文章可以得出结论是，Fresco在网络加载PNG的时候有一个BUG导致流解析为bitmap的时候抛出异常。

那就解决问题吧。这边我说一下我的解决思路。

第一种方案，就是链接里面说的修改Fresco的源码。可以翻到1.3.0版本的432行，我这里给你找出来了。

public class DecodeProducer implements Producer<CloseableReference<CloseableImage>> {
//......
@Override
protected synchronized boolean updateDecodeJob(EncodedImage encodedImage, boolean isLast) {
boolean ret = super.updateDecodeJob(encodedImage, isLast);
if (!isLast && EncodedImage.isValid(encodedImage) &&
encodedImage.getImageFormat() == DefaultImageFormats.JPEG) {
if (!mProgressiveJpegParser.parseMoreData(encodedImage)) {
return false;
}
int scanNum = mProgressiveJpegParser.getBestScanNumber();
if (scanNum <= mLastScheduledScanNumber) {
// We have already decoded this scan, no need to do so again
return false;
}
if (scanNum < mProgressiveJpegConfig.getNextScanNumberToDecode(mLastScheduledScanNumber)
&& !mProgressiveJpegParser.isEndMarkerRead()) {
// We have not reached the minimum scan set by the configuration and there
// are still more scans to be read (the end marker is not reached)
return false;
}
mLastScheduledScanNumber = scanNum;
}
return ret;
}
//......
}

把下面这个if里面加上 &&encodedImage.getImageFormat() == DefaultImageFormats.PNG

if (scanNum < mProgressiveJpegConfig.getNextScanNumberToDecode(mLastScheduledScanNumber)
&& !mProgressiveJpegParser.isEndMarkerRead()) {
// We have not reached the minimum scan set by the configuration and there
// are still more scans to be read (the end marker is not reached)
return false;
}

由于现在已经是1.8.0版本的Fresco了，你可以找一下这个类定位到这个地方改。

第二种方案

用反射修改第一个方案里面的这个方法，这可以不修改Fresco的源码

第三种方案，我就是用这个方案的

由于我们App的图片是存在七牛的所以只要在你的Url的后面加上

"?imageView2/1/format/jpg"

把png通过七牛转成jpg，这样就完美解决了。

不过不可能整个app都是用jpg格式的图片，所以可能还是得需要上面两个方案里面的其中一个。如果你有更好的方案，请告诉我一下哈。开源万岁。