Zwracanie vector

0

Mam funkcję, w której są tablice vectorowe. Wszystko było dobrze, do czasu gdy nie dopisałem kolejnej funkcji w której musiałem wczytać te tablice. Okazało się, że nie można tego zwrócić returnem.

	vector < vector <double > > macierz_alfa;
	vector <double> macierz_beta;
	bool dziel_przez_zero;



int funkcja1()
{
       
 macierz_alfa.resize(n+1);
   for(int i=0; i<macierz_alfa.size(); i++)
      macierz_alfa[i].resize(n+1);
 
 macierz_beta.resize(n+1);
 
 for (int i = 1; i < n+1; i++)
 {
         if (A[i][i]!=0)
         {
 
                macierz_beta[i] = (B[i]/A[i][i]);
                for (int j = 1; j < n+1; j++)
                {
                        if (i==j) macierz_alfa[i][j]=0;
                        else macierz_alfa[i][j]= - ( (A[i][j]) / (A[i][i]) );
                }
         }
         else
         {
                 dziel_przez_zero=true;
                  break;
         }
       
 
 }
 return macierz_alfa;
 return macierz_beta;
}

1
  1. jezeli wektory sa globalne to czemu chcesz je zwracac?
  2. jezeli chcesz zwracac vector to czemu typ zwracany funkcji to int?
  3. poczytaj sobie przekazywaniu argumentow funkcji przez kopie, referencje oraz wskaznik. Zastanow sie jak to moze Ci pomoc przy zwracaniu vectora
0

Dzięki, zwracanie faktycznie nie było potrzebne. Sam sobie skomplikowałem. Użycie zmiennych globalnych rozwiązuje problem

P.S. Skopiowałem fragment kodu. A zmienne wcześniej nie były globalne.

1

to jezeli nie byly globalne to bardzo dobrze. Powinienes je przekazywac do funkcji a pozniej wynik zwracać. Używanie globalnych zmiennych jest ZŁE.
a kod wkleiłeś tak, że to się wydawało globalne

0

Witam! Mam problem z dodawaniem zmiennych do vectora. Coś ewidentnie jest nie tak, bo niby zmienne dodaje poprawnie, ale pozniej nie moge tych zmiennych wyświetlić. Z tego co wiem to po vectorze poruszamy sie jak po tablicy. Vector jest globalny wiec funkcja nie musi nic zwracać. Do vectora mozna dodawać zmienne typu stukturalnego.
Poniżej udostępniam kod..

#include <iostream>
#include <string>
#include <vector>
#include <conio.h>
#include <windows.h>
#include <fstream>

using namespace std;

struct osoba
{
    string imie;
    string nazwisko;
    int rok_urodzenia;
};

struct student: osoba
{
    int indeks;
    string kierunek;
} zmienna;

struct nauczyciel: osoba
{
    string tytul_naukowy;
} zmienna2;

void add_student(vector<student> &);
void add_nauczyciel(vector<nauczyciel> &);
void out_student(vector<student> &);
void out_nauczyciel(vector<nauczyciel> &);
void wyswietl(vector<student> &);

int main()
{
    vector<nauczyciel> nauczyciele;
    vector<student> studenci;
    int x;
    do
    {
        system("cls");
        cout<<"1 - dodaj nowego studenta"<<endl;
        cout<<"2 - dodaj nowego nauczyciela"<<endl;
        cout<<"3 - eksportuj liste studentow do pliku"<<endl;
        cout<<"4 - eksportuj liste wykladowcow do pliku"<<endl;
        cout<<"5 - wyswietl wszystkich studentow"<<endl;
        cout<<"6 - wyjscie"<<endl;
        cin>>x;
        switch(x)
        {
        case 1:
            add_student(studenci);
            break;
        case 2:
            add_nauczyciel(nauczyciele);
            break;
        case 3:
            out_student(studenci);
            break;
        case 4:
            out_nauczyciel(nauczyciele);
            break;
        case 5:
            wyswietl(studenci);
            break;
        }
        getch();
    }while(x!=6);

    return 0;
}

void add_student(vector<student> &studenci)
{
    int i=studenci.size();
    studenci.push_back(zmienna);
    cout<<"Podaj kolejno: imie, nazwisko, rok urodzenia, nr indeksu, kierunek"<<endl;
    cin>>studenci[i].imie>>studenci[i].nazwisko>>studenci[i].rok_urodzenia>>studenci[i].indeks>>studenci[i].kierunek;
    cout<<endl<<"Dodano nowego studenta!"<<endl;
}

void add_nauczyciel(vector<nauczyciel> &nauczyciele)
{
    int i=nauczyciele.size();
    nauczyciele.push_back(zmienna2);
    cout<<"Podaj kolejno: imie, nazwisko, rok urodzenia, tytul naukowy"<<endl;
    cin>>nauczyciele[i].imie>>nauczyciele[i].nazwisko>>nauczyciele[i].rok_urodzenia>>nauczyciele[i].tytul_naukowy;
    cout<<endl<<"Dodano nowego nauczyciela!"<<endl;
}

void out_student(vector<student> &studenci)
{
    fstream plik;
    plik.open("studenci.txt", ios::out);
    if(plik.good()==false)
    {
        cout<<"Blad otwarcia pliku"<<endl;
    }
    else
    {
        for(int j=0; j<studenci.size(); j++)
        {
            plik<<studenci[j].imie<<"\t"<<studenci[j].nazwisko<<"\t"<<studenci[j].rok_urodzenia<<"\t"
                <<studenci[j].indeks<<"\t"<<studenci[j].kierunek<<endl;
        }
        plik.close();
        cout<<"Export listy studentow przebiegl pomyslnie"<<endl;
    }
}

void out_nauczyciel(vector<nauczyciel> &nauczyciele)
{
    fstream plik;
    plik.open("nauczyciele.txt", ios::out);
    if(plik.good()==false)
    {
        cout<<"Blad otwarcia pliku"<<endl;
    }
    else
    {
        for(int j=0; j<nauczyciele.size(); j++)
        {
            plik<<nauczyciele[j].imie<<"\t"<<nauczyciele[j].nazwisko<<"\t"
                <<nauczyciele[j].rok_urodzenia<<"\t"<<nauczyciele[j].tytul_naukowy<<endl;
        }
        plik.close();
        cout<<"Export listy nauczycieli przebiegl pomyslnie"<<endl;
    }
}

void wyswietl(vector<student> &studenci)
{
    for(int j=0; j<studenci.size(); j++)
    {
        cout<<studenci[j].imie<<"\t"<<studenci[j].nazwisko<<"\t"<<studenci[j].rok_urodzenia
            <<studenci[j].indeks<<"\t"<<studenci[j].kierunek<<endl;
    }
}

 
1

Przekazuj przez referencję.

0

jak wyżej, czyli np: void add_student(vector<student>&);

0

Dzięki ;D A czy ktoś widzi jak poprawić/ulepszyć taki programik? Czy zrobiłem coś źle/niepoprawnie? Czy można napisac to inaczej/lepiej?
czy np zamiast tego

 void add_student(vector<student> &studenci)
{
    int i=studenci.size();
    studenci.push_back(zmienna);
    cout<<"Podaj kolejno: imie, nazwisko, rok urodzenia, nr indeksu, kierunek"<<endl;
    cin>>studenci[i].imie>>studenci[i].nazwisko>>studenci[i].rok_urodzenia>>studenci[i].indeks>>studenci[i].kierunek;
    cout<<endl<<"Dodano nowego studenta!"<<endl;
}

nie lepiej zrobić cos takiego..

void add_student(vector<student> &studenci)
{
    cout<<"Podaj kolejno: imie, nazwisko, rok urodzenia, nr indeksu, kierunek"<<endl;
    cin>>zmienna.imie>>zmienna.nazwisko>>zmienna.rok_urodzenia>>zmienna.indeks>>zmienna.kierunek;
    studenci.push_back(zmienna);
    cout<<endl<<"Dodano nowego studenta!"<<endl;
} 
0

Lepiej zrobić coś takiego:

student read_student_from_cin()
  {
   student zmienna;
   cout<<"Podaj kolejno: imie, nazwisko, rok urodzenia, nr indeksu, kierunek"<<endl;
   cin>>zmienna.imie>>zmienna.nazwisko>>zmienna.rok_urodzenia>>zmienna.indeks>>zmienna.kierunek;
   return student;
  }

Natomiast wywoływać: studenci.push_back(read_student_from_cin());

1 użytkowników online, w tym zalogowanych: 0, gości: 1