Random Array of long

Question

Может кто-нибудь сказать мне, почему во время компиляции не было ошибки, но во время работы программы она давала ложное возвращение. Я хочу создать случайный массив long и найти max. Спасибо вам большое и извините за мой английский.

#include <iostream> 
#include <stdlib.h> 
#include <ctime> 
#include <conio.h> 
#include <malloc.h> 

using namespace std; 

void main() 
{ 
long max,dem; 
//srand(time(0)); 
long *A=(long *)malloc(1000000*sizeof(long)); 
srand(time(NULL)); 
for(long i=0;i<1000000;i++) 
{ 
    A[i]=rand(); 
} 
max=A[0]; 
for(long i=0;i<1000000;i++) 
{ 
    if(A[i]<max) 
      max=A[i]; 
} 
dem=0; 
for(long i=0;i<1000000;i++) 
    {if(A[i]==max) dem++;} 
cout <<"\n Gia tri lon nhat:  "<<max; 
cout <<"\n Tan so xuat hien:  "<<dem; 
getch(); 
} 

Answer 1

Этот

if(A[i]<max) 
     max=A[i]; 

выглядит так, как будто это должно быть

//  v-- here 
if(A[i]>max) 
     max=A[i]; 

Side Примечание: Вы не очистить память вы выделить. По крайней мере, вы должны

free(A); 

после того, как вы закончите с этим. Кроме того, не делайте этого, потому что обработка голой памяти редко бывает хорошей идеей. Вместо этого используйте std::vector:

#include <vector> 

... 

std::vector<long> A(1000000); 

вместо

long *A=(long *)malloc(1000000*sizeof(long)); 

Тогда вам не придется делать какую-либо ручную очистку, так как деструктор std::vector «s будет обрабатывать Deallocation когда A выходит из области видимости. std::vector перегружает оператор [], чтобы сделать его пригодным для использования (для многих целей), как простой массив, поэтому вам не придется менять остальную часть кода для работы.

... больше одной вещи, за исключением:

void main() 

не является стандартом C++; Из-за этого gcc и clang отказываются компилировать ваш код. Вы должны использовать

int main() 

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