Cleaning up my code

Question

I am trying to condense and improve my java code. I have almost everything in one class and want to break it out into more manageable classes. So, I created this class:
public class MyTableModels extends DefaultTableModel {
  public MyTableModels() {
    class MySTableModel extends DefaultTableModel {
      public MySTableModel(Object[] columnNames, int numRows)
        super(columnNames, numRows);
      public boolean isCellEditable(int row, int column) {
        if (column == 6) {
          return false;
        return true;
    class MyOTableModel extends DefaultTableModel {
      public MyOTableModel(Object[] columnNames, int numRows)
        super(columnNames, numRows);
      public boolean isCellEditable(int row, int column) {
        if (column == 0 || column == 4 || column == 5) {
          return false;
        return true;
    class MyPTableModel extends DefaultTableModel {
      public Class getColumnClass(int c) {
        return getValueAt(0, c).getClass();
      public MyPTableModel(Object[] columnNames, int numRows)
        super(columnNames, numRows);
      public boolean isCellEditable(int row, int column) {
        if (column == 7) {
          return true;
        return false;
    class MyFTableModel extends DefaultTableModel {
      public Class getColumnClass(int c) {
        return getValueAt(0, c).getClass();
      public MyFTableModel(Vector columnNames, int numRows)
        super(columnNames, numRows);
      public boolean isCellEditable(int row, int column) {
        return false;
}Now, I want to call MyFTableModel from my main class like this:
MyFTableModel modelF = new MyFTableModel(columnNamesF,0);The first class is in a file called MyTableModels.java and the call is in FDASFrame.java. They are both in the package sdh. Do I need to change the call to MyFTableModel or add some include or something? Thanks.
Allyson

Answer 1

I guess you are being sarcastic??? I am sorry. I guess my class is not written correctly??? Is this any better?:
public class MyTableModels extends DefaultTableModel {
    class MySTableModel extends DefaultTableModel {
      public MySTableModel(Object[] columnNames, int numRows)
        super(columnNames, numRows);
      public boolean isCellEditable(int row, int column) {
        if (column == 6) {
          return false;
        return true;
    class MyOTableModel extends DefaultTableModel {
      public MyOTableModel(Object[] columnNames, int numRows)
        super(columnNames, numRows);
      public boolean isCellEditable(int row, int column) {
        if (column == 0 || column == 4 || column == 5) {
          return false;
        return true;
    class MyPTableModel extends DefaultTableModel {
      public Class getColumnClass(int c) {
        return getValueAt(0, c).getClass();
      public MyPTableModel(Object[] columnNames, int numRows)
        super(columnNames, numRows);
      public boolean isCellEditable(int row, int column) {
        if (column == 7) {
          return true;
        return false;
    class MyFTableModel extends DefaultTableModel {
      public Class getColumnClass(int c) {
        return getValueAt(0, c).getClass();
      public MyFTableModel(Vector columnNames, int numRows)
        super(columnNames, numRows);
      public boolean isCellEditable(int row, int column) {
        return false;
}I am just trying to create some new classes outside the main class that will make the program more readable. The whole main file is over 5000 lines long and I want to break it up into individual files of classes. Is this the wrong way to do it? Sorry for my confusion.
Allyson