Въпрос относно функций и масиви

sun_seeker 15 Точки

Въпрос относно функций и масиви

Здравейте,

Уточнявам, че говоря конкретно за 2-ра и 3-та задача, от домашното за judge, но предполагам ще имам грешки и за напред.

Описвам едната грешка. Когато изнеса във функция масив и там му променям стойностите на елементите, при компилиране ми дава грешка точно на този ред, където презаписвам стойност в масива. Ако го оставя в int main(), всичко си работи.

Другата грешка е подобна. Когато кода е във функция, този път int, всичко си работи, не дава грешка, но резултата е грешен. Ако го върна в main(), речултата е верен.

Гледах лекцията отново и отново, уж всичко правя както трябва пък явно не е. Не пускам кода тук за да не нарушавам правилата, но някакъв съвет къде може да греша?

Тагове:

30/11/2017 12:01:39 C++ Fundamentals

Виж всички отговори

georgi.stef.georgiev 921 Точки

Здравей,

Пробвай да изолираш грешката, нещо различно е от това, че подаваш масива във функция. Ето код, който приема масив и му променя елементите във функция, и след това го печата - компилира се нормално без грешки:

#include<iostream>
using namespace std;

void modify(int arr[], int arrSize) {
    for (int i = 0; i < arrSize; i++) {
        arr[i] = i;
    }
}

int main() {
    int arr[10]{};

    modify(arr, 10);

    for (int i = 0; i < 10; i++) {
        cout << arr[i] << " ";
    }

    return 0;
}

Подаваш ли правилно масива като параметър? По правилен начин ли му достъпваш елементи? Да не би нещото, което се мъчиш да присвоиш върху елемент на масива да дава грешка, а не самото присвояване (примерно да имаш някакъв израз, който има грешка в себе си)?

Ако искаш може да публикуваш кода, но просто изтрий от него частта от функцията, която ти работи, и остави само тази, която ти дава грешка. Примерно:

void modify(int arr[], int arrSize) {
    ...
    arr[i] = i; // Тук ми дава грешка. Грешката е: [текст на грешката тук]
    ...
}

int main() {
    int arr[10]{};

    ...

    modify(arr, 10);

    ...

    return 0;
}

Обърни внимание, че в горния пример е ясно всяка променлива каква е (масивът, параметрите, само i не е ясно какво е, но го оставих така без цикъла защото кодът на функцията е само това, а исках да илюстрирам как да орежеш код от това, което публикуваш). И е много важно да ни кажеш каква е грешката. "Дава ми грешка" изобщо не помага, има причина тези грешки да ти се изписват долу, да си имат съобщение, понякога даже да си имат и номер - те ти помагат да разбереш какво точно не е наред на този ред.

Поздрави,

Жоро

30/11/2017 13:16:03

sun_seeker 15 Точки

Ето кода, поорязох го, но на последната лекция го обяснихте много добре, та едва ли има някой не разбрал как да реши задачата.

#include <iostream>
#include <vector>
using namespace std;
void inputNumbers(vector<vector<int> > arrays, int numArrays, int numElements);
void multiplication(vector<vector<int> > arrays, int numArrays, int numElements);
void sumElements(vector<vector<int> > arrays, int numArrays, int numElements);
int main() {
vector<vector<int> > arrays;
int numArrays;
int numElements;
cin >> numArrays >> numElements;
inputNumbers(arrays, numArrays, numElements);
multiplication(arrays, numArrays, numElements);
sumElements(arrays, numArrays, numElements);
return 0;
}
void inputNumbers(vector<vector<int> > arrays, int numArrays, int numElements){
..............
}
// Грешката е на 36-ти ред: "Thread 1: EXC_BAD_ACCESS (code=1, address=0x0)"
void multiplication(vector<vector<int> > arrays, int numArrays, int numElements){
int something = 0;
for(int b ......){
cin >> something;
for(int c ......){
arrays[b][c] *= something; // error?!
}
}
}
void sumElements(vector<vector<int> > arrays, int numArrays, int numElements){
...............
}

30/11/2017 13:55:10

georgi.stef.georgiev 921 Точки

Ясно, значи проблемът не е компилационна грешка, а грешка по време на изпълнение - това също е важно.

Грешката, която получаваш, казва, че се опитваш да достъпиш памет, която не е твоя (по-конкретно изглежда сякаш достъпваш адрес 0). Това ме навежда на мисълта, че този vector<vector<int> > arrays няма толкова елементи, колкото смяташ че има.

Тук вече трябва да дебъгваш стъпка по стъпка. Първо хвани и си изпечатай всички стойности на arrays (т.е. изпечатай си входа, след като си го прочел). Виж дали arrays съдържа това, което очакваш, че съдържа като брой елементи. След това до колко стигат b и c (с които достъпваш елементи от arrays в циклите) - дали винаги стойностите им са по-малки от размерите на масивите.

Всъщност отсега виждам къде ти е грешката, като погледнах дефинициите на методите. За ориентир - пробвай да изпечаташ стойностите на arrays в края на inputNumbers (там сигурно ще е вярно), и след това пак ги изпечатай в началото на multiplication. Има ли разлика (изобщо има ли нещо в arrays когато влезеш в multiplication - даже, има ли нещо в arrays между извикването на inputNumbers и multiplication, тоест между 18 и 19 ред)? Масивите се подават по референция винаги, но vector как се подава на функция? inputNumbers върху оригиналния arrays ли работи или върху локално копие на arrays?

Ориентира ли се къде е проблемът?

30/11/2017 14:03:46

sun_seeker 15 Точки

Така, в inputNumbers на финала всичко си е в arreys, но когато вляза в multiplication и искам да изпечатам arreys ми дава същата грешка.

Дори сега направих нова функция, веднага след nputNumbers да ми изпечата arreys и пак ми дава същата грешка.

Мисля, че се ориентирах. Ще прегледам пак лекцията и ще пробвам да оправя сам проблема. Благодаря за съвета.

30/11/2017 14:31:13

georgi.stef.georgiev 921 Точки

Точно. Проблемът ти идва от това, че векторът се приема по копие, не по референция. Все едно да направиш функция swap(int a, int b), вместо функция swap(int& a, int& b). При теб проблемът е едно към едно с този пример.

Успех!

30/11/2017 14:33:31

georgi.stef.georgiev 921 Точки

Успех!

30/11/2017 14:33:35

sun_seeker 15 Точки

Готово! Отново благодаря!

30/11/2017 15:30:06

Виж всички отговори