Исправить программу на Си

[Максим28]

Программа запускается но выводит не то, что нужно.
Задание:
Дан массив из 18 вещественных чисел, все элементы которого различны.
1) Найти произведение (ненулевых) элементов массива, находящихся на позициях, кратных числу 3.
2) Найти квадраты всех чисел, расположенных между минимальным и максимальным значениями.
3) Упорядочить по возрастанию элементы, находящиеся на местах, не кратных числу 3.

#include <stdio.h>
#define SIZE 18
 
void search(int [], int);
void MaxMin(int [], int);
void increase(int [], int);
 
int main()
{
int arr[SIZE] = {0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15, 16, 17};
 
search(arr, SIZE);
MaxMin(arr, SIZE);
increase(arr, SIZE);
 
return 0;
}
 
void search(int a[], int size) //произведение на позициях кратных трем
{
int f1 = 1;
int i;
for (i = 0; i < size; i++)
{
if (i % 3 == 0)
{
f1 *= a[i];
}
}
 for(i = 0; i < size; i++)
    printf("% d", a[i]);
}
 
void MaxMin(int a[], int size) //нахождение квадратов
{
int max = a[0], min = a[0];
int A[size]; //в этом массиве будут храниться все квадраты нужных нам чисел
int i;
 
for (i = 0; i < size; i++)
{
if (max < a[i])
{
max = i;
}
if (min > a[i])
{
min = i;
}
}
 
int begin = min < max ? min : max;
int end = min > max ? min : max;
 
for (i = 0; begin <= end; begin++)
{
A[i] = a[begin] * a[begin];
i++;
}
for(i = 0; i < size; i++)
    printf("% d", A[i]);
}
void increase(int a[], int size) //сортировка по возрастанию всех элементов не кратных трем
{
int hold;
int i, j;
 
for (i = 0; i < size; i++)
 
for (j = 0; j < size; j++)
if (i % 3 != 0 && a[i] > a[i + 1])
{
hold = a[i];
a[i] = a[i + 1];
a[i + 1] = hold;
}
for( i = 0; i < size; i++)
printf("% d", a[i]);
}

[Unique_Maks]

а через отладчик смотреть не пробовали?

[pitlis]

Проблема с последней функцией?
Попробуйте так:

for (i = 0; i < size; i++)
 
for (j = 0; j < size; j++)
if (j % 3 != 0 && (j+1) % 3 != 0  && a[j] > a[j + 1])
{
hold = a[j];
a[j] = a[j + 1];
a[j + 1] = hold;
}
if (j % 3 != 0 && (j+1) % 3 == 0 && a[j] > a[j + 2])
{
hold = a[j];
a[j] = a[j + 2];
a[j + 2] = hold;
}

[Максим28]

Сообщение от pitlis

Проблема с последней функцией?

Тоже самое, скорее всего проблемы во всех функциях .

[Максим28]

Сообщение от Unique_Maks

а через отладчик смотреть не пробовали?

Если вы про лог с ошибками, то их там нет.

[pitlis]

Первые две функции вроде правильные. Неправильная сортировка была в третьей. Вы учитываете, что в заданном изначально массиве (0-17) - все элементы и так стоят по возрастанию - упорядочивать там нечего.
PS Зачем нужен вывод массива целиком в первой функции - отдельный вопрос. Нужно также добавить проверку, не является ли число нулем.

[Unique_Maks]

Сообщение от Максим28

Если вы про лог с ошибками, то их там нет.

я про трассировку

[Unique_Maks]

void search(int a[], int size) // 
{
  int f1 = 1;
  int i;
  for (i = 0; i < size; i++)
  {
    if (i % 3 == 0)
    {
      f1 *= a[i];
    }
  }
  for(i = 0; i < size; i++)
    printf("% d", a[i]);
}

0 % 3 = 0 -> f1 = 0, в итоге произведение = 0
если int arr[SIZE] = {0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15, 16, 17};

Возможно, как-то так было бы лучше

int search(int a[], int s)
{
	int res = 1;
	for (int i = 1; i <= s; i++)
		if (i % 3 == 0)
			res *= a[i-1];
	return res;
}

или нулевая позиция должна считаться...