Problem z dodawaniem

Question

Witam program działa poprawnie wyświetla liczby jednak pierwszy zapisane na stosie. Chciałem dodać do programu możliwość sumowania wszystkich podanych liczb z danej kategorii.

#include <iostream> // poprawic !!!!
#include <string>
#include <conio.h>

int main()
{
	std::string dziekuje_prosze, gleboki_plytki;
	int ilosc_slow, id , licznik_plytkie = 0, licznik_glebokie = 0;
	int glebokie[1000], plytkie[1000];

	std::cin >> ilosc_slow;

	for (int i = 0; i < ilosc_slow; i++)
	{
		std::cin >> dziekuje_prosze >> gleboki_plytki >> id;

		if (dziekuje_prosze == "dziekuje")
		{
			if (gleboki_plytki == "plytki")
			{
				plytkie[licznik_plytkie] = id ;
				licznik_plytkie++;
			}
			else if (gleboki_plytki == "gleboki")
			{
				glebokie[licznik_glebokie] = id;
				licznik_glebokie++;
			}
		}
		if (dziekuje_prosze == "prosze")
		{
			if (gleboki_plytki == "plytki") 
			{
				licznik_plytkie--;
				plytkie[licznik_plytkie] = id;
			}
			else if (gleboki_plytki == "gleboki")
			{
				licznik_glebokie--;
				glebokie[licznik_glebokie] = id;
			}
		}
	}

	for (int i = 0; i < licznik_plytkie; i++)
	{
		std::cout << plytkie[i] << std::endl;
		//musi je sumowac
	}

	for (int i = 0; i < licznik_glebokie; i++)
	{
		std::cout << glebokie[i] << std::endl; // alokacja na stos (zdejmuje pierwszy zapisany) 
		//musi je sumowac 
	}
	getch();
	return 0;
}

 

Comments

Jeśli chciałeś to po prostu dodaj. Mam wrażenie, że to zżyna od kogoś na informatykę, którą musisz uzupełnić.

---

Poprawiam program dla siebie i zacząłem modyfikować, mam problem bo nie mogę dodać wszystkich przypisanych wartości.

---

Jesteś pewien że program dobrze działa. Bo kod mi bardzo przypomina ten z https://forum.pasja-informatyki.pl/288531/problem-dotyczacy-zadania-ze-strony-szkopul-c-bar-apis#a288883 a w tym wątku wymagał delikatnych poprawek. Za bardzo nie rozumiem co miało by być liczone ilość przyjętych talerzy wydanych czy coś innego.

Answer 1

zacznij od przejrzystości kodu, bo jak na razie w takim chaosie nic nie widać.

1. Użyj using namespace std;

2.Popraw nazwy zmiennych ( polecam ang można zawrzeć bardziej jasną informacje w krótszych słowach niż w jezyku polskim)

Jak to zrb czytelność się poprawi, w tedy znacznie łatwiej będzie szukać błędów

Comments

Nie używaj using namespace std;

---

Przy tej wielkości projektu? Używaj :D

---

W programach składających się z jednego pliku można śmiało używać using namespace std;. W programach obiektowych składających się z wielu plików warto używać std::, aby nie włączać wielokrotnie do plików tej samej przestrzeni nazw.

Odpowiedź użytkownika Evelek w temacie: 

https://forum.pasja-informatyki.pl/266789/using-namespace-std-vs-std

---

Bo cały trik polega na dostosowaniu narzędzia do potrzeby, a nie potrzeby do narzędzia 

---

Nie ma znaczenia ile plików jest w projekcie. Dyrektywa using jest zła i tyle. Zaśmieca globalną przestrzeń nazw przez co powoduje konflikty nazw. Jedyny sens używania dyrektywy using, to w małych programach, w których testujesz jakąś klasę lub funkcję. Ale nawet wtedy możesz się natknąć na konflikt nazw.

---

to w małych programach no tak, bo ten to prawdziwy c++ moloch