argcを使ってforループの範囲を制御する

Question

これは私のコードです。なんらかの理由で、無限ループ（最初のforループ）に詰まってしまいます。このコマンドの目的は、コマンドライン引数を取得するためにそれをコード化し、ユーザーがその多くの引数を入力し、残りのプログラムが最小の番号を出力します。前もって感謝します！

#include<stdio.h> 

int main(int argc, char *argv[]) 
{ 
    int array[argc]; 
    int smallest; 
    for(int i=0; i < argc; i++) 
    { 
     printf("Welcome to number comparator!\n"); 
     printf("Enter command line # of numbers(enter delimitater):"); 
     scanf("%d",&array[i]); 
    } 
    for(int i=0; i < sizeof(array); i++) 
    { 
     if(i == 0) 
      smallest=array[i]; 
     else if(array[i] < array[i-1]) 
     { 
      smallest=array[i]; 
     } 
    } 
} 
printf("The smallest number is %d", smallest); 

    return 0; 

} 

編集：これは、cmd行から整数の配列を取得するのと同じではありません。 cmd行の引数は、入力する必要がある引数の数を入力するだけです。本質的に、私が./a.out 5.を実行すると、その後に5つの数字を入力し、最小値を返すべきです。

Answer 1

Cがコマンドライン引数をどのように渡すかについて完全に誤解しました。あなたはまた、大括弧よりも多くの閉じ括弧を持っています。そして、あなたの声明命令は、ある調整を使うことができます。ここで

は、上記の問題に対処するあなたのコードの手直しです：

#include <stdio.h> 
#include <stdlib.h> 

int main(int argc, char *argv[]) 
{ 
    int length = atoi(argv[1]); 

    int array[length]; 

    printf("Welcome to number comparator!\n"); 
    printf("Enter %d number(s): ", length); 

    for (int i = 0; i < length; i++) 
    { 
     (void) scanf("%d", &array[i]); 
    } 

    int smallest = array[0]; 

    for (int i = 1; i < length; i++) 
    { 
     if (array[i] < smallest) 
     { 
      smallest = array[i]; 
     } 
    } 

    printf("The smallest number is %d\n", smallest); 

    return 0; 
} 

これは、完成したコードではありません、あなたはまだいくつかのエラーチェックを実行する必要があります。コマンドラインの値はargc > 1を渡されたことを確認してください。コマンドラインの値が数値であることを確認してください。 scanf()が正常に番号を読み取ることを確認してください。