1

Тема: Зацініть програму

#include <stdio.h>

#define Z 14
int data[Z] = {-100, 13, -200, 1, 2, 26, -13, 50, 14, 5, 6, 1, 100, -100};

int main(void)
{
int a, b, c;
   
   for(int i=0; i<(Z-2); i++)
   {
     a = data[i+0];
     b = data[i+1];
     c = data[i+2];
      
        if (a > b && a < c || a < b && a > c)
            printf("%d\n", a);
                else
                    if (b > a && b < c || b < a && b > c)
                       printf("%d\n", b);
                        else
                            if (c > a && c < b || c < a && c > b )
                            printf("%d\n", c);
    }
   
return 0;
}

2

Re: Зацініть програму

1. Немає тегів code
2. Нерівні відступи
3. Незрозумілі імена змінних
4. Взагалі невідомо, що програма має робити

Досить?

Подякували: cheappi386, Master_Sergius2

3

Re: Зацініть програму

Amper

А що вона має робити по завданню?

4

Re: Зацініть програму

А чому масив  - як глобальна змінна?
Всі інші мінуси Вам описав пан koala:)

5

Re: Зацініть програму

Такі умови

c > a && c < b || c < a && c > b

краще писати так

a < c && c < b || b < c && c < a

тоді їх легше читати.

6 Востаннє редагувалося koala (27.11.2015 11:49:46)

Re: Зацініть програму

Я підозрюю, що тут їх взагалі не треба писати, але залежить від завдання ж. Бо "знайти медіанне значення для кожних трьох елементів, що йдуть поспіль" - якось дивно. А

min(a,b) < c && c < max(a,b)

ще красивіше.

7

Re: Зацініть програму

Хоспадє!   -  така оцінка програми згодиться?

Подякували: koala1

8

Re: Зацініть програму

...ковзна медіана з шириною вікна на три точки, то й що.

9

Re: Зацініть програму

a = data[i+0];

навіщо нуль?

10 Востаннє редагувалося koala (27.11.2015 16:39:50)

Re: Зацініть програму

raxp написав:

...ковзна медіана з шириною вікна на три точки, то й що.

За умови, що це вона. Якщо чесно, ніколи не стикався з такою характеристикою, принаймні на практиці (включно з навчанням). Але відчуваю, що в автора трохи інше завдання.

FakiNyan написав:
a = data[i+0];

навіщо нуль?

Для однорідності. Іноді має сенс. Хоча, звісно, не тут. Тут мало що має сенс.

11

Re: Зацініть програму

Хоча, звісно, не тут.

У данному випакду зеро тут задля форматування коду. Так читати легше. А в цілом згоден.
Більш того: Хтось відповість на моє питання вишче?

12

Re: Зацініть програму

За умови, що це вона

вона, бо в нього задано вiкно, яке ковзає по масиву (в даному випадку заданий статично) і видає в консоль медіанне значення залежно від попередніх і наступних значень. Алгоритм фільтрує різкі кидки в послідовності даних.

http://raxp2.blogspot.com/2015/10/blog-post_13.html

13 Востаннє редагувалося raxp (27.11.2015 18:19:42)

Re: Зацініть програму

видалити, не в цю тему.

14

Re: Зацініть програму

Пане raxp, я чудово бачу, що тут відбувається, я тільки висловлюю сумнів, що це було завданням автора.
Втім, схоже, автора ця тема зовсім не цікавить.