Odwracanie kolejności elementów tablicy

Question

Witam mam problem z odwracaniem kolejnosci wyświetlania liczb. Poniżej wkleje mój kod, gdy wpisuje liczby np 1, 2, 3 chce żeby funkcja wyświetliła mi je od tylu czyli 3, 2, 1, ale jest taki problem ze jako pierwsze wypisuje mi jakaś liczbe której nie podałem a jak zakończę wprowadzanie wcześniej tzn jak napisze np q to jako pierwsze wyświetla mi 0 i w obu przypadkach przez to nie wyświetla mi ostatniej liczby która wprowadziłem.

#include <iostream>
int fill_array(double arr[], int size);
void show_array(const double arr[], int size);
void reverse_array(double arr[], int size);
int main()
{
    using namespace std;
    double tablica [6];
    int size;
    size = fill_array(tablica, 6);
    show_array(tablica, size);
    cout << "Odwrocone wartosci.\n";
    reverse_array(tablica, size);
    
    return 0;
}
int fill_array(double arr[], int size)
{
    using namespace std;
    double temp;
    int i;
    for (i = 0; i < size; i++)
    {
        cout << "Podaj wartosc nr " << (i + 1) << ": ";
        cin >> temp;
        if (!cin)
        {
            cin.clear();
            while (cin.get() != '\n')
                continue;
            cout << "Bledne dane, wprowadzanei danych przerwane.\n";
            break;
        }
        else if (temp < 0)
            break;
        arr[i] = temp;
    }
    return i;
}
void show_array(const double arr[], int size)
{

    for (int i = 0; i < size; i++)
    {
        std::cout << "wartosc " << (i + 1) << ": ";
        std::cout << arr[i] << std::endl;
    }
}
void reverse_array(double arr[], int size)
{
    for (int i = size; i > 0; i--)
    {
        std::cout << "wartosc " << i << ": ";
        std::cout << arr[i] << std::endl;
    }
}

 

Answer 1

1. Bardziej czytelne jest (i w całym programie jest do "użycia")

int main()
{
    using namespace std;

    ...
}

na

#include <iostream>
using namespace std;

int fill_array(double arr[], int size);
void show_array(const double arr[], int size);
void reverse_array(double arr[], int size);

int main()
{ 
    ...
}

2. Skoro i tak "ręcznie" podajesz wielkość tablicy to po co tworzysz zmienną i pobierasz wielkość tablicy

int main()
{
    using namespace std;
    double tablica [6];
    int size;
    size = fill_array(tablica, 6);
    show_array(tablica, size);

    ...
}

na 

int main()
{
    int size=6;
    double tablica [size];    
    fill_array(tablica, size);
    show_array(tablica, size);

    ....
}

3. "Zapełniasz" tablice od indeksu 0 (zero)

int fill_array(double arr[], int size)
{

    double temp;
    int i;
    for (i = 0; 

   ...
}

no to rewers musi też "osiągnąć" 0 (zero)  

void reverse_array(double arr[], int size)
{
    for (int i=size - 1; i >= 0; i--)
    {

 

Comments

bo ta funkcja zwraca liczbe wprowadzonych danych - size. I jak to ustawie na wielkość tablicy to wyświetli mi tylko te dane które sa wprowadzone a jak ustawie wielkość która jest na początku to przy tych wartościach gdzie nic nie podałem wyświetli mi nie zero.

---

@VBService, tez to sprawdzałem ale dalej wyświetla mi nie jakaś liczba na początku 

 

---

@VBService, oo mam, ustawiłem i na początku na wartość size - 1

---

 

#include <iostream>
using namespace std;


void fill_array(double arr[], int size);
void show_array(double arr[], int size);
void reverse_array(double arr[], int size);

int main()
{
    int size=6;
    double tablica[size];    
    fill_array(tablica, size);
    show_array(tablica, size);
    cout << "Odwrocone wartosci.\n";
    reverse_array(tablica, size);
     
    return 0;
}

void fill_array(double arr[], int size)
{
    double temp;
    for (int i=0; i<size; i++)
    {
        cout << "Podaj wartosc nr " << (i + 1) << ": ";
        cin >> temp;
        if (!cin)
        {
            cin.clear();
            while (cin.get() != '\n')
                continue;
            cout << "Bledne dane, wprowadzanei danych przerwane.\n";
            break;
        }
        else if (temp < 0)
            break;
        arr[i] = temp;
    }
}
void show_array(double arr[], int size)
{
    for (int i=0; i<size; i++)
    {
        cout << "wartosc " << (i + 1) << ": ";
        cout << arr[i] << endl;
    }
}
void reverse_array(double arr[], int size)
{
    for (int i=size-1; i>=0; i--)
    {
        cout << "wartosc " << (i + 1) << ": ";
        cout << arr[i] << endl;
    }
}

 

---

@VBService, jeszcze mam pytanie, czy dałoby nie zrobić tak żeby funkcja show_array wyświetlała mi te elementy w odwrotnej kolejnosci? Bo wyświetla mi je tak samo.

---

chcesz mieć jedna funkcję show, która wyświetli dwa warianty? normalnie i rewers?

---

dokładnie

---

Tak "na już"  mało elegancki sposób, ale działa  

#include <iostream>
using namespace std;


void fill_array(double arr[], int size);
void show_array(double arr[], int size, int direction);

int main()
{
    int size=6;    
    double tablica[size]; 
   
    fill_array(tablica, size);
    show_array(tablica, size, 0); // 0 - DIRECTION_NORMAL
    cout << "Odwrocone wartosci.\n";
    show_array(tablica, size, 1); // 1 - DIRECTION_REVERSE
     
    return 0;
}

void fill_array(double arr[], int size)
{
    double temp;
    for (int i=0; i<size; i++)
    {
        cout << "Podaj wartosc nr " << (i + 1) << ": ";
        cin >> temp;
        if (!cin)
        {
            cin.clear();
            while (cin.get() != '\n')
                continue;
            cout << "Bledne dane, wprowadzanei danych przerwane.\n";
            break;
        }
        else if (temp < 0)
            temp = temp * -1; // Zamiana liczby ujemnej na dodatnia
        arr[i] = temp;
    }
}
void show_array(double arr[], int size, int direction)
{
    if (! direction) {
        for (int i=0; i<size; i++)
        {
            cout << "wartosc " << (i + 1) << ": ";
            cout << arr[i] << endl;
        }
    } else {
        for (int i=size-1; i>=0; i--)
        {
            cout << "wartosc " << (i + 1) << ": ";
            cout << arr[i] << endl;
        }        
    }
}

lub z użyciem reverse() method  

#include <iostream>
#include <algorithm>
using namespace std;


void fill_array(double arr[], int size);
void show_array(double arr[], int size, int direction);

int main()
{
    int size=6;
    
    double tablica[size];    
    fill_array(tablica, size);
    show_array(tablica, size, 0); // 0 - DIRECTION_NORMAL
    cout << "Odwrocone wartosci.\n";
    show_array(tablica, size, 1); // 1 - DIRECTION_REVERSE
     
    return 0;
}

void fill_array(double arr[], int size)
{
    double temp;
    for (int i=0; i<size; i++)
    {
        cout << "Podaj wartosc nr " << (i + 1) << ": ";
        cin >> temp;
        if (!cin)
        {
            cin.clear();
            while (cin.get() != '\n')
                continue;
            cout << "Bledne dane, wprowadzanei danych przerwane.\n";
            break;
        }
        else if (temp < 0)
            temp *= (-1); // Zamiana liczby ujemnej na dodatnia
        arr[i] = temp;
    }
}
void show_array(double arr[], int size, int direction)
{
    if (direction) reverse(arr, arr + size);
    for (int i=0; i<size; i++)
    {
        cout << "wartosc " << (i + 1) << ": ";
        cout << arr[i] << endl;
    }
}

 

---

Dzięki za pomoc ;)

---

Nie musisz przerywać programu w linii 34-35  

        else if (temp < 0)
            temp = temp * -1; // Zamiana liczby ujemnej na dodatnia

lub

        else if (temp < 0)
            temp *= (-1); // Zamiana liczby ujemnej na dodatnia

how to make negative numbers into positive