Как исправить ошибку «Нет совпадений для оператора []» (С++)

У вас опечатка в вашем условии while. смотрите ar[12].

Ваша проблема заключается в передаче массива:

Вы не передаете массив следующим образом:

function(arr[10])

Строка выше передает 11-й элемент в function()

Чтобы передать весь массив, вы передаете имя массива.

function(arr)

В твоем случае:

largestOfArray(arr[12])

проходит 13-й элемент arr

это должно быть:

largestOfArray(arr)

в этой строке:

while (sum > 500) ar.remove(largestOfArray(arr[10]));

Обновлено: Теперь, когда я вижу, что происходит, у std::list нет оператора []https://en.cppreference.com/w/cpp/container/list, почему его нет? Это может помочь: https://thispointer.com/difference-between-vector-and-list-in-c/

в основном

largestOfArray() принимает массив, поэтому я не знаю, вы хотели передать arr или ar.

Во-первых, вы используете как ar, так и arr, а arr — это массив, а ar — список. Это сбивает с толку и читателя, и (как оказалось) автора кода.

В C++ std::list не имеет произвольного доступа. Вы не можете использовать ar[], так как оператор [] не определен для списков. Кроме того, largestOfArray ожидает массив, а не список. Вероятно, вы намеревались использовать arr вместо ar.

Используя ответ std::list:

1. Сделать наибольший из массивов для получения списка
2. Заставить наибольший из массивов возвращать наибольшее значение
3. Чтобы удалить элемент, вы должны вызвать ar.erase(find(value));

Расширенный ответ:

Здесь есть множество вещей, которые можно улучшить.

1. Обычно предпочитают std::vector вместо std::list
2. Используйте std::accumulate для вычисления суммы
3. Ваш код хрупок и не обрабатывает разные размеры массивов (используйте std::size)
4. Элементы std::list не могут быть доступны по индексу
5. std::max_element можно использовать для поиска максимального элемента
6. Если у вас не может быть дубликатов, используйте std::set, а не std::vector
7. Если вас не волнует порядок, я бы использовал вектор, (обратно) отсортировал вектор, а затем продолжал выталкивать заднюю часть вектора, пока ваша сумма превышает 500

Код, который вы не пишете, не может содержать ошибок (вообще говоря). Вы должны просмотреть все функции внутри заголовка <algorithm>, чтобы увидеть, что может вам помочь.

В вашем коде много ошибок.

• Пожалуйста, посмотрите, как список определяется в C++.
• Проверьте, что такое статический массив, что такое динамический массив.
• Используйте {} для каждого цикла. Особенно новичку будет легче читать и находить ошибки.
• Узнайте, как передавать аргументы функциям в C++.

Вот простое решение:

#include <iostream>
#include <algorithm>
#include <list>

using namespace std;

int largestInList(const list<int> &numbers);
int sumOfList(const list<int> &numbers);

int main()
{
    int i,c = 0;
    int arr[] = {60,70,30,15,17,80,16,75,90,85,40,75};
    list<int> mylist(arr,arr+12);
    int sum = sumOfList(mylist);
    while (sum > 500)
    {
        int largest = largestInList(mylist);
        for (list<int>::iterator i=mylist.begin(); i!=mylist.end(); i++)
        {
            if (*i == largest)
            {
                mylist.erase(i);
                break;
            }

        }
        sum = sumOfList(mylist);
    }
    cout << sum << endl;
    return 0;
}

int sumOfList(const list<int> &numbers)
{
    int sum = 0;
    //For Loop before C++11:
    //for (list<int>::const_iterator it = numbers.begin(); it != numbers.end(); ++it)
    //{
    //    sum += *it;
    //}
    for (const auto &lel : numbers) 
        sum += lel;
    return sum;
}

int largestInList(const list<int> &numbers){
    list<int>::const_iterator it = numbers.begin();
    int largest = *it;
    for (const auto &n : numbers){
        if (largest < n) largest = n;
    }
    return largest;
}