Question

package homework4;
import java.util.Scanner;
public class Prog4 {
static Scanner scanner = new Scanner(System.in);
public static void main(String[] args)
{   CreditCardNumber[] cred1;
    CreditCardNumber cred2 = getInput();
    Display("The complete number from your input:", cred2);
    cred1 = getInputArray();
    DisplayArray(cred1);
    TryAnother(cred1);
}

public static CreditCardNumber getInput() {
    String ID;
    String accNum;
    CreditCardNumber tempCred;      
    System.out.printf("Please enter issuer ID:");
    ID = scanner.next();
    System.out.printf("Please enter account number:");
    accNum = scanner.next();
    tempCred = new CreditCardNumber(ID, accNum);

    return tempCred;
}
public static void Display(String ch, CreditCardNumber cred2)
{

    System.out.println(ch);
    System.out.println(cred2.toString().replaceAll(".{4}", "$0   "));       
}

public static CreditCardNumber[] getInputArray()
{
    CreditCardNumber[] tempArray;
    String tempID;
    int size;       
    System.out.printf("Please enter size of the aray:");
    size = scanner.nextInt();
    if(size < 1)
    {
        size = 1;
    }
    tempArray = new CreditCardNumber[size];
    System.out.printf("Please enter issuer ID:");
    tempID = scanner.next();
    for(int i = 0; i < tempArray.length; i++)
    {
        tempArray[i] = new CreditCardNumber();
        tempArray[i].CreateCred(tempID);
    }

    return tempArray;
}

public static void DisplayArray(CreditCardNumber[] cred1)
{
    for(int i = 0; i< cred1.length; i++)
    {
        Display("Credit Card # " + i+":" + '\n', cred1[i]);
    }
    System.out.println();
}

public static boolean TryAnother(CreditCardNumber[] cred1)   
{
    String s;
    System.out.printf("Get more credit card numbers? (n for no):");
    s = scanner.next();
    while(s.charAt(0) != 'N' && s.charAt(0) != 'n')
    {                                       
            cred1 = getInputArray();
            DisplayArray(cred1);    
            if(s.charAt(0) != 'N' && s.charAt(0) != 'n')
            TryAnother(cred1);
    }       


    return false;   
}
}

hi i have a problem with the Tryanother method when i answer no the process doesn’t terminate but instead prompt user to ente the size of array again like this:

Please enter issuer ID:321321
Please enter account number:654654654
The complete number from your input:
3213   2165   4654   6549   
Please enter size of the aray:7
Please enter issuer ID:789789
Credit Card # 0:

7897   8987   8895   1653   
Credit Card # 1:

7897   8935   5880   5197   
Credit Card # 2:

7897   8997   8152   8134   
Credit Card # 3:

7897   8996   1605   8067   
Credit Card # 4:

7897   8953   2006   5521   
Credit Card # 5:

7897   8964   3319   8191   
Credit Card # 6:

7897   8986   1830   7068   

Get more credit card numbers? (n for no):y
Please enter size of the aray:3
Please enter issuer ID:345345
Credit Card # 0:

3453   4590   2392   9734   
Credit Card # 1:

3453   4549   5025   1905   
Credit Card # 2:

3453   4504   2989   0927   

Get more credit card numbers? (n for no):n
Please enter size of the aray:

as you can see when I enter ‘y’ the program work, it prompt user to enter the size of array, but when i enter ‘n’ instead of terminating the program prompt user to enter the size of the array

how can i fix this problem?

thank you in advance

Answer 1

Just noticed you have s.charAt(0) != 'N' && s.charAt(0) != 'n' inside your while loop and also calling the same function recursively within the loop.

Change it to if(s.charAt(0) != 'N' && s.charAt(0) != 'n')