Как вывести сумму всех элементов для каждого столбца, которые оканчиваются тройкой?

Question

[m000gg m000gg]

У меня есть задача: надо вывести сумму всех элементов для каждого столбца, которые оканчиваются тройкой.

Вот мой код на с++

#include <iostream>
#include <ctime>
#include <vector>



using namespace std;

void Randmass(int**, int, int, int);
void Printmass(int**, int);
vector<int> Col_3(int**,int);


int main()
{
    int n,first_value,last_value;
    cout<<"vvedite diapazn:"<<endl;
    cin>>first_value>>last_value;
    srand(time(0));
    cout<<"vvedite razmer matr:"<<endl;cin>>n;
    int matrix[n][n]={0};
    int *stroki[n];
    for (int i = 0; i<n; i++)stroki[i] =&matrix[i][0];
    Randmass(stroki,n,first_value,last_value);
    cout<<endl;
    Printmass(stroki,n);
    vector<int> v = Col_3(stroki,n);
    cout<< endl;
    for(int i=0;i<n;i++)
    cout<<"summa:"<<i<<"="<<v[i]<<endl;

    
}


void Randmass(int** mass,int sizematr,int first_value, int last_value)
{
    cout<<"initsializavia massiva:"<<endl;
    for (int i = 0; i<sizematr; i++){
        for (int j = 0; j<sizematr; j++){
            mass[i][j] = first_value + rand()% (last_value - first_value + 1);
        }
    }
    
}

void Printmass(int** mass,int sizematr)
{
    cout<<"Pechat massiva:"<<endl;
    for (int i = 0; i<sizematr; i++){
            cout<<endl;
        for (int j = 0; j<sizematr; j++){
            cout<<"\t"<<mass[i][j];
        }
    }
    
}



vector<int> Col_3(int**x, int n)
{
    vector<int> v;
    for (int j = 0; j<n;j++)
    {   v. push_back(0);
        for( int i= 0; i<n; i++)
        {  
           if(x[i][j]%10 == 3){
              v[i] += x[i][j]; 
           }
        }
    }
    return v;
}

Но что- то не очень это получается, что нужно изменить в коде?

Comments

[Adamos]

Не жалеть пробелов в коде, но ставить их одинаково, а не как получится. Код должен читаться, а не выглядеть выкинутым из окна.

Использовать вектора вместо голых массивов. Писать больше, но читать и обрабатывать - легче, и труднее запутаться.

Конкретно по задаче - внимательно посмотреть на главную функцию и мысленно покрутить оба вложенных цикла, обращая внимание на то, что и куда заносится. Одна ошибка там видна невооруженным глазом, но она у вас должна и в дебаге выскакивать...

[m000gg m000gg]

Adamos,дебаг ничего не выдает

[Adamos]

m000gg m000gg, может, его стоит включить?

[m000gg m000gg]

Adamos, дебаг включен

[Adamos]

m000gg m000gg, и молча выполняет v[1] += x[1][0], когда v == { 0 }?
Или у вас после всей этой кашицы просто нет в матрице элементов, оканчивающихся тройкой?

[res2001]

m000gg m000gg, Не привыкайте к VLA.
Вот это int matrix[n][n]; массив VLA (variable length array). В С++ их нет. Они тянутся из Си, потому что gcc позволяет это делать по умолчанию. Но если задать строгое соответствие стандарту параметрами компилятора, то ваш код не соберется. Так же он не соберется и микрософтовским компилятором, т.к. там в принципе нет поддержки VLA.
Вместо массивов VLA рекомендую использовать вектор или динамичиские массивы.

[res2001]

m000gg m000gg, У вас выход за пределы массива тут v[i] += x[i][j]; на второй итерации внутреннего цикла. Я бы рекомендовал в v сразу добавить нужное количество нулевых элементов и только потом заходить в циклы.

[Александр Лисин]

В качестве альтернативы: есть мнение, что в задачах подобного рода, когда на каком-то этапе нужно найти числа по каким-то признакам, которые очевидны человеческому глазу, но от машины требует если не перебора, то каких-то громоздких вычислений, лучше пользоваться текстовыми контейнерами а-ля QString или std::string и их методами. Изначально, в силу зрительного восприятия, человек (программист) воспринимает такую задачу не математически, а как поиск отклонений от текстового шаблона (или наоборот - совпадений с шаблоном). Более того, для решения таких задач программистами же и созданы вышеупомянутые классы и их методы (number, find, contains и т.п.), так почему бы не воспользоваться? Впрочем, это imho, конечно же.