spoj -srednia arytmetyczna

Question

float minimum=srednia-tab[0];
 
		for(int k(1);k<n;k++)
		{
			if (abs(srednia-tab[k-1])<abs(srednia-tab[k])) minimum=tab[k-1];
		}
 
		//if (minimum=srednia-tab[0]) cout<<tab[0]<<endl;
		 cout<<minimum<<endl;

Proszę o pomoc w znalezieniu błędu, wydaje mi się, że problem leży w tej części kodu,  bo odpowiedzi się wyświetlają, tyle że złe 

link do zadania: http://pl.spoj.com/problems/PP0604A/

Answer 1

A  co ma na celu warunek if? że tak zapytam liczysz zauważ, że  iterując tak jak piszesz  nigdy do minimum nie przypiszesz wartosci k=4 ;] . [ zakladajac ze na takiej pozycji bedzie szukana srednia]  Sugeruje pokombinować troche z iterowaniem pętli

Comments

ten w komentarzu jest dlatego, bo gdyby go nie było a minimum , które ustanowiłam na początku okazało się faktycznym minimum wyświetliło liczbę najbliższą średniej, a nie różnicę miedzę średnią a nią

a reszta ifów jest to takie klasyczne (tak mi się zdaje) wyszukiwanie wartości minimalnej

Answer 2

1. Nie używasz funkcji abs wszędzie gdzie powinnaś np tu:

float minimum = srednia-tab[0];

2. Dlaczego porównujesz pomiędzy dwoma kolejnymi elementami tablicy, a  nie każdy element z minimum?

3. Zapisujesz w minimum liczbę, a nie różnicę srednia - tab[k]. Dodaj jakąś nową zmienną, która będzie zapisywała znalezioną liczbę.

4. Ostatecznie kod powinien wyglądać tak:

        float minimum = fabs( srednia_arytmetyczna-tablica[0] );
        int ktory_minimum = tablica[0];

        for(int k(1);k<n;k++)
        {
            if( fabs(srednia_arytmetyczna-tablica[k]) < minimum )
            {
                minimum = fabs( srednia_arytmetyczna-tablica[k] );
                ktory_minimum = tablica[k];
            }
        }

Comments

1. Racja, muszę poprawić

2. No wydawało mi się,że to jest takie klasyczne wyszukiwanie wartości minimalnej, czy maksymalnej, tak przynajmniej zawsze jej szukałam

3.Stwierdziłam, że gdy zapiszę w minimum od razu liczbę (odpowiadającą minimum tej różnicy) będzie szybciej
4.Czym różni się fabs od abs?

---

Byłaś blisko napisania dobrego programu tylko zgubiła Ci się koncepcja. 

2. Tak się nie do końca szuka minimalnej, bo porównuje dwa elementy lokalnie. Nie uwzględniałaś, że gdzieś wcześniej zaistniało minimum... Zobacz taki przykład:

#include <iostream>
using namespace std;

int main()
{
    int tab[5] = {1,3,2,4,2};
    int n = 5;

    float minimum = tab[0];
    for(int k(1);k<n;k++)
    {
        if( tab[k-1] < tab[k] ) minimum=tab[k-1];
    }
    
    cout<<minimum<<endl; // Wychodzi 2 zamiast 1, bo warunek nie miał w sobie odwołania do "globalnego" minimum, a tylko do bieżących elementów
    return false;
}

3. No, ale minimum było potrzebne i jakaś informacja o tej liczbie osobno też ;)

4. Ktoś mi kiedyś powiedział, że do liczb zmiennoprzecinkowych bezpieczniej stosować funkcję fabs, bo teoretycznie funkcja abs powinna zwracać liczbę całkowitą. Natomiast u większości kompilatorów dzisiaj (w tym na spoju) nie powinno być z tym problemu. Dlatego nie uznałem tego za błąd. Po prostu tak sobie dopisałem :D

---

Dziękuję bardzo za pomoc