перестановка эл. одномерного массива

[quartzz]

error C4716: 'fill_array' : must return a value

-ошибка указывает на конец блока fill_array.

[Sazary]

Ну так заголовок функции вы изменили, но от этого ничего возвращать она не будет, пока сами не пропишете.
Вам нужно возвращать указатель на int, вот и добавьте

return mas;

[quartzz]

Всё равно ситуация прежняя - ничего не сортирует по возрастанию, массив остаётся прежним. Выкладываю ВЕСЬ код программы из Visual Studio 2005. Может быть посмотрите на своём компиляторе, где там не так что-то.

#include "stdafx.h"
#include <iostream>
#include <cmath>


using namespace std;
static int size = 0;
static int *mas = NULL;

  int* fill_array()
 
{
	cout << "Vvedite razmer massiva : ";
	cin >> size;
	cout << endl;
	
	mas = new int [size];
	for (int i = 0; i < size; i++)
	{
		cout << "Vvedite element massiva " << (i + 1) << ": ";
		cin >> mas[i];
	}
   cout << endl;
   return mas;
  }

void view_array()
{
	cout<<"Nomer elementa massiva :\tZnachenije elementa massiva : \n";
	for(int i=0; i<size; i++)
  {
    cout<<i<<"                        \t"<< mas[i]<<"\n";
  }

}

void find_numbers()
{
	 cout<<"Vvedite iskomoe chislo ";
     int find;
	 cin>>find; //искомое число записывается в переменную find.
	 int count=0; //начальное кол-во искомых элементов инициализируется нулём.
    
	for(int i=0;i<size;i++)// проходит по всему массиву (от нуля до size).
    {
        if(mas[i]==find)/*если элемент массива совпадает с искомым
						- увеличиваем счётчик на 1.*/
        {
          count++;
        }
    } 
    cout<<"\nKolichestvo iskomih chisel v massive - "<<count;
    cout<<endl;
}



  void delete_null()
{
	
    int n,i,k; 
    n=size;// размер исходного массива
    k = -1; 
  for(i=0; i<n; i++)
      if(mas[i]==0)  // ищем нулевой элемент
     {
       k = i;  // запоминаем его индекс.
       break;
     }  
		  
	      if(k>-1) // если нулевой элемент был найден
     {
          n--;  // уменьшаем размер массива
         for(i=k;i<n;i++) 
          mas[i]=mas[i+1];  // и сдвигаем все, что правее нуля, влево
     } else // в противном случае выводим :
		  {
	        cout << "Nulevoj element ne bil najden, massiv ostalsa bez izmenenij " <<endl;
	      } 
   
		  cout << "Znachenije massiva na dannij moment :     ";
	   
		  for(i=0; i<n; i++)//до n (n - уменьшенное значение массива.
	   {
		   cout<<mas[i]<<" "; //выводим массив на текущий момент
	   }
      
	   cout<<endl; //переход на новую строчку
   }


  void elements(int* mas, int size)

 
 {
	 int k = 0;
     int i = 0;
     int temp1 = 0;
	 int temp2 = 0;
	 
	 while (k<size)  // выполняется столько раз, сколько всего записей в массиве (size).
   {
      while (i<size-1) //выполняется на 1 раз меньше, т.к. появляется лишний символ.
	{
	   if (mas[i]>mas[i+1]) //сравниваем два рядом стоящих числа. если 2-ое число меньше  
          
 	   {
	     temp1 = mas[i];   // большее число
	     temp2 = mas[i+1]; // меньшее число
	     mas[i] = temp2;  // меняем местами большее и меньшее числа.
	     mas[i+1] = temp1;
	   }
	      i++;
	 }
    i=0;
    k++;
   }

    
 }


int main()
{
	fill_array();
	int action;
	while (true)
	{
		action = 0;
		cout << "Viberete dejstvije: " << endl;
		
		while ( action < 1 || action > 5)
		{
			cout << "1: Pokazatj massiv" << endl;
		    cout << "2: Najti kol-vo zadannogo elementa v massive" << endl;
			cout << "3: Udalitj nulevoj element massiva" << endl;
			cout << "4: Perestavitj elementi massiva po vozrastaniju" << endl;
			cout << "5: Zakritj programmu" << endl;
			cin >> action;
			cout << endl;
			switch (action)
			{
			case 1:
				view_array();
				break;
			
			case 2:
				find_numbers();
				break;
			
			case 3:
				delete_null();
				break;
			
			case 4:
				void elements();
				break;
			
			case 5:
				exit(0);
				break;
			default:
				cout << "Viberete dejstvije: " << endl;
			}
		}
		cout << endl;
	}

	return 0;
}

[Sazary]

  void elements(int* mas, int size)

если у вас массив глобальный, то и незачем передавать его в качестве аргумента.
К тому же вы все равно ему ничего не передаете при вызове (вас должно было насторожить уже то, что все компилится без проблем).

case 4:
  void elements();
  break;

смотрим на эти строчки и ищем подвох. Не замечаете? void уберите.

[quartzz]

действительно, просто скопировал в блок свич не убрав, поставил там

case 4:
				int elements();
				break;

но всё также - никакой сортировки не происходит.

[Sazary]

А теперь подумайте, что вы сделали.
При вызове функции не нужно указывать ее тип.

[quartzz]

я это сделал потому что если не указывать появляется ошибка :

error C2660: 'elements' : function does not take 0 arguments.

[MaTBeu]

Правильно, потому что у вас функция принимает 2 аргумента, а передаете вы 0. Передайте 2 аргумента и все скомпилится.

[quartzz]

сделал так

case 4:
				elements(mas,size);
				break;

и всё отлично сортирует! всё, программа готова. Всем огромное спасибо! Буду надеятся что прокатит =)

[Sazary]

quartzz, пара замечаний.
Как я уже сказал, раз у вас массив глобальный, то вам незачем передавать его в качестве параметра.
В функции fill_array вы возвращаете ваш массив (mas), но нигде этим не пользуетесь, т.к. не ничему не присваиваете то, что возвращает функция:

fill_array();

Аналогично и с elements().