CODEWITHSUNDEEP
java
George K.
George K.

Reputation: 3151

Java - Can't find logic error

So I have this piece of code that prompts for space delimited input of the two sides and the hypotenuse of a triangle. The method below main is supposed to return true if the triangle is right, false if not.

For some reason, it still prints that a triangle is not a right one even if I know the measurments I entered are for a right triangle.

I've been trying to detect a logic error in this code for a while, can comeone help/call me out?

import java.util.Scanner;

public class VerifyRightTriangle { public static void main(String[] args) { Scanner sc = new Scanner(System.in);

    System.out.print("Please enter the two sides and the hypotenuse: ");
    String input = sc.nextLine();

    String[] values = input.split(" ");

    int[] nums = new int[values.length];

    for (int i = 0; i < nums.length; i++) {
        nums[i] = Integer.parseInt(values[i]);
    }

    double s1 = nums[0];
    double s2 = nums[1];
    double hyp = nums[2];

    System.out.printf("Side 1: %.2f Side 2: %.2f Hypotenuse: %.2f\n", s1, s2, hyp);

    boolean result = isRightTriangle(s1, s2, hyp);

    if (result == true) {
        System.out.println("The given triangle is a right triangle.");
    } else if (result == false) {
        System.out.println("The given triangle is not a right triangle.");
    }
}

/**
 * Determine if the triangle is a right triangle or not, given the shorter side, the longer
 * side, and the hypotenuse (in that order), using the Pythagorean theorem.
 * @param s1 the shorter side of the triangle
 * @param s2 the longer side of the triangle
 * @param hyp the hypotenuse of the triangle
 * @return true if triangle is right, false if not
 */

private static boolean isRightTriangle(double s1, double s2, double hyp) {
    double leftSide = s1 * s1 + s2 * s2;
    double rightSide = hyp * hyp;
    if (Math.sqrt(leftSide) == Math.sqrt(rightSide)) {
        return true;
    } else {
        return false;
    }
}

}

Upvotes: 0

Views: 292

Answers (3)

Peter Lawrey
Peter Lawrey

Reputation: 533530

Given the inputs are int values you shouldn't have an error. If I test

public static void main(String... args) {
    for (int i = 3; i <= 11; i += 2) {
        int side1 = (i * i - 1) / 2;
        int side2 = side1 + 1;
        System.out.println(i + "," + (side1 - 1) + " and " + (side2 - 1) + " is " + isRightTriangle(i, side1 - 1, side2 - 1));
        System.out.println(i + "," + side1 + " and " + side2 + " is " + isRightTriangle(i, side1, side2));
        System.out.println(i + "," + (side1 + 1) + " and " + (side2 + 1) + " is " + isRightTriangle(i, side1 + 1, side2 + 1));
    }
}

it prints

3,3 and 4 is false
3,4 and 5 is true
3,5 and 6 is false
5,11 and 12 is false
5,12 and 13 is true
5,13 and 14 is false
7,23 and 24 is false
7,24 and 25 is true
7,25 and 26 is false
9,39 and 40 is false
9,40 and 41 is true
9,41 and 42 is false
11,59 and 60 is false
11,60 and 61 is true
11,61 and 62 is false

which is correct.

Can you provide a use case where this method fails?

Try this instead to allow for a small error.

private static boolean isRightTriangle(double s1, double s2, double hyp) {
    double leftSide = s1 * s1 + s2 * s2;
    double rightSide = hyp * hyp;
    return Math.abs(leftSide - rightSide) < 1e-9;
}

if the sqrt of two values are equal, then the two values are equal.

Math.sqrt() has some error, so if you can avoid it, you should.

Upvotes: 1

Ankur
Ankur

Reputation: 12774

Floating point calculations are not precise, thats why the numbers never match, try using BigDecimal instead of double

Edit: On second thoughts as you are already parsing integer nums[i] = Integer.parseInt(values[i]);

simply use int in place of double and don't use Math.sqrt, simply use leftSide==rightside

Upvotes: 1

Ridcully
Ridcully

Reputation: 23655

You have to check for s1*s1 + s2*s2 == hyp*hyp without using sqrt. See pythagorean theorem.

Upvotes: 1

Related Questions