Kod źródłowy w języku C, ciąg arytmetyczny

Question

Kod źródłowy w języku C, który polega na zbadaniu ciągu czy jest arytmetyczny czy nie. Wyskakuje mi, że nie jest arytmetyczny, a powinien być. Nie wiem co jest źle

#include<stdio.h

#include<stdio.h>
#include<stdlib.h>
int czy_arytmetyczny(int ciag[],int rozmiar);
int main()
{
  int ciag[]={2,4,6,8,10,12,14,16,18,20,22,24};
  int rozmiar=sizeof(ciag)/sizeof(ciag[0]);
  int arytmetyczny=czy_arytmetyczny(ciag,rozmiar);
  if(arytmetyczny==1)
  {
    printf("Ciąg jest arytmetyczny");
  }
  else
  {
    printf("Ciąg nie jest arytmetyczny");
  }
  return 0;
}
int czy_arytmetyczny(int ciag[], int rozmiar)
{
  int i;
  int r;
  for(i=0;i<rozmiar;i++)
    {
      if(r!=ciag[i+1]-ciag[i]);
      {
        return 0;
      }
    }
  return 1;
}

>
 

Answer 1

Oprócz tego co napisał @Great:

1. Dlaczego nie stosujesz typu bool? Od standardu C99 jest on obecny w języku.

2. Funkcja czy_arytmetyczny() nie broni się przed podaniem długości ciągu poniżej 2

3. W linii 4, mail powinien wyglądać tak: int main(void). Takiego jak wpisałeś, standard nie dopuszcza ale kompilator jest liberalny :) https://en.cppreference.com/w/c/language/main_function

4. O ile to możliwe, stosuj tylko 1 return z funkcji https://softwareengineering.stackexchange.com/questions/118703/where-did-the-notion-of-one-return-only-come-from https://spin.atomicobject.com/2011/07/26/in-defence-of-misra/

5. Jeśli masz złożony warunek do sprawdzenia, stosuj nawiasy ( i ) aby grupować warunki. C i C++ ma dość "egzotyczne priorytety operacji".

6. W linii 23, iteracja powinna zaczynać się od 2 a wcześniej wylicz pierwszy r.

Comments

6. Powinna zaczynać się od 1, bo inaczej różnica drugiej pary (i = 1, i + 1 = 2) będzie pomijana. Kod od autora pytania (z prywatnej wiadomości):

#include<stdio.h>
#include<stdlib.h>
int czy_arytmetyczny(int ciag[], int rozmiar);

int main()
{
    int ciag[] = { 2, 4, 9000, 9002 };
    int rozmiar = sizeof(ciag) / sizeof(ciag[0]);
    int arytmetyczny = czy_arytmetyczny(ciag, rozmiar);
    if (arytmetyczny == 1)
    {
        printf("Ciąg jest arytmetyczny");
    }
    else
    {
        printf("Ciąg nie jest arytmetyczny");
    }
    return 0;
}

int czy_arytmetyczny(int ciag[], int rozmiar)
{
    int i = 0;
    int r = ciag[i + 1] - ciag[i];
    for (int i = 2; i < rozmiar - 1; i++)
    {
        if (r != (ciag[i + 1] - ciag[i]))
        {
            return 0;
        }
    }
    return 1;
}

---

@Great, Nie sądzę.

#include <stdio.h>
#include <stdlib.h>
#include <stdbool.h>
#include <assert.h>

bool czy_arytmetyczny (int ciag[], int rozmiar);

int main (void)
{
    int ciag[] = { 2, 4, 6, 8, 10, 12, 14, 16, 18, 20, 22, 24 };
    const int rozmiar = sizeof (ciag) / sizeof (*ciag);

    bool arytmetyczny = czy_arytmetyczny(ciag, rozmiar);

    printf("Ciąg%s jest arytmetyczny\n", arytmetyczny ? "": " nie");

    return EXIT_SUCCESS;
}

bool czy_arytmetyczny (int ciag[], int rozmiar)
{
    assert(rozmiar >= 2);

    bool rezultat = true;

    int r = ciag[0] - ciag[1];
    for (int i = 2; i < rozmiar; ++i)
    {
        if (r != (ciag[i - 1] - ciag[i]))
        {
            rezultat = false;
            break;
        }
    }

    return rezultat;
}

 

Answer 2

1. Zmienna r nie jest zainicjalizowana.

2. Instrukcja warunkowa jest przerwana średnikiem.

3. Pętla wychodzi poza zakres tablicy (jeżeli naprawisz 1 i 2).