查找数组中出现最多的数字-C问题

Question

正如你们在问题中看到的，我有这个问题。我的意思是，我真的看不出有什么问题，也许是一些愚蠢的事情，但我不能解决它。这是代码:

#include <stdio.h>
int main() {
    int i,n;
    int niz[100];
    int brojac[100]={0};
    int maxi,mini;
    printf("Sada unesi clanove niza: \n");
    for(i=0;i<100;i++) {
        scanf("%d",&niz[i]);
        if(niz[i]==-1) {
            printf("Unijeli ste -1, a to znaci kraj unosenja clanova niza.\n");
            break;
        }
        if(niz[i]<0 || niz[i]>100){
            printf("Pogresan unos. On se nece pikati.\n");
            i--;
        }
    }
    n=i;

    //POVECAVAMO VRIJEDNOST NEKIH CLANOVA HISTOGRAMA(SVI CLANOVI VEC POSTOJE, I INICIJALIZOVANI SU NULOM)
    for(i=0;i<n;i++) {
        brojac[niz[i]]++;
    }
    maxi=0;
    mini=-1;

    //Nadji INDEX NAJVECEG ELEMENTA HISTOGRAMA
    for(i=0;i<=100;i++) {
        if(brojac[i]>brojac[maxi]) {
            maxi=i;
        }
        if(brojac[i]>0 && (mini==-1 || brojac[i]<brojac[mini])) { 
            mini=i;
        }
    }

    printf("Element niza koji se ponavlja najveci broj puta(a koji je i najamnji ako takvih brojeva ima vise je: %d\n",maxi);
    printf("Element niza koji se ponavlja najmanji broj puta(a koji je i najamnji ako takvih brojeva ima vise je: %d",mini);
    return 0;
}

Answer 1

for(i=0;i<=100;i++) -> for(i=0;i<100;i++)

Answer 2

因此，该程序试图确定在1到100(包括1和100)之间输入的最常出现的数字。您的html" target="_blank">方法是将100个数字存储在数组niz[]中，然后遍历该数组，并将一个数字在另一个数组brojac[]的等价索引中出现的次数存储起来。所以如果nix[1]= 8那么bro JAC[8]；

另一种解决方案是将数字读入临时值，如果它在范围内，则增加存储在数组中数字索引处的值。确保首先将 nix[] 上的所有值初始化为零。

示例:输入了3。3存储在temp中。3在范围内，因此niz[temp]= niz[temp]；

但无论如何，问题是

maxi=0;
mini=-1;

//Nadji INDEX NAJVECEG ELEMENTA HISTOGRAMA
for(i=0;i<=100;i++) {// here
    if(brojac[i]>brojac[maxi]) {
        maxi=i;
    }
    if(brojac[i]>0 && (mini==-1 || brojac[i]<brojac[mini])) { // and  here
        mini=i;
    }
}

那一行'if（brojac[i]

对于 i=0 且 mini = -1 读取“如果（布罗杰克[0]

brojac[-1]不存在，因为您是引用内存中的某个随机点，可以包含任何内容。

更正是:

maxi=1;
mini=1;

//Nadji INDEX NAJVECEG ELEMENTA HISTOGRAMA
for(i=1;i<=100;i++) {// i was set to zero, it's set to one cause the range the code accepts is between 1 and 100
    if(brojac[i]>brojac[maxi]) {
        maxi=i;
    }
    if(brojac[i]<brojac[mini]) { 
        mini=i;
    }
}