java - 为什么我的上一个方法没有打印出来？我怎样才能让它执行

Question

我正在尝试执行最后一个方法，但我不确定需要添加什么才能将该方法添加到我的主方法中。目前我的程序执行第一个方法，然后执行第二个方法，然后程序结束。我对 java 很陌生，任何见解都会很有帮助!感谢您查看我的代码。

import java.util.Arrays;
import java.util.Scanner;

public class dot {
    static int n = 3;

    public static void main(String[] args) {

        getArrayOne();
        getArrayTwo();

    }

    public static double[] getArrayOne() {
        Scanner scan = new Scanner(System.in);
        System.out.println("Please enter the length of your first vector :");
        int lengthOne = Integer.parseInt(scan.nextLine());
        double[] vOne = new double[lengthOne];
        for (int i = 0; i < lengthOne; i++) {
            System.out.println("Please enter a vector value of your first vector :");
            double valuesOne = Double.parseDouble(scan.nextLine());
            vOne[i] = valuesOne;
        }
        System.out.println("Your first vector values are: " + Arrays.toString(vOne));
        return vOne;

    }

    public static double[] getArrayTwo() {
        Scanner scan = new Scanner(System.in);
        System.out.println("Please enter the length of your second vector :");
        int lengthTwo = Integer.parseInt(scan.nextLine());
        double[] vTwo = new double[lengthTwo];
        for (int i = 0; i < lengthTwo; i++) {
            System.out.println("Please enter a vector value :");
            double valuesTwo = Double.parseDouble(scan.nextLine());
            vTwo[i] = valuesTwo;
        }
        System.out.println("Your second vector values are: " + Arrays.toString(vTwo));
        return vTwo;
    }

    public static double dotProduct(double vOne[], double vTwo[]) {
        double product = 0.0;
        for (int i = 0; i < n; i++) {
            product = product + vOne[i] * vTwo[i];

        }
        System.out.println("Dot Product: ");
        System.out.println(dotProduct(vOne, vTwo));
        System.out.println(product);
        return product;
    }
}

Answer 1

将主方法中的代码更改为

double[] arr1 = getArrayOne();
double[] arr2 = getArrayTwo();
double dotProduct = dotProduct(arr1, arr2);

问题是您实际上并没有调用您的 dotProduct 方法。

除此之外，您的代码中还有一些其他值得一提的事情。

1. 您可以通过一些重构来消除一些冗余。如果您将逻辑更改为通用 getArray(int, Scanner)。这将帮助您保持代码更易于维护。 int 类型参数将采用数组的长度，而 Scanner 是您用来从键盘获取输入的扫描器。由于您正在计算点/标量积，因此 vector 必须具有相同的长度。

2. 您需要关闭您创建的 Scanner 对象以防止内存泄漏。

3. 您的 dotProduct 方法在没有基本情况的情况下调用自身(不受控制的递归)。这是个坏主意。

考虑以下因素:

public static void main(String[] args) {

    Scanner scan = new Scanner(System.in);
    System.out.println("Please enter the length of the vectors whose dot product you wish to calculate :");
    int length = Integer.parseInt(scan.nextLine());

    System.out.println("Getting values for the first vector");
    double[] arr1 = getArray(length, scan);

    System.out.println("Getting values for the second vector");
    double[] arr2 = getArray(length, scan);

    System.out.println("Calculating the dot product");
    double dotProduct = getDotProduct(arr1, arr2);

    // TODO do something with the dot product

    scan.close();
}

private static double[] getArray(int length, Scanner scan) {

    double[] vector = new double[length];
    for (int i = 0; i < length; i++) {
        System.out.println("Please enter a vector value :");
        double valuesOne = Double.parseDouble(scan.nextLine());
        vector[i] = valuesOne;
    }
    System.out.println("Your vector values are: " + Arrays.toString(vector));

    return vector;
}

private static double getDotProduct(double vOne[], double vTwo[]) {
    double product = 0.0;
    for (int i = 0; i < vOne.length; i++) {
        product = product + vOne[i] * vTwo[i];
    }
    System.out.println("Dot Product: ");
    System.out.println(product);
    return product;
}

另一件值得一读的事情是 Java 的编码约定。