Java OOP入门求助：计算HardwareComponent集合元素总和报错

Hey there! Let's work through this problem step by step. The error you're seeing has two key causes, and fixing them is straightforward:

1. Why get(i) isn't working #

The Collection interface in Java doesn't define an index-based get(int) method — that's a feature exclusive to its subinterface List. Since your hwComponents is declared as Collection<HardwareComponent>, the compiler can't locate the get(i) method you're trying to use.

2. You can't add a HardwareComponent object directly to a float #

Even if you had a valid way to access the element, sum += hwComponents.get(i) would throw another error. You can't add an object directly to a numeric type; you need to call a method on HardwareComponent that returns its price (like getPrice() — I’ll assume this method exists in your class, or that you can add it).

Solution 1: Use an enhanced for-each loop (simplest approach) #

This is the standard, clean way to iterate over any Collection in Java, no indexes required:

private Collection<HardwareComponent> hwComponents = new ArrayList<>();

public float calculatePrice() {
    float sum = 0;
    // Iterate over every component in the collection
    for (HardwareComponent component : hwComponents) {
        sum += component.getPrice(); // Fetch the component's price and add to sum
    }
    return sum;
}

Solution 2: Convert to List if you need index access #

If you specifically need index-based iteration (for edge cases), convert the Collection to a List first. Also, note your original loop started at i=1 — that would skip the first element! Always start index loops at 0:

private Collection<HardwareComponent> hwComponents = new ArrayList<>();

public float calculatePrice() {
    float sum = 0;
    List<HardwareComponent> componentList = new ArrayList<>(hwComponents);
    for (int i = 0; i < componentList.size(); i++) {
        sum += componentList.get(i).getPrice();
    }
    return sum;
}

Solution 3: Use Java 8+ Stream API (modern, concise approach) #

For a more streamlined solution, use streams to handle the sum in one line:

private Collection<HardwareComponent> hwComponents = new ArrayList<>();

public float calculatePrice() {
    return hwComponents.stream()
                       .mapToFloat(HardwareComponent::getPrice) // Convert each component to its price value
                       .sum(); // Calculate the total sum of all prices
}

Just make sure your HardwareComponent class includes a method to return the price, like this:

public class HardwareComponent {
    // ... your existing fields and methods ...
    private float price; // Example price field

    public float getPrice() {
        return this.price; // Return the component's price
    }
}

内容的提问来源于stack exchange，提问作者Teardrop