ArrayList füllen

[Leon]

Hallo Leute

Komme nicht darauf wie ich mein ArrayList mit den Daten in der while schleife füllen soll 🙁

import java.sql.Connection;
import java.sql.ResultSet;
import java.sql.SQLException;
import java.sql.Statement;
import java.util.ArrayList;
import java.util.Collection;


public class UserMsg
{
	private int id;

	private String date;

	private String title;

	private String role;

	private String msg;

	private String altMsg;	
	
	private ArrayList newsArrayList;
	
	private static DBUser dbUser;

	public UserMsg () 
	{
		createNews ();		
	}

	public String getAltMsg ()
	{
		return altMsg;
	}

	public void setAltMsg (String altMsg)
	{
		this.altMsg = altMsg;
	}

	public String getDate ()
	{
		return date;
	}

	public void setDate (String date)
	{
		this.date = date;
	}

	public int getId ()
	{
		return id;
	}

	public void setId (int id)
	{
		this.id = id;
	}

	public String getMsg ()
	{
		return msg;
	}

	public void setMsg (String msg)
	{
		this.msg = msg;
	}

	public String getRole ()
	{
		return role;
	}

	public void setRole (String role)
	{
		this.role = role;
	}

	public String getTitle ()
	{
		return title;
	}

	public void setTitle (String title)
	{
		this.title = title;
	}
	
	public void createNews () 
	{		
		try
		{	
			dbUser = new DBUser();			
			final Connection conn = dbUser.getConnection ();	
			newsArrayList = new ArrayList ();
			Statement statement = conn.createStatement ();
			ResultSet rSet = statement.executeQuery ("SELECT * from News");

			while (rSet.next ())
			{
				setId (rSet.getInt ("id"));
				setAltMsg (rSet.getString ("altMsg"));
				setDate (rSet.getString ("date"));
				setMsg (rSet.getString ("msg"));
				setRole (rSet.getString ("role"));
				setTitle (rSet.getString ("title"));

			newsArrayList.add (); //???????????
			}
			rSet.close ();
			statement.close ();
			conn.close ();

		}
		catch (SQLException e)
		{

		}
	}

	public Collection getNews ()
	{
		return newsArrayList;
	}

	

}

Dankbar für jede Hilfe

 

[maki]

         while (rSet.next ())
         {
            setId (rSet.getInt ("id"));
            setAltMsg (rSet.getString ("altMsg"));
            setDate (rSet.getString ("date"));
            setMsg (rSet.getString ("msg"));
            setRole (rSet.getString ("role"));
            setTitle (rSet.getString ("title"));

         newsArrayList.add (); //???????????
         }

du überschreibst de ganze Zeit die Eigenschaften desselben objektes (this) und fügst nix deiner liste hinzu.

sollte heissen:

         UserMsg userMsg;
         while (rSet.next ())
         {
            usrMsg= new UserMsg();
            
            usrMsg.setId (rSet.getInt ("id"));
            usrMsg.setAltMsg (rSet.getString ("altMsg"));
            usrMsg.setDate (rSet.getString ("date"));
            usrMsg.setMsg (rSet.getString ("msg"));
            usrMsg.setRole (rSet.getString ("role"));
            usrMsg.setTitle (rSet.getString ("title"));

         newsArrayList.add(usrMsg); 
         }

Gibt es einen Grund, warum createNews nicht static sein sollte und eine Liste zurückgeben sollte?
Dann aber die Threads(?) synchronisieren.

Asserdem wäre es besser

private List newsArrayList;

anstatt

private ArrayList newsArrayList;

zu benutzen.

 

[Leon]

Vielen Dank maki

Gibt es einen Grund, warum createNews nicht static sein sollte und eine Liste zurückgeben sollte?
Dann aber die Threads(?) synchronisieren.

Asserdem wäre es besser

private List newsArrayList;

Werde gleich ändern :wink: