listModle.modif y

[no1zson]

I have been adding buttons to my GUI to manipulate list data.
I added a Delete button this morning in case I decide I no longer needed a particular element.
I am now working on a modify button, in case I want to keep the element, but only modify a field or two.
This seemed easy after my Delete, but now I am having issues with it.
Is there a listModel command out there that I am unfamiliar with that will help me with this?
Right now I have modify set up exactly like delete, just to put architecture in place without creating an error.
Any tips on what I should change to make it work correctly? Here is entire class.

import java.util.*;
import java.awt.*;
import java.awt.event.*;
import javax.swing.*;
import javax.swing.event.*;
import java.text.*;
import java.lang.*;
import javax.swing.JFrame;
 
public class Inventory2 extends JFrame
{
    private JLabel cdNameLabel; // name label 
    private JLabel artistLabel; // item number label 
    private JLabel nstockLabel; // units in stock label 
    private JLabel priceLabel; // price each label 
    private JLabel itemLabel; // item number label 
    private JTextField cdNameField; // name display 
    private JTextField artistField; // artist display 
    private JFormattedTextField nstockField; // units in stock display 
    private JFormattedTextField priceField; // price each display 
    private JTextField itemField; // item number display 
    private NumberFormat nstockFormat; // format field and parse numbers 
    private NumberFormat priceFormat; // format field and parse numbers 
    private JButton btnAdd; // first button 
    private JButton btnPrev; // previous button 
    private JButton btnNext; // next button 
    private JButton btnDel; // last button 
    private JButton btnFirst; // first button
    private JButton btnLast; // last button
    private JButton btnModify; // modify button
    private JButton btnSave; // save button
    private JButton btnSearch; // search button
    private JPanel buttonJPanel; // JPanle to hold buttons 
    private JPanel fieldJPanel; // JPanel to hold labels and displays 
    private JPanel fontJPanel; // JPanel to display logo 
    private int currCD; 
    private double total = 0; // variable for total inventory 
    private JList Inventorylist; // JList to take place of old array
    private DefaultListModel listModel;
    private JScrollPane jScrollPanel;  
    private float Invtotal = .00f;
 
     DecimalFormat formatter = new DecimalFormat("$0.00");
 
    public Inventory2() // create class and method to perform GUI build 
    { 
        initComponents(); 
    } 
 
    private void initComponents() 
    { 
 
        // create label names 
        cdNameLabel = new JLabel("CD Name:"); 
        artistLabel = new JLabel("Artist:");
        nstockLabel = new JLabel("In Stock:"); 
        priceLabel = new JLabel("Each Item Cost:$"); 
        itemLabel = new JLabel("Item Number:"); 
 
 
        // initial fields
        cdNameField = new JTextField(25);
        cdNameField.setEditable(true);
        artistField = new JTextField(15);
        artistField.setEditable(true);
        nstockField = new JFormattedTextField(nstockFormat);
        nstockField.setEditable(true);
        nstockField.setColumns(5);
        priceField = new JFormattedTextField(priceFormat);
        priceField.setEditable(true);
        priceField.setColumns(5);
        itemField = new JTextField(4);
        itemField.setEditable(true);
 
        // JList
        jScrollPanel = new JScrollPane();
        Inventorylist = new JList(); 
        currCD = 0;
 
 
        // buttons 
        btnAdd = new JButton(); 
        btnNext = new JButton(); 
        btnPrev = new JButton(); 
        btnDel = new JButton();
        btnLast = new JButton();
        btnFirst = new JButton();
        btnModify = new JButton(); 
        btnSave = new JButton();
        btnSearch = new JButton();
 
        getContentPane().setLayout(new FlowLayout());
        setDefaultCloseOperation(WindowConstants.EXIT_ON_CLOSE);
 
        // add shapesJPanel to frame
        ShapesJPanel shapesJPanel = new ShapesJPanel();    
        getContentPane().add( new ShapesJPanel() );            
 
        // place textFields and labels
 
        //artist
        artistLabel.setText("Artist"); 
        getContentPane().add(artistLabel); 
 
        artistField.setMinimumSize(new Dimension(70,20)); 
        artistField.setPreferredSize(new Dimension(70,20));
        getContentPane().add(artistField); 
 
        // cd name
        cdNameLabel.setText("CD Name"); 
        getContentPane().add(cdNameLabel); 
 
        cdNameField.setMinimumSize(new Dimension(70,20)); 
        cdNameField.setPreferredSize(new Dimension(70,20));
        getContentPane().add(cdNameField);
 
        // copies in stock
        nstockLabel.setText("Copies In Stock"); 
        getContentPane().add(nstockLabel); 
 
        nstockField.setMinimumSize(new Dimension(5,20)); 
        nstockField.setPreferredSize(new Dimension(5,20));
        getContentPane().add(nstockField); 
 
        //price of cd
        priceLabel.setText("Price"); 
        getContentPane().add(priceLabel); 
 
        priceField.setMinimumSize(new Dimension(20,20)); 
        priceField.setPreferredSize(new Dimension(20,20));
        getContentPane().add(priceField); 
 
        //item number of cd
        itemLabel.setText("Item Number"); 
        getContentPane().add(itemLabel); 
 
        itemField.setMinimumSize(new Dimension(5,20)); 
        itemField.setPreferredSize(new Dimension(5,20));
        getContentPane().add(itemField); 
 
 
 
        // add listeners
 
        btnAdd.setText("Add");
        btnAdd.addActionListener(new ActionListener()
        {
            public void actionPerformed(ActionEvent evt)
            {
                btnAddActionPerformed(evt);
            }
        });
        getContentPane().add(btnAdd);
 
        // PREVIOUS
        btnPrev.setText("Previous");
        btnPrev.addActionListener(new ActionListener()
        {
            public void actionPerformed(ActionEvent evt)
            {
                btnPrevActionPerformed(evt);
            }
        });
        getContentPane().add(btnPrev);
 
        // NEXT
        btnNext.setText("Next");
        btnNext.addActionListener(new ActionListener()
        {
            public void actionPerformed(ActionEvent evt)
            {
                btnNextActionPerformed(evt);
            }
        });
        getContentPane().add(btnNext);
 
        // SEARCH
        btnSearch.setText("Search");
        btnSearch.addActionListener(new ActionListener()
        {
            public void actionPerformed(ActionEvent evt)
            {
                btnAddActionPerformed(evt);
            }
        });
        getContentPane().add(btnSearch);
 
        // FIRST
        btnFirst.setText("First");
        btnFirst.addActionListener(new ActionListener()
        {
            public void actionPerformed(ActionEvent evt)
            {
                btnFirstActionPerformed(evt);
            }
        });
        getContentPane().add(btnFirst);
 
        // LAST
        btnLast.setText("Last");
        btnLast.addActionListener(new ActionListener()
        {
            public void actionPerformed(ActionEvent evt)
            {
                btnLastActionPerformed(evt);
            }
        });
        getContentPane().add(btnLast);
 
        // MODIFY
        btnModify.setText("Modify");
        btnModify.addActionListener(new ActionListener()
        {
            public void actionPerformed(ActionEvent evt)
            {
                btnModifyActionPerformed(evt);
            }
        });
        getContentPane().add(btnModify);
 
        // SAVE
        btnSave.setText("Save");
        btnSave.addActionListener(new ActionListener()
        {
            public void actionPerformed(ActionEvent evt)
            {
                btnAddActionPerformed(evt);
            }
        });
        getContentPane().add(btnSave);
 
        // DELETE
        btnDel.setText("Delete");
        btnDel.addActionListener(new ActionListener()
        {
            public void actionPerformed(ActionEvent evt)
            {
                btnDeleteActionPerformed(evt);
            }
        });
        getContentPane().add(btnDel);
 
 
        // new Jlist model
        listModel = new DefaultListModel();
        Inventorylist.setModel(listModel);
 
        jScrollPanel.setViewportView(Inventorylist);
 
        getContentPane().add(jScrollPanel);
 
        pack();
    }// close
 
 
        private void btnAddActionPerformed(ActionEvent evt)
        {
            // Create cd to add
            CdwArtist newCD = new CdwArtist();
            newCD.setArtist(artistField.getText());
            newCD.setName(cdNameField.getText());    
            newCD.setItemno(Integer.parseInt(itemField.getText()));
            newCD.setNstock(Integer.parseInt(nstockField.getText()));
            newCD.setPrice(Float.parseFloat(priceField.getText()));
 
            // Add cd to list
            listModel.addElement(newCD);
            currCD = listModel.size()-1;  // sets currCD to added index
 
 
            // Clear the text fields after add
            artistField.setText(null);
            cdNameField.setText(null);    
            itemField.setText(null);
            nstockField.setText(null);
         priceField.setText(null);
 
            }// end ADD
 
        private void btnPrevActionPerformed(ActionEvent evt)
        {
            // Grab Previous cd 
            if (--currCD<0) currCD = listModel.size()-1;
            CdwArtist newCD = (CdwArtist) listModel.get( currCD );
 
 
            artistField.setText(newCD.getArtist());
            cdNameField.setText(newCD.getName());    
            itemField.setText(String.valueOf(newCD.getItemno()));
            nstockField.setText(String.valueOf(newCD.getNstock()));
            priceField.setText(formatter.format(newCD.getPrice()));
 
 
            }// end PREV
 
                private void btnNextActionPerformed(ActionEvent evt)
            {
            // Grab Next cd 
            if (++currCD >= listModel.size()) currCD= 0;
            CdwArtist newCD = (CdwArtist) listModel.get( currCD );
 
 
            artistField.setText(newCD.getArtist());
            cdNameField.setText(newCD.getName());    
            itemField.setText(String.valueOf(newCD.getItemno()));
            nstockField.setText(String.valueOf(newCD.getNstock()));
            priceField.setText(formatter.format(newCD.getPrice()));
 
 
            }// end NEXT
 
 
                private void btnFirstActionPerformed(ActionEvent evt)
            {
            // Grab First cd 
            CdwArtist newCD = (CdwArtist) listModel.get(0);
 
 
            artistField.setText(newCD.getArtist());
            cdNameField.setText(newCD.getName());    
            itemField.setText(String.valueOf(newCD.getItemno()));
            nstockField.setText(String.valueOf(newCD.getNstock()));
            priceField.setText(formatter.format(newCD.getPrice()));
 
 
            }// end FIRST
 
 
                private void btnLastActionPerformed(ActionEvent evt)
            {
            // Grab Last cd 
            CdwArtist newCD = (CdwArtist) listModel.lastElement();
 
 
            artistField.setText(newCD.getArtist());
            cdNameField.setText(newCD.getName());    
            itemField.setText(String.valueOf(newCD.getItemno()));
            nstockField.setText(String.valueOf(newCD.getNstock()));
            priceField.setText(formatter.format(newCD.getPrice()));
 
 
            }// end LAST
 
                private void btnDeleteActionPerformed(ActionEvent evt)
            {
            // Delete cd 
            listModel.remove(currCD);
 
 
            // Clear the text fields after delete
            artistField.setText(null);
            cdNameField.setText(null);    
            itemField.setText(null);
            nstockField.setText(null);
         priceField.setText(null);
 
 
            }// end DELETE
 
                private void btnModifyActionPerformed(ActionEvent evt)
            {
            // Modify cd 
            listModel.remove(currCD);
 
 
            // Clear the text fields after delete
            artistField.setText(null);
            cdNameField.setText(null);    
            itemField.setText(null);
            nstockField.setText(null);
         priceField.setText(null);
 
 
            }// end Modify
 
        // run it
        public static void main(String args[])
        {
        //JFrame frame = new JFrame( "CD Inventory Logo" );
      //frame.setDefaultCloseOperation( JFrame.EXIT_ON_CLOSE );
 
      // create ShapesJPanel
      //ShapesJPanel shapesJPanel = new ShapesJPanel();
      //frame.add( shapesJPanel ); // add shapesJPanel to frame
      //frame.setSize( 125, 200 ); // set frame size
      //frame.setVisible( true ); // display frame
        java.awt.EventQueue.invokeLater(new Runnable()
            {
            public void run()
                {
                new Inventory2().setVisible(true);
               }
            });
        }
 
} // close class

[JosAH]

Of course that was the simpler of my two problems. I have some debugging left to try on the Search button I am trying to implement.

Thanks again for your help.

You're welcome of course; you also have some refactoring to do because re-
populating all those text fields over and over again deserves a little method of
its own and not to be repeated in every single btnSoAndSoActio nPerformed
method. (Same goes for clearing the fields in your add and update methods).
You copy and paste too much ;-)

kind regards,

Jos

[r035198x]

I know it has been commented on earlier in the thread, but I did not understand the solution.
I know the list does not like the information I am trying to pass to it.

The $ comes from the decimal formatter I have put in.
That was the issue. I took the $ out of the formatter and just left it at 0.00, put the $ into the label instead, and all is well.

You guys are great.

Of course that was the simpler of my two problems. I have some debugging left to try on the Search button I am trying to implement.

Thanks again for your help.

What I was trying to tell you earlier is something like that. You store the price as a double say 2.00. When diplsying to the user you want to show them that it is money by displaying the $ as well, $2.00. A JLabel is just for display, so you use the $2.00 version. A JTextField, however, is for taking in input. So either you force the user to enter it as 2.00 (using a validation) or allow the user to enter the $2.00 version and remove that $ in the background (code) so that you store it correctly as a double.

[no1zson]

You are right Jos. I have only 2 or 3 more things to do in order to close out this app and get it doing what I want it to.
I will then go back through it and "clean it up" ... this is my first experience with Java and I am just trying to see what is what right now.

Validation is also a concern when I go through on my second sweep. I fear if I try and do everything at once then I may just confuse myself further and do more harm than good.

[JosAH]

I will then go back through it and "clean it up" ... this is my first experience with Java and I am just trying to see what is what right now.

Good; do one thing first before you do the next thing. You're on the right track.
Otherwise you end up asking about Collections while File input still doesn't
work and curly brackets still don't match up and being puzzled why A$ is
not considered a valid identifier and a foo String == bar String still won't work ;-)

kind regards,

Jos