Сортировка одномерного массива. С

[McFrey]

Прошу помощи, но не судите строго...Почему происходит сортировка только двух первых элементов массива?...(массив должен быть отсортирован в порядке возрастания)

# include <stdio.h>
# include <conio.h>

double sortirovka(double a[], double b, int n)
  {   
      int i,j;
    for (i=0; i<n-1; i++)
          for (j=i+1; j<n; j++)
            if (a[i]>a[j])
              {
                 b=a[i];
                 a[i]=a[j];
                 a[j]=b;
               }
           } 


int input( double a[])
{ int n,j;
while(1)
      {
              printf("\n Количество элементов массива n=");
              scanf("%i",&n);
              if ( n>1 & n<=100 ) break;
              printf("\n Недопустимое значение 1<n<=100)");
       } 
       printf("\n Значения элементов массива: \n");
       for (j=0; j<n; j++)
        {
        printf("a[%d]=", j+1);
        scanf("%lf", &a[j]);
      }
 } 
 
double output (double a[], int j, int n)
 {
         printf("\n Упорядоченный массив: \n");
          for (j=0; j<n; j++)
            printf("a[%d]=%g\n", j+1, a[j]);
 }

 int main()
{ 
      int n,i,j;
      double a[100],b;
     
   input(a);             
   sortirovka(a,b,n);                 
   output(a,j,n);        

         getch();
         return 0;
         
}

[Stilet]

for (j=i+1; j<n; j++)

По-моему тут наоборот нужно: for (j=0; j<i; j++)

[McFrey]

Stilet, тоже самое, сортирует только 2 первых, а остальные даже не выводит
Когда программа без разбиения на подпрограммы, то она работает, а вот я попытался разбить и вот нарвался на это...

[Stilet]

Ладно. Лови ликбез:
1)

input(a);
sortirovka(a,b,n);
output(a,j,n);

Функции описаны как возвращающие значния, а ты из вызываешь без выражения. Некошерно. Не всем компиляторам это понравится.

2)

if ( n>1 & n<=100 ) break;

Это не логический оператор а битовый. Нужно &&. Читай книги по основам.

3)

double output (double a[], int j, int n)

j? Зачем??? Эта переменка должна быть локальной. Возможно именно из-за того что она у тебя не константный параметр (значение ее сохраняется для остальных функций) твоя сортировка проходит только кончик массива.

И так далее - недочетов море.

Вот подправил на скорую руку.

// uu.cpp: определяет точку входа для консольного приложения.
//

#include "stdafx.h"
#include <conio.h>
#include<stdlib.h>

void sortirovka(double a[],  int n)
  {   
      int i,j,b;
    for (i=0; i<n-1; i++)
          for (j=i+1; j<n; j++)
            if (a[i]>a[j])
              {
                 b=a[i];
                 a[i]=a[j];
                 a[j]=b;
               }
           } 


void input( double a[],int n)
{ 
       for (int j=0; j<n; j++)        {
        a[j]=rand()%100;
        printf("%g\t",a[j]);
	   }
} 
 
void output (double a[], int n)
 {
          for (int j=0; j<n; j++)
            printf("a[%d]=%g\n", j+1, a[j]);
 }
int _tmain(int argc, _TCHAR* argv[])
{
     int n=10,i,j;
      double a[10],b;
     
   input(a,n);             
   sortirovka(a,n);                 
   output(a,n);        

         getch();
         return 0;
}

Особо не оптимизировал и не вникал. Оставляю изучение основ тебе. Будь внимателен, смотри что пишешь и где.

[McFrey]

Stilet, большое спасибо, попробую вникнуть