Czy liczba ma więcej 0 czy 1?

pytanie zadane 7 listopada 2015 w C i C++ przez Aisekai Nałogowiec (42,190 p.)
edycja 7 listopada 2015 przez Aisekai

Witam.

Napisałem programik który miał za zadanie sprawdzić ile liczb w pliku ma więcej 0 niż 1. Tylko za każdym razem pokazuje 1 albo 0, a tych liczb w pliku jest 432 bodajże. Pomógłby ktoś zlokalizować błąd?


#include <cstdlib>
#include <iostream>
#include <fstream>
#include <string>
using namespace std;
int main()
{
fstream plik;
string linia;
int i;
int b,a,y;
b=a=y=0;
plik.open("liczby.txt", ios::in);
if(plik.good()==false)
{
                      cout<<"Nie udało się otworzyć pliku"<<endl;
                      exit(0);
}
                      while(getline(plik,linia))
   {
                                  
                                               for(i=0; i<linia.length(); i++)
                                            {
                                              if(linia[i] == '0')
                                              { 
                                                          a++;
                                              }
				                              else
                                              {
                                                   b++;
                                              }
                                            }
			                                 if(a>b) y++;
                                         
   }
                      cout<<y<<endl;
                      
                      system("PAUSE");
                      return 0;
}

ps: tak przy okazji, poleciłby ktoś jakiś dobry kompilator do c++? i dałby linka? Bo w devie strasznie boli to, że się w którymś momencie zaczynam gubić który nawias co zamyka

4 odpowiedzi

odpowiedź 7 listopada 2015 przez niezalogowany

Mógłbyś zedytować wpis, tak aby wcięcia były zrobione normalnie?

komentarz 7 listopada 2015 przez Aisekai Nałogowiec (42,190 p.)

O to chodziło?

komentarz 7 listopada 2015 przez niezalogowany

Nie do końca, chodzi mi o to aby poszczególne sekcje programu były w sensownych odległościach, bo teraz ten kod analizuje się dość średnio, chodzi mi o takie coś:

#include <...>

int main()
{
  int a,b;
  if(...)
  {
    dosomething();
  }
}

odpowiedź 7 listopada 2015 przez furas Maniak (53,800 p.)
edycja 7 listopada 2015 przez furas

Po wczytaniu każdej linii powinieneś wyzerować `a` i `b` bo inaczej zliczasz ile jest zer i jedynek we wszystkich liczbach w sumie.

---

Odnośnie kompilatora to każdy kompilator C/C++ będzie się czepiać gdy bedziesz gubił nawiasy. Jeśli byś porządnie sformatował ten kod (tak jak to sugerował Szymon Siarkiewicz) to było by widać który nawias co zamyka.

---

Sformatowałem po swojemu ale nie sprawdzałem w działaniu po poprawkach kodu.

#include <cstdlib>
#include <iostream>
#include <fstream>
#include <string>

using namespace std;

int main()
{
  fstream plik;
  string linia;
  int i;
  int b, a, y;

  plik.open("liczby.txt", ios::in);
  
  if(plik.good() == false)
  {
    cout << "Nie udało się otworzyć pliku" << endl;
    exit(0);
  }

  y = 0;

  while(getline(plik, linia))
  {
    b = a = 0;
    
	for(i = 0; i < linia.length(); i++)
	{
	  if(linia[i] == '0')
      { 
	    a++;
	  }
	  else
	  {
	    b++;
	  }
	}

	if(a > b)
	  y++;
  }

  cout << y << endl;
                       
  system("PAUSE");
}