Newbie help w/2-D Array and GUI

[Bruce Laker]

Im using Netbeans to create a program/project for school that takes user input from two text fields, stores it in an array. I have created the GUI and Array but get errors when defining the on-enter event.

Here is the array definition:

    int[][] dataArray = new int[200][2];
    int i = 0;
    int j = 0;

And here is the textfield retrieval:

private void jEnterBtnActionPerformed(java.awt.event.ActionEvent evt) {                                         
        // Collects user input from the Minutes text area.
        int minutes = Integer.parseInt(jMinutesTxtFld.getText());
        //Exception handling, verify minutes >=0 and <=240)
        if (minutes <= 0 || minutes > 240) {
            jMinutesTxtFld.setText("Invalid minutes");
        } else {
            //After the user enters their minutes and presses Enter, add the 
            //data to the array.
        }
            dataArray[i++] = minutes;                                         (Here I get error: incompatible types required int[], found int)
        // Collects user input from the Monies text area.
            int monies = Integer.parseInt(jMoniesTxtFld.getText());
            //Exception handling, verify Monies >0)
            if (monies <= 0) {
                jMoniesTxtFld.setText("Invalid amount");
            } else {
            //After the user enters their monies and presses Enter, add the 
            //data to the array and increment.
            }
            dataArray[j++] = monies;                                         (Here I get error: incompatible types required int[], found int)

Looking forward to learning more. Thanks for any help.

[fred rosenberger]

Bruce ofBruce wrote:I don't know how to post code on this forum but will try.

Here you go: How to post code.

You may want to also read our FAQ on How to Ask Questions on this site.

[Phil English]

To locate a point in a 2 dimensional space you need two coordinates. When you reference a point in your array you only use a single index this returns a slice (1D array) not a point.

[dennis deems]

dataArray[i++] = minutes;

I think this coding style is ill-advised. Why embed the incrementing of a variable -- a significant event by any standard -- within an array access instruction? Your intent is not entirely clear. Can you tell me the value of i when the array access occurs?

When incrementing a variable, do so in an instruction that contains nothing else but incrementing the variable, unless there is a compelling and immediately comprehensible reason to do otherwise.

[Bruce Laker]

I figured it out. Disregard my question.

[Campbell Ritchie]

Well done solving it. How did you do it?