Segmentation fault c++?

Question

[danila_prokopenko]

#include <stdio.h>
#include <iostream>

using namespace std;

int main()
{
    int n;
    int u=0;
    int a[n];
    
    // do{
    //     if (u != 0)
    //     {
    //         cerr << "Ошибка"; cin.clear(); cin.seekg(0);
    //     }
    //     cout << "Сообщите размерность массива N: "; u++;
    //     cin >> n;
    // } while (cin.get() != '\n');
    
    cout << "Сообщите размерность массива N: "; u++;
    cin >> n;
    
    // рандомное заполнение массива
    srand(time(0));
    for(int i=0; i < n; i++){
        a[i] = 1 + rand() % 100;
        std::cout << a[i] << " ";
    }
    
    for(int k = 0; k < n; k++)
    {
        if(a[k]%2==0)
        {
            cout << a[k] << " ";
        }
    }
    
    
    
    
    

    return 0;
}

выдает ошибку, как исправить?

Comments

[Евгений Шатунов]

danila_prokopenko ,

как исправить?

Отлаживать. Иначе исправить никак.

Answer 1

[Wataru]

Чему равно n, когда вы заводите массив a[n]?

Comments

[pfemidi]

Что за идиотская манера использовать VLA толком не понимая что это и игнорируя напрочь что "In C99, the length parameter must come before the variable-length array parameter in function calls."?

[Wataru]

pfemidi, ну что вы, эти страшные заклинания явно автору непонятны.

[Евгений Шатунов]

pfemidi, ключевое тут - это In C99. А стандарт C++ никогда не поддерживал C99.

[pfemidi]

Wataru, Евгений Шатунов, но я заметил тенденцию что 99% новичков, студентов и просто мамкиных хакеров почему-то используют именно VLA вместо тех же malloc/free или new/delete или на крйний случай обычных статических масстивов.

[Wataru]

pfemidi, Ну, потому что не понимают, как надо. Видели в примерах int a[10]; и когда им надо n элементов - тупо меняют 10 на n. Стек, куча - это все какие-то непонятные для них слова.

[pfemidi]

Wataru, вот посему пусть сначала книжки читают (ну или лекции не прогуливают), а после уже 10 на n меняют.

[Армянское Радио]

Так целая тонна училкиных методичек содержит две мои любимые ржаки:
Намба уан: int n; int a[n];
Намба ту: system("pause");

И тиражируется эта гадость из головы в голову и конца-края этому днищу нет.