我如何让这个 for 循环工作？

Question

我创建了一个辅助方法来对包含数字的数组进行排序。

然后我在我的“main”方法中调用该方法并让它返回一个使用 string.format。

当 string.format 部分在 for 循环和 if 语句之外，但当它在 i 中时，当我告诉它时它不返回任何内容时，此代码有效。

我知道问题可能与我的 if 语句有关，但我找不到解决方案。

public static String getSmallestSalaryString(String[] names, int[] ages, double[] salaries) {
    String str= "";
    String str2="";
    String str3="";
    double[] sal= smallestSal(salaries);

    for(int i= 0; i < sal.length; i++) {
        if(sal[i]== 0) {
             str= String.format("Smallest salary:$%,.2f, Name:%s, age:%d" , sal[0], names[0], ages[0]);
             return str;
        }

        else if(sal[i]== 1) {
             str2= String.format("Smallest salary:$%,.2f, Name:%s, age:%d" , sal[1], names[1], ages[1]);
            return str2;
        }

        else if(sal[i] == 2) {
             str3= String.format("Smallest salary:$%,.2f, Name:%s, age:%d" , sal[2], names[2], ages[2]);
            return str3;
        }
    }


    return str;
}

public static double[] smallestSal(double[] salaries) {
    Arrays.sort(salaries);

    return salaries;
}

Answer 1

解决方案要求您找到“地址”而不是“内容”。

我在这里假设salaries 包含所有员工的工资，names 是他们的名字，ages 是他们的年龄。我还假设ages、names 和salaries 具有相同的大小并且与每个索引相关。然后利用这些信息，构造一个字符串来表示公司中薪水最低的员工。

由于这些信息都不是在单个对象中结构化的，因此您只需找到最低薪水INDEX。有了这些信息，您可以通过访问ages[INDEX] 和names[INDEX] 提供他/她的姓名和年龄。

你目前正在做的是找到最低工资，然后在循环中猜测它的 INDEX，这会更复杂。相反，在您的方法“smallestSal”中，获取最低薪水数组中的索引。然后，您的循环变得简单。请看下面我的意思。

public static String getSmallestSalaryString(String[] names, int[] ages, double[] salaries) {
    int lowestSalaryIdx= getLowestSalaryIndex(salaries);

    return String.format("Smallest salary:$%,.2f, Name:%s, age:%d" , salaries[lowestSalaryIdx], names[lowestSalaryIdx], ages[lowestSalaryIdx]);
}

public static int getLowestSalaryIndex(double[] salaries) {
    int lowestSalaryIndex = 0;
    double lowestSalary = Double.MAX_VALUE;
    for(int i=0; i<salaries.length; ++i) {
      if (salaries[i] < lowestSalary) {
        lowestSalaryIndex = i;
        lowestSalary = salaries[i];
      }
    }

    return lowestSalaryIndex;
}

Answer 2

排序后，您将在第一个元素（索引 0）中获得最小的薪水。然后您应该在原始数组中找到该值的实际索引，并从该索引中获取详细信息。所以你的代码应该修改如下。

public static String getSmallestSalaryString(String[] names, int[] ages, double[] salaries) {
    String str = "";
    String str2 = "";
    String str3 = "";
    ArrayList sal = smallestSal(salaries);

    double smallestSalary = (double) sal.get(0);
    int salIndex = 0;

    for (int i = 0; i < salaries.length; i++) {
        if (salaries[i] == smallestSalary) {
            salIndex = i;
            break;
        }
    }
    str = String.format("Smallest salary:$%,.2f, Name:%s, age:%d", salaries[salIndex], names[salIndex], ages[salIndex]);

    return str;
}

public static ArrayList smallestSal(double[] salaries) {
    ArrayList salaryList = new ArrayList();
    for (int i = 0; i < salaries.length; i++) {
        salaryList.add(salaries[i]);
    }
    Collections.sort(salaryList);

    return salaryList;
}

Answer 3

由于您正在遍历您的数组，您应该将 sal[0], names[0],ages[0] 更改为 sal[i], names[i],ages[i]。

for(int i= 0; i < sal.length; i++) {
    if(sal[i]== 0) {
         str= String.format("Smallest salary:$%,.2f, Name:%s, age:%d" , sal[i], names[i], ages[i]);
         return str;
    }