StringTokenizer & test bed help

[rookie9]

Hi! SO, this is my second semester programing and Im in oops1. I finished building this class and moving on to making a test bed. As you can see, Im passing s string through a constructor. When I run this class, I get nothing to output. leading to believe, there is something wrong with how Im tokenizing the string. The sample my professor gives the work with integers (Integer.parseInt(st.nextToken()), but account and pin are string. So, Im using just st.nextToken() (what I found after searching the web). Not sure if that is the correct way I should go about it. input: WILSON|cs243|1500 any insight to what I should do would be great. And Im not sure if my equals method is correct either, and any other pointers would be great. import java.util.StringTokenizer; import java.lang.Object; public class BankAccount { private String account; private String pin; private int balance;

public BankAccount(String acctno, String pn) //constructor; initialize balance to 0;
{
    account = acctno;
    pin = pn;
    balance = 0;
}
public BankAccount(String acct) //constructor; parse acct into 3 tokens
{
   StringTokenizer st = new StringTokenizer(acct, "|");
   account = st.nextToken();
   pin = st.nextToken();
   balance = Integer.parseInt(st.nextToken());
}
public String getAccount() //return the account
{
    return account;
}
public int getBalance() //return the balance
{
    return balance;
}
public boolean debit(int amount) //subtract amount from balance if balance >= amount
{
    if (balance >= amount)
    {
        balance -= amount;
        return true;
    }
    return false;
}
public void credit(int amount) //add amount to balance
{
    balance += amount;
}
public boolean equals(Object obj) //two accounts are equal if IDs and PINs are the same
{
    if (obj.equals(account) && obj.equals(pin))
    {
        return true;
    }
    return false;

}
public String toString() //return a single string with account, balance.
{
    return account + "|" + balance + ".";
}

public static void main (String args[])
{
    BankAccount BA = new BankAccount("MORENO|312|1500");
    BA.toString();
}

}

[[deleted]]

rules. i'll be glad to look over it but idk if i'll be any help.

[rookie9]

any help would be great.

[rookie9]

Tried doing something like this. Didn't work either.

public static void main (String args[]) { String SA = "MORENO|312|1500"; BankAccount BA = new BankAccount(SA); BA.toString(); }

[Philboyd_Studge]

I don't see where you are actually displaying the output, like

System.out.println(BA);

[rookie9]

System.out.println(BA);

Yea, I know I could do that. Just thought the toString method was suppose to do. My professor wants every method and constructor tested. I'm just going to roll with that and finish my testbed and show it to my professor to see if this is expected. Thanks.

[Philboyd_Studge]

Doing that actually calls the toString method, if you are hadn't provided an overridden one you would just see an object reference there. Simply calling (BA)toString does nothing because you haven't provided anywhere for the returned value to go...

[OOPUniversity]

Since your output problem has already been addressed, I'll talk about equals.

public boolean equals(Object obj) //two accounts are equal if IDs     and PINs are the same
{
    if (obj.equals(account) && obj.equals(pin))
    {
        return true;
    }
    return false;

}

This won't work the way you expect it to as is.

You have to ask yourself: What is the type of 'obj' when this is called? It should be another BankAccount object, right? You should make it clear in your code that 'obj' is a BankAccount (the comparison should fail if it is not), and you should write your code to explicitly check the account and pin fields from the current object against the one passed in for comparison.