Reputation: 11
Here is the code to my LoginDatabase class. I cant find an error. It crashes whenever I open it. I'm a beginner in android studio. I tried viewing all sites to compare.What am I missing? The database opens as soon as the first activity is created
package androiddev.task_one;
import android.content.ContentValues;
import android.content.Context;
import android.database.Cursor;
import android.database.sqlite.SQLiteDatabase;
import android.database.sqlite.SQLiteException;
import android.widget.Toast;
/**
* Created by Mahe on 7/9/2017.
*/
public class LoginDatabase {
final static String TABLE_NAME = "REGISTERED";
final static String DATABASE_CREATE = "create table" + TABLE_NAME + "(" +"ID"+ "int PRIMARY KEY AUTOINCREMENT" + "NAME text" + "Age int" + "EMAIL text" + "USERNAME text" + "PASSWORD text" + ")";
DataBaseHelper dbh;
Context context;
public LoginDatabase(Context context1){
this.context = context1;
dbh = new DataBaseHelper(context, TABLE_NAME, null, 1);
}
SQLiteDatabase db;
public LoginDatabase open() throws SQLiteException{
db = dbh.getWritableDatabase();
return this;
}
public void insertentry(String name, String age, String email, String username, String password){
ContentValues Values = new ContentValues();
Values.put("NAME", name);
Values.put("AGE", age);
Values.put("EMAIL", email);
Values.put("USERNAME", username);
Values.put("PASSWORD", password);
db.insert("REGISTERED", null, Values);
}
public String getPassword(String Username){
Cursor cursor = db.query("REGISTERED", null, "USERNAME", new String[]{Username}, null, null, null);
if (cursor.getCount()<1)
{ cursor.close();
return "Username does not exist";
}
else {
cursor.moveToFirst();
String pass = cursor.getString(cursor.getColumnIndex("PASSWORD"));
cursor.close();
return pass;
}
}
}
Log-
07-10 14:08:47.404 5443-5443/androiddev.task_one E/SQLiteLog: (1) near "tableREGISTERED": syntax error 07-10 14:08:47.404 5443-5443/androiddev.task_one D/AndroidRuntime: Shutting down VM 07-10 14:08:47.405 5443-5443/androiddev.task_one E/AndroidRuntime: FATAL EXCEPTION: main Process: androiddev.task_one, PID: 5443 java.lang.RuntimeException: Unable to start activity ComponentInfo{androiddev.task_one/androiddev.task_one.MainActivity}: android.database.sqlite.SQLiteException: near "tableREGISTERED": syntax error (code 1): , while compiling: create tableREGISTERED(IDint PRIMARY KEY AUTOINCREMENTNAME textAge intEMAIL textUSERNAME textPASSWORD text) at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:2646) at android.app.ActivityThread.handleLaunchActivity(ActivityThread.java:2707) at android.app.ActivityThread.-wrap12(ActivityThread.java) at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1460) at android.os.Handler.dispatchMessage(Handler.java:102) at android.os.Looper.loop(Looper.java:154) at android.app.ActivityThread.main(ActivityThread.java:6077) at java.lang.reflect.Method.invoke(Native Method) at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:865) at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:755) Caused by: android.database.sqlite.SQLiteException: near "tableREGISTERED": syntax error (code 1): , while compiling: create tableREGISTERED(IDint PRIMARY KEY AUTOINCREMENTNAME textAge intEMAIL textUSERNAME textPASSWORD text) at android.database.sqlite.SQLiteConnection.nativePrepareStatement(Native Method) at android.database.sqlite.SQLiteConnection.acquirePreparedStatement(SQLiteConnection.java:889) at android.database.sqlite.SQLiteConnection.prepare(SQLiteConnection.java:500) at android.database.sqlite.SQLiteSession.prepare(SQLiteSession.java:588) at android.database.sqlite.SQLiteProgram.(SQLiteProgram.java:58) at android.database.sqlite.SQLiteStatement.(SQLiteStatement.java:31) at android.database.sqlite.SQLiteDatabase.executeSql(SQLiteDatabase.java:1675) at android.database.sqlite.SQLiteDatabase.execSQL(SQLiteDatabase.java:1606) at androiddev.task_one.DataBaseHelper.onCreate(DataBaseHelper.java:20) at android.database.sqlite.SQLiteOpenHelper.getDatabaseLocked(SQLiteOpenHelper.java:251) at android.database.sqlite.SQLiteOpenHelper.getWritableDatabase(SQLiteOpenHelper.java:163) at androiddev.task_one.LoginDatabase.open(LoginDatabase.java:25) at androiddev.task_one.MainActivity.onCreate(MainActivity.java:20) at android.app.Activity.performCreate(Activity.java:6664) at android.app.Instrumentation.callActivityOnCreate(Instrumentation.java:1118) at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:2599) at android.app.ActivityThread.handleLaunchActivity(ActivityThread.java:2707) at android.app.ActivityThread.-wrap12(ActivityThread.java) at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1460) at android.os.Handler.dispatchMessage(Handler.java:102) at android.os.Looper.loop(Looper.java:154) at android.app.ActivityThread.main(ActivityThread.java:6077) at java.lang.reflect.Method.invoke(Native Method) at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:865) at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:755)
Upvotes: 0
Views: 1428
Reputation: 43
In the String DATABASE_CREATE use this
"create table " + TABLE_NAME + "(" +"ID"+ " int PRIMARY KEY AUTOINCREMENT, " + " NAME text," + " Age int," + " EMAIL text," + " USERNAME text," + " PASSWORD text" + ")";
Upvotes: 1
Reputation: 5063
You can write code bellow like
import android.content.ContentValues;
import android.content.Context;
import android.database.Cursor;
import android.database.sqlite.SQLiteDatabase;
import android.database.sqlite.SQLiteOpenHelper;
public class LoginDatabase extends SQLiteOpenHelper{
private static final String DBNAME = "dbname.db";
private static final int VERSION = 1;
public LoginDatabase(Context context) {
super(context, DBNAME, null, VERSION);
// TODO Auto-generated constructor stub
}
@Override
public void onCreate(SQLiteDatabase db) {
// TODO Auto-generated method stub
db.execSQL("create table REGISTERED(ID integer primary key autoincrement,NAME text,Age text,EMAIL text,USERNAME text,PASSWORD text)");
}
@Override
public void onUpgrade(SQLiteDatabase db, int oldVersion, int newVersion) {
db.execSQL("DROP TABLE IF EXISTS REGISTERED");
onCreate(db);
}
public long insertentry(String name, String age, String email, String username, String password) {
long rValue = -1;
SQLiteDatabase sqldb = this.getWritableDatabase();
ContentValues cv = new ContentValues();
cv.put("NAME", name);
cv.put("Age", age);
cv.put("EMAIL",email);
cv.put("USERNAME",username);
cv.put("PASSWORD",password);
rValue = sqldb.insert("REGISTERED", null, cv);
// /closeDB();
return rValue;
}
public String getPassword(String Username) {
String rValue = null;
String sql = "SELECT * FROM REGISTERED where USERNAME='"
+Username + "' ";
SQLiteDatabase sqdb = this.getReadableDatabase();
Cursor c = sqdb.rawQuery(sql, null);
if (c.moveToFirst()) {
String pass= c.getString(c.getColumnIndex("PASSWORD"));
rValue = pass;
}
return rValue;
}
}
Upvotes: 0
Reputation: 86
try replacing this line :
final static String DATABASE_CREATE = "create table" + TABLE_NAME + "(" +"ID"+ "int PRIMARY KEY AUTOINCREMENT" + "NAME text" + "Age int" + "EMAIL text" + "USERNAME text" + "PASSWORD text" + ")";
with this :
final static String DATABASE_CREATE = "create table " + TABLE_NAME + "(" +"ID"+ " int PRIMARY KEY AUTOINCREMENT, " + " NAME text," + " Age int," + " EMAIL text," + " USERNAME text," + " PASSWORD text" + ")";
Upvotes: 1
Reputation: 5312
Change:
final static String DATABASE_CREATE = "create table" + TABLE_NAME
To:
final static String DATABASE_CREATE = "create table " + TABLE_NAME
Upvotes: 0