java - return array from method

Question

I created this snippet:

public static String[] get_data()
{
    conn = getInstance();
    String[] data_array = null;
    if(conn != null)
    {
        Statement query;
        try 
        {
            query = conn.createStatement();

            String sql = "SELECT data_x FROM table_x";
            ResultSet result = query.executeQuery(sql);

            result.next();
            int count = result.getInt("data_x");
            result.close();

            data_x_array = new String[count];

            for (int x = 1; x <= count; x++)
            {
                String data_x = result.getString(x);
                data_x_array[x] = data_x;
            }
        } 
        catch (SQLException e) 
        {
            e.printStackTrace();
        }
    }
    return data_x_array;
}

I just created a class, where data from the database is collected in an array.

Now i just want to return the array from this method.

But what i get is:

data_array cannot be resolved to a variable

Anybody could help me ?

Greetings!

UPDATE:

i changed snippet to:

public static String[] get_data()
{
    conn = getInstance();
    String[] data_array = null;
    if(conn != null)
    {
        Statement query;
        try 
        {
            query = conn.createStatement();

            String sql = "SELECT data_x FROM table_x";
            ResultSet result = query.executeQuery(sql);

            result.next();
            int count = result.getInt("data_x");
            result.close();

            data_array = new String[count];

            for (int x = 1; x <= count; x++)
            {
                String data_x = result.getString(x);
                data_x_array[x] = data_x;
            }
        } 
        catch (SQLException e) 
        {
            e.printStackTrace();
        }
    }
    return data_x_array;
}

When i compile just:

Invalid value for getInt() - 'value_in_table'

Anybody know this?

Greetings!

data_array is not in the same scope as return(data_array); — August
– August, Commented Oct 19, 2014 at 16:24

SparkOn · Accepted Answer · 2014-10-19 16:39:16Z

6

One more thing everyone forgot to mention

String[] data_array = new String[999];
for (int x = 0; x <= 999; x++){}

will throw an ArrayIndexOutOfBoundsException. Possible solution

String[] data_array = new String[999];
for (int x = 0; x < 999; x++){}

answered Oct 19, 2014 at 16:39

SparkOn

8,9665 gold badges31 silver badges36 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

davek · Accepted Answer · 2014-10-19 16:25:24Z

3

You have defined your variable inside the while loop i.e. it is not visible to the return statement. Plus, you have defined your method as static void, meaning no return value is expected. Use static String [] instead.

answered Oct 19, 2014 at 16:25

davek

23k11 gold badges80 silver badges97 bronze badges

1 Comment

user3683881 Over a year ago

Updated my post, could you refer to it?

SMA · Accepted Answer · 2014-10-19 16:26:54Z

Since its not accessible outside the block of while loop, compile is complaining about the same. Try this:

public static String[] get_data()
{
conn = getInstance();
String[] data_array = null;
if(conn != null)
{
    Statement query;
    try 
    {
        query = conn.createStatement();

        String sql = "SELECT data_x FROM table_x";
        ResultSet result = query.executeQuery(sql);

        while (result.next()) 
        {
            String data_x = result.getString("data_x");
            data_array = new String[999];
            for (int x = 0; x <= 999; x++)
            {
                data_array[x] = data_x;
            }
        }
    } 
    catch (SQLException e) 
    {
        e.printStackTrace();
    }
}
return data_array;
}

Blackwood · Accepted Answer · 2016-02-09 20:30:12Z

2

Your declaration of the String[] is not in the same scope as the return statement.

You need to declare it in the beginning of the scope.

And you need to change the header of the function to:

public static String[] get_data()

edited Feb 9, 2016 at 20:30

Blackwood

4,53216 gold badges34 silver badges41 bronze badges

answered Oct 19, 2014 at 16:25

nonamer92

1,9371 gold badge13 silver badges24 bronze badges

Comments

Tom · Accepted Answer · 2014-10-19 16:33:55Z

There are two things to fix here:

public static void get_data()

This method is declared to return nothing. Change it to:

public static String[] get_data()

Your variable String[] data_array is declared in the while loop, therefore it is only known there. Your return statement is outside that loop, so it doesn't have access to it.

Move the variable outside the loop:

String sql = "SELECT data_x FROM table_x";
ResultSet result = query.executeQuery(sql);
String[] data_array = new String[999];

Mind that you need to move the declaration and the initialization outside the while loop, or you will overwrite the previously stored data of that array by initializing it again. And also mind, that your for loop will overwrite the current data anyway ... you should think about storing the row data in another array so, or it will be lost.

Collectives™ on Stack Overflow

java - return array from method

5 Answers 5

Comments

1 Comment

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

5 Answers 5

Comments

1 Comment

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related