Повторное использование метода модернизации

У меня есть метод, который вызывает мой сервер, используя Retrofit:

public class MainActivity extends AppCompatActivity {

    // ... activity methods here, removed for simplicity ...

    // Used to subscribe to a user given their userId
    public void subscribeToUser(int userId) {
        final ApiInterface apiService = ApiClient.createService(ApiInterface.class);

        Call<BasicResponse> call = apiService.subscribe(userId);
        call.enqueue(new Callback<BasicResponse>() {
            @Override
            public void onResponse(Call<BasicResponse> call, Response<BasicResponse> response) {
                if (response.isSuccessful()) {
                    Toast.makeText(MainActivity.this, "Success", Toast.LENGTH_LONG).show();
                } else {
                    Toast.makeText(MainActivity.this, "Failed", Toast.LENGTH_LONG).show();
                }
            }

            @Override
            public void onFailure(Call<BasicResponse> call, Throwable t) {
                Log.e(TAG, t.toString());
            }
        });
    }

}

Теперь мне нужно использовать этот же метод (subscribeToUser()) в другом действии, но нет смысла копировать и вставлять метод в другое действие. Тогда у меня будет один и тот же код дважды.

Итак, могу ли я поместить метод в одно место и сообщить действиям, был ли вызов успешным или неудачным? Как мне это организовать?

Вот мой класс ApiClient.java:

public class ApiClient {

    public static final String API_BASE_URL = "http://www.website.com/api/";

    private static OkHttpClient.Builder httpClient = new OkHttpClient.Builder();

    private static Retrofit.Builder builder =
            new Retrofit.Builder()
                    .baseUrl(API_BASE_URL)
                    .addConverterFactory(GsonConverterFactory.create());

    public static <S> S createService(Class<S> serviceClass) {
        Retrofit retrofit = builder.client(httpClient.build()).build();

        return retrofit.create(serviceClass);
    }

    public static <S> S createService(Class<S> serviceClass, final String authToken) {
        if (authToken != null) {
            httpClient.addInterceptor(new Interceptor() {
                @Override
                public Response intercept(Interceptor.Chain chain) throws IOException {
                    Request original = chain.request();

                    // Request customization: add request headers
                    Request.Builder requestBuilder = original.newBuilder()
                            .header("Authorization", "Bearer " + authToken)
                            .method(original.method(), original.body());

                    Request request = requestBuilder.build();
                    return chain.proceed(request);
                }
            });
        }

        OkHttpClient client = httpClient.build();
        Retrofit retrofit = builder.client(client).build();

        return retrofit.create(serviceClass);
    }
}

А вот мой ApiInterface.java класс:

public interface ApiInterface {
    @FormUrlEncoded
    @POST("subscribe")
    Call<BasicResponse> subscribe(@Field("userId") Integer userId);
}

Спасибо.


android android-activity retrofit retrofit2
person user7669706    schedule 07.03.2017    source источник
comment
Вы пытаетесь использовать BaseActivity extend AppCompatActivity, содержащий метод subscribeToUser, а после этого MainActivity и другие действия расширяют BaseActivity.   -  person RoShan Shan    schedule 07.03.2017
comment
@RoShanShan Что делать, если мне нужно использовать метод subscribeToUser() вне действия, например, в адаптере?   -  person user7669706    schedule 07.03.2017
comment
Я думаю, вы можете использовать этот метод в Activity и отправить результат вашему адаптеру. Или вы можете отправить context на свой адаптер и использовать ((YouActivity)context).subscribeToUser().   -  person RoShan Shan    schedule 07.03.2017

Ответы (1)


arrow_upward
4
arrow_downward

На мой взгляд, createService(ApiInterface.class) не следует вызывать несколько раз. Это не обязательно и замедляет ваше приложение. Вы можете попытаться создать UserService с шаблоном singleton, как показано ниже:

public class UserService {

private UserService userService;
final ApiInterface apiService;
//Contructor private to prevent init object from outside directly.
private UserService() {
    apiService = ApiClient.createService(ApiInterface.class);
}
//use this method when you need to use UserService 
public static UserService getInstance() {
    if(userService ==  null) {
        userService = new UserService();
    }
}

// Used to subscribe to a user given their userId
public void subscribeToUser(int userId, ServiceCallBack serviceCallBack) {
    final ApiInterface apiService = ApiClient.createService(ApiInterface.class);

    Call<BasicResponse> call = apiService.subscribe(userId);
    call.enqueue(new Callback<BasicResponse>() {
        @Override
        public void onResponse(Call<BasicResponse> call, Response<BasicResponse> response) {
            if (response.isSuccessful()) {
                Toast.makeText(MainActivity.this, "Success", Toast.LENGTH_LONG).show();
                serviceCallBack.successful(response);
            } else {
                Toast.makeText(MainActivity.this, "Failed", Toast.LENGTH_LONG).show();
            }
        }

        @Override
        public void onFailure(Call<BasicResponse> call, Throwable t) {
            Log.e(TAG, t.toString());
            serviceCallBack.fail(t);
        }
    });
}
//this is callback interface, help you know whether success from outside.
interface ServiceCallBack {
     void successful(Response response);
    void fail(Throwable t);
}
}

Как пользоваться:

 UserService.getInstance(1, new ServiceCallBack(){

        @Override
        public void successful(Response response) {
            //process successful
        }

        @Override
        public void fail(Throwable t) {
            //process fail
        }
    });

Теперь вы можете поместить все методы, относящиеся к User API, в класс UserService для повторного использования.

person Luong Dinh    schedule 07.03.2017