0

Здравствуйте! Имею вот такой фрагмент кода, который применяется более чем в 10-ти фрагментах, подскажите как его можно стандартизировать, что бы его можно было вызвать одной-двумя строками. Я так понимаю необходимо реализовывать интерфейс обратного вызова, но не совсем понимаю как это реализовать.

public class MyFragment extends Fragment {
private RecyclerView recyclerView;
private LinearLayoutManager layoutManager;

private BookListAdapter bookListAdapter;
private List<Book> bookList = new ArrayList<>();

public MyFragment() { }

@Override
public void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
}

@Override
public View onCreateView(LayoutInflater inflater, ViewGroup container,
                         Bundle savedInstanceState) {
    final View view = inflater.inflate(R.layout.fragment, container, false);
    final Context context = getActivity();

    tests();
    Log.e("Hrre ","qwertyuio " + bookList.get(2).getAuthor());

    bookListAdapter = new BookListAdapter(context, bookList);

    recyclerView = (RecyclerView) view.findViewById(R.id.recyclerView);

    layoutManager = new LinearLayoutManager(getActivity());
    layoutManager.setOrientation(LinearLayoutManager.VERTICAL);

    recyclerView.setItemAnimator(new DefaultItemAnimator());

    recyclerView.setAdapter(bookListAdapter);
    recyclerView.setLayoutManager(layoutManager);

    // bookListAdapter.notifyDataSetChanged();
    return view;
}

}

9
  • не совсем понятен вопрос. Вы хотите узнать, как вставить его 10 раз в разметку или программно? Если фрагмент прописан и должным образом вставлен, дополнительно обращаться к нему не нужно.
    – Jarvis_J
    13 апр 2018 в 5:29
  • а какие-то функции во фрагменте будут изменяться, а какие-то оставаться базовыми?
    – Jarvis_J
    13 апр 2018 в 5:41
  • @Jarvis_J, Хотел исключить копирование кода во всех фрагментах, и упростить будущую поддержку. хотел сделать что-то на подобие: class Fragment_1 extends MyFragment { public void myMethods() { bookList = getBookList(); myFragment(bookList); } }
    – Valeriy
    13 апр 2018 в 5:42
  • по правилам хорошего кода booklist вообще лучше хранить отдельно и создавать фрагмент через newInstance, передавая туда аргументом позицию списка. И при необходимости получать booklist оттуда, где он хранится. Удобный паттерн MCV. Если вам нужна реализация именно с наследованием, напишите, что именно не получается в вашем варианте (с extends)?
    – Jarvis_J
    13 апр 2018 в 5:48
  • 1
    вот хорошая большая книжка по этому (и не только) поводу: infasoft.com/library/books/… . Ну и касательно вашего вопроса, сейчас напишу ответ на то как я понял вопрос
    – Jarvis_J
    13 апр 2018 в 6:19

2 ответа 2

1

Делаете BaseFragment (например) который в свою очередь наследуется от Fragmen. В этом BaseFragment реализуете всю логика которая повторяется, потом каждый новый фрагмент в котором есть такая же логика наследуете от BaseFragment и все. Если Вам необходимы какие-то методы BaseFragment вызывать непосредственно из наследника задайте свойство видимости для нужных методов protected. Это базовые вещи Java ООП.

6
  • Видимо здесь в вопросе классическая проблема XY. Дело в том, что человеку в вопросе нужны полностью одинаковые классы, в которых отличаются только данные (разные наборы книг), которые нужно вывести на экран. Это решается не наследованием, а передачей данных в один и тот же класс. Человек не очень понимает, как именно решается его задача, стоит подождать, когда он снова появится и прояснит, в чем именно его проблема, а не его [неправильное] представление о ее решении.
    – pavlofff
    13 апр 2018 в 8:06
  • )) Ну тогда ждем пояснений автора 13 апр 2018 в 8:21
  • BaseFragment, это то что мне нужно. Только я не могу понять как его мне реализовать и как вызвать.
    – Valeriy
    13 апр 2018 в 12:10
  • Вот что должно получиться: есть одно активити, которое имеет несколько фрагментов: жанры, поиск, новинки и т.д., есть второе активити: которое также имеет жанры, новинки, и т.д., но уже с другой логикой работы. Но они все будут визуально иметь одинаковое оформление, это будет RecyclerView с CardView, которые будут программно формироваться из нескольких разметок (что-то на подобие динамической разметки в вебпрограммировании). Соответственно зачем мне писать в каждом фрагменте реализацию фрагмента которая приведена выше, хотя она будет идентичной для всех.
    – Valeriy
    13 апр 2018 в 12:17
  • @Valeriy если это будет 10 идентичных фрагментов, зачем их от чего то наследовать. Используйте один и тот же фрагмент, это так трудно понять разве. Никто не делает отдельный фрагмент, чтобы выести на экран жанры, другой - новинки и тд, если все это одно и тоже, но отсортированно или отфильтровано по разному.
    – pavlofff
    13 апр 2018 в 12:32
1

Храните ваш список отдельно от всего. В упрощении паттерна MVC:

public class BookLab {

private static BookLab bookLab;
private ArrayList<Book> books;
private ArrayList<Book> books2;
private ArrayList<Book> books3;
...

public static BookLab get(Context ctx) {
    if (bookLab== null) bookLab= new BookLab (ctx);
    return bookLab;
}

private BookLab (Context ctx) {
    books = new ArrayList<>();
    String [] books_names = ctx.getResources().getStringArray(R.array.ваш array из ресурсов);

    for (int i=0;i<book_names.lenght;i++){
        Book book = new Book();
        book.setName(book_names[i]);
        books.add(book);
    }

    повторить для остальных books

}

ArrayList<Book> getBooks (int number){
    switch(number){
         case 1: return books;
         case 2: return books1;
         case 3: return books2;
    }
}

Класс Book:

public class Book {
private String name, description;

public String getName() {
    return name;
}

public void setName(String name) {
    this.name = name;
}

public String getDescription() {
    return description;
}

public void setDescription(String description) {
    this.description = description;
}
}

Во фрагменте:

public static MyFragment newInstance(int number) {
    Bundle args = new Bundle();
    args.putInt("number", number);
    MyFragment fragment = new MyFragment ();
    fragment.setArguments(args);
    return fragment;
}

onCreateView(...){
    number_of_list = getArguments().getInt("number",0)
    myList = BookLab.get(getActivity()).getBooks(number_of_list);

}

int getNumberOfList(){
    return number_of_list;
}

Достать список из другого места: BookLab.get(getActivity()).getBooks(MyFragment.getNumberOfList());

7
  • благодарю, но это немного не то, хотя для меня будет полезно. Я хотел упростить непосредственно реализацию фрагментов, что бы в каждом фрагменте не создавать 30-50 строк одинакового кода, которые в будущем будет проблематично изменять и дополнять, ведь их таких будет более 10 классов (фрагментов). А так нужно изменить анимацию, изменил в родительском и всё отлично, нужно переделать тени, пожалуйста и т.д. Ведь все фрагменты находятся в одном приложении и должны себя вести плюс/минус одинаково, что-бы не вызывать ошеломление у пользователя
    – Valeriy
    13 апр 2018 в 6:55
  • А) а в чем тогда проблема с extends?
    – Jarvis_J
    13 апр 2018 в 6:56
  • я так понимаю вы создаёте класс содержащий в себе все массивы, и оттуда уже достаёте их. Это делается для упрощения доступа к данным?
    – Valeriy
    13 апр 2018 в 6:57
  • проблема в том что я не могу понять как в дочернем классе вызвать родительский класс и передать ему выводимые значения. Вызывать нужно onCreate() или onCreateView() и что им необходимо передавать? И как им передать мой массив?
    – Valeriy
    13 апр 2018 в 7:00
  • если вы работаете с наследником к родителю обращаться не нужно. Если вы хотите переписать какую-то родительскую переменную или функцию - делайте @Override myFunction (или myVar, определенная в паренте, = нужное значение.
    – Jarvis_J
    13 апр 2018 в 7:06

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими условиями использования и подтверждаете, что прочитали и поняли наши политику конфиденциальности и нормы поведения.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.