Fehler beim öffnen der Datenbank

ChristophB

Hallo,

ich habe ein kleines Problem beim ausführenden der Folgenden Funktion:

Code

public ArrayList&lt;SportActivity&gt; getAllActivities(){
		
		try{
			//open readable db
			dbActivitiesTracker = this.getReadableDatabase();
			
			//add new sportActivity
			Cursor activityCursor = dbActivitiesTracker.query(
					TABLE_NAME,
					new String[] {
							TABLE_IDENTIFYER,
							TABLE_COLUMN_ACTIVITYNAME},
					null, null, null, null, null);
			
			//create ArrayList&lt;SportActivity&gt;
			ArrayList sportActivities = new ArrayList&lt;SportActivity&gt;();
			
			while(activityCursor.isAfterLast()){
				//define new SportActivity
				SportActivity sportActivity = new SportActivity();
				sportActivity.setActivityID(activityCursor.getInt(0));
				sportActivity.setActivityName(activityCursor.getString(1));
				
				//add SportActivity to ArrayList
				sportActivities.add(sportActivity);
				
				//increment cursor
				activityCursor.moveToNext();
			}
			
			//close db
			dbActivitiesTracker.close();
			
			//return ArrayList
			return sportActivities;
		}
		catch(SQLException e)
		{
			return null;
		}
	}

Alles anzeigen

Diese Funktion habe ich einer Klasse implementiert die von SQLiteOpenHelper erbt.
Sobald die Zeile dbActivitiesTracker = this.getReadableDatabase(); erreicht wird der try-Block verlassen.

Was mache ich dabei falsch?

-Edit-
Der Fehler ist noch woanders aufgetreten... und lag an etwas anderem. Aber danke fü eure Mühe!

titus

Hi ChristophB,

Java

catch(SQLException e)
{
	return null;
}

ein return null beim Exception Handling solltest du vermeiden weil du so keine Ahnung hast warum dein Code nicht ausgeführt wird.

besser wäre:

Java

try {
   
       //dein Code


} catch (Exception e) {
	// TODO Auto-generated catch block
	e.printStackTrace();


       return null;
}

Alles anzeigen

Mfg Titus

Edit: Hab eben nochmal deinen Code angesehen und festgestellt das der Try/catch - block innerhalb eines Funktionsaufruf ist und falls ein Fehler auftritt deshalb das return null zurück geben wird...die Exception ausgabe sollteste aber trotzdem drin haben ist einfach besser für die Fehlersuche

Jetzt mitmachen!