Почему массив malloc не наполняется?

Question

[you_go_home]

#include <locale.h>
#include <stdio.h>
#include <time.h>
#include <malloc.h>
#include <stdlib.h>
#define N 1000
int sort(int mas_otr, int n);

int main()

{
	setlocale(LC_ALL, "RUS"); 
    int i, n, *mas, *mas_otr;
    setlocale(0, "rus");
    printf("Введите размер массива: ");
    scanf("%d", &n);
    srand(time(0));
    mas = malloc(n * sizeof(int));
    mas_otr = malloc(n * sizeof(int));
    printf("Сгенерированный массив:\n");
    for (i = 0; i < n; i++) {
        mas[i] = rand() % 100;
        printf("%d ", mas[i]);
    }
    for (int b = 0; b < n; n++) {
        if (mas[b] < 0) { mas_otr[b] = mas[b]; printf("%d", mas_otr[b]); //НЕ ВЫВОДИТ МАССИВ
        }
    }
    sort(mas_otr,n);
    free(mas);
    free(mas_otr);
    return 0;

	getchar(); 

}

int sort(int mas_otr, int n) {
    int i, *j, tmp;
    for (i = 0; i < n; i++) { 
        for (j = 0; j < n - i - 1; j++) {
            if (mas_otr[j] > mas_otr[j + 1]) {
                 tmp = mas_otr[j];
                 mas_otr[j] = mas_otr[j + 1];
                 mas_otr[j + 1] = tmp;
            }
        }
    }
    puts("Массив с отрицательными значениями");
    for (int *c = 0; c < n; c++) {
        printf("%d", mas_otr[c]); //НЕ ВЫВОДИТ МАССИВ
    }
}

Comments

[Roman]

А ещё *
sort(int mas_otr, int n)

int c = 0

for (int *c = 0; c < n; c++) {
        printf("%d", mas_otr[c]); //НЕ ВЫВОДИТ МАССИВ
}

[you_go_home]

Roman,

[res2001]

you_go_home, смотри мой ответ п.4

[Roman]

you_go_home, ещё раз по-порядку

А ещё *
sort(int mas_otr, int n)

https://en.cppreference.com/w/c/language/pointer

Answer 1

[res2001]

1.rand() генерирует только положительные числа: https://en.cppreference.com/w/c/numeric/random/rand
2.Даже если бы с п.1 было бы все нормально, то после того как выделена память для mas_otr ее нужно обнулить, т.к. malloc никакой инициализации памяти не производит и там будет содержаться мусор. А вы заполняете mas_otr только выборочно. Так что итоговый результат очевидно то же будет не тем, что вы ожидаете.
3.нужно в правильных местах в вывести переносы строк, а то сейчас у вас все будет лепиться в одну строку.
4.В sort() вместо массива вы передаете целое число.

Comments

[res2001]

5.разберитесь с sort() вы там запутались с указателями и индексами.
i,j,c должны быть int
mas_otr - int*

[you_go_home]

Как последовательно заполнить массив? Теперь все под растчет создается, но не заполняется.

#include <locale.h>
#include <stdio.h>
#include <time.h>
#include <malloc.h>
#include <stdlib.h>
#define N 1000
int sort(int mas_otr, int n);

int main()

{
	setlocale(LC_ALL, "RUS"); 
    int i, n, *mas, *mas_otr,l=0;
    setlocale(0, "rus");
    printf("Введите размер массива: ");
    scanf("%d", &n);
    srand(time(0));
    mas = malloc(n * sizeof(int));
    
    printf("Сгенерированный массив:\n");
    for (i = 0; i < n; i++) {
        mas[i] = rand() % 100 - 32;
        printf("%d ", mas[i]);
    }
    for (int b = 0; b < n; n++) {
        if (mas[b] < 0) {
            l++;
        }
    }
    mas_otr = malloc(l * sizeof(int));
    for (int b = 0; b < n; n++) {
        if (mas[b] < 0) { mas_otr[b] = mas[b]; //он должен последовательно заполнять элементы нового массива 
        }
    }
    sort(mas_otr,n);
    free(mas);
    free(mas_otr);
    return 0;

	getchar(); 

}

int sort(int mas_otr, int n) {
    int i, *j, tmp;
    for (i = 0; i < n; i++) { 
        for (j = 0; j < n - i - 1; j++) {
            if (mas_otr[j] > mas_otr[j + 1]) {
                 tmp = mas_otr[j];
                 mas_otr[j] = mas_otr[j + 1];
                 mas_otr[j + 1] = tmp;
            }
        }
    }
    puts("Массив с отрицательными значениями");
    for (int *c = 0; c < n; c++) {
        printf("%d", mas_otr[c]);
    }


}

[res2001]

Прикольно, что в текущей комбинации типов j и mas_otr в первом цикле sort() все работает правильно (только для х32 варианта сборки).
Но это только из-за того, что вам повезло так ошибится. Вряд ли имелось ввиду то что написано. Так что исправляйте.

[res2001]

you_go_home,
В mas_otr[b] = mas[b] - выход за границы массива в случае если b > l

[res2001]

you_go_home,

Теперь все под растчет создается, но не заполняется.

Потому что b - это индекс по массиву mas, по mas_otr нужен свой собственный индекс в третьем цикле main().
Вы еще видимо отладчиком не научились пользоваться? Пора!

[you_go_home]

res2001,

for (int b = 0; b < n; b++) {
        if (mas[b] < 0) { mas_otr[n-b] = mas[b];
        }

[you_go_home]

res2001, Он же должен с первого заполнять. Я теперь написал!))))))))))))))))))))))))

[you_go_home]

res2001,

#include <locale.h>
#include <stdio.h>
#include <time.h>
#include <malloc.h>
#include <stdlib.h>
#define N 1000
int sort(int mas_otr, int n);

int main()

{
	setlocale(LC_ALL, "RUS"); 
    int i, n, *mas, *mas_otr,l=0;
    setlocale(0, "rus");
    printf("Введите размер массива: ");
    scanf("%d", &n);
    mas = malloc(n * sizeof(int));
    
    printf("Сгенерированный массив:\n");
    for (i = 0; i < n; i++) {
        mas[i] = rand() % 100 - 32;
        printf("%d ", mas[i]);
    }
    for (int b = 0; b < n; n++) {
        if (mas[b] < 0) {
            l++;
        }
    }
    mas_otr = malloc(l * sizeof(int));
    for (int b = 0; b < n; b++) {
        if (mas[b] < 0) { mas_otr[n-b] = mas[b];
        }
    }
    sort(*mas_otr,l);
    free(mas);
    free(mas_otr);
    return 0;

	getchar(); 

}

int sort(int *mas_otr, int l) {
    int i, j, tmp;
    for (i = 0; i < l; i++) { 
        for (j = 0; j < l - i - 1; j++) {
            if (mas_otr[j] > mas_otr[j + 1]) {
                 tmp = mas_otr[j];
                 mas_otr[j] = mas_otr[j + 1];
                 mas_otr[j + 1] = tmp;
            }
        }
    }
    puts(" \n Массив с отрицательными значениями");
    for (int c = 0; c < l; c++) {
        printf("%d", mas_otr[c]);
    }


}

[you_go_home]

res2001, да уж, ошибок дофига. Исправляю )

[res2001]

for (int b = 0, c = 0; b < n; b++) {
        if (mas[b] < 0) { 
            mas_otr[c] = mas[b]; 
            ++c;
        }

[you_go_home]

res2001, благодарю!