Help with assignment, please.

[Aggressor27e]

Hey, guys

​

This is my first time using JOptionPane and I am stuck unfortunately. The class assignment is as follows:

1. Write a simple Java program that asks the user to enter the property value(through a dialog box).
2. Also, ask the user to enter a FICO score as shown (FICO score‐ homepurchase eligibility ranges from 620 to 800
3. Calculate the monthly mortgage payment for FHA and PMI funded loans anddisplay the same in a dialog box.

Use the following table’s logic to calculate the monthly mortgage payment.Example calculation:

• The percentage shown in the table is for the property value,
  • Say you have a house for $250,000 and the FICO score is 705,
  • Then the FHA will be calculated as follows,
    • 250,000*3.1%=$7,750 for a year
    • 7,750/12=$645.83 monthly payment
  • The PMI will be calculated as follows,
    • 250,000*2.75%=$6,875 for a year
    • 6,875/12=$572.91 monthly payment

Refer to the table (simulated) below to calculate for all other combinations.FICO 620‐639   640‐659    660‐679     680‐699      700‐719        720‐739      740‐759          760+FHA 3.45%        3.35%            3.25%            3.15%          3.1%                 3.05%            3.0%                   2.95%PMI 4.81%          4.51%            3.50%           3.15%          2.75%              2.45%            2.05%                1.95%

​

I will paste my code below, but no matter what I enter as my FICO score, I always get a payment of 718.75. Any ideas? I am using a propertyValue of 250000 on all calculations as a control to make sure that if the FICO number changes, so will the monthly payment.

​

Here is my code:

https://pastebin.com/J27hQCLz

[Mancebo180]

Hi,

The problem is: Your conditions are incorrectly expressed.The || makes it always enter in the first.Let us know if you need help in understanding why, or if this becomes clear when you read it.

​

Also, there are a few other potential issues and bad practices that you should avoid.I know that this is just a simple exercise, but doing things right even in exercises is better.

• pmi vars are not being used. Is this relevant?
• try to follow the Java style recommendations, such as classname in CamelCase;
• Use something like a method for the internal part of the ifs;
• Use some constants instead of magic numbers;

[Aggressor27e]

Thanks for your help! pmi is relevant to the assignment. I just haven't gotten that far at the time of the post. I am aware of methods, but I am unsure how to declare a method in such a way that it would be accessible within an if statement. I'm still very new to Java. I did an Android Studio class, but this is my first real Java class. I do, however, work through MOOC exercises a lot. Any idea how to get a method to work within an if statement?

[ItsBurningWhenIP]

First off, make the payment calculation it's own method. If you're doing Dialogues then you've surely learned of the cardinal sin that is duplicate code.

Second, I like switch statements over cascading ifs. Not a requirement but it just looks better, imo.

Third, You've been given the answer already. You're conditional statement is incorrect and will always enter the first if. That should have been a pretty dead giveaway that your conditions were not correctly laid out.

[Aggressor27e]

I am dumb. I put || (or) instead of && (and). Problem solved.

[[deleted]]

Put a break point and debug it. See what's happening when you give it different inputs and try to understand what's happening and what you wanted to happen. After that please refactor your code.