Произведение всех отрицательных чисел,убрать ошибки

@ssapers · Регистрация: 03.10.2015

Author24 — интернет-сервис помощи студентам

Помогите убрать ошибки,где должен объявляться массив для того чтобы программа не крушилась?Условие задачи,нужно чтобы пользователь сам объявлял длину массива и считал произведение всех отрицательных чисел массива

C

#include <stdio.h>
#include <conio.h>
#include <locale.h>
 int main(){
 
    int N, i, k=0;
    float A[N],mid=1;
    setlocale(LC_ALL,"Russian");
    printf("\Программа произведения отрицательных чисел");
    printf("\nВведите длину массива=");
    scanf("%d", &N);
    printf("\Введите элементы массива=");
    for (i=0;i<N;i++){
            scanf("%f", &A[i]);
        }
    for (i=0;i<N;i++){  
        if (A[i]<=0){
                mid=mid*A[i];
                k++;
 
        }
    }
    if(k==0){
        printf("В массиве нет отрицательных чисел");
    }else{
        printf("Произведение отрицательных чисел=%.1f",mid);
    }
    
    getchar();
}

@Catstail · 03.10.2015, 16:17

ssapers, а она компилируется? Сомневаюсь. Массивы так не описывают (N должна в этом случае быть константой).

@Fonduee · 03.10.2015, 16:38

Сообщение от Catstail

а она компилируется?

Да. Ибо пока N не инициализирован, значение этой области памяти может быть любым

@Catstail · 03.10.2015, 16:52

Сообщение от Fonduee

значение этой области памяти может быть любым

- а размер?

@Fonduee · 03.10.2015, 17:08

Сообщение от Catstail

- а размер?

Как уже сказал, при декларации переменной выделяется область, что в ней стоит до инициализации есть UB и зависит уже от используемого компилятора. Значение может быть 0, может и 523113409475275873578874385. Это и будет размер массива. Компилятор максимум возмутится варнингом, но скомпилирует программу.

@bob2015 · 03.10.2015, 17:12

Используй функции malloc, calloc, free. Или введи ограничение на максимальную длину массива.

C

#include <stdio.h>
#include <conio.h>
#include <locale.h>
#include <stdlib.h>
 
 int main(){
 
    int N, i, k = 0;
    float *A, mid = 1;
 
    setlocale(LC_ALL,"Russian");
 
    printf("\Программа произведения отрицательных чисел");
    printf("\nВведите длину массива = ");
    scanf("%d", &N);
 
    A = (float *)calloc(N, sizeof(float));
 
    printf("\Введите элементы массива = ");
 
    for (i = 0; i < N; i++)
        scanf("%f", &A[i]);
 
    for (i = 0; i < N; i++)
    {  
        if (A[i] <= 0)
        {
            mid *= A[i];
            k++;
        }
    }
 
    if(!k)
        printf("В массиве нет отрицательных чисел");
    else
        printf("Произведение отрицательных чисел = %.1f",mid);
    
    getchar();
    free(A);
}

@ssapers · 03.10.2015, 18:19 **[ТС]**

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

@Fonduee · 03.10.2015, 18:48

ssapers, Объявляй массив после ввода пользователем его размера.

@ssapers · 03.10.2015, 19:57 **[ТС]**

Так?

C

#include <stdio.h>
#include <conio.h>
#include <locale.h>
 int main(){
 
    int N, i, k=0;
    setlocale(LC_ALL,"Russian");
    printf("\Программа произведения отрицательных чисел");
    printf("\nВведите длину массива=");
    scanf("%d", &N);
    float A[N],mid=1;
    printf("\Введите элементы массива=");
    for (i=0;i<N;i++){
            scanf("%f", &A[i]);
        }
    for (i=0;i<N;i++){  
        if (A[i]<=0){
                mid=mid*A[i];
                k++;
 
        }
    }
    if(k==0){
        printf("В массиве нет отрицательных чисел");
    }else{
        printf("Произведение отрицательных чисел=%.1f",mid);
    }
    
    getchar();
}

@Fonduee · 03.10.2015, 22:19

Сообщение от ssapers

Так?

Да. В таких вопросах можно и самому проверить, работает или нет.

@Catstail Модератор 36601 / 20330 / 4220 Регистрация: 12.02.2012 Сообщений: 33,641 Записей в блоге: 13
	03.10.2015, 16:17	2
	ssapers, а она компилируется? Сомневаюсь. Массивы так не описывают (N должна в этом случае быть константой). 0

@Fonduee Музыка нас Связала 232 / 232 / 52 Регистрация: 26.03.2008 Сообщений: 616
	03.10.2015, 16:38	3
	Сообщение от Catstail а она компилируется? Да. Ибо пока N не инициализирован, значение этой области памяти может быть любым 0

@Catstail Модератор 36601 / 20330 / 4220 Регистрация: 12.02.2012 Сообщений: 33,641 Записей в блоге: 13
	03.10.2015, 16:52	4
	Сообщение от Fonduee значение этой области памяти может быть любым - а размер? 0

@Fonduee Музыка нас Связала 232 / 232 / 52 Регистрация: 26.03.2008 Сообщений: 616
	03.10.2015, 17:08	5
	Сообщение от Catstail - а размер? Как уже сказал, при декларации переменной выделяется область, что в ней стоит до инициализации есть UB и зависит уже от используемого компилятора. Значение может быть 0, может и 523113409475275873578874385. Это и будет размер массива. Компилятор максимум возмутится варнингом, но скомпилирует программу. 0

@ssapers 0 / 0 / 0 Регистрация: 03.10.2015 Сообщений: 13
	03.10.2015, 18:19 [ТС]	7
	Пользователь сам должен вводить длину массива и его значения.Но мне сказали,что нельзя так объявлять массив,как это сделал я,ведь она приводит к краху.Как объявить массив,чтобы это не вело к краху? 0

@Fonduee Музыка нас Связала 232 / 232 / 52 Регистрация: 26.03.2008 Сообщений: 616
	03.10.2015, 18:48	8
	ssapers, Объявляй массив после ввода пользователем его размера. 0

@Fonduee Музыка нас Связала 232 / 232 / 52 Регистрация: 26.03.2008 Сообщений: 616
	03.10.2015, 22:19	10
	Сообщение от ssapers Так? Да. В таких вопросах можно и самому проверить, работает или нет. 0