Lesen von stdin und vergleichen Zahlen in C

Question

Das Ziel hier ist es, 1 Zahl von stdin und dann eine Reihe von Zahlen zu lesen und dann zu überprüfen, ob die beiden eine Bijektion sind. Wenn ich es richtig verstanden habe, bedeutet das, dass die erste Zahl in der Menge der Zahlen sein muss und dass es keine doppelten Zahlen in der Menge geben darf.

Probeneingang

3 
1 2 3 

Beispielausgabe

YES 

Probeneingang

5 
2 3 4 5 2 

Beispielausgabe

NO 

Mein Eingang

3 
1 2 3 

Meine Ausgabe

NO 

Es scheint, dass mein Fehler fr kommt om mein Array, das nach Duplikaten sucht. Es wird immer checkDups bis 1. Dieses Stück Code einstellen:

for(x=0; x < 20; x++) { 
    if(n == numbers[x]) checkNums = 1; 
    for(y=0; y < 20; y++) { 
     if(x != y && numbers[x] == numbers[y]) { 
      checkDups = 1; 
     } 
    } 
} 

vollständigen Code

int n; 
int numbers[21]; 
int i = 0; 
int x = 0; 
int y = 0; 
int checkDups = 0; 
int checkNums = 0; 

scanf("%d", &n); 

while(i < 20 && scanf("%d", &numbers[i]) == 1) i++; 

for(x=0; x < 20; x++) { 
    if(n == numbers[x]) checkNums = 1; 
    for(y=0; y < 20; y++) { 
     if(x != y && numbers[x] == numbers[y]) { 
      checkDups = 1; 
     } 
    } 
} 

if(checkNums == 1 && checkDups == 0) printf("YES"); 
else printf("NO"); 

Answer 1

Da es eine ist nicht initialisiertes Array (numbers), kann das Verhalten von == nicht sicher sein.

In C11 Standarddokument

ein L-Wert, ein Objekt des automatischen Lagerdauer bezeichnen, die mit dem Register Speicherklasse wird in einem Kontext verwendet, erklärt worden sein könnte, der den Wert des bezeichneten erfordert Objekt, aber das Objekt ist nicht initialisiert. (6.3.2.1).

So das Ergebnis mit numbers[x] (numbers[y]), wo x >= i (y >= i) ist nicht definiert.

Bitte verwenden Sie keine nicht initialisierten Bereich (in diesem Fall die Indizes über i)

Answer 2

Diese for Schleife funktioniert für mich geändert:

for(x=0; x < i; x++) { 
    if(n == numbers[x]) 
     checkNums = 1; 
    for(y=x; y < i; y++) { 
     if(x != y && numbers[x] == numbers[y]) { 
      checkDups = 1; 
     } 
    } 
}