Integer Field in JList ... renumbering

[no1zson]

I do not even know how to correctly ask this question.
I have an item field in the code I am about to post. Simple intger meant to be an item number for a cd. The user enters this number.
Over the last week I have played with Add buttons, Del buttons, Modify and everything else and it brought to light a problem I want to address today.
I have always just used 1 for the first item, 2 for the second and so on ...
well after I am done will all the button nonsence the item numbers are all over the place.
How could I make it so that the item number field would be restructured and stay in ascending order no matter what was added or deleted?
If I delete cd number 4, then make 5 number 4, 6 number 5 and so on ... the next cd added should just take the next highest number ...
Does that make sense?
Here is my primary class. I can post the others if needed.

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 logo = new ShapesJPanel();
        logo.setPreferredSize(new Dimension(200,200));
        getContentPane().add( logo );
 
        // 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);
            currCD = 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();
            currCD = listModel.size()-1;
 
 
            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);
 
            // 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 Modify
 
            // Search the Name Field
            cdNameField.addCaretListener(new CaretListener()
            {
            public void caretUpdate(CaretEvent e)
                {
                JTextField f = (JtextField)e.getSource();
                final String s = f.getText();
 
 
        // run it
        public static void main(String args[])
        {
        JFrame frame = new JFrame( "CD Inventory Logo" );
      frame.setDefaultCloseOperation( JFrame.EXIT_ON_CLOSE );
 
         java.awt.EventQueue.invokeLater(new Runnable()
            {
            public void run()
                {
                new Inventory2().setVisible(true);
               }
            });
        }
 
} // close class

[JosAH]

Don't allow the user to enter that (integer) key him/herself. Users are stupid
beings that make mistakes all the time. Have you had a look at the Bitset
class yet? I might come in handy.

kind regards,

Jos

[no1zson]

I did. It is what I thought of useing, but the API just confuses me.
I do not yet understand how to use the API properly, so I have been scanning the internet for code examples of one that is actually being used, but I cannot find a model to try and use to set mine up.

[JosAH]

I did. It is what I thought of useing, but the API just confuses me.

It just allows you to set bits or reset them; either single bits or a range of them.
You can also go fancy and 'or', 'and', 'xor' and whatever with two entire bit sets.
You just need the simple stuff: get the first bit that wasn't set to true and set
a bit to true or false. Compare it with a boolean[] array except that a BitSet is
more clever than that.

kind regards,

Jos

[no1zson]

wow. Now I feel dumb. I know nothing about arrays, and boolean is just a term to me at this point for true or false.

I understand the basic concept, but have no idea how to code it.

Am I reformatting the field? Am I creating a whole new field? Am I creating this bit and then applying it to my itemno field?

I am lost.

[JosAH]

wow. Now I feel dumb. I know nothing about arrays, and boolean is just a term to me at this point for true or false.

I understand the basic concept, but have no idea how to code it.

Am I reformatting the field? Am I creating a whole new field? Am I creating this bit and then applying it to my itemno field?

I am lost.

Read all about that BitSet class and judge for yourself whether you can use it
or not; maybe my suggestion was all wrong. You won't know until you've understood
what that set is all about ;-)

kind regards,

Jos

[no1zson]

I have read it, and I do think it will help, but reading about it does not show me how to code it in to my existing program. That is what I do not understand.

I understand that you do not wish to just hand out code,

but could you show code examples of what I might try? I have never seen this implemented anywhere, it is hard for me to see how I would put it in action with what I already have.

[JosAH]

I have read it, and I do think it will help, but reading about it does not show me how to code it in to my existing program. That is what I do not understand.

I understand that you do not wish to just hand out code,

but could you show code examples of what I might try? I have never seen this implemented anywhere, it is hard for me to see how I would put it in action with what I already have.

There's a saying: "code to the interface, not to the implementation" . You want a
'thing' that gives you a unique integer number when you ask for it and you can
return a number (previously given to you by that 'thing') telling it that it can be
used again in the future if you ask for another number again.

Those numbers are your primary keys for your CD collection.

Here goes:

public interface PrimaryKeyGenerator {
   public int give();
   public void take();
}
 

If you pass a PrimaryKeyGener ator (an interface!) to your CD collection at
construction time (store it as a member variable), you can use it all over the
place, i.e. when you insert a new CD you ask it to give() you a number; when
you delete a CD you let that object take() it back.

You should be able to program and compile your CD collection class using this
interface; of course it won't run (yet) because there is no implementation of this
interface yet.

Here's a stupid implementation: i.e. it generates the numbers 0, 1, 2 ... and
it doesn't do anything when you make it take() a number back again. You
obviously don't want this implementation as a final implementation but it is
enough to get you started. We'll implement a clever PrimaryKeyGener ator
later when you understand that BitSet API. Here's the stupid interface implementation:

public class StupidPrimaryKeyGenerator implements PrimaryKeyGenerator {
   private int key;
   public synchronized int give() { return key++; }
   public void take() { }
} 
 

kind regards,

Jos

[no1zson]

OK, that is an explaination I THINK I understand.

So, I do not want the user entering anything in to my itemField, I should probably just set editable to false and be done with it right off the bat.

I should then impliment the public int give(); every time a cd is added, which stupidkeygenera tor will give the next available number?

Is that about the long and short of it, or am I twisted?

If all that is right, my only remaining question (at the moment) would be how does int give() know to give its number to itemField?
should I not also have to change the way the field is built and initialize?

currently I have this:

private JTextField itemField;
and
itemField = new JTextField(4);
itemField.setEditable(true);
and
newCD.setItemno(Integer.parseInt(itemField.getText()));
when a cd is added

If I am to code the interface, and not the implementation, will not most of that need to change also?

[r035198x]

OK, that is an explaination I THINK I understand.

So, I do not want the user entering anything in to my itemField, I should probably just set editable to false and be done with it right off the bat.

I should then impliment the public int give(); every time a cd is added, which stupidkeygenera tor will give the next available number?

Is that about the long and short of it, or am I twisted?

If all that is right, my only remaining question (at the moment) would be how does int give() know to give its number to itemField?
should I not also have to change the way the field is built and initialize?

currently I have this:

private JTextField itemField;
and
itemField = new JTextField(4);
itemField.setEditable(true);
and
newCD.setItemno(Integer.parseInt(itemField.getText()));
when a cd is added

If I am to code the interface, and not the implementation, will not most of that need to change also?

But now in that class you have a KeyGenerator object, so instead of setting itemNo with value entered by the user, you simply call the give method and use the number that it has generated as the itemNo