Help with program

[bergadur]

i need help with a project which is a encrypting program. the file is attached and the deadline is right around the corner. I can't get it to work in the codeMessage() method. What im trying to do is replace the letters in the message with whatever you type in the fields below and then print it out in the outputField. Please help.

[sunde887]

Most people here aren't going to download attached files, you should post the code here. If it's a lot of code, post relevant code, use code tags also.
[code]
YOUR CODE HERE
[/code]

Ask specific questions.

[Junky]

Even better create an SSCCE (google it) and post that here.

[bergadur]

sorry if i do this wrong.

public void actionPerformed(ActionEvent e)   {
        System.out.println("");
        characterizeMessage();
        changeFields();
        checkCharacters();
        codeMessage();
    }
    
public void codeMessage(){ 
    System.out.println("");   
    
    for (int counter5 = 0; counter5 < messageFieldChar.length; counter5++){
        System.out.print("");
        if (messageFieldChar[counter5] == 'A' || messageFieldChar[counter5] == 'a'){
            messageFieldChar[counter5] = letters[0];
        }
    }
        for (int counter6 = 0; counter6 < messageStr.length(); counter6++){
            System.out.print(messageFieldChar[counter6]);
        }
}


public void characterizeMessage(){
        
        messageStr = inputField.getText();
        char[] messageFieldChar = messageStr.toCharArray();
        
        for (int counter1 = 0; counter1 < messageStr.length(); counter1++){
            System.out.print(messageFieldChar[counter1]);
        }
        
        
}

public void checkCharacters() {
        letters = new char[27];
        letters[0] = aChar;
        letters[1] = bChar;
        letters[2] = cChar;
        letters[3] = dChar;
        letters[4] = eChar;
        letters[5] = fChar;
        letters[6] = gChar;
        letters[7] = hChar;
        letters[8] = iChar;
        letters[9] = jChar;
        letters[10] = kChar;
        letters[11] = lChar;
        letters[12] = mChar;
        letters[13] = nChar;
        letters[14] = oChar;
        letters[15] = pChar;
        letters[16] = qChar;
        letters[17] = rChar;
        letters[18] = sChar;
        letters[19] = tChar;
        letters[20] = uChar;
        letters[21] = vChar;
        letters[22] = wChar;
        letters[23] = xChar;
        letters[24] = yChar;
        letters[25] = zChar;
        letters[26] = _Char;
        
        System.out.println("");
        for (int counter2 = 0; counter2 < letters.length; counter2++){
            System.out.print(letters[counter2]);
        }
}

public void changeFields() {
        aStr = aField.getText();
        char[] aFieldChar = aStr.toCharArray();
        aChar = aFieldChar[0];
        
        bStr = bField.getText();
        char[] bFieldChar = bStr.toCharArray();
        bChar = bFieldChar[0];
        
        cStr = cField.getText();
        char[] cFieldChar = cStr.toCharArray();
        cChar = cFieldChar[0];
        
        dStr = dField.getText();
        char[] dFieldChar = dStr.toCharArray();
        dChar = dFieldChar[0];
        
        eStr = eField.getText();
        char[] eFieldChar = eStr.toCharArray();
        eChar = eFieldChar[0];
        
        fStr = fField.getText();
        char[] fFieldChar = fStr.toCharArray();
        fChar = fFieldChar[0];
        
        gStr = gField.getText();
        char[] gFieldChar = gStr.toCharArray();
        gChar = gFieldChar[0];
        
        hStr = hField.getText();
        char[] hFieldChar = hStr.toCharArray();
        hChar = hFieldChar[0];
        
        iStr = iField.getText();
        char[] iFieldChar = iStr.toCharArray();
        iChar = iFieldChar[0];
        
        jStr = jField.getText();
        char[] jFieldChar = jStr.toCharArray();
        jChar = jFieldChar[0];
        
        kStr = kField.getText();
        char[] kFieldChar = kStr.toCharArray();
        kChar = kFieldChar[0];
        
        lStr = lField.getText();
        char[] lFieldChar = lStr.toCharArray();
        lChar = lFieldChar[0];
        
        mStr = mField.getText();
        char[] mFieldChar = mStr.toCharArray();
        mChar = mFieldChar[0];
        
        nStr = nField.getText();
        char[] nFieldChar = nStr.toCharArray();
        nChar = nFieldChar[0];
        
        oStr = oField.getText();
        char[] oFieldChar = oStr.toCharArray();
        oChar = oFieldChar[0];
        
        pStr = pField.getText();
        char[] pFieldChar = pStr.toCharArray();
        pChar = pFieldChar[0];
        
        qStr = qField.getText();
        char[] qFieldChar = qStr.toCharArray();
        qChar = qFieldChar[0];
        
        rStr = rField.getText();
        char[] rFieldChar = rStr.toCharArray();
        rChar = rFieldChar[0];
        
        sStr = sField.getText();
        char[] sFieldChar = sStr.toCharArray();
        sChar = sFieldChar[0];
        
        tStr = tField.getText();
        char[] tFieldChar = tStr.toCharArray();
        tChar = tFieldChar[0];
        
        uStr = uField.getText();
        char[] uFieldChar = uStr.toCharArray();
        uChar = uFieldChar[0];
        
        vStr = vField.getText();
        char[] vFieldChar = vStr.toCharArray();
        vChar = vFieldChar[0];
        
        wStr = wField.getText();
        char[] wFieldChar = wStr.toCharArray();
        wChar = wFieldChar[0];
        
        xStr = xField.getText();
        char[] xFieldChar = xStr.toCharArray();
        xChar = xFieldChar[0];
        
        yStr = yField.getText();
        char[] yFieldChar = yStr.toCharArray();
        yChar = yFieldChar[0];
        
        zStr = zField.getText();
        char[] zFieldChar = zStr.toCharArray();
        zChar = zFieldChar[0];
        
        _Str = _Field.getText();
        char[] _FieldChar = _Str.toCharArray();
        _Char = _FieldChar[0];
}
}

[Fubarable]

First of all, couldn't you use arrays to eliminate 99% of the redundancy in your code?

edit: or perhaps a HashMap<Character, JTextField>.

[Junky]

aStr = aField.getText();
char[] aFieldChar = aStr.toCharArray();
aChar = aFieldChar[0];

:eek:

aChar = aField.getText().charAt(0);

[Fubarable]

aStr = aField.getText();
char[] aFieldChar = aStr.toCharArray();
aChar = aFieldChar[0];

:eek:

aChar = aField.getText().charAt(0);

// in a loop
charMap.put(charIndex, charFieldMap.get(charIndex).getText().charAt(0));

[Junky]

Only if you know maps ;)

[bergadur]

thanks for the code simplicity but i am unable to get the message to exchange it's characters with the corresponding textFields

[Fubarable]

Here's a complex version of what I meant:

import java.awt.*;
import java.awt.event.*;
import java.util.HashMap;
import java.util.Map;

import javax.swing.*;
import javax.swing.text.*;

@SuppressWarnings("serial")
public class CharMapGUI extends JPanel {
   public static final String AVAIL_CHARS = "ABCDEFGHIJKLMNOPQRSTUVWXYZ_";
   private static final int CHAR_FIELD_COLUMNS = 5;
   private JPanel charFieldPanel = new JPanel(new GridLayout(0, 3, 25, 10));
   private Map<Character, Character> charMap = new HashMap<Character, Character>();
   private Map<Character, JTextField> charFieldMap = new HashMap<Character, JTextField>();

   public CharMapGUI() {
      for (char c : AVAIL_CHARS.toCharArray()) {
         charMap.put(c, c);
         JTextField charField = new JTextField("" + c, CHAR_FIELD_COLUMNS);
         ((PlainDocument) charField.getDocument()).setDocumentFilter(new SingleCharDocFilter());
         charField.addFocusListener(new CharFieldFocusListener());
         JPanel labelFieldPanel = new JPanel(new BorderLayout());
         labelFieldPanel.add(new JLabel("" + c + ":"), BorderLayout.WEST);
         labelFieldPanel.add(charField, BorderLayout.EAST);
         charFieldPanel.add(labelFieldPanel);
         charFieldMap.put(c, charField);
      }

      setBorder(BorderFactory.createEmptyBorder(5, 5, 5, 5));
      setLayout(new BorderLayout());
      add(charFieldPanel, BorderLayout.CENTER);
      add(new JPanel() {{
         add(new JButton(new AssignCharactersAction()));
      }}, BorderLayout.PAGE_END);
   }

   private class SingleCharDocFilter extends DocumentFilter {
      private static final int TEXT_LENGTH = 1;

      @Override
      public void insertString(FilterBypass fb, int offset, String insertText, AttributeSet attr)
               throws BadLocationException {
         Document doc = fb.getDocument();
         StringBuilder sb = new StringBuilder(doc.getText(0, doc.getLength()));
         sb.insert(offset, insertText);
         if (checkText(sb)) {
            super.insertString(fb, offset, insertText.toUpperCase(), attr);
         }
      }

      @Override
      public void replace(FilterBypass fb, int offset, int length, String replaceText,
               AttributeSet attrs) throws BadLocationException {
         Document doc = fb.getDocument();
         StringBuilder sb = new StringBuilder(doc.getText(0, doc.getLength()));
         sb.replace(offset, offset + length, replaceText);
         if (checkText(sb)) {
            super.replace(fb, offset, length, replaceText.toUpperCase(), attrs);
         }
      }

      @Override
      public void remove(FilterBypass fb, int offset, int length) throws BadLocationException {
         Document doc = fb.getDocument();
         StringBuilder sb = new StringBuilder(doc.getText(0, doc.getLength()));

         sb.delete(offset, offset + length);
         if (checkText(sb)) {
            super.remove(fb, offset, length);
         }
      }

      private boolean checkText(StringBuilder sb) {
         if (sb.length() != TEXT_LENGTH) {
            return false;
         }
         if (!CharMapGUI.AVAIL_CHARS.contains(sb.toString().toUpperCase())) {
            return false;
         }
         return true;
      }

   }

   private class CharFieldFocusListener extends FocusAdapter {
      @Override
      public void focusGained(FocusEvent e) {
         ((JTextComponent) e.getComponent()).selectAll();
      }
   }

   private class AssignCharactersAction extends AbstractAction {
      AssignCharactersAction() {
         super("Assign Characters");
      }

      @Override
      public void actionPerformed(ActionEvent arg0) {
         for (char c : AVAIL_CHARS.toCharArray()) {
            String fieldText = charFieldMap.get(c).getText().trim();
            if (fieldText.length() == 1) {
               charMap.put(c, charFieldMap.get(c).getText().charAt(0));
            } else {
               charMap.put(c, null); 
            }
         }

         // show it works:
         for (char c : AVAIL_CHARS.toCharArray()) {
            System.out.printf("%s: %s%n", c, charMap.get(c));
         }
      }
   }

   private static void createAndShowUI() {
      JFrame frame = new JFrame("Foo001");
      frame.getContentPane().add(new CharMapGUI());
      frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
      frame.pack();
      frame.setLocationRelativeTo(null);
      frame.setVisible(true);
   }

   public static void main(String[] args) {
      java.awt.EventQueue.invokeLater(new Runnable() {
         public void run() {
            createAndShowUI();
         }
      });
   }
}

Don't try to use this code or you'll definitely be busted. :)

[bergadur]

thanks that works great but the problem i keep running into is trying to go through the message array and change the letters with what ever shows up in that field. The codeMessage() method won't work. I think it is because it doesn't like the if statement inside a for loop.

It's this I don't think it likes:

for (int counter5 = 0; counter5 < messageFieldChar.length; counter5++){
        System.out.print("");
        if (messageFieldChar[counter5] == 'A' || messageFieldChar[counter5] == 'a'){
            messageFieldChar[counter5] = letters[0];
        }
    }

[bergadur]

Solved!!!!!!!!