ビジュアルスタジオCプログラミング関数defenition

Question

以下のプログラムは、関数を使用して昇順にソートします。これはVisual Studioで書かれています。

を上記のmainとし、mainを宣言しているので、私は関数宣言で間違いを犯したことを知っています。

プログラムはエラーなく実行されますが、関数が呼び出されていないためソートは行われません。

プログラムはC++として保存されます。

誰でも私がこの関数を呼び出すのを助け、下のプログラムを編集してソートすることはできますか？

メインファイル

#include <stdio.h> 
#include <stdafx.h> 
#include "sort.h" 
#include <conio.h> 
#include <iostream> 

int LinearSort(); 

int main() 
{ 
    int sort[50]; 
    int i=0; 
    int j=0; 
    int k=0; 
    int a = 0; 

    printf("Enter 10 Numbers"); 

    for (i = 0; i < 10; i++) 
    { 
     scanf_s("%d",&sort[i]); 
    } 

    for (i = 0; i < 10; i++) 
    { 
     printf("%d\n",sort[i]); 
    } 


    return 0; 
} 

.Cファイル

#include "stdafx.h" 
#include "sort.h" 
#include <stdio.h> 
#include <conio.h> 
#include <iostream> 

void LinearSort(int i, int j, int k, int a, int sort[]) 
{ 
    for (j=0; j < i-1; j++) 
    { 
     for (k=0; k < i-j-1; k++) 
     { 
     if(sort[k] < sort[k+1]) 
     { 
      a = sort[k]; 
      sort[k] = sort[k+1]; 
      sort[k+1] = a; 
     } 

     else 
     { 
      sort[j] = sort[j]; 
     } 
    } 
    } 
    for (j = 0; j < i; j++) 
    { 
     printf("ascending %d\n",sort[j]); 
    } 
    _getch(); 

} 

ヘッダファイル

#pragma once 
#include <stdio.h> 

extern void LinearSort(int i, int j, int k, int a, int sort[]); 

Answer 1

あなたは正しい方法であり、あなたのコードはちょっとしたことしか必要としません。他の人たちは貴重な提案をしてくれました。

最初にあなたのLinearSort()の定義。私たちがという局所変数と呼ぶ多数の変数（i..k）を渡していて、渡すべきではありません。ローカル変数は関数でのみ使用され、関数内で宣言されます。適切な定義は次のようになります。

void LinearSort(int a, int sort[]); // prototype; put in header file or above main 

void LinearSort(int a, int sort[]) // function itself 
{ 
    int i, j, k;      // local variables 

次に、すべてのデータを読み込んだ後、メインから呼び出す必要があります。好きそれを呼び出す：

LinearSort(10, sort[]); 

あなたが整数の固定数を読んでいるのであなたがaのための10のpasss。任意の数値（50未満）を読んでいれば、この量の変数を渡したでしょう。

あなたの情報：配列変数sort[]を渡しますが、この名前sortは、あなたのメインとあなたの関数で同じであることに注意してください。必要はなく、これはちょうど偶然です。

ソートアルゴリズムは、バブルソートに基づいているようですが、for-loopsを使用しています。それは少なくとも珍しいし、間違っているかもしれません。外側のループは、それ以上要素が交換されなくなるまで必要な回数だけ行わなければならない。しかし、forループは通常、一定回数実行されるため、これがおそらく間違っているのがわかります。バブルソートについてお読みください。

Answer 2

まず、このS削除ttyement

int LinearSort(); 

すでにヘッダファイルとヘッダの機能が宣言されているので、すべてのファイルに含まれています。 scanf()の後にmain()に適切な引数を使用してLinearSort()を呼び出す必要があります。そして、あなたのLinearSort()機能では、この

int main() 
{ 
    int sort[50]; 
    int i=0; 
    int j=0; 
    int k=0; 
    int a = 0; 

    printf("Enter 10 Numbers"); 

    for (i = 0; i < 10; i++) 
    { 
     scanf_s("%d",&sort[i]); 
    } 

    LinearSort(i, j, k, a, sort); 

    for (i = 0; i < 10; i++) 
    { 
     printf("%d\n",sort[i]); 
    } 
    return 0; 
} 

とLinearSortで（）のように、forループ内で、私はelse文の任意の特定の使用が表示されません。