Thread: Exception (payword protocol)

Originally Posted by copeg

Without code posted we all have to rely on guesses. Here's my guess: you are accessing an ArrayList for index -1, which will throw an exception. Based upon the stack trace, I'd guess you are trying to get the selected row or column within a table - but there are none selected so the returned selected row is -1 (JTable (Java 2 Platform SE 5.0)). Again beyond that its anyone's guess...

Client to Vendor Connection

package payword.user;
 
import payword.User;
import java.io.ObjectInputStream;
import java.io.ObjectOutputStream;
import java.net.Socket;
import payword.ClientCommitment;
 
/**
 *
 * @author Thodoris
 */
public class ClientToVendorConnection {
    private Socket socket;
    private String address;
    private int port;
    private ClientWindow parent;
    private ClientCommitment commitment;
    private User user;
 
    public ClientToVendorConnection(String address, int port, ClientWindow parent, ClientCommitment commitment, User user) {
        this.address = address;
        this.port = port;
        this.parent = parent;
        this.commitment = commitment;
        this.user = user;
 
        connect();
    }
 
 
    public void connect(){
        Thread t = new Thread(new Runnable() {
 
            public void run() {
                try {
                    socket = new Socket(address, port);
                    parent.log("Connected to: " + address + ":" + port);
 
                    // Στέλνω το commitment για να γράψει ο vendor τα στοιχεία του
                    ObjectOutputStream out = new ObjectOutputStream(socket.getOutputStream());
                    out.writeObject(commitment);
                    out.flush();
 
                    // Παίρνω πίσω το commitment μαζί με τα στοιχεία του vendor και υπολογίζω την υπογραφή και το paychain
                    ObjectInputStream iin = new ObjectInputStream(socket.getInputStream());
                    commitment = (ClientCommitment)iin.readObject();
                    commitment.calculatePayChain(user.getUserPrivateKey());
 
                    commitment.setWRoot(null);
 
                    // Στέλνω στον έμπορο το τελικό commitment.
                    out.writeObject(commitment);
 
 
                    parent.log("User commitment sent.");
 
                    socket.close();
 
                } catch (Exception ex) {
                    parent.log("ERROR: " + ex.getMessage());
                } 
 
            }
        });
        t.run();
    }
 
 
}

Vendor to Client connection

package payword.vendor;
 
import java.io.IOException;
import java.io.ObjectInputStream;
import java.io.ObjectOutputStream;
import java.net.ServerSocket;
import java.net.Socket;
import java.util.logging.Level;
import java.util.logging.Logger;
import payword.ClientCommitment;
 
/**
 *
 * @author Thodoris
 */
public class VendorToClientConnection {
    private VendorWindow parent;
    private ServerSocket serverSocket;
    private ClientCommitment commitment;
    private int port;
 
    public VendorToClientConnection(int port, VendorWindow parent, ClientCommitment commitment) {
        this.parent = parent;
        this.commitment = commitment;
        this.port = port;
    }
 
 
    public void startListening(){
        Thread thread = new Thread(new Runnable() {
            public void run() {
                try {
                    serverSocket = new ServerSocket(port);
                    parent.log("VendorToClient: Start listening port " + port);
 
                } catch (IOException ex) {
                    parent.log("ERROR: Could not start listening port: " + port);
                }
 
                try {
                    Socket socket = serverSocket.accept();
                    parent.log("Accepted connection.");
 
                    // Παίρνω το commitment και γράφω το vendorInfo
                    ObjectInputStream iin = new ObjectInputStream(socket.getInputStream());
                    ClientCommitment commitment = (ClientCommitment)iin.readObject();
                    parent.log("Commitment received.");
 
                    commitment.setVendorInfo("Vendor One");
                    // Στέλνω πίσω το commitment για να μπορέσει ο client να υπολογίσει το wRoot
                    ObjectOutputStream out = new ObjectOutputStream(socket.getOutputStream());
                    out.writeObject(commitment);
                    parent.log("Sent.");
 
                    commitment = (ClientCommitment)iin.readObject();
                    parent.log("Final commitment received.");
 
                    parent.log("vendorInfo: " + commitment.getVendorInfo());
                    parent.log("userInfo: " + commitment.getCertificate().getUserName());
                    parent.log("ammount: " + commitment.getAmmount());
 
                    parent.log("Vendor: Commitment received.");
 
                    VendorDbConnection con = new VendorDbConnection();
                    con.saveCommitment(commitment);
 
                    socket.close();
                } catch (Exception ex) {
                    Logger.getLogger(VendorToClientConnection.class.getName()).log(Level.SEVERE, null, ex);
                }
 
            }
        });
 
        thread.start();
    }
 
    public void disconnect(){
        try {
            serverSocket.close();
        } catch (IOException ex) {
            parent.log("ERROR: Could not disconnect.");
        }
    }
 
}

And this is the Certificate Table Model .. that comes the error

package payword.user;
 
import payword.User;
import java.util.ArrayList;
import java.util.List;
import javax.swing.table.AbstractTableModel;
import payword.ClientCertificate;
 
/**
 *
 * @author Thodoris
 */
public class CertificateTableModel extends AbstractTableModel {
    int lines = 0;
    List<ClientCertificate> contents = new ArrayList<ClientCertificate>();
 
    public void refreshTable(User user){
        ClientDbConnection connection = new ClientDbConnection();
        contents = connection.getCertificateList(user);
        lines = contents.size();
        fireTableDataChanged();
        connection.close();
    }
 
    public int getRowCount() {
        return lines;
    }
 
    public int getColumnCount() {
        return 3;
    }
 
    public Object getValueAt(int rowIndex, int columnIndex) {
        ClientCertificate cert = contents.get(rowIndex);
        if(columnIndex == 0){
            return cert.getUserName();
        }else if(columnIndex == 1){
            return cert.getExpirationDate();
        }else{
            return cert.getAmmount();
        }
    }
 
    @Override
    public String getColumnName(int column){
        if(column == 0){
            return "Broker";
        }else if(column==1){
            return "Expires";
        }else{
            return "Ammount";
        }
    }
 
    public ClientCertificate getSelected(int i){
        return contents.get(i);
    }
}

And here I thought you may have actually taken my first suggestion...Read the API for getSelectedRow() JTable (Java 2 Platform SE 5.0)

 int i = certificateTable.getSelectedRow();

i will be -1 if there are no rows selected, and you cannot access an ArrayList index at -1 in the following line of code

model.getSelected(-1) ///will throw an exception

Fix is to make sure i is within the bounds of your list