Skip to content

G
glide

沉迷打码的小凳子 / glide
与 Fork 源项目一致

从无法访问的项目Fork

通知 3
Star 0
Fork 0
G
glide

体验新版 GitCode，发现更多精彩内容 >>

提交 8c7ae0d1 编写于 作者: S Sam Judd
浏览文件
操作

Revamp Glide interface to be more flexible. 

Conflicts:
	library/src/com/bumptech/glide/presenter/target/ImageViewTarget.java
上级 896af010
展开全部 隐藏空白更改
内联 并排
Showing with 365 addition and 443 deletion
library/src/com/bumptech/glide/ListPreloader.java
浏览文件 @ 8c7ae0d1
...@@ -26,7 +26,6 @@ import java.util.Queue; ...@@ -26,7 +26,6 @@ import java.util.Queue;
*/ */
public abstract class ListPreloader<T> implements AbsListView.OnScrollListener { public abstract class ListPreloader<T> implements AbsListView.OnScrollListener {
private final int maxPreload; private final int maxPreload;
private final Context context;
private final PreloadTargetQueue preloadTargetQueue; private final PreloadTargetQueue preloadTargetQueue;
private int lastEnd; private int lastEnd;
...@@ -39,11 +38,9 @@ public abstract class ListPreloader<T> implements AbsListView.OnScrollListener { ...@@ -39,11 +38,9 @@ public abstract class ListPreloader<T> implements AbsListView.OnScrollListener {
/** /**
* Constructor for the preloader. * Constructor for the preloader.
* *
* @param context A context
* @param maxPreload The maximum number of items in the list to load ahead (corresponds to adapter positions). * @param maxPreload The maximum number of items in the list to load ahead (corresponds to adapter positions).
*/ */
public ListPreloader(Context context, int maxPreload) { public ListPreloader(int maxPreload) {
this.context = context;
this.maxPreload = maxPreload; this.maxPreload = maxPreload;
preloadTargetQueue = new PreloadTargetQueue(maxPreload); preloadTargetQueue = new PreloadTargetQueue(maxPreload);
} }
...@@ -134,7 +131,7 @@ public abstract class ListPreloader<T> implements AbsListView.OnScrollListener { ...@@ -134,7 +131,7 @@ public abstract class ListPreloader<T> implements AbsListView.OnScrollListener {
final T item = items.get(position); final T item = items.get(position);
final int[] dimensions = getDimensions(item); final int[] dimensions = getDimensions(item);
if (dimensions != null) { if (dimensions != null) {
getRequest(item).into(preloadTargetQueue.next(dimensions[0], dimensions[1])).with(context); getRequest(item).into(preloadTargetQueue.next(dimensions[0], dimensions[1]));
} }
} }
......
library/src/com/bumptech/glide/presenter/ImagePresenter.java
浏览文件 @ 8c7ae0d1
...@@ -9,13 +9,13 @@ import android.content.res.Resources; ...@@ -9,13 +9,13 @@ import android.content.res.Resources;
import android.graphics.Bitmap; import android.graphics.Bitmap;
import android.graphics.drawable.Drawable; import android.graphics.drawable.Drawable;
import android.util.Log; import android.util.Log;
import android.widget.ImageView;
import com.bumptech.glide.loader.bitmap.BitmapLoadFactory; import com.bumptech.glide.loader.bitmap.BitmapLoadFactory;
import com.bumptech.glide.loader.bitmap.transformation.TransformationLoader;
import com.bumptech.glide.loader.image.ImageLoader; import com.bumptech.glide.loader.image.ImageLoader;
import com.bumptech.glide.loader.bitmap.model.ModelLoader; import com.bumptech.glide.loader.bitmap.model.ModelLoader;
import com.bumptech.glide.presenter.target.ImageViewTarget;
import com.bumptech.glide.presenter.target.Target; import com.bumptech.glide.presenter.target.Target;
import com.bumptech.glide.resize.BitmapLoadTask; import com.bumptech.glide.resize.BitmapLoadTask;
import com.bumptech.glide.resize.load.Transformation;
/** /**
* Wraps an {@link Target} to display arbitrary Bitmaps and provides a framework for fetching and * Wraps an {@link Target} to display arbitrary Bitmaps and provides a framework for fetching and
...@@ -28,9 +28,8 @@ import com.bumptech.glide.resize.BitmapLoadTask; ...@@ -28,9 +28,8 @@ import com.bumptech.glide.resize.BitmapLoadTask;
* @param <T> The type of the model that contains information necessary to display an image. Can be as simple * @param <T> The type of the model that contains information necessary to display an image. Can be as simple
* as a String containing a path or a complex data type. * as a String containing a path or a complex data type.
*/ */
public class ImagePresenter<T> { public class ImagePresenter<T, Y extends Target> {
private static final String TAG = "ImagePresenter"; private static final String TAG = "ImagePresenter";
private final BitmapLoadFactory<T> loadFactory;
/** /**
* A builder for an {@link ImagePresenter}. * A builder for an {@link ImagePresenter}.
...@@ -43,10 +42,10 @@ public class ImagePresenter<T> { ...@@ -43,10 +42,10 @@ public class ImagePresenter<T> {
* @param <T> The type of the model that the presenter this builder will produce requires to load an image. * @param <T> The type of the model that the presenter this builder will produce requires to load an image.
*/ */
@SuppressWarnings("unused") @SuppressWarnings("unused")
public static class Builder<T> { public static class Builder<T, Y extends Target> {
private int placeholderResourceId; private int placeholderResourceId;
private Drawable placeholderDrawable; private Drawable placeholderDrawable;
private ImageReadyCallback<T> imageReadyCallback; private ImageReadyCallback<T, Y> imageReadyCallback;
private ImagePresenterCoordinator coordinator; private ImagePresenterCoordinator coordinator;
private ImageLoader imageLoader; private ImageLoader imageLoader;
private Context context; private Context context;
...@@ -61,15 +60,16 @@ public class ImagePresenter<T> { ...@@ -61,15 +60,16 @@ public class ImagePresenter<T> {
private int errorResourceId; private int errorResourceId;
private Drawable errorDrawable; private Drawable errorDrawable;
private Target target; private BitmapLoadFactory<T> loadFactory;
BitmapLoadFactory<T> loadFactory; private TransformationLoader<T> transformationLoader;
private Y target;
/** /**
* Builds an ImagePresenter. * Builds an ImagePresenter.
* *
* @return A new ImagePresenter * @return A new ImagePresenter
*/ */
public ImagePresenter<T> build(){ public ImagePresenter<T, Y> build(){
if (target == null) { if (target == null) {
throw new IllegalArgumentException("cannot create presenter without a target"); throw new IllegalArgumentException("cannot create presenter without a target");
} }
...@@ -80,36 +80,17 @@ public class ImagePresenter<T> { ...@@ -80,36 +80,17 @@ public class ImagePresenter<T> {
throw new IllegalArgumentException("cannot create presenter without a bitmap load factory"); throw new IllegalArgumentException("cannot create presenter without a bitmap load factory");
} }
return new ImagePresenter<T>(this); return new ImagePresenter<T, Y>(this);
} }
/** /**
* Required - Sets the {@link android.widget.ImageView} the presenter will use to display any loaded bitmaps. * Required - Sets the {@link Target} the presenter will use to display any loaded bitmaps.
* Alternatively you can instead set a more general target for an object other than an ImageView using
* {@link #setTarget(com.bumptech.glide.presenter.target.Target, android.content.Context)}.
*
* @see #setTarget(Target, Context)
*
* @param imageView The {@link android.widget.ImageView} to wrap
* @return This Builder object
*/
public Builder<T> setImageView(final ImageView imageView) {
setTarget(new ImageViewTarget(imageView), imageView.getContext());
return this;
}
/**
* Required - Sets the {@link Target} the presenter will use to display any loaded bitmaps. If you are loading
* bitmaps into an {@link ImageView}, you can instead use {@link #setImageView(android.widget.ImageView)}.
*
* @see #setImageView(ImageView)
* *
* @param target The {@link Target} to wrap * @param target The {@link Target} to wrap
* @param context A context that can be held for the duration of the load * @param context A context that can be held for the duration of the load
* @return This builder object * @return This builder object
*/ */
public Builder<T> setTarget(Target target, Context context) { public Builder<T, Y> setTarget(Y target, Context context) {
this.target = target; this.target = target;
this.context = context; this.context = context;
...@@ -123,9 +104,8 @@ public class ImagePresenter<T> { ...@@ -123,9 +104,8 @@ public class ImagePresenter<T> {
* @param loadFactory The {@link BitmapLoadFactory} to use to obtain the id and InputStreams * @param loadFactory The {@link BitmapLoadFactory} to use to obtain the id and InputStreams
* @return This Builder object * @return This Builder object
*/ */
public Builder<T> setBitmapLoadFactory(BitmapLoadFactory<T> loadFactory) { public Builder<T, Y> setBitmapLoadFactory(BitmapLoadFactory<T> loadFactory) {
this.loadFactory = loadFactory; this.loadFactory = loadFactory;
return this; return this;
} }
...@@ -136,7 +116,7 @@ public class ImagePresenter<T> { ...@@ -136,7 +116,7 @@ public class ImagePresenter<T> {
* @param imageLoader The {@link com.bumptech.glide.loader.image.ImageLoader} to use to load an image * @param imageLoader The {@link com.bumptech.glide.loader.image.ImageLoader} to use to load an image
* @return This Builder object * @return This Builder object
*/ */
public Builder<T> setImageLoader(ImageLoader imageLoader) { public Builder<T, Y> setImageLoader(ImageLoader imageLoader) {
this.imageLoader = imageLoader; this.imageLoader = imageLoader;
return this; return this;
} }
...@@ -149,7 +129,7 @@ public class ImagePresenter<T> { ...@@ -149,7 +129,7 @@ public class ImagePresenter<T> {
* @param resourceId The id of the resource to show * @param resourceId The id of the resource to show
* @return This Builder object * @return This Builder object
*/ */
public Builder<T> setPlaceholderResource(int resourceId) { public Builder<T, Y> setPlaceholderResource(int resourceId) {
if (resourceId != 0 && placeholderDrawable != null) { if (resourceId != 0 && placeholderDrawable != null) {
throw new IllegalArgumentException("Can't set both a placeholder drawable and a placeholder resource"); throw new IllegalArgumentException("Can't set both a placeholder drawable and a placeholder resource");
} }
...@@ -166,7 +146,7 @@ public class ImagePresenter<T> { ...@@ -166,7 +146,7 @@ public class ImagePresenter<T> {
* @param placeholderDrawable The drawable to show * @param placeholderDrawable The drawable to show
* @return This Builder object * @return This Builder object
*/ */
public Builder<T> setPlaceholderDrawable(Drawable placeholderDrawable) { public Builder<T, Y> setPlaceholderDrawable(Drawable placeholderDrawable) {
if (placeholderDrawable != null && placeholderResourceId != 0) { if (placeholderDrawable != null && placeholderResourceId != 0) {
throw new IllegalArgumentException("Can't set both a placeholder drawable and a placeholder resource"); throw new IllegalArgumentException("Can't set both a placeholder drawable and a placeholder resource");
} }
...@@ -182,7 +162,7 @@ public class ImagePresenter<T> { ...@@ -182,7 +162,7 @@ public class ImagePresenter<T> {
* @param resourceId The id of the resource to show * @param resourceId The id of the resource to show
* @return This Builder object * @return This Builder object
*/ */
public Builder<T> setErrorResource(int resourceId) { public Builder<T, Y> setErrorResource(int resourceId) {
if (resourceId != 0 && errorDrawable != null) { if (resourceId != 0 && errorDrawable != null) {
throw new IllegalArgumentException("Can't set both an error drawable and an error resource"); throw new IllegalArgumentException("Can't set both an error drawable and an error resource");
} }
...@@ -198,7 +178,7 @@ public class ImagePresenter<T> { ...@@ -198,7 +178,7 @@ public class ImagePresenter<T> {
* @param drawable The drawable to show * @param drawable The drawable to show
* @return This Builder object * @return This Builder object
*/ */
public Builder<T> setErrorDrawable(Drawable drawable) { public Builder<T, Y> setErrorDrawable(Drawable drawable) {
if (errorResourceId != 0 && drawable != null) { if (errorResourceId != 0 && drawable != null) {
throw new IllegalArgumentException("Can't set both an error drawable and an error resource"); throw new IllegalArgumentException("Can't set both an error drawable and an error resource");
} }
...@@ -215,7 +195,7 @@ public class ImagePresenter<T> { ...@@ -215,7 +195,7 @@ public class ImagePresenter<T> {
* @param cb The callback to call * @param cb The callback to call
* @return This Builder object * @return This Builder object
*/ */
public Builder<T> setImageReadyCallback(ImageReadyCallback<T> cb) { public Builder<T, Y> setImageReadyCallback(ImageReadyCallback<T, Y> cb) {
this.imageReadyCallback = cb; this.imageReadyCallback = cb;
return this; return this;
} }
...@@ -229,7 +209,7 @@ public class ImagePresenter<T> { ...@@ -229,7 +209,7 @@ public class ImagePresenter<T> {
* @param coordinator The coordinator to set * @param coordinator The coordinator to set
* @return This Builder object * @return This Builder object
*/ */
public Builder<T> setImagePresenterCoordinator(ImagePresenterCoordinator coordinator) { public Builder<T, Y> setImagePresenterCoordinator(ImagePresenterCoordinator coordinator) {
this.coordinator = coordinator; this.coordinator = coordinator;
return this; return this;
} }
...@@ -242,21 +222,35 @@ public class ImagePresenter<T> { ...@@ -242,21 +222,35 @@ public class ImagePresenter<T> {
* @param exceptionHandler The exception handler to set * @param exceptionHandler The exception handler to set
* @return This builder object * @return This builder object
*/ */
public Builder<T> setExceptionHandler(ExceptionHandler<T> exceptionHandler) { public Builder<T, Y> setExceptionHandler(ExceptionHandler<T> exceptionHandler) {
this.exceptionHandler = exceptionHandler; this.exceptionHandler = exceptionHandler;
return this; return this;
} }
/**
* Optional - Sets a transformation loader to use to obtain a transformation to apply to images on a per
* model basis.
*
* @see Transformation
*
* @param transformationLoader A {@link TransformationLoader} for this model type
* @return This builder object
*/
public Builder<T, Y> setTransformationLoader(TransformationLoader<T> transformationLoader) {
this.transformationLoader = transformationLoader;
return this;
}
} }
@SuppressWarnings("all") @SuppressWarnings("all")
private Object imageToken; //this is just a reference we may need to keep, otherwise unused private Object imageToken; //this is just a reference we may need to keep, otherwise unused
private final Target target; private final Y target;
private final ImageLoader imageLoader; private final ImageLoader imageLoader;
private final Drawable placeholderDrawable; private final Drawable placeholderDrawable;
private final ImageReadyCallback<T> imageReadyCallback; private final ImageReadyCallback<T, Y> imageReadyCallback;
private final ImagePresenterCoordinator coordinator; private final ImagePresenterCoordinator coordinator;
private final ExceptionHandler<T> exceptionHandler; private final ExceptionHandler<T> exceptionHandler;
...@@ -266,6 +260,7 @@ public class ImagePresenter<T> { ...@@ -266,6 +260,7 @@ public class ImagePresenter<T> {
private boolean isImageSet = false; private boolean isImageSet = false;
private boolean isErrorSet = false; private boolean isErrorSet = false;
private final BitmapLoadFactory<T> loadFactory;
private boolean loadedFromCache = false; private boolean loadedFromCache = false;
private final Drawable errorDrawable; private final Drawable errorDrawable;
...@@ -311,7 +306,7 @@ public class ImagePresenter<T> { ...@@ -311,7 +306,7 @@ public class ImagePresenter<T> {
* A callback interface used to perform some action when an {@link ImagePresenter} sets a new bitmap in an * A callback interface used to perform some action when an {@link ImagePresenter} sets a new bitmap in an
* {@link android.widget.ImageView} * {@link android.widget.ImageView}
*/ */
public interface ImageReadyCallback<T> { public interface ImageReadyCallback<T, Y extends Target> {
/** /**
* The method called when a bitmap is set * The method called when a bitmap is set
...@@ -319,10 +314,10 @@ public class ImagePresenter<T> { ...@@ -319,10 +314,10 @@ public class ImagePresenter<T> {
* @param target The target that will display the bitmap * @param target The target that will display the bitmap
* @param fromCache True iff the load completed without a placeholder being shown. * @param fromCache True iff the load completed without a placeholder being shown.
*/ */
public void onImageReady(T model, Target target, boolean fromCache); public void onImageReady(T model, Y target, boolean fromCache);
} }
protected ImagePresenter(Builder<T> builder) { protected ImagePresenter(Builder<T, Y> builder) {
this.imageLoader = builder.imageLoader; this.imageLoader = builder.imageLoader;
final Resources res = builder.context.getResources(); final Resources res = builder.context.getResources();
...@@ -342,6 +337,11 @@ public class ImagePresenter<T> { ...@@ -342,6 +337,11 @@ public class ImagePresenter<T> {
this.exceptionHandler = builder.exceptionHandler; this.exceptionHandler = builder.exceptionHandler;
this.loadFactory = builder.loadFactory; this.loadFactory = builder.loadFactory;
this.target = builder.target; this.target = builder.target;
final ImagePresenter previous = builder.target.getImagePresenter();
if (previous != null) {
previous.clear();
}
builder.target.setImagePresenter(this);
} }
/** /**
......
library/src/com/bumptech/glide/presenter/ThumbImagePresenter.java
浏览文件 @ 8c7ae0d1
...@@ -4,8 +4,9 @@ ...@@ -4,8 +4,9 @@
package com.bumptech.glide.presenter; package com.bumptech.glide.presenter;
import android.content.Context;
import android.graphics.drawable.Drawable; import android.graphics.drawable.Drawable;
import android.widget.ImageView; import com.bumptech.glide.presenter.target.Target;
/** /**
*Wraps a pair of {@link ImagePresenter} objects and uses them to load and display an image and a thumbnail for that *Wraps a pair of {@link ImagePresenter} objects and uses them to load and display an image and a thumbnail for that
...@@ -15,7 +16,7 @@ import android.widget.ImageView; ...@@ -15,7 +16,7 @@ import android.widget.ImageView;
* shown until the image load completes. If the image load completes first, the thumbnail will never be shown. * shown until the image load completes. If the image load completes first, the thumbnail will never be shown.
*/ */
@SuppressWarnings("unused") @SuppressWarnings("unused")
public class ThumbImagePresenter<A, B> implements ImagePresenter.ImagePresenterCoordinator { public class ThumbImagePresenter<A, B, C extends Target> implements ImagePresenter.ImagePresenterCoordinator {
/** /**
* A builder for a {@link ThumbImagePresenter}. Has a few convenience methods to avoid identical calls on both * A builder for a {@link ThumbImagePresenter}. Has a few convenience methods to avoid identical calls on both
...@@ -26,25 +27,26 @@ public class ThumbImagePresenter<A, B> implements ImagePresenter.ImagePresenterC ...@@ -26,25 +27,26 @@ public class ThumbImagePresenter<A, B> implements ImagePresenter.ImagePresenterC
* @param <A> The type of the model that the full {@link ImagePresenter} requires * @param <A> The type of the model that the full {@link ImagePresenter} requires
* @param <B> The type of the model that the thumb {@link ImagePresenter} requires * @param <B> The type of the model that the thumb {@link ImagePresenter} requires
*/ */
public static class Builder<A, B> { public static class Builder<A, B, C extends Target> {
private ImagePresenter.Builder<A> fullPresenterBuilder; private ImagePresenter.Builder<A, C> fullPresenterBuilder;
private ImagePresenter.Builder<B> thumbPresenterBuilder; private ImagePresenter.Builder<B, C> thumbPresenterBuilder;
private ImageView imageView;
private Drawable placeholderDrawable; private Drawable placeholderDrawable;
private int placeholderResourceId; private int placeholderResourceId;
private Context context;
private C target;
public ThumbImagePresenter<A, B> build(){ public ThumbImagePresenter<A, B, C> build(){
if (fullPresenterBuilder == null) { if (fullPresenterBuilder == null) {
throw new IllegalArgumentException("you must include a builder for the full image presenter"); throw new IllegalArgumentException("you must include a builder for the full image presenter");
} }
if (thumbPresenterBuilder == null) { if (thumbPresenterBuilder == null) {
throw new IllegalArgumentException("you must include a builder for the thumb image presenter"); throw new IllegalArgumentException("you must include a builder for the thumb image presenter");
} }
if (imageView == null){ if (target == null){
throw new IllegalArgumentException("cannot create presenter without an image view"); throw new IllegalArgumentException("cannot create presenter without an image view");
} }
return new ThumbImagePresenter<A, B>(this); return new ThumbImagePresenter<A, B, C>(this);
} }
/** /**
...@@ -54,7 +56,7 @@ public class ThumbImagePresenter<A, B> implements ImagePresenter.ImagePresenterC ...@@ -54,7 +56,7 @@ public class ThumbImagePresenter<A, B> implements ImagePresenter.ImagePresenterC
* and {@link ImagePresenter.Builder ImagePresenter.Builder#setImageLoader} must have been called * and {@link ImagePresenter.Builder ImagePresenter.Builder#setImageLoader} must have been called
* @return This builder object * @return This builder object
*/ */
public Builder<A, B> setFullPresenterBuilder(ImagePresenter.Builder<A> builder) { public Builder<A, B, C> setFullPresenterBuilder(ImagePresenter.Builder<A, C> builder) {
this.fullPresenterBuilder = builder; this.fullPresenterBuilder = builder;
return this; return this;
} }
...@@ -66,7 +68,7 @@ public class ThumbImagePresenter<A, B> implements ImagePresenter.ImagePresenterC ...@@ -66,7 +68,7 @@ public class ThumbImagePresenter<A, B> implements ImagePresenter.ImagePresenterC
* and {@link ImagePresenter.Builder ImagePresenter.Builder#setImageLoader} must have been called * and {@link ImagePresenter.Builder ImagePresenter.Builder#setImageLoader} must have been called
* @return This builder object * @return This builder object
*/ */
public Builder<A, B> setThumbPresenterBuilder(ImagePresenter.Builder<B> builder) { public Builder<A, B, C> setThumbPresenterBuilder(ImagePresenter.Builder<B, C> builder) {
this.thumbPresenterBuilder = builder; this.thumbPresenterBuilder = builder;
return this; return this;
} }
...@@ -74,15 +76,16 @@ public class ThumbImagePresenter<A, B> implements ImagePresenter.ImagePresenterC ...@@ -74,15 +76,16 @@ public class ThumbImagePresenter<A, B> implements ImagePresenter.ImagePresenterC
/** /**
* @see ImagePresenter.Builder ImagePresenter.Builder#setImageView * @see ImagePresenter.Builder ImagePresenter.Builder#setImageView
*/ */
public Builder<A, B> setImageView(ImageView imageView) { public Builder<A, B, C> setTarget(C target, Context context) {
this.imageView = imageView; this.target = target;
this.context = context;
return this; return this;
} }
/** /**
* @see ImagePresenter.Builder ImagePresenter.Builder#setPlaceholderDrawable * @see ImagePresenter.Builder ImagePresenter.Builder#setPlaceholderDrawable
*/ */
public Builder<A, B> setPlaceholderDrawable(Drawable drawable) { public Builder<A, B, C> setPlaceholderDrawable(Drawable drawable) {
if (drawable != null && this.placeholderResourceId != 0) { if (drawable != null && this.placeholderResourceId != 0) {
throw new IllegalArgumentException("Can't set both a placeholder drawable and a placeholder resource"); throw new IllegalArgumentException("Can't set both a placeholder drawable and a placeholder resource");
} }
...@@ -94,7 +97,7 @@ public class ThumbImagePresenter<A, B> implements ImagePresenter.ImagePresenterC ...@@ -94,7 +97,7 @@ public class ThumbImagePresenter<A, B> implements ImagePresenter.ImagePresenterC
/** /**
* @see ImagePresenter.Builder ImagePresenter.Builder#setPlaceholderResource * @see ImagePresenter.Builder ImagePresenter.Builder#setPlaceholderResource
*/ */
public Builder<A, B> setPlaceholderResource(int resourceId) { public Builder<A, B, C> setPlaceholderResource(int resourceId) {
if (resourceId != 0 && placeholderDrawable != null) { if (resourceId != 0 && placeholderDrawable != null) {
throw new IllegalArgumentException("Can't set both a placeholder drawable and a placeholder resource"); throw new IllegalArgumentException("Can't set both a placeholder drawable and a placeholder resource");
} }
...@@ -104,20 +107,21 @@ public class ThumbImagePresenter<A, B> implements ImagePresenter.ImagePresenterC ...@@ -104,20 +107,21 @@ public class ThumbImagePresenter<A, B> implements ImagePresenter.ImagePresenterC
} }
} }
private final ImagePresenter<A> fullPresenter; private final ImagePresenter<A, C> fullPresenter;
private final ImagePresenter<B> thumbPresenter; private final ImagePresenter<B, C> thumbPresenter;
private ThumbImagePresenter(Builder<A, B> builder) { private ThumbImagePresenter(Builder<A, B, C> builder) {
fullPresenter = builder.fullPresenterBuilder fullPresenter = builder.fullPresenterBuilder
.setImagePresenterCoordinator(this) .setImagePresenterCoordinator(this)
.setImageView(builder.imageView) .setTarget(builder.target, builder.context)
.setPlaceholderResource(0) .setPlaceholderResource(0)
.setPlaceholderDrawable(builder.placeholderDrawable == null ? .setPlaceholderDrawable(builder.placeholderDrawable == null ?
builder.imageView.getResources().getDrawable(builder.placeholderResourceId) : builder.placeholderDrawable) builder.context.getResources().getDrawable(builder.placeholderResourceId)
: builder.placeholderDrawable)
.build(); .build();
thumbPresenter = builder.thumbPresenterBuilder thumbPresenter = builder.thumbPresenterBuilder
.setImagePresenterCoordinator(this) .setImagePresenterCoordinator(this)
.setImageView(builder.imageView) .setTarget(builder.target, builder.context)
.setPlaceholderDrawable(null) .setPlaceholderDrawable(null)
.setPlaceholderResource(0) .setPlaceholderResource(0)
.build(); .build();
......
library/src/com/bumptech/glide/presenter/target/BaseTarget.java 0 → 100644
浏览文件 @ 8c7ae0d1
package com.bumptech.glide.presenter.target;
import com.bumptech.glide.presenter.ImagePresenter;
public abstract class BaseTarget implements Target {
private ImagePresenter imagePresenter;
@Override
public void setImagePresenter(ImagePresenter imagePresenter) {
this.imagePresenter = imagePresenter;
}
@Override
public ImagePresenter getImagePresenter() {
return imagePresenter;
}
}
library/src/com/bumptech/glide/presenter/target/ImageViewTarget.java
浏览文件 @ 8c7ae0d1
...@@ -10,7 +10,6 @@ import android.view.ViewTreeObserver; ...@@ -10,7 +10,6 @@ import android.view.ViewTreeObserver;
import android.view.WindowManager; import android.view.WindowManager;
import android.view.animation.Animation; import android.view.animation.Animation;
import android.widget.ImageView; import android.widget.ImageView;
import com.bumptech.glide.presenter.ImagePresenter;
import java.lang.ref.WeakReference; import java.lang.ref.WeakReference;
...@@ -19,17 +18,20 @@ import static android.view.ViewGroup.LayoutParams; ...@@ -19,17 +18,20 @@ import static android.view.ViewGroup.LayoutParams;
/** /**
* A target wrapping an ImageView. Obtains the runtime dimensions of the ImageView. * A target wrapping an ImageView. Obtains the runtime dimensions of the ImageView.
*/ */
public class ImageViewTarget implements Target { public class ImageViewTarget extends BaseTarget {
private static final String TAG = "ImageViewTarget"; private static final String TAG = "ImageViewTarget";
private final ImageView imageView; private final ImageView imageView;
private final SizeDeterminer sizeDeterminer; private final SizeDeterminer sizeDeterminer;
private ImagePresenter imagePresenter;
public ImageViewTarget(ImageView imageView) { public ImageViewTarget(ImageView imageView) {
this.imageView = imageView; this.imageView = imageView;
this.sizeDeterminer = new SizeDeterminer(imageView); this.sizeDeterminer = new SizeDeterminer(imageView);
} }
public ImageView getView() {
return imageView;
}
@Override @Override
public void onImageReady(Bitmap bitmap) { public void onImageReady(Bitmap bitmap) {
imageView.setImageBitmap(bitmap); imageView.setImageBitmap(bitmap);
...@@ -52,16 +54,6 @@ public class ImageViewTarget implements Target { ...@@ -52,16 +54,6 @@ public class ImageViewTarget implements Target {
imageView.startAnimation(animation); imageView.startAnimation(animation);
} }
@Override
public void setImagePresenter(ImagePresenter imagePresenter) {
this.imagePresenter = imagePresenter;
}
@Override
public ImagePresenter getImagePresenter() {
return imagePresenter;
}
@Override @Override
public int hashCode() { public int hashCode() {
return imageView.hashCode(); return imageView.hashCode();
......
library/src/com/bumptech/glide/presenter/target/SimpleTarget.java
浏览文件 @ 8c7ae0d1
...@@ -7,8 +7,7 @@ import com.bumptech.glide.presenter.ImagePresenter; ...@@ -7,8 +7,7 @@ import com.bumptech.glide.presenter.ImagePresenter;
/** /**
* A simpler interface for targets with default (usually noop) implementations of non essential methods. * A simpler interface for targets with default (usually noop) implementations of non essential methods.
*/ */
public abstract class SimpleTarget implements Target { public abstract class SimpleTarget extends BaseTarget {
private ImagePresenter imagePresenter;
@Override @Override
public void setPlaceholder(Drawable placeholder) { } public void setPlaceholder(Drawable placeholder) { }
...@@ -16,16 +15,6 @@ public abstract class SimpleTarget implements Target { ...@@ -16,16 +15,6 @@ public abstract class SimpleTarget implements Target {
@Override @Override
public void startAnimation(Animation animation) { } public void startAnimation(Animation animation) { }
@Override
public void setImagePresenter(ImagePresenter imagePresenter) {
this.imagePresenter = imagePresenter;
}
@Override
public ImagePresenter getImagePresenter() {
return imagePresenter;
}
/** /**
* A default implementation that calls {@link com.bumptech.glide.presenter.target.Target.SizeReadyCallback} * A default implementation that calls {@link com.bumptech.glide.presenter.target.Target.SizeReadyCallback}
* synchronously with {@link #getSize()} * synchronously with {@link #getSize()}
......
library/src/com/bumptech/glide/presenter/target/Target.java
浏览文件 @ 8c7ae0d1
...@@ -46,19 +46,7 @@ public interface Target { ...@@ -46,19 +46,7 @@ public interface Target {
*/ */
public void startAnimation(Animation animation); public void startAnimation(Animation animation);
/**
* A method that is used to keep a reference to the ImagePresenter. Increases the efficiency of loads if it is
* possible that this target object will be used more than once.
*
* @param imagePresenter The ImagePresenter to store
*/
public void setImagePresenter(ImagePresenter imagePresenter); public void setImagePresenter(ImagePresenter imagePresenter);
/**
* A method for retrieving any current ImagePresenter. Increases the efficiency of loads if it is possible that this
* target object will be used more than once.
*
* @return The current ImagePresenter or null
*/
public ImagePresenter getImagePresenter(); public ImagePresenter getImagePresenter();
} }
library/tests/src/com/bumptech/glide/GlideTest.java
浏览文件 @ 8c7ae0d1
package com.bumptech.glide; package com.bumptech.glide;
import android.content.Context;
import android.graphics.Bitmap; import android.graphics.Bitmap;
import android.net.Uri; import android.net.Uri;
import android.test.ActivityTestCase; import android.test.ActivityTestCase;
...@@ -23,17 +24,23 @@ import java.net.URL; ...@@ -23,17 +24,23 @@ import java.net.URL;
*/ */
public class GlideTest extends ActivityTestCase { public class GlideTest extends ActivityTestCase {
private ImageView imageView; private ImageView imageView;
private ImageViewTarget imageViewTarget;
@Override @Override
protected void setUp() throws Exception { protected void setUp() throws Exception {
super.setUp(); super.setUp();
imageView = new ImageView(getInstrumentation().getContext()); imageView = new ImageView(getContext());
//this is a quick hack to get the SizeDeterminer in ImagePresenter to think the view has been measured //this is a quick hack to get the SizeDeterminer in ImagePresenter to think the view has been measured
imageView.setLayoutParams(new ViewGroup.LayoutParams(100, 100)); imageView.setLayoutParams(new ViewGroup.LayoutParams(100, 100));
imageViewTarget = new ImageViewTarget(imageView);
} }
private void checkImagePresenter(Object model) { private Context getContext() {
ImagePresenter imagePresenter = getImagePresenterFromView(); return getInstrumentation().getContext();
}
private void checkImagePresenter(Target target, Object model) {
ImagePresenter imagePresenter = target.getImagePresenter();
imagePresenter.setModel(model); imagePresenter.setModel(model);
boolean caughtException = false; boolean caughtException = false;
...@@ -46,58 +53,56 @@ public class GlideTest extends ActivityTestCase { ...@@ -46,58 +53,56 @@ public class GlideTest extends ActivityTestCase {
assertTrue(caughtException); assertTrue(caughtException);
} }
private ImagePresenter getImagePresenterFromView() {
return ((ImageViewTarget) imageView.getTag()).getImagePresenter();
}
public void testFileDefaultLoader() { public void testFileDefaultLoader() {
File file = new File("fake"); File file = new File("fake");
Glide.load(file).into(imageView); Target target = Glide.with(getContext()).load(file).into(imageViewTarget);
checkImagePresenter(file); checkImagePresenter(target, file);
} }
public void testUrlDefaultLoader() throws MalformedURLException { public void testUrlDefaultLoader() throws MalformedURLException {
URL url = new URL("http://www.google.com"); URL url = new URL("http://www.google.com");
Glide.load(url).into(imageView); Target target = Glide.with(getContext()).load(url).into(imageViewTarget);
checkImagePresenter(url); checkImagePresenter(target, url);
} }
public void testUriDefaultLoader() { public void testUriDefaultLoader() {
Uri uri = Uri.fromFile(new File("Fake")); Uri uri = Uri.fromFile(new File("Fake"));
Glide.load(uri).into(imageView); Target target = Glide.with(getContext()).load(uri).into(imageViewTarget);
checkImagePresenter(uri); checkImagePresenter(target, uri);
} }
public void testStringDefaultLoader() { public void testStringDefaultLoader() {
String string = "http://www.google.com"; String string = "http://www.google.com";
Glide.load(string).into(imageView); Target target = Glide.with(getContext()).load(string).into(imageViewTarget);
checkImagePresenter(string); checkImagePresenter(target, string);
} }
public void testIntegerDefaultLoader() { public void testIntegerDefaultLoader() {
int integer = 1234; int integer = 1234;
Glide.load(integer).into(imageView); Target target = Glide.with(getContext()).load(integer).into(imageViewTarget);
checkImagePresenter(integer); checkImagePresenter(target, integer);
} }
public void testGlideDoesNotReplaceIdenticalPresenters() { public void testGlideDoesNotReplaceIdenticalPresenters() {
Glide.load("fake") Target target = Glide.with(getContext())
.load("fake")
.asIs() .asIs()
.centerCrop() .centerCrop()
.animate(android.R.anim.fade_in) .animate(android.R.anim.fade_in)
.placeholder(com.bumptech.glide.tests.R.raw.ic_launcher) .placeholder(com.bumptech.glide.tests.R.raw.ic_launcher)
.error(com.bumptech.glide.tests.R.raw.ic_launcher) .error(com.bumptech.glide.tests.R.raw.ic_launcher)
.into(imageView); .into(imageViewTarget);
ImagePresenter first = getImagePresenterFromView(); ImagePresenter first = target.getImagePresenter();
Glide.load("fake2") Target target2 = Glide.with(getContext())
.load("fake2")
.asIs() .asIs()
.centerCrop() .centerCrop()
.animate(android.R.anim.fade_in) .animate(android.R.anim.fade_in)
.placeholder(com.bumptech.glide.tests.R.raw.ic_launcher) .placeholder(com.bumptech.glide.tests.R.raw.ic_launcher)
.error(com.bumptech.glide.tests.R.raw.ic_launcher) .error(com.bumptech.glide.tests.R.raw.ic_launcher)
.into(imageView); .into(imageViewTarget);
ImagePresenter second = getImagePresenterFromView(); ImagePresenter second = target2.getImagePresenter();
assertSame(first, second); assertSame(first, second);
} }
...@@ -105,23 +110,23 @@ public class GlideTest extends ActivityTestCase { ...@@ -105,23 +110,23 @@ public class GlideTest extends ActivityTestCase {
public void testCanHandleWrapContent() { public void testCanHandleWrapContent() {
imageView.setLayoutParams(new ViewGroup.LayoutParams(ViewGroup.LayoutParams.WRAP_CONTENT, imageView.setLayoutParams(new ViewGroup.LayoutParams(ViewGroup.LayoutParams.WRAP_CONTENT,
ViewGroup.LayoutParams.WRAP_CONTENT)); ViewGroup.LayoutParams.WRAP_CONTENT));
Glide.load("fake").into(imageView); Target target = Glide.with(getContext()).load("fake").into(imageViewTarget);
assertNotNull(getImagePresenterFromView()); assertNotNull(target.getImagePresenter());
} }
public void testCanHandleWrapContentMatchParent() { public void testCanHandleWrapContentMatchParent() {
imageView.setLayoutParams(new ViewGroup.LayoutParams(ViewGroup.LayoutParams.WRAP_CONTENT, imageView.setLayoutParams(new ViewGroup.LayoutParams(ViewGroup.LayoutParams.WRAP_CONTENT,
ViewGroup.LayoutParams.MATCH_PARENT)); ViewGroup.LayoutParams.MATCH_PARENT));
Glide.load("fake").into(imageView); Target target = Glide.with(getContext()).load("fake").into(imageViewTarget);
assertNotNull(getImagePresenterFromView()); assertNotNull(target.getImagePresenter());
} }
public void testDifferentModelTypesReplacesPresenters() { public void testDifferentModelTypesReplacesPresenters() {
assertDifferentPresenters( assertDifferentPresenters(
Glide.load(4), Glide.with(getContext()).load(4),
Glide.load("fake") Glide.with(getContext()).load("fake")
); );
} }
...@@ -182,31 +187,31 @@ public class GlideTest extends ActivityTestCase { ...@@ -182,31 +187,31 @@ public class GlideTest extends ActivityTestCase {
final Object object = new Object(); final Object object = new Object();
assertDifferentPresenters( assertDifferentPresenters(
Glide.using(first).load(object), Glide.with(getContext()).using(first).load(object),
Glide.using(second).load(object) Glide.with(getContext()).using(second).load(object)
); );
} }
public void testDifferentDownsamplersReplacesPresenter() { public void testDifferentDownsamplersReplacesPresenter() {
assertDifferentPresenters( assertDifferentPresenters(
Glide.load("a").approximate(), Glide.with(getContext()).load("a").approximate(),
Glide.load("a").asIs() Glide.with(getContext()).load("a").asIs()
); );
} }
public void testDifferentTransformationsReplacesPresenter() { public void testDifferentTransformationsReplacesPresenter() {
final File file = new File("fake"); final File file = new File("fake");
assertDifferentPresenters( assertDifferentPresenters(
Glide.load(file).centerCrop().fitCenter(), Glide.with(getContext()).load(file).centerCrop().fitCenter(),
Glide.load(file).centerCrop() Glide.with(getContext()).load(file).centerCrop()
); );
} }
public void testDifferentPlaceholdersReplacesPresenter() { public void testDifferentPlaceholdersReplacesPresenter() {
final File file = new File("fake"); final File file = new File("fake");
assertDifferentPresenters( assertDifferentPresenters(
Glide.load(file).placeholder(com.bumptech.glide.tests.R.raw.ic_launcher), Glide.with(getContext()).load(file).placeholder(com.bumptech.glide.tests.R.raw.ic_launcher),
Glide.load(file) Glide.with(getContext()).load(file)
); );
} }
...@@ -214,25 +219,18 @@ public class GlideTest extends ActivityTestCase { ...@@ -214,25 +219,18 @@ public class GlideTest extends ActivityTestCase {
public void testDifferentAnimationsReplacesPresenter() { public void testDifferentAnimationsReplacesPresenter() {
final File file = new File("fake"); final File file = new File("fake");
assertDifferentPresenters( assertDifferentPresenters(
Glide.load(file).animate(android.R.anim.fade_in), Glide.with(getContext()).load(file).animate(android.R.anim.fade_in),
Glide.load(file).animate(android.R.anim.fade_out) Glide.with(getContext()).load(file).animate(android.R.anim.fade_out)
); );
} }
public void testDifferentErrorIdsReplacesPresenter() { public void testDifferentErrorIdsReplacesPresenter() {
assertDifferentPresenters( assertDifferentPresenters(
Glide.load("b").error(R.raw.ic_launcher), Glide.with(getContext()).load("b").error(R.raw.ic_launcher),
Glide.load("b").error(android.R.drawable.btn_star) Glide.with(getContext()).load("b").error(android.R.drawable.btn_star)
); );
} }
public void testClearingTagReplacesPresenter() {
Glide.load("a").into(imageView);
assertNotNull(imageView.getTag());
imageView.setTag(null);
Glide.load("b").into(imageView);
}
public void testObtainAndOfferToBitmapPool() { public void testObtainAndOfferToBitmapPool() {
Bitmap small = Bitmap.createBitmap(100, 100, Bitmap.Config.ARGB_8888); Bitmap small = Bitmap.createBitmap(100, 100, Bitmap.Config.ARGB_8888);
Bitmap large = Bitmap.createBitmap(512, 512, Bitmap.Config.RGB_565); Bitmap large = Bitmap.createBitmap(512, 512, Bitmap.Config.RGB_565);
...@@ -244,21 +242,9 @@ public class GlideTest extends ActivityTestCase { ...@@ -244,21 +242,9 @@ public class GlideTest extends ActivityTestCase {
assertEquals(large, bitmapPool.get(large.getWidth(), large.getHeight(), large.getConfig())); assertEquals(large, bitmapPool.get(large.getWidth(), large.getHeight(), large.getConfig()));
} }
public void testThrowExceptionIfTagReplaced() {
imageView.setTag(1234);
Exception exception = null;
try {
Glide.load("a").into(imageView);
} catch (Exception e) {
exception = e;
}
assertNotNull(exception);
}
public void testDifferentRequestListenersReplacesPresenter() { public void testDifferentRequestListenersReplacesPresenter() {
assertDifferentPresenters( assertDifferentPresenters(
Glide.load("a").listener(new Glide.RequestListener<String>() { Glide.with(getContext()).load("a").listener(new Glide.RequestListener<String>() {
@Override @Override
public void onException(Exception e, String model, Target target) { public void onException(Exception e, String model, Target target) {
...@@ -269,7 +255,7 @@ public class GlideTest extends ActivityTestCase { ...@@ -269,7 +255,7 @@ public class GlideTest extends ActivityTestCase {
public void onImageReady(String model, Target target) { public void onImageReady(String model, Target target) {
} }
}), }),
Glide.load("a").listener(new Glide.RequestListener<String>() { Glide.with(getContext()).load("a").listener(new Glide.RequestListener<String>() {
@Override @Override
public void onException(Exception e, String model, Target target) { public void onException(Exception e, String model, Target target) {
} }
...@@ -350,14 +336,9 @@ public class GlideTest extends ActivityTestCase { ...@@ -350,14 +336,9 @@ public class GlideTest extends ActivityTestCase {
// } // }
private void assertDifferentPresenters(Glide.Request a, Glide.Request b) { private void assertDifferentPresenters(Glide.Request a, Glide.Request b) {
a.into(imageView); ImagePresenter first = a.into(imageViewTarget).getImagePresenter();
ImagePresenter first = getImagePresenterFromView(); ImagePresenter second = a.into(imageViewTarget).getImagePresenter();
ImagePresenter third = b.into(imageViewTarget).getImagePresenter();
a.into(imageView);
ImagePresenter second = getImagePresenterFromView();
b.into(imageView);
ImagePresenter third = getImagePresenterFromView();
assertSame(first, second); assertSame(first, second);
assertNotSame(first, third); assertNotSame(first, third);
......
library/tests/src/com/bumptech/glide/ImagePresenterTest.java
浏览文件 @ 8c7ae0d1
...@@ -11,6 +11,7 @@ import com.bumptech.glide.loader.bitmap.BitmapLoadFactory; ...@@ -11,6 +11,7 @@ import com.bumptech.glide.loader.bitmap.BitmapLoadFactory;
import com.bumptech.glide.loader.image.ImageLoader; import com.bumptech.glide.loader.image.ImageLoader;
import com.bumptech.glide.loader.image.ImageManagerLoader; import com.bumptech.glide.loader.image.ImageManagerLoader;
import com.bumptech.glide.presenter.ImagePresenter; import com.bumptech.glide.presenter.ImagePresenter;
import com.bumptech.glide.presenter.target.BaseTarget;
import com.bumptech.glide.presenter.target.Target; import com.bumptech.glide.presenter.target.Target;
import com.bumptech.glide.resize.BitmapLoadTask; import com.bumptech.glide.resize.BitmapLoadTask;
import com.bumptech.glide.resize.bitmap_recycle.BitmapPool; import com.bumptech.glide.resize.bitmap_recycle.BitmapPool;
...@@ -69,10 +70,8 @@ public class ImagePresenterTest extends AndroidTestCase { ...@@ -69,10 +70,8 @@ public class ImagePresenterTest extends AndroidTestCase {
public void testMustProvideATarget() { public void testMustProvideATarget() {
Exception exception = null; Exception exception = null;
try { try {
new ImagePresenter.Builder<Object>() new ImagePresenter.Builder<Object, Target>()
.setBitmapLoadFactory(new MockBitmapLoadFactory()) .setBitmapLoadFactory(new MockBitmapLoadFactory())
.setBitmapLoadFactory(new MockBitmapLoadFactory())
.setImageLoader(new MockImageLoader())
.build(); .build();
} catch (Exception e) { } catch (Exception e) {
exception = e; exception = e;
...@@ -83,7 +82,7 @@ public class ImagePresenterTest extends AndroidTestCase { ...@@ -83,7 +82,7 @@ public class ImagePresenterTest extends AndroidTestCase {
public void testMustProvideAModelLoader() { public void testMustProvideAModelLoader() {
Exception exception = null; Exception exception = null;
try { try {
new ImagePresenter.Builder<Object>() new ImagePresenter.Builder<Object, Target>()
.setTarget(target, getContext()) .setTarget(target, getContext())
.setImageLoader(new MockImageLoader()) .setImageLoader(new MockImageLoader())
.build(); .build();
...@@ -96,7 +95,7 @@ public class ImagePresenterTest extends AndroidTestCase { ...@@ -96,7 +95,7 @@ public class ImagePresenterTest extends AndroidTestCase {
public void testMustProvideAnImageLoader() { public void testMustProvideAnImageLoader() {
Exception exception = null; Exception exception = null;
try { try {
new ImagePresenter.Builder<Object>() new ImagePresenter.Builder<Object, Target>()
.setTarget(target, getContext()) .setTarget(target, getContext())
.setBitmapLoadFactory(new MockBitmapLoadFactory()) .setBitmapLoadFactory(new MockBitmapLoadFactory())
.build(); .build();
...@@ -108,7 +107,7 @@ public class ImagePresenterTest extends AndroidTestCase { ...@@ -108,7 +107,7 @@ public class ImagePresenterTest extends AndroidTestCase {
public void testPlaceholderIsSetWithNullModel() { public void testPlaceholderIsSetWithNullModel() {
Drawable placeholder = new ColorDrawable(Color.RED); Drawable placeholder = new ColorDrawable(Color.RED);
ImagePresenter<Object> imagePresenter = new ImagePresenter.Builder<Object>() ImagePresenter<Object, Target> imagePresenter = new ImagePresenter.Builder<Object, Target>()
.setTarget(target, getContext()) .setTarget(target, getContext())
.setBitmapLoadFactory(new MockBitmapLoadFactory()) .setBitmapLoadFactory(new MockBitmapLoadFactory())
.setImageLoader(new ImageManagerLoader(getContext())) .setImageLoader(new ImageManagerLoader(getContext()))
...@@ -121,7 +120,7 @@ public class ImagePresenterTest extends AndroidTestCase { ...@@ -121,7 +120,7 @@ public class ImagePresenterTest extends AndroidTestCase {
public void testPlaceholderIsSetWithNullBitmapLoadTask() { public void testPlaceholderIsSetWithNullBitmapLoadTask() {
Drawable placeholder = new ColorDrawable(Color.RED); Drawable placeholder = new ColorDrawable(Color.RED);
final Bitmap result = Bitmap.createBitmap(1, 1, Bitmap.Config.ALPHA_8); final Bitmap result = Bitmap.createBitmap(1, 1, Bitmap.Config.ALPHA_8);
ImagePresenter<Object> imagePresenter = new ImagePresenter.Builder<Object>() ImagePresenter<Object, Target> imagePresenter = new ImagePresenter.Builder<Object, Target>()
.setTarget(target, getContext()) .setTarget(target, getContext())
.setBitmapLoadFactory(new MockBitmapLoadFactory() { .setBitmapLoadFactory(new MockBitmapLoadFactory() {
@Override @Override
...@@ -144,7 +143,7 @@ public class ImagePresenterTest extends AndroidTestCase { ...@@ -144,7 +143,7 @@ public class ImagePresenterTest extends AndroidTestCase {
public void testPlaceholderIsSetDuringAsynchronousLoad() { public void testPlaceholderIsSetDuringAsynchronousLoad() {
Drawable placeholder = new ColorDrawable(Color.RED); Drawable placeholder = new ColorDrawable(Color.RED);
ImagePresenter<Object> imagePresenter = new ImagePresenter.Builder<Object>() ImagePresenter<Object, Target> imagePresenter = new ImagePresenter.Builder<Object, Target>()
.setTarget(target, getContext()) .setTarget(target, getContext())
.setBitmapLoadFactory(new MockBitmapLoadFactory()) .setBitmapLoadFactory(new MockBitmapLoadFactory())
.setImageLoader(new MockImageLoader()) .setImageLoader(new MockImageLoader())
...@@ -160,7 +159,7 @@ public class ImagePresenterTest extends AndroidTestCase { ...@@ -160,7 +159,7 @@ public class ImagePresenterTest extends AndroidTestCase {
final Bitmap result = Bitmap.createBitmap(1, 1, Bitmap.Config.ARGB_4444); final Bitmap result = Bitmap.createBitmap(1, 1, Bitmap.Config.ARGB_4444);
final AtomicBoolean wasPlaceholderEverSet = new AtomicBoolean(false); final AtomicBoolean wasPlaceholderEverSet = new AtomicBoolean(false);
ImagePresenter<Object> imagePresenter = new ImagePresenter.Builder<Object>() ImagePresenter<Object, Target> imagePresenter = new ImagePresenter.Builder<Object, Target>()
.setTarget(new MockTarget() { .setTarget(new MockTarget() {
@Override @Override
public void setPlaceholder(Drawable placeholder) { public void setPlaceholder(Drawable placeholder) {
...@@ -184,7 +183,7 @@ public class ImagePresenterTest extends AndroidTestCase { ...@@ -184,7 +183,7 @@ public class ImagePresenterTest extends AndroidTestCase {
public void testErrorPlaceholderIsSetOnException() { public void testErrorPlaceholderIsSetOnException() {
Drawable errorDrawable = new ColorDrawable(Color.RED); Drawable errorDrawable = new ColorDrawable(Color.RED);
ImagePresenter<Object> imagePresenter = new ImagePresenter.Builder<Object>() ImagePresenter<Object, Target> imagePresenter = new ImagePresenter.Builder<Object, Target>()
.setTarget(target, getContext()) .setTarget(target, getContext())
.setBitmapLoadFactory(new MockBitmapLoadFactory()) .setBitmapLoadFactory(new MockBitmapLoadFactory())
.setImageLoader(new MockImageLoader().onCallback(new MockImageLoader.CallbackAction() { .setImageLoader(new MockImageLoader().onCallback(new MockImageLoader.CallbackAction() {
...@@ -202,7 +201,7 @@ public class ImagePresenterTest extends AndroidTestCase { ...@@ -202,7 +201,7 @@ public class ImagePresenterTest extends AndroidTestCase {
public void testBitmapIsSetWhenLoaded() { public void testBitmapIsSetWhenLoaded() {
final Bitmap result = Bitmap.createBitmap(1, 1, Bitmap.Config.ARGB_8888); final Bitmap result = Bitmap.createBitmap(1, 1, Bitmap.Config.ARGB_8888);
ImagePresenter<Object> imagePresenter = new ImagePresenter.Builder<Object>() ImagePresenter<Object, Target> imagePresenter = new ImagePresenter.Builder<Object, Target>()
.setTarget(target, getContext()) .setTarget(target, getContext())
.setBitmapLoadFactory(new MockBitmapLoadFactory()) .setBitmapLoadFactory(new MockBitmapLoadFactory())
.setImageLoader(new MockImageLoader().onCallback(new MockImageLoader.CallbackAction() { .setImageLoader(new MockImageLoader().onCallback(new MockImageLoader.CallbackAction() {
...@@ -221,7 +220,7 @@ public class ImagePresenterTest extends AndroidTestCase { ...@@ -221,7 +220,7 @@ public class ImagePresenterTest extends AndroidTestCase {
final Bitmap second = Bitmap.createBitmap(1, 1, Bitmap.Config.ARGB_4444); final Bitmap second = Bitmap.createBitmap(1, 1, Bitmap.Config.ARGB_4444);
final List<ImageLoader.ImageReadyCallback> cbs = new ArrayList<ImageLoader.ImageReadyCallback>(); final List<ImageLoader.ImageReadyCallback> cbs = new ArrayList<ImageLoader.ImageReadyCallback>();
ImagePresenter<Object> imagePresenter = new ImagePresenter.Builder<Object>() ImagePresenter<Object, Target> imagePresenter = new ImagePresenter.Builder<Object, Target>()
.setTarget(target, getContext()) .setTarget(target, getContext())
.setBitmapLoadFactory(new MockBitmapLoadFactory()) .setBitmapLoadFactory(new MockBitmapLoadFactory())
.setImageLoader(new MockImageLoader().onCallback(new MockImageLoader.CallbackAction() { .setImageLoader(new MockImageLoader().onCallback(new MockImageLoader.CallbackAction() {
...@@ -245,7 +244,7 @@ public class ImagePresenterTest extends AndroidTestCase { ...@@ -245,7 +244,7 @@ public class ImagePresenterTest extends AndroidTestCase {
final Object actualModel = new Object(); final Object actualModel = new Object();
final AtomicBoolean wasOnExceptionCalled = new AtomicBoolean(false); final AtomicBoolean wasOnExceptionCalled = new AtomicBoolean(false);
ImagePresenter<Object> imagePresenter = new ImagePresenter.Builder<Object>() ImagePresenter<Object, Target> imagePresenter = new ImagePresenter.Builder<Object, Target>()
.setTarget(target, getContext()) .setTarget(target, getContext())
.setBitmapLoadFactory(new MockBitmapLoadFactory()) .setBitmapLoadFactory(new MockBitmapLoadFactory())
.setImageLoader(new MockImageLoader().onCallback(new MockImageLoader.CallbackAction() { .setImageLoader(new MockImageLoader().onCallback(new MockImageLoader.CallbackAction() {
...@@ -273,7 +272,7 @@ public class ImagePresenterTest extends AndroidTestCase { ...@@ -273,7 +272,7 @@ public class ImagePresenterTest extends AndroidTestCase {
final Object model = new Object(); final Object model = new Object();
final AtomicInteger timesMockImageLoaderCalled = new AtomicInteger(); final AtomicInteger timesMockImageLoaderCalled = new AtomicInteger();
ImagePresenter<Object> imagePresenter = new ImagePresenter.Builder<Object>() ImagePresenter<Object, Target> imagePresenter = new ImagePresenter.Builder<Object, Target>()
.setTarget(target, getContext()) .setTarget(target, getContext())
.setBitmapLoadFactory(new MockBitmapLoadFactory()) .setBitmapLoadFactory(new MockBitmapLoadFactory())
.setImageLoader(new MockImageLoader().onCallback(new MockImageLoader.CallbackAction() { .setImageLoader(new MockImageLoader().onCallback(new MockImageLoader.CallbackAction() {
...@@ -293,7 +292,7 @@ public class ImagePresenterTest extends AndroidTestCase { ...@@ -293,7 +292,7 @@ public class ImagePresenterTest extends AndroidTestCase {
public void testImageSetCallbackIsCalledWhenImageIsSet() { public void testImageSetCallbackIsCalledWhenImageIsSet() {
final AtomicBoolean wasImageSetCallbackCalled = new AtomicBoolean(false); final AtomicBoolean wasImageSetCallbackCalled = new AtomicBoolean(false);
ImagePresenter<Object> imagePresenter = new ImagePresenter.Builder<Object>() ImagePresenter<Object, Target> imagePresenter = new ImagePresenter.Builder<Object, Target>()
.setTarget(target, getContext()) .setTarget(target, getContext())
.setBitmapLoadFactory(new MockBitmapLoadFactory()) .setBitmapLoadFactory(new MockBitmapLoadFactory())
.setImageLoader(new MockImageLoader().onCallback(new MockImageLoader.CallbackAction() { .setImageLoader(new MockImageLoader().onCallback(new MockImageLoader.CallbackAction() {
...@@ -302,7 +301,7 @@ public class ImagePresenterTest extends AndroidTestCase { ...@@ -302,7 +301,7 @@ public class ImagePresenterTest extends AndroidTestCase {
cb.onImageReady(Bitmap.createBitmap(1, 1, Bitmap.Config.ARGB_4444)); cb.onImageReady(Bitmap.createBitmap(1, 1, Bitmap.Config.ARGB_4444));
} }
})) }))
.setImageReadyCallback(new ImagePresenter.ImageReadyCallback<Object>() { .setImageReadyCallback(new ImagePresenter.ImageReadyCallback<Object, Target>() {
@Override @Override
public void onImageReady(Object model, Target target, boolean fromCache) { public void onImageReady(Object model, Target target, boolean fromCache) {
wasImageSetCallbackCalled.set(true); wasImageSetCallbackCalled.set(true);
...@@ -314,7 +313,7 @@ public class ImagePresenterTest extends AndroidTestCase { ...@@ -314,7 +313,7 @@ public class ImagePresenterTest extends AndroidTestCase {
assertTrue(wasImageSetCallbackCalled.get()); assertTrue(wasImageSetCallbackCalled.get());
} }
private static class MockTarget implements Target { private static class MockTarget extends BaseTarget {
private Bitmap bitmap = null; private Bitmap bitmap = null;
private Drawable placeholder = null; private Drawable placeholder = null;
...@@ -347,15 +346,6 @@ public class ImagePresenterTest extends AndroidTestCase { ...@@ -347,15 +346,6 @@ public class ImagePresenterTest extends AndroidTestCase {
@Override @Override
public void startAnimation(Animation animation) { public void startAnimation(Animation animation) {
} }
@Override
public void setImagePresenter(ImagePresenter imagePresenter) {
}
@Override
public ImagePresenter getImagePresenter() {
return null;
}
} }
private static class MockImageLoader implements ImageLoader { private static class MockImageLoader implements ImageLoader {
......
library/tests/src/com/bumptech/glide/ListPreloaderTest.java
浏览文件 @ 8c7ae0d1
...@@ -168,10 +168,12 @@ public class ListPreloaderTest extends AndroidTestCase { ...@@ -168,10 +168,12 @@ public class ListPreloaderTest extends AndroidTestCase {
} }
private static class ListPreloaderAdapter extends ListPreloader<Object> { private static class ListPreloaderAdapter extends ListPreloader<Object> {
private final Context context;
public Bitmap bitmap = Bitmap.createBitmap(100, 100, Bitmap.Config.ARGB_8888); public Bitmap bitmap = Bitmap.createBitmap(100, 100, Bitmap.Config.ARGB_8888);
public ListPreloaderAdapter(Context context, int maxPreload) { public ListPreloaderAdapter(Context context, int maxPreload) {
super(context, maxPreload); super(maxPreload);
this.context = context;
} }
@Override @Override
...@@ -188,7 +190,7 @@ public class ListPreloaderTest extends AndroidTestCase { ...@@ -188,7 +190,7 @@ public class ListPreloaderTest extends AndroidTestCase {
@Override @Override
protected Glide.Request getRequest(Object item) { protected Glide.Request getRequest(Object item) {
return Glide.using(new StreamModelLoader<Object>() { return Glide.with(context).using(new StreamModelLoader<Object>() {
@Override @Override
public ResourceFetcher<InputStream> getResourceFetcher(final Object model, int width, int height) { public ResourceFetcher<InputStream> getResourceFetcher(final Object model, int width, int height) {
return new ResourceFetcher<InputStream>() { return new ResourceFetcher<InputStream>() {
......
samples/flickr/src/com/bumptech/glide/samples/flickr/FlickrPhotoGrid.java
浏览文件 @ 8c7ae0d1
...@@ -18,7 +18,7 @@ import com.bumptech.glide.loader.bitmap.model.Cache; ...@@ -18,7 +18,7 @@ import com.bumptech.glide.loader.bitmap.model.Cache;
import com.bumptech.glide.loader.bitmap.transformation.CenterCrop; import com.bumptech.glide.loader.bitmap.transformation.CenterCrop;
import com.bumptech.glide.loader.image.ImageManagerLoader; import com.bumptech.glide.loader.image.ImageManagerLoader;
import com.bumptech.glide.presenter.ImagePresenter; import com.bumptech.glide.presenter.ImagePresenter;
import com.bumptech.glide.presenter.target.Target; import com.bumptech.glide.presenter.target.ImageViewTarget;
import com.bumptech.glide.resize.load.Downsampler; import com.bumptech.glide.resize.load.Downsampler;
import com.bumptech.glide.samples.flickr.api.Photo; import com.bumptech.glide.samples.flickr.api.Photo;
...@@ -72,9 +72,11 @@ public class FlickrPhotoGrid extends SherlockFragment implements PhotoViewer { ...@@ -72,9 +72,11 @@ public class FlickrPhotoGrid extends SherlockFragment implements PhotoViewer {
private class FlickrPreloader extends ListPreloader<Photo> { private class FlickrPreloader extends ListPreloader<Photo> {
private final int[] dimens = new int[] { photoSize, photoSize }; private final int[] dimens = new int[] { photoSize, photoSize };
private final Context context;
public FlickrPreloader(Context context, int toPreload) { public FlickrPreloader(Context context, int toPreload) {
super(context, toPreload); super(toPreload);
this.context = context;
} }
@Override @Override
...@@ -88,8 +90,9 @@ public class FlickrPhotoGrid extends SherlockFragment implements PhotoViewer { ...@@ -88,8 +90,9 @@ public class FlickrPhotoGrid extends SherlockFragment implements PhotoViewer {
} }
@Override @Override
protected Glide.Request getRequest(Photo item) { protected Glide.Request<Photo> getRequest(Photo item) {
return Glide.using(new FlickrModelLoader(getActivity(), urlCache)) return Glide.with(context)
.using(new FlickrModelLoader(getActivity(), urlCache))
.load(item) .load(item)
.centerCrop(); .centerCrop();
} }
...@@ -126,7 +129,7 @@ public class FlickrPhotoGrid extends SherlockFragment implements PhotoViewer { ...@@ -126,7 +129,7 @@ public class FlickrPhotoGrid extends SherlockFragment implements PhotoViewer {
@Override @Override
public View getView(int position, View view, ViewGroup container) { public View getView(int position, View view, ViewGroup container) {
final Photo current = photos.get(position); final Photo current = photos.get(position);
final ImagePresenter<Photo> imagePresenter; final ImagePresenter<Photo, ImageViewTarget> imagePresenter;
if (view == null) { if (view == null) {
ImageView imageView = (ImageView) inflater.inflate(R.layout.flickr_photo_grid_item, container, false); ImageView imageView = (ImageView) inflater.inflate(R.layout.flickr_photo_grid_item, container, false);
ViewGroup.LayoutParams params = imageView.getLayoutParams(); ViewGroup.LayoutParams params = imageView.getLayoutParams();
...@@ -135,18 +138,18 @@ public class FlickrPhotoGrid extends SherlockFragment implements PhotoViewer { ...@@ -135,18 +138,18 @@ public class FlickrPhotoGrid extends SherlockFragment implements PhotoViewer {
final Context context = getActivity(); final Context context = getActivity();
//this is an example of how one might use ImagePresenter directly, there is otherwise no particular // This is an example of how one might use ImagePresenter directly, there is otherwise no particular
//reason why ImagePresenter is used here and not in FlickrPhotoList. // reason why ImagePresenter is used here and not in FlickrPhotoList.
final Animation fadeIn = AnimationUtils.loadAnimation(context, R.anim.fade_in); final Animation fadeIn = AnimationUtils.loadAnimation(context, R.anim.fade_in);
imagePresenter = new ImagePresenter.Builder<Photo>() imagePresenter = new ImagePresenter.Builder<Photo, ImageViewTarget>()
.setBitmapLoadFactory(new BaseBitmapLoadFactory<Photo, InputStream, Void>( .setBitmapLoadFactory(new BaseBitmapLoadFactory<Photo, InputStream, Void>(
new FlickrModelLoader(context, urlCache), Downsampler.AT_LEAST, new FlickrModelLoader(context, urlCache), Downsampler.AT_LEAST,
new CenterCrop<Photo>())) new CenterCrop<Photo>()))
.setImageView(imageView) .setTarget(new ImageViewTarget(imageView), context)
.setImageLoader(new ImageManagerLoader(context)) .setImageLoader(new ImageManagerLoader(context))
.setImageReadyCallback(new ImagePresenter.ImageReadyCallback<Photo>() { .setImageReadyCallback(new ImagePresenter.ImageReadyCallback<Photo, ImageViewTarget>() {
@Override @Override
public void onImageReady(Photo photo, Target target, boolean fromCache) { public void onImageReady(Photo photo, ImageViewTarget target, boolean fromCache) {
if (!fromCache) { if (!fromCache) {
target.startAnimation(fadeIn); target.startAnimation(fadeIn);
} }
...@@ -156,7 +159,7 @@ public class FlickrPhotoGrid extends SherlockFragment implements PhotoViewer { ...@@ -156,7 +159,7 @@ public class FlickrPhotoGrid extends SherlockFragment implements PhotoViewer {
view = imageView; view = imageView;
view.setTag(imagePresenter); view.setTag(imagePresenter);
} else { } else {
imagePresenter = (ImagePresenter<Photo>) view.getTag(); imagePresenter = (ImagePresenter<Photo, ImageViewTarget>) view.getTag();
} }
imagePresenter.setModel(current); imagePresenter.setModel(current);
......
samples/flickr/src/com/bumptech/glide/samples/flickr/FlickrPhotoList.java
浏览文件 @ 8c7ae0d1
...@@ -13,6 +13,7 @@ import com.actionbarsherlock.app.SherlockFragment; ...@@ -13,6 +13,7 @@ import com.actionbarsherlock.app.SherlockFragment;
import com.bumptech.glide.Glide; import com.bumptech.glide.Glide;
import com.bumptech.glide.ListPreloader; import com.bumptech.glide.ListPreloader;
import com.bumptech.glide.loader.bitmap.model.Cache; import com.bumptech.glide.loader.bitmap.model.Cache;
import com.bumptech.glide.presenter.target.ImageViewTarget;
import com.bumptech.glide.samples.flickr.api.Photo; import com.bumptech.glide.samples.flickr.api.Photo;
import java.net.URL; import java.net.URL;
...@@ -50,20 +51,22 @@ public class FlickrPhotoList extends SherlockFragment implements PhotoViewer { ...@@ -50,20 +51,22 @@ public class FlickrPhotoList extends SherlockFragment implements PhotoViewer {
} }
private static class ViewHolder { private static class ViewHolder {
private final ImageView imageView;
private final TextView titleText; private final TextView titleText;
private final ImageViewTarget imageViewTarget;
public ViewHolder(ImageView imageView, TextView titleText) { public ViewHolder(ImageView imageView, TextView titleText) {
this.imageView = imageView;
this.titleText = titleText; this.titleText = titleText;
this.imageViewTarget = new ImageViewTarget(imageView);
} }
} }
private class FlickrListPreloader extends ListPreloader<Photo> { private class FlickrListPreloader extends ListPreloader<Photo> {
private final Context context;
private int[] photoDimens = null; private int[] photoDimens = null;
public FlickrListPreloader(Context context, int maxPreload) { public FlickrListPreloader(Context context, int maxPreload) {
super(context, maxPreload); super(maxPreload);
this.context = context;
} }
public boolean isDimensSet() { public boolean isDimensSet() {
...@@ -88,7 +91,8 @@ public class FlickrPhotoList extends SherlockFragment implements PhotoViewer { ...@@ -88,7 +91,8 @@ public class FlickrPhotoList extends SherlockFragment implements PhotoViewer {
@Override @Override
protected Glide.Request getRequest(Photo item) { protected Glide.Request getRequest(Photo item) {
return Glide.using(new FlickrModelLoader(getActivity(), urlCache)) return Glide.with(context)
.using(new FlickrModelLoader(getActivity(), urlCache))
.load(item) .load(item)
.centerCrop(); .centerCrop();
} }
...@@ -144,11 +148,12 @@ public class FlickrPhotoList extends SherlockFragment implements PhotoViewer { ...@@ -144,11 +148,12 @@ public class FlickrPhotoList extends SherlockFragment implements PhotoViewer {
viewHolder = (ViewHolder) view.getTag(); viewHolder = (ViewHolder) view.getTag();
} }
Glide.using(new FlickrModelLoader(getActivity(), urlCache)) Glide.with(getActivity())
.using(new FlickrModelLoader(getActivity(), urlCache))
.load(current) .load(current)
.centerCrop() .centerCrop()
.animate(R.anim.fade_in) .animate(R.anim.fade_in)
.into(viewHolder.imageView); .into(viewHolder.imageViewTarget);
viewHolder.titleText.setText(current.title); viewHolder.titleText.setText(current.title);
return view; return view;
......
Markdown is supported
0% 或 .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑！
想要评论请 注册 或