0

Не могу разобраться как часть кода разместить в другом классе и потом получить к нему доступ.

В классе Resurs я получаю из файла string.xml данные и загружаю их в массив String[] title, так же в этом же классе создаю метод findResurs(), где загружаю в List arrImage картинки, картинок в приложении более четырехсот, здесь для примера только две

public class Resurs {
    private Context context;
    List<Person> persons;

    public Resurs(Context context) {
        Resources res = context.getResources();
        // загружаю в массив
        String[] title = res.getStringArray(R.array.titleString);
    } 

    public void findResurs(){ 
        List arrImage = new ArrayList<>(); 
        arrImage.add(R.drawable.img1); 
        arrImage.add(R.drawable.img2);}
    }

Затем в Активити MainActivity подключаюсь к классу

Resurs resurs = new Resurs(this);

Теперь как можно загрузить данные из Resurs в persons, другими словами получить из Resurs title и arrImage1

int length = title.length;
persons = new ArrayList<>();
for(int i = 0; i < length; i++)
    persons.add(new Person(i, title[i]  arrImage1.get(i)));

Класс Person

public class Person {

long id;
String title;
int photoId;

// пустой конструктор
public Person() {
}

// конструктор с переменными

public Person(long id, String title,  int photoId1) {
    this.id = id;
    this.title = title;
    this.photoId = photoId;
}}
6
  • 2
    К вопросу не относится, но хранение ссылки на активити чревато утечками памяти.
    – post_zeew
    15 мая 2017 в 8:23
  • Да вроде ничего секретного в приложении нет да данных пользователя в приложении нет. Я не знаю, а что может утеч с приложение
    –  Artsait
    15 мая 2017 в 8:26
  • Называйте классы правильно, в английском языке есть слово Resource, но никак не Resurs
    – Silento
    15 мая 2017 в 8:26
  • 1
    Речь не про утечку конфиденциальной информации, а про memory leak.
    – post_zeew
    15 мая 2017 в 8:28
  • Так может стоит просто сделать title и arrImage полями класса Resurs, и создать геттеры? 15 мая 2017 в 8:48

1 ответ 1

1

Я бы предложил класс Resources(такое название правильнее Resurs) считать набором инструментов для загрузки данных из ресурсов, а метод получения персон реализовать статичным. Например так:

public class Resources {

    public static Person[] loadPersons(Context ctx){
        Resources res = context.getResources();
        // загружаю в массив
        String[] title = res.getStringArray(R.array.titleString);
        // статичное определение списка картинок
        int[] images = {R.drawable.img1, R.drawable.img2, ..., R.drawable.imgN};

        if(title.lenght != images.lenght)
            // количество картинок не соответствует количеству имён
            throw new IllegalStateException();

        Person[] persons = new Person[title.lenght];
        for(int i = 0; i < persons.length; i++)
            persons.add(new Person(i, title[i],  images.get(i)));

        return persons;
    }
}

Что касается кэширования - считаю, что это нужно оставить на усмотрение вызывающего кода. Не стоит перегружать класс Resources ещё и этой ответственностью.

1
  • Спасибо за ответ, но это немного не то. Я так понял вопрос лучше разбить на несколько частей и опубликовать отдельно, а то сейчас наверно не очень понятно, что именно нужно
    –  Artsait
    15 мая 2017 в 9:47

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

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