C# Program Validation

Question

I need some help making sure that when the user enters nothing, or not a number, it doesn't crash. I have it down so that it will tell them to enter a number if they don't, but I don't know how to make it go back to the original question and give them a chance to input correctly. Thanks for the help.

        Console.WriteLine("How much is rent: ");
        string  strRent = Console.ReadLine();
        double dblRent = 0.0;
        if (double.TryParse(strRent, out dblRent))
        {
            Console.WriteLine("How much is the car payment: ");
            string strCarPayment = Console.ReadLine();
            double dblCarPayment = Convert.ToDouble(strCarPayment);
        }
        else
        {
            Console.WriteLine("Enter a number");
        }

        Console.WriteLine("How much is student loan payment: ");
        string strStudentLoan = Console.ReadLine();

        Console.WriteLine("How much is phone bill: ");
        string strPhoneBill = Console.ReadLine();

        Console.WriteLine("How much is electric bill: ");
        string strElectricBill = Console.ReadLine();

        Console.WriteLine("Fraction deposited: ");
        string strFractionDeposited = Console.ReadLine();

        Console.WriteLine("Amount leftover: ");
        string strAmountLeft = Console.ReadLine();



        double dblStudentLoan = Convert.ToDouble(strStudentLoan);
        double dblPhoneBill = Convert.ToDouble(strPhoneBill);
        double dblElectricBill = Convert.ToDouble(strElectricBill);
        double dblFractionDeposited = Convert.ToDouble(strFractionDeposited);
        double dblAmountLeft = Convert.ToDouble(strAmountLeft);

        double dblBillSum = dblRent + dblCarPayment + dblStudentLoan + dblPhoneBill + dblElectricBill;
        double afterBills = dblAmountLeft / (1 - dblFractionDeposited);
        double totalPaycheck = afterBills + dblBillSum;
        Console.WriteLine("Total Paycheck: " + totalPaycheck.ToString("C"));

        Console.WriteLine("Enter wage $/hr: ");
        string strWage = Console.ReadLine();

        Console.WriteLine("Enter hours worked overtime: ");
        string strHoursOT = Console.ReadLine();

        Console.WriteLine("Enter overtime multiplier: ");
        string strOTWage = Console.ReadLine();

        double dblWage = Convert.ToDouble(strWage);
        double dblHoursOT = Convert.ToDouble(strHoursOT);
        double dblOTWage = Convert.ToDouble(strOTWage);

        double OTPay = (dblWage * dblOTWage) * dblHoursOT;
        Console.WriteLine("Overtime Pay: " + OTPay.ToString("C"));
        Console.ReadLine();

    }
}

}

Kar · Accepted Answer · 2015-10-08 21:27:23Z

2

You need a loop, something like this

        Console.WriteLine("How much is rent: ");
        string strRent = Console.ReadLine();
        double dblRent = 0.0;
        while (!double.TryParse(strRent, out dblRent))
        {
            Console.WriteLine("Enter a number");
            strRent = Console.ReadLine();
        }
        Console.WriteLine("How much is the car payment: ");
        string strCarPayment = Console.ReadLine();
        double dblCarPayment = Convert.ToDouble(strCarPayment);

answered Oct 8, 2015 at 21:27

Kar

5051 gold badge4 silver badges9 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

MikeH Over a year ago

Since there are so many questions that follow a similar format, I'd recommend putting this solution into another method and pass the "question string" to the method, and return the entered value.

Crizz Over a year ago

I used this and it worked beautifully. Thank you so much!

jdweng · Accepted Answer · 2015-10-08 21:28:57Z

0

Try this

        double dblRent = 0.0;
        Boolean valid = false;
        while (!valid)
        {
            if (double.TryParse(strRent, out dblRent))
            {
                Console.WriteLine("How much is the car payment: ");
                string strCarPayment = Console.ReadLine();
                double dblCarPayment = Convert.ToDouble(strCarPayment);
                valid = true;
            }
            else
            {
                Console.WriteLine("Enter a number");
            }
        }

answered Oct 8, 2015 at 21:28

jdweng

34.5k3 gold badges17 silver badges19 bronze badges

Comments

Olivier Jacot-Descombes · Accepted Answer · 2015-10-08 21:47:03Z

Since you are reading many doubles, you could create a method encapsulating reading and processing the number

private static double? ReadLineDouble()
{
    while(true) {
        string s = Console.ReadLine();
        if (String.IsNullOrWhitespace(s)) {
            return null; // The user wants to abort
        }
        double d;
        if (Double.TryParse(s, out d)) {
            return d;
        }
        Console.WriteLine("Please enter a valid number");
    }
}

If the user just hits Enter probably he wants to abort. In this case the method returns null. This is possible, because the return type is a Nullable<double>. The shorthand notation for it is double?.

Now you can read a number like this:

double? rent = ReadLineDouble();
if (double == null) return; // Abort the program.
// Otherwise continue.

Collectives™ on Stack Overflow

C# Program Validation

3 Answers 3

2 Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

2 Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related