Recyclerview Adapter

redfox · 25. Juli 2023

Hallo,

ich möchte gerne in einem Recyclerview Item den letzten Eintrag von jeweils 3 Array Listen ausgeben. In den folgenden Items natürlich die nächsten Einträge der 3 Listen usw.. Leider verstehe ich nicht wieso nicht alle Daten vom letzten Eintrag in einem Recyclerview Item ausgegeben werden sondern die Einträge verteilt werden auf die nächsten Recyclerviews. Wo könnte das Problem liegen?

Code

public class EntryAdapter extends RecyclerView.Adapter<EntryAdapter.EntryViewHolder> {

    private List<Entry_Pagination> lagerhausEntries;
    private List<Entry_Pagination> jetEntries;
    private List<Entry_Pagination> shellEntries;

    public EntryAdapter(List<Entry_Pagination> lagerhausEntries, List<Entry_Pagination> jetEntries, List<Entry_Pagination> shellEntries) {
        this.lagerhausEntries = lagerhausEntries;
        this.jetEntries = jetEntries;
        this.shellEntries = shellEntries;
    }

    @NonNull
    @Override
    public EntryViewHolder onCreateViewHolder(@NonNull ViewGroup parent, int viewType) {
        View itemView = LayoutInflater.from(parent.getContext()).inflate(R.layout.entry_item_layout, parent, false);
        return new EntryViewHolder(itemView);
    }

    @Override
    public void onBindViewHolder(@NonNull EntryViewHolder holder, int position) {
        // Determine which tankstelle (gas station) the entry belongs to
        Entry_Pagination entry;
        int itemViewType = getItemViewType(position);
        if (itemViewType == 0) {
            entry = lagerhausEntries.get(position);
            holder.bindEntry("liezen_lagerhaus_tankstelle", entry);
        } else if (itemViewType == 1) {
            entry = jetEntries.get(position - lagerhausEntries.size());
            holder.bindEntry("liezen_jet_tankstelle", entry);
        } else {
            entry = shellEntries.get(position - lagerhausEntries.size() - jetEntries.size());
            holder.bindEntry("trautenfels_shell_tankstelle", entry);
        }
    }


    @Override
    public int getItemViewType(int position) {
        if (position < lagerhausEntries.size()) {
            return 0; // Lagerhaus Tankstelle
        } else if (position < lagerhausEntries.size() + jetEntries.size()) {
            return 1; // Jet Tankstelle
        } else {
            return 2; // Shell Tankstelle
        }
    }

    @Override
    public int getItemCount() {
        // Total count is the sum of entries from all tankstellen
        return lagerhausEntries.size() + jetEntries.size() + shellEntries.size();
    }


    public static class EntryViewHolder extends RecyclerView.ViewHolder {

        private CardView cardView;
        private TextView priceTextView;
        private TextView fuelTypeTextView;
        private TextView tankstelleTextView;
        private TextView adresseTextView;
        private TextView zeitTextView;

        private TextView priceTextView2;
        private TextView fuelTypeTextView2;
        private TextView tankstelleTextView2;
        private TextView adresseTextView2;
        private TextView zeitTextView2;

        private TextView priceTextView3;
        private TextView fuelTypeTextView3;
        private TextView tankstelleTextView3;
        private TextView adresseTextView3;
        private TextView zeitTextView3;

        public EntryViewHolder(@NonNull View itemView) {
            super(itemView);

            cardView = itemView.findViewById(R.id.cv);

            // Initialize views for Lagerhaus Tankstelle
            priceTextView = itemView.findViewById(R.id.tvPreis);
            fuelTypeTextView = itemView.findViewById(R.id.tvTreibstoff);
            tankstelleTextView = itemView.findViewById(R.id.tvTankstelle);
            adresseTextView = itemView.findViewById(R.id.tvAdresse);
            zeitTextView = itemView.findViewById(R.id.tvZeit);

            // Initialize views for Jet Tankstelle
            priceTextView2 = itemView.findViewById(R.id.tvPreis2);
            fuelTypeTextView2 = itemView.findViewById(R.id.tvTreibstoff2);
            tankstelleTextView2 = itemView.findViewById(R.id.tvTankstelle2);
            adresseTextView2 = itemView.findViewById(R.id.tvAdresse2);
            zeitTextView2 = itemView.findViewById(R.id.tvZeit2);

            // Initialize views for Shell Tankstelle
            priceTextView3 = itemView.findViewById(R.id.tvPreis3);
            fuelTypeTextView3 = itemView.findViewById(R.id.tvTreibstoff3);
            tankstelleTextView3 = itemView.findViewById(R.id.tvTankstelle3);
            adresseTextView3 = itemView.findViewById(R.id.tvAdresse3);
            zeitTextView3 = itemView.findViewById(R.id.tvZeit3);
        }

        public void bindEntry(String tankstelleName, Entry_Pagination entry) {
            // Bind entry data to views here

            if (tankstelleName.equals("liezen_lagerhaus_tankstelle")) {
                priceTextView.setText(entry.getPrice());
                fuelTypeTextView.setText(entry.getFuelType());
                tankstelleTextView.setText(entry.getStation());
                adresseTextView.setText(entry.getAddress());
                zeitTextView.setText(entry.getTime());
            } else if (tankstelleName.equals("liezen_jet_tankstelle")) {
                priceTextView2.setText(entry.getPrice());
                fuelTypeTextView2.setText(entry.getFuelType());
                tankstelleTextView2.setText(entry.getStation());
                adresseTextView2.setText(entry.getAddress());
                zeitTextView2.setText(entry.getTime());
            } else if (tankstelleName.equals("trautenfels_shell_tankstelle")) {
                priceTextView3.setText(entry.getPrice());
                fuelTypeTextView3.setText(entry.getFuelType());
                tankstelleTextView3.setText(entry.getStation());
                adresseTextView3.setText(entry.getAddress());
                zeitTextView3.setText(entry.getTime());
            }
        }
    }
}

Alles anzeigen

nono124 · 25. Juli 2023

Zitat

ich möchte gerne in einem Recyclerview Item den letzten Eintrag von jeweils 3 Array Listen ausgeben.

Das heist für mich du willst die Liste von unten nach oben durch gehen.

Zitat

einem Recyclerview Item ausgegeben werden sondern die Einträge verteilt werden auf die nächsten Recyclerviews.

Du hast nur ein Recyclerview nicht mehrere.

So wie ich es sehen hasst du alle angaben der drei Listen in einem Item. In deinem Layout R.layout.entry_item_layout sind bestimmt 16 TextViews enthalten. Das Ist dann für das Recycler ein Eintrag eine Pos.

somit ist auch die Rückgabe der Anzahl der Items falsch „getItemCount()“

Da reicht die Anzahl einer Liste. wenn in der Liste 10 Einträge sind werden es auch nur 10 Zeilen sein. Auch wenn dein Layout vielleicht 5 Zeilen hat für das Recycler ist es eine Pos.

zeige mal das Layout

Wenn ich falsch liege beschreibe dein Problem genauer.

nono124 · 25. Juli 2023

getItemViewType ist sicher nicht das was du eigentlich willst.

How to create RecyclerView with multiple view types

From Create dynamic lists with RecyclerView: When we create a RecyclerView.Adapter we have to specify ViewHolder that will bind with the adapter. public class…

stackoverflow.com

redfox · 27. Juli 2023

Sorry, hab erst jetzt bemerkt das wer auf meinen Post geantwortet hat.

Ich möchte jeweils den letzten Eintrag der 3 Listen im entry_item_layout ausgeben.

Hier ist das XML davon:

XML

<?xml version="1.0" encoding="utf-8"?>
<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:app="http://schemas.android.com/apk/res-auto"
    android:layout_width="match_parent"
    android:layout_height="wrap_content"
    android:orientation="vertical">

    <androidx.cardview.widget.CardView
        android:id="@+id/cv"
        android:layout_width="match_parent"
        android:layout_height="match_parent">

        <LinearLayout
            android:layout_width="match_parent"
            android:layout_height="match_parent"
            android:layout_marginTop="10dp"
            android:orientation="vertical"
            android:paddingLeft="30dp"
            android:paddingEnd="30dp">

            <LinearLayout
                android:id="@+id/layoutLagerhaus"
                android:layout_width="match_parent"
                android:layout_height="match_parent"
                android:orientation="horizontal">

                <TextView
                    android:id="@+id/tvPreis"
                    android:layout_width="wrap_content"
                    android:layout_height="wrap_content"
                    android:layout_gravity="right|end"
                    android:text="preis"
                    android:textSize="34sp" />

                <TextView
                    android:id="@+id/tvTreibstoff"
                    android:layout_width="wrap_content"
                    android:layout_height="wrap_content"
                    android:layout_weight="1"
                    android:gravity="right"
                    android:text="treibstoff"
                    android:textSize="24sp" />
            </LinearLayout>

            <TextView
                android:id="@+id/tvTankstelle"
                android:layout_width="match_parent"
                android:layout_height="wrap_content"
                android:text="tankstelle"
                android:textSize="24sp" />

            <TextView
                android:id="@+id/tvAdresse"
                android:layout_width="match_parent"
                android:layout_height="wrap_content"
                android:text="adresse" />

            <LinearLayout
                android:layout_width="match_parent"
                android:layout_height="match_parent"
                android:layout_marginTop="10dp"
                android:layout_marginBottom="10dp"
                android:orientation="horizontal">

                <TextView
                    android:id="@+id/tvZeit"
                    android:layout_width="match_parent"
                    android:layout_height="wrap_content"
                    android:gravity="right"
                    android:text="Zeit"
                    android:textSize="16sp" />

            </LinearLayout>

            <View
                android:id="@+id/divider2"
                android:layout_width="match_parent"
                android:layout_height="1dp"
                android:layout_weight="1"
                android:background="?android:attr/listDivider" />

            <LinearLayout
                android:id="@+id/layoutJet"
                android:layout_width="match_parent"
                android:layout_height="match_parent">

                <TextView
                    android:id="@+id/tvPreis2"
                    android:layout_width="wrap_content"
                    android:layout_height="wrap_content"
                    android:layout_gravity="right|end"
                    android:layout_weight="1"
                    android:text="preis"
                    android:textSize="34sp" />

                <TextView
                    android:id="@+id/tvTreibstoff2"
                    android:layout_width="wrap_content"
                    android:layout_height="wrap_content"
                    android:layout_weight="1"
                    android:gravity="right"
                    android:text="treibstoff"
                    android:textSize="24sp" />
            </LinearLayout>

            <TextView
                android:id="@+id/tvTankstelle2"
                android:layout_width="match_parent"
                android:layout_height="wrap_content"
                android:text="tankstelle"
                android:textSize="24sp" />

            <TextView
                android:id="@+id/tvAdresse2"
                android:layout_width="match_parent"
                android:layout_height="wrap_content"
                android:text="adresse" />

            <LinearLayout
                android:layout_width="match_parent"
                android:layout_height="match_parent">

                <TextView
                    android:id="@+id/tvZeit2"
                    android:layout_width="match_parent"
                    android:layout_height="wrap_content"
                    android:layout_weight="1"
                    android:gravity="right"
                    android:text="Zeit"
                    android:textSize="16sp" />

            </LinearLayout>

            <View
                android:id="@+id/divider3"
                android:layout_width="match_parent"
                android:layout_height="1dp"
                android:background="?android:attr/listDivider" />

            <LinearLayout
                android:id="@+id/layoutShell"
                android:layout_width="match_parent"
                android:layout_height="match_parent">

                <TextView
                    android:id="@+id/tvPreis3"
                    android:layout_width="match_parent"
                    android:layout_height="wrap_content"
                    android:layout_gravity="right|end"
                    android:layout_weight="1"
                    android:text="preis"
                    android:textSize="34sp" />

                <TextView
                    android:id="@+id/tvTreibstoff3"
                    android:layout_width="match_parent"
                    android:layout_height="wrap_content"
                    android:layout_weight="1"
                    android:gravity="right"
                    android:text="treibstoff"
                    android:textSize="24sp" />
            </LinearLayout>

            <TextView
                android:id="@+id/tvTankstelle3"
                android:layout_width="match_parent"
                android:layout_height="wrap_content"
                android:text="tankstelle"
                android:textSize="24sp" />

            <TextView
                android:id="@+id/tvAdresse3"
                android:layout_width="match_parent"
                android:layout_height="wrap_content"
                android:text="adresse" />

            <LinearLayout
                android:layout_width="match_parent"
                android:layout_height="match_parent">

                <TextView
                    android:id="@+id/tvZeit3"
                    android:layout_width="match_parent"
                    android:layout_height="wrap_content"
                    android:layout_weight="1"
                    android:gravity="right"
                    android:text="Zeit"
                    android:textSize="16sp" />
            </LinearLayout>

        </LinearLayout>
    </androidx.cardview.widget.CardView>

    <View
        android:id="@+id/divider"
        android:layout_width="match_parent"
        android:layout_height="10dp"
        android:layout_marginTop="10dp"
        android:layout_marginBottom="10dp"
        android:background="?android:attr/listDivider"
        android:backgroundTint="#FF1100" />

</LinearLayout>

Alles anzeigen

nono124 · 27. Juli 2023

Zitat

Ich möchte jeweils den letzten Eintrag der 3 Listen im entry_item_layout ausgeben.

Wenn du nur den letzten Eintag in der Liste ausgeben willst dann Brauchst du doch gar kein Recyclerview.

nono124 · 27. Juli 2023

Code

String st = lagerhausEntries.get(lagerhausEntries.size()-1).getPrice;
// sollte dir den letzten eintrag der liste geben.

redfox · 27. Juli 2023

Nein so meine ich das nicht. Ich möchte jeweils die letzten Einträge der 3 Listen in einem entry_item_layout bzw einem Reylcerview Item ausgeben. Die nächsten Einträge der 3 Listen in folgende Recyclerviews bzw in einem neuen entry_item_layout. Bin nicht sicher ob das verständlich ist aber hier ein Screenshot um das zu verdeutlichen:

recycler hosted at ImgBB

Image recycler hosted in ImgBB

ibb.co

Die Werte von "liezen_lagerhaus_tankstelle" sollen in das Layout "layoutLagerhaus", die Werte von "liezen_jet_tankstelle" in das Layout "layoutJet" und die Werte von "trautenfels_shell_tankstelle" in das Layout "layoutShell" der entry_item_layout.xml.

Da wo noch "preis" steht sollen die entsprechenden Werte eingefügt werden also noch "liezen_jet_tankstelle" und "trautenfels_shell_tankstelle".

nono124 · 27. Juli 2023

Das mit dem getItemViewType ist mist und brauchst du in deinem Fall auch nicht.

Du willst doch alle Felder von deinem layout füllen. Also musst du das auch auf einmal in der

onBindViewHolder -> bindEntry Methode machen ohne den getItemView…

Da musst du allen TextView bei einem durchlauf setzen.

Und der Rückgabewert getItemCount() ist nur die Anzahl einer Liste.

also wenn in einer Liste 10 sind ist der Wert auch nur 10 und nicht 30

nono124 · 27. Juli 2023

Code

@Override
    public void onBindViewHolder(@NonNull EntryViewHolder holder, int position) {
        // Determine which tankstelle (gas station) the entry belongs to
           
        
                holder.priceTextView.setText(lagerhausEntries.get(position).getPrice());
                holder.fuelTypeTextView.setText(lagerhausEntries.get(position).getFuelType());
                //  ....
                //alle 15
        
       
    }

Alles anzeigen

nathanroberts · 4. Oktober 2024

It sounds like the issue with your RecyclerView is that you are not properly handling the position when binding data from different lists. Since you're trying to display the last entries of each list, the logic for determining which entry to show is misaligned.

You can try the below solution:

Change how you handle positions in onBindViewHolder: Instead of assuming the same position across different lists, you should calculate the position based on the sizes of the lists.

Bind data only if the position is within the bounds of the list: Check if you are trying to access an index that exists in each list.

Here’s an updated version of your onBindViewHolder method:

Code

@Override
public void onBindViewHolder(@NonNull EntryViewHolder holder, int position) {
    Entry_Pagination entry;

    if (position < lagerhausEntries.size()) {
        entry = lagerhausEntries.get(position);
        holder.bindEntry("liezen_lagerhaus_tankstelle", entry);
    } else if (position < lagerhausEntries.size() + jetEntries.size()) {
        entry = jetEntries.get(position - lagerhausEntries.size());
        holder.bindEntry("liezen_jet_tankstelle", entry);
    } else {
        entry = shellEntries.get(position - lagerhausEntries.size() - jetEntries.size());
        holder.bindEntry("trautenfels_shell_tankstelle", entry);
    }
}

Alles anzeigen

By ensuring that each position corresponds to the correct entry in its respective list, you should see the last entries displayed correctly in the RecyclerView.

Recyclerview Adapter

Jetzt mitmachen!

Teilen

Ähnliche Themen

RecyclerView ItemClick Position aus ViewAdapter an Fragment übergeben ?

Hilfe beim einbinden von RecyclerView oder LiestView in Fragment

Tags

Benutzer online in diesem Thema