Improving speed in a java program

Question

i have a fairly simple content blocking program, it accesses a database, this puts a user profile in an array to compare it against a web page classification.
The database used is Oracle and I'm connecting with a JDBC thin client. The web page classifications are currently been loaded from a text file and inserted into an 2D array.
My problem, the program is really slow.. i need to speed it up, any idea's or suggestion, area's to research.

Answer 1

The code is a bit long but any thoughts would be appreciated
import java.sql.*;
import java.io.*;
import java.util.*;
* This is a small program that will retreive the user profile data from the
* database and checked it against random page classifiers created. This will
* supply the number of pages blocked or accepted.
* @author
public class SetUpDatabase extends Object {
private Connection conn; //A database connection
int userId = -1; //The user id of the user who requested the internet content.
//User Profile Variables used to store data from the user profile database
boolean userProfileLocation = false; //The value of the location filter option in the user profile database.
boolean userProfileKey =false; //The value of the Keyword filter option in the user profile database.
boolean userProfileSkin =false; //The value of the Skin filter option in the user profile database.
boolean userProfileIcra = false; //The value of the ICRA filter option in the user profile database.
boolean userProfileRsaci = false; //The value of the RSACi filter option in the user profile database.
boolean userIcra[] = new boolean [46]; //An array to store the Icra profile retrieved from the user profile database.
int userRsaci[] = new int[6]; //An array to store the RSACi profile retrieved from the user profile database
// 2D array variables used to store data retreived from the Files
int incomingFileUser[][] = new int[100000][6]; //This 2d array is used to store the user profile extracted from a text file.
int incomingFileIcra[][] = new int[100000][46]; //This 2d array is used to store the ICRA rating extracted from a text file.
int incomingFileRsaci[][] = new int [100000][4]; //This 2d array is used to store the RSACi rating extracted from a text file.
// Array variables to hold converted incoming web page classifiers
boolean incomingPageClassifiers[] = new boolean[5]; //An array to store the internet content classifiers
boolean incomingIcraRating[] = new boolean [48]; //An array to store the incoming ICRA rating
int incomingRsaciRating[] = new int[5]; //An array to store the incoming RSACi rating
// Counters
//The amount of pages and records that have been processed
int numBlocked = 0; //The number of pages not blocked.
int numAllowed = 0; //The number of pages blocked.
// Icra Filter Counters
int icraCalled = 0; //The number of pages blocked with ICRA labels
int icraAccepted = 0; //The number of pages blocked with ICRA labels
int icraBlocked = 0; //The number of pages blocked with ICRA labels
//Rsaci Filter Counters
int rsaciCalled = 0; //To keep count of the total number of times the icra filter is called.
int rsaciAccepted = 0; //The number of pages blocked with RSACi labels
int rsaciBlocked=0; //The number of pages blocked with RSACi labels
//Misc
Random random = new Random(); //An object to create random numbers
boolean blocked = false; //A switch to indicate if the page should be blocked.
int numPages = 0; //A variable to hold the number of pages that have been put in the text box
* Constructor
* This connects to the database. It has methods that create random users, icra
* and rsaci profile. It also access the user profile database to collect user profile
* data.
* @param numPages, this is an integer value from the User Interface specifying
* the number of pages that are to be created.
public SetUpDatabase(int incomingNumPages, String option) {
numPages = (incomingNumPages-1);
try {
Class.forName("oracle.jdbc.driver.OracleDriver");
} catch (ClassNotFoundException c){
System.out.println("Problem:Could not load database drivers");
if (option == "Random"){
try {
for(int i= 0;i <= numPages;i++) {
userId = (int)((random.nextFloat()) * numPages);
GetUserData();
CreateRandomClassifiers();
RandomFilter();
} catch (SQLException h) {
System.out.println ("Problem with the Random Option.");
}else{
try {
int row = 0;
int col = 0;
int classifierIndex = 0;
GetFileUser(numPages);
GetFileIcra(numPages);
GetFileRsaci(numPages);
for(row= 0;row <= numPages;row++) {
userId = (int)(incomingFileUser[row][0]);
classifierIndex = 0;
for(col=1;col<=5;col++){
incomingPageClassifiers[classifierIndex]= ConvertBoolean(incomingFileUser[row][col]);
classifierIndex++;
col=0;
for(col=0;col<=45;col++){
incomingIcraRating[col] = ConvertBoolean(incomingFileIcra[row][col]);
col=0;
for (col=0;col<=3;col++){
incomingRsaciRating[col] = incomingFileRsaci[row][col];
col=0;
String dburl="jdbc:oracle:thin:@192.168.0.1:1521:up";
//Class.forName("oracle.jdbc.driver.OracleDriver");
conn = DriverManager.getConnection(dburl,"gary","happygary");
// conn = DriverManager.getConnection("jdbc:oracle:oci8:@UP","gary","happygary");
GetUserData();
FileFilter();
conn.close();
//System.out.println("row= "+row+" the user ="+userId + "numALlowed =" + numAllowed);
} catch (SQLException h) {
System.out.println ("Method calling problem");
* Method - This goes to the database and gets the user profile information,
* it uses the random user id from CreateRandomUser method.
void GetUserData () throws SQLException {
try {
Statement s = conn.createStatement();
String state = ("SELECT * FROM user_profile WHERE user_id=" + userId);
ResultSet rset = s.executeQuery (state);
while (rset.next()) {
int temp[] = new int[5];
int col=6;
for (int i = 0;i<=4;i++){
temp= rset.getInt(col);
col++;
userProfileLocation =ConvertBoolean(temp[0]);
userProfileKey = ConvertBoolean(temp[1]);
userProfileSkin = ConvertBoolean(temp[2]);
userProfileIcra = ConvertBoolean(temp[3]);
userProfileRsaci = ConvertBoolean(temp[4]);
rset.close(); // close the resultSet
s.close(); // Close the statement
} catch (SQLException g) {
System.out.println ("did not work");
* Method - This creates a random user profile
void CreateRandomClassifiers() {
float randomNumber = 0;
randomNumber = random.nextFloat(); // This creates a random number between 0.0 - 1.0, this is for the user_id By multiplying the random number by 1000 we can create a random number between 0 and 1000
userId = ((int)(randomNumber *1000));
for (int i =0;i<4;i++){
incomingPageClassifiers[i] = CreateRandomBoolean(10.5);
* Method - Filter
* This compares the incoming internet classification with the retrieved database information.
void RandomFilter () throws SQLException {
try {
if ((userProfileLocation ==true && incomingPageClassifiers[0]==true)||(userProfileKey==true && incomingPageClassifiers[1]==true)||(userProfileSkin==true && incomingPageClassifiers[2]==true)){
numBlocked ++;
} else if ((incomingPageClassifiers[3]==true && userProfileIcra == true)) {
GetUserIcra();
CreateRandomIcra();
IcraFilter();
} else if ((incomingPageClassifiers[4]==true && userProfileRsaci == true)){
GetUserRsaci();
CreateRandomRsaci();
RsaciFilter();
} else {
numAllowed ++;
}catch (SQLException l){
System.out.println ("filter problem");
* Method - Filter
* This compares the incoming internet classification with the retrieved database information.
void FileFilter () throws SQLException {
try {
if ((userProfileLocation == true) && (incomingPageClassifiers[0]==true)){
blocked = true;
} else if ((userProfileKey == true) && (incomingPageClassifiers[1]==true)){
blocked = true;
}else if ((userProfileSkin==true) && (incomingPageClassifiers[2]==true)){
blocked = true;
}else if ((userProfileIcra == true) && (incomingPageClassifiers[3]== true)) {
GetUserIcra();
IcraFilter();
} else if ((userProfileRsaci == true) && (incomingPageClassifiers[4]==true)){
GetUserRsaci();
RsaciFilter();
} else {
blocked = false;
if (blocked == true ){
numBlocked++;
blocked = false;
} else {
numAllowed++;
}catch (SQLException l){
System.out.println ("Filter problem");
* Method - GetUserICra
* This accesses the User Profile database and gets the ICRA profile details
void GetUserIcra() throws SQLException {
try {
Statement s = conn.createStatement();
String state = ("SELECT * FROM Icra WHERE user_id=" + userId);
ResultSet rset = s.executeQuery (state);
ResultSetMetaData rsmd = rset.getMetaData();// get the number of columns
int numCols = rsmd.getColumnCount ();
int icraIndex=0;
while (rset.next()) {
for (int colNum =2;colNum<=46;colNum++){
userIcra[icraIndex] = rset.getBoolean (colNum);
icraIndex++;
rset.close(); // Close the resultSet
s.close(); // Close the statement
} catch (SQLException g) {
System.out.println ("Problem with Obtaining ICRA profile from database");
* Method - GetUserRSACi
* This gets the Rsaci profile data from the User Profile Database
void GetUserRsaci() throws SQLException {
try {
Statement s = conn.createStatement();
String state = ("SELECT * FROM Rsaci WHERE user_id=" + userId);
ResultSet rset = s.executeQuery (state);
ResultSetMetaData rsmd = rset.getMetaData(); // get the number of columns
int numCols = rsmd.getColumnCount ();
int rsaciIndex=0;
while (rset.next()) {
for (int colNum =2;colNum<=5;colNum++){
userRsaci[rsaciIndex] = rset.getInt (colNum);
rsaciIndex++;
rset.close(); // close the resultSet
s.close(); // Close the statement
} catch (SQLException g) {
System.out.println ("Problem with Rsaci");
* Method - CreateRandomIcra
* This creates a random icra classification
* @return It stores a boolean value in the pageIcra array variable
void CreateRandomIcra () {
for (int i = 2; i<=45;i++) {
incomingIcraRating[i]=CreateRandomBoolean(1.1);
* Method - CreateRandomRsaci
* This creates the integer values between 0-9 for a rsaci profile.
* @return It stores the random Rsaci profile in a pageRsaci variable
void CreateRandomRsaci () {
for (int i = 2; i<=5;i++) {
incomingRsaciRating[i]=CreateRandomInteger(10);
* Method - IcraFilter
* The Icra Filter
void IcraFilter() {
icraCalled++;
for (int i = 0;i<=45;i++) {
if ((userIcra[i] == true) && (incomingIcraRating[i] == true)){
blocked = true;
if (blocked == true) {
icraBlocked++;
} else {
icraAccepted++;
* Method -RsaciFilter
* The Rsaci Filter
void RsaciFilter() {
rsaciCalled++;
for (int i = 0;i<=3;i++) {
if ((userRsaci[i] >= 1) && (incomingRsaciRating[i] <= userRsaci[i])){
blocked = true;
if (blocked == true) {
rsaciBlocked++;
} else {
rsaciAccepted++;
* Method -This creates a random boolean.
* @param - percentage of likelyhood that boolean is true
* @return - boolean value
boolean CreateRandomBoolean(double percentage) {
double indicator = (random.nextFloat())*100;
if (indicator <= percentage) {
return true;
} else {
return false;
* Method - CreateRandomInteger
* @return Creates a random integer between the value of 0-9.
private int CreateRandomInteger(int percentage){
int indicator = 0;
indicator = (int)((random.nextFloat())*100);
if (indicator <= percentage) {
return(int)((random.nextFloat())*10);
} else {
return 0;
void GetFileUser(int numPages){
// assuming doubles and assuming you know the number
// of rows is m and the number of columns is n...
int numRows = numPages;
String line = null;
StringTokenizer st = null;
int row = 0;
int column = 0;
try {
BufferedReader in = new BufferedReader(new FileReader("C:\\java\\user.txt"));
// assuming each line contains one row of n values
try{
while( (( line = in.readLine())!= null)&&(row <=numRows)){
st = new StringTokenizer(line, ", ");
while(st.hasMoreTokens()) {
try{
incomingFileUser[row][column] = new Double(st.nextToken()).intValue();
} catch(NumberFormatException nfe) {
incomingFileUser[row][column] = 0;
column++;
column=0;
row++;
}catch (IOException ioe){
System.out.println("File reading error");
} catch (FileNotFoundException e){
System.out.println("File Not Found");
void GetFileIcra(int numPages){
// assuming doubles and assuming you know the number
// of rows is m and the number of columns is n...
int numRows = numPages;
String line = null;
StringTokenizer st = null;
int row = 0;
int column = 0;
try {
BufferedReader in = new BufferedReader(new FileReader("C:\\java\\icra.txt"));
// assuming each line contains one row of n values
try{
while( (( line = in.readLine())!= null)&&(row <numRows)){
st = new StringTokenizer(line, ", ");
while(st.hasMoreTokens()) {
try{
incomingFileIcra[row][column] = new Double(st.nextToken()).intValue();
} catch(NumberFormatException nfe) {
incomingFileRsaci[row][column] = 0;
column++;
column=0;
row++;
}catch (IOException ioe){
System.out.println("file reading error");
} catch (FileNotFoundException e){
System.out.println("File Not Found");
void GetFileRsaci(int numPages){
// assuming doubles and assuming you know the number
// of rows is m and the number of columns is n...
int numRows = numPages;
String line = null;
StringTokenizer st = null;
int row = 0;
int column = 0;
try {
BufferedReader in = new BufferedReader(new FileReader("C:\\java\\rsaci.txt"));
// assuming each line contains one row of n values
try{
while( (( line = in.readLine())!= null)&&(row <numRows)){
st = new StringTokenizer(line, ", ");
while(st.hasMoreTokens()) {
try{
incomingFileRsaci[row][column] = new Integer(st.nextToken()).intValue();
} catch(NumberFormatException nfe) {
incomingFileRsaci[row][column] = 0;
column++;
column = 0;
row++;
}catch (IOException ioe){
System.out.println("file reading error");
} catch (FileNotFoundException e){
System.out.println("File Not Found");
boolean ConvertBoolean(int number) {
if (number == 1){
return true;
} else{
return false;
* Method - This opens the database connection
//accessor methods
public int GetNumberBlocked (){
return numBlocked;
public int GetNumberAllowed(){
return numAllowed;
public int GetIcraCalled(){
return icraCalled;
public int GetIcraAccepted(){
return icraAccepted;
public int GetIcraBlocked(){
return icraBlocked;
public int GetRsaciCalled(){
return rsaciCalled;
public int GetRsaciAccepted(){
return rsaciAccepted;
public int GetRsaciBlocked(){
return rsaciBlocked;