Visual studio program składający się z trzech plików

pytanie zadane 29 września 2016 w C i C++ przez Kasztan Dyskutant (8,080 p.)
zamknięte 30 września 2016 przez Kasztan

Otóż mam taki program składający się z trzech plików

Nagłówek:

#ifndef Bamboo_h
#define Bamboo_h

namespace SALES
{
	const int QUARTERS = 4;
	struct Sales
	{
		double sales[QUARTERS];
		double average;
		double max;
		double min;

	};

	void setSales(Sales &s, const double ar[], int n);
	void setSales(Sales &s);
	void showSales(const Sales &s);
}

#endif // !Bamboo

Pierwszy plik

#include "stdafx.h"
#include "Header4.h"
#include <iostream>



int main()
{
	using namespace std;
	using namespace SALES;

	Sales s;
	const double tablica[4] = { 1.2,3.2,5.5,0.1 };
	setSales(s, tablica[], 4);
	showSales(s);
	setSales(s);
	showSales(s);
	

    return 0;
}

drugi plik

#include "stdafx.h"
#include "Header4.h"
#include <iostream>


void setSales(Sales &s, const double ar[], int n)
{	
	using namespace SALES;
	using namespace std;
	double srednia = 0;
	for (int i = 0; i < n; i++)
	{
		s.sales[i] = ar[i];
		srednia += ar[i];
	}
	s.average = (srednia/n);
	double najwieksza = ar[0];
	for (int i = 0; i < n; i++)
	{
		if (najwieksza < ar[i])
		{
			najwieksza = ar[i];
		}
	}
	s.max = najwieksza;
	double najmniejsza = ar[0];
	for (int i = 0; i < n; i++)
	{
		if (najmniejsza > ar[i])
		{
			najmniejsza = ar[i];
		}
	}
	s.min = najmniejsza;
	//zerowanie danych w tablicy;
	for (int i = 0; i < n; i++)
	{
		if (ar[i] != najwieksza || ar[i] != najmniejsza)
		{
			ar[i] = 0;
		}
	}
}

void setSales(Sales &s)
{	
	using namespace SALES;
	using namespace std;
	double srednia = 0;
	for (int i = 0; i < 4; i++)
	{
		cout << "Wpiesz wartosc numer: " << i << endl;
		cin >> s.sales[i];
		srednia += s.sales[i];

	}
	s.average = (srednia / 4);
	double najwieksza = s.sales[0];
	for (int i = 0; i < 4; i++)
	{
		if (najwieksza < s.sales[i])
		{
			najwieksza = s.sales[i];
		}
	}
	s.max = najwieksza;
	double najmniejsza = s.sales[0];
	for (int i = 0; i < 4; i++)
	{
		if (najmniejsza > s.sales[i])
		{
			najmniejsza = s.sales[i];
		}
	}
	s.min = najmniejsza;
	//zerowanie danych w tablicy;
	for (int i = 0; i < 4; i++)
	{
		if (s.sales[i] != najwieksza || s.sales[i] != najmniejsza)
		{
			s.sales[i] = 0;
		}
	}

}

void showSales(const Sales &s)
{	
	using namespace SALES;
	using namespace std;
	cout << s.average << endl;
	cout << s.max << endl;
	cout << s.min << end;

}

I teraz moje pytanie czy nagłówek jest dobrze napisany chodzi mi czy do tworzenia przestrzeni nazw używa się infdef i define

Następnie mam wrażenie, że ten drugi plik wgl nie jest odnajdowany przez pierwszy(definicje funkcji nie widnieją w funkcji main)

W pliku wygląda to tak

oraz tak od strony IDE

komentarz zamknięcia: Uzyskałem odpowiedź

2 odpowiedzi

odpowiedź 30 września 2016 przez Kasztan Dyskutant (8,080 p.)
wybrane 30 września 2016 przez Kasztan

Najlepsza

Odniosłem się do wszystkich powyższych rad, które były trafne jednak nadal mam problem żeby uruchomić ten program.

Severity    Code    Description    Project    File    Line    Suppression State
Error    LNK2019    unresolved external symbol "void __cdecl SALES::showSales(struct SALES::Sales const &)" (?showSales@SALES@@YAXABUSales@1@@Z) referenced in function 4

Severity    Code    Description    Project    File    Line    Suppression State
Error    LNK1120    3 unresolved externals    4

komentarz 30 września 2016 przez criss Mędrzec (172,570 p.)

Przed wszystkim co zostało zadeklarowane w namespace musisz dodawać operator zasięgu. Tutaj chodzi o typ SALES::Sales (w definicji masz samo Sales). Powtarzam - najprościej będzie jak wszystko w pliku cpp gdzie definiujesz te funkcje zamkniesz w klamrach namespace SALES { }

komentarz 30 września 2016 przez Kasztan Dyskutant (8,080 p.)

No ale dalej to nie pomaga

odpowiedź 29 września 2016 przez criss Mędrzec (172,570 p.)

1. Tak, dobrze.

(definicje funkcji nie widnieją w funkcji main)

Nie mam pojęcia o co ci chodzi. Jak to "masz wrażenie"? Tzn. co się dzieje?

komentarz 29 września 2016 przez Kasztan Dyskutant (8,080 p.)

w nazwy funkcji są podkreślone jako błąd i że not found a jak widać w tym 2 pliku zdefiniowałem je

komentarz 29 września 2016 przez Pascal Bywalec (2,720 p.)

nie jestem pewien czy dobrze rozumiem ale w tym drugim pliku przed każdą nazwą funkcji np:

void setSales(Sales &s, const double ar[], int n)

{

//...

}

dodaj

SALES::

chodzi mi o to aby wyglądało to mniej więcej tak:

void SALES::setSales(Sales &s, const double ar[], int n)

{

//...

}

komentarz 29 września 2016 przez criss Mędrzec (172,570 p.)

Ah, już wiem. W nagłówku funkcje masz w namespacie. Czyli deklarujesz SALES::setSales... itd. Definiowane przez ciebie funkcje nie są w namespace-ie. Także przez nazwą funkcji w cpp dopisz SALES:: albo wszystkie definicje zamknij w klamrach namespace-u podobnie jak deklaracje w .h

komentarz 29 września 2016 przez Kasztan Dyskutant (8,080 p.)

poprawiłem to

ale dalej jest problem

IDE sugeruje zdefiniowanie funkcji w 1 pliku

komentarz 29 września 2016 przez criss Mędrzec (172,570 p.)

?? Tzn co dokładnie mowi?

komentarz 29 września 2016 przez Kasztan Dyskutant (8,080 p.)

tak jak na screenie

komentarz 29 września 2016 przez criss Mędrzec (172,570 p.)

Tzn. nie zrobiles tego co mowilem.

PS: Forum jest od tego, żeby pisać a nie wrzucać screeny. Mam dosyc 200% zooma i domyslania sie jaka literka są 2 pixele.

komentarz 29 września 2016 przez Pascal Bywalec (2,720 p.)

Spróbuj zmienić nazwę tego pliku cpp na Header4.cpp

komentarz 29 września 2016 przez Kasztan Dyskutant (8,080 p.)

tak jak pisałem zrobiłem

komentarz 29 września 2016 przez Pascal Bywalec (2,720 p.)

pokaż kod po tych zmianach tylko błagam nie jako screen...

komentarz 29 września 2016 przez Kasztan Dyskutant (8,080 p.)

#ifndef Bamboo_h
#define Bamboo_h

namespace SALES
{
	const int QUARTERS = 4;
	struct Sales
	{
		double sales[QUARTERS];
		double average;
		double max;
		double min;

	};

	void setSales(Sales &s, const double ar[], int n);
	void setSales(Sales &s);
	void showSales(const Sales &s);
}

#endif // !Bamboo

#include "stdafx.h"
#include "Header4.h"
#include <iostream>



int main()
{
	using namespace std;
	using namespace SALES;

	Sales s;
	const double tablica[4] = { 1.2,3.2,5.5,0.1 };
	setSales(s, tablica[], 4);
	showSales(s);
	setSales(s);
	showSales(s);
	

    return 0;
}

#include "stdafx.h"
#include "Header4.h"
#include <iostream>


void Sales::setSales(Sales &s, const double ar[], int n)
{	
	
	using namespace SALES;
	using namespace std;
	double srednia = 0;
	for (int i = 0; i < n; i++)
	{
		s.sales[i] = ar[i];
		srednia += ar[i];
	}
	s.average = (srednia/n);
	double najwieksza = ar[0];
	for (int i = 0; i < n; i++)
	{
		if (najwieksza < ar[i])
		{
			najwieksza = ar[i];
		}
	}
	s.max = najwieksza;
	double najmniejsza = ar[0];
	for (int i = 0; i < n; i++)
	{
		if (najmniejsza > ar[i])
		{
			najmniejsza = ar[i];
		}
	}
	s.min = najmniejsza;
	//zerowanie danych w tablicy;
	for (int i = 0; i < n; i++)
	{
		if (ar[i] != najwieksza || ar[i] != najmniejsza)
		{
			ar[i] = 0;
		}
	}
}

void Sales::setSales(Sales &s)
{	
	using namespace SALES;
	using namespace std;
	double srednia = 0;
	for (int i = 0; i < 4; i++)
	{
		cout << "Wpiesz wartosc numer: " << i << endl;
		cin >> s.sales[i];
		srednia += s.sales[i];

	}
	s.average = (srednia / 4);
	double najwieksza = s.sales[0];
	for (int i = 0; i < 4; i++)
	{
		if (najwieksza < s.sales[i])
		{
			najwieksza = s.sales[i];
		}
	}
	s.max = najwieksza;
	double najmniejsza = s.sales[0];
	for (int i = 0; i < 4; i++)
	{
		if (najmniejsza > s.sales[i])
		{
			najmniejsza = s.sales[i];
		}
	}
	s.min = najmniejsza;
	//zerowanie danych w tablicy;
	for (int i = 0; i < 4; i++)
	{
		if (s.sales[i] != najwieksza || s.sales[i] != najmniejsza)
		{
			s.sales[i] = 0;
		}
	}

}

void Sales::showSales(const Sales &s)
{	
	using namespace SALES;
	using namespace std;
	cout << s.average << endl;
	cout << s.max << endl;
	cout << s.min << end;

}

komentarz 29 września 2016 przez Pascal Bywalec (2,720 p.)
edycja 29 września 2016 przez Pascal

c++ rozpoznaje wielkość znaków masz tu podane Sales:: a powinno być SALES:: gdyż nazwałeś przestrzeń nazw SALES a nie Sales, lub też zmień nazwę tej przestrzeni nazw SALES na Sales i będzie git :P No i wyszło mi masło maślane :D