Zakończenie programu

Witam wszystkich.
Zabrałem się za pisanie prostego programu używając swojej dotychczasowej wiedzy i mam problem.
Dałem warunek, że jeżeli wcisnę klawisz z to program się wyłączy i tutaj jest problem. Jeżeli wcisnę ten klawisz zaraz po uruchomieniu programu to wszystko jest ok i się zamknie, a jeśli wpisuje kody wpisuje i wpisuje i nagle chce wyłączyć program i wciskam klawisz z to program się zacina i nic nie wpisze oraz się nie zamknie.

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

int main()
{
    int kod;
    string zakoncz;
    string z;
    cout<<"Podaj numery kodow kreskowych\nAby zakonczyc program przycisnij przycisk 'z'\n\n";
    ofstream zapis("kody.txt");
    cin>>zakoncz;
    if(zakoncz=="z")
    {
           cin.ignore();
    }
    else
    {
        while(zakoncz!="z")
        {
          cin>>kod;
        }
    }
    zapis << kod;
    zapis.close();
    return 0;
}

W petli while zapisujesz caly czas do zmiennej 'kod', a warunek sprawdzasz dla zmiennej 'zakoncz', wiec po wejsciu do else'a petla zawsze bedzie spelniona, a program zawiesza sie dlatego, ze probujesz wpisac znak do zmiennej liczbowej.

Więc co zrobić?
A przy okazji jak zrobić, żeby zapisywały się wszystkie wpisane liczby do pliku tekstowego a nie tylko pierwsza?

Jesli chcesz to robic tym sposobem, to najprosciej chyba bedzie zapisywac kody jako lancuchy znakowe.
Co do zapisywania wszystkich podanych kodow, to mozesz uzyc tablicy, ale wtedy najlepiej wiedziec ile sie tych kodow bedzie wpisywac, albo mozesz skorzystac z vectora jesli chcesz dynamicznie zwiekszac wielkosc tablicy.
O vectorze mozesz poczytac tu - http://pl.wikibooks.org/wiki/C%2B%2B/Vector i tu - http://www.cplusplus.com/reference/stl/vector/

Ok zrobiłem to zamykanie, ale działa jak podam małe liczby i góra dwie:

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

int main()
{
    int kod;
    string zakoncz;
    string z;
    cout<<"Podaj numery kodow kreskowych\nAby zakonczyc program przycisnij przycisk 'z'\n\n";
    ofstream zapis("kody.txt");
    cin>>zakoncz;
    if(zakoncz=="z")
    {
           cin.ignore();
    }
    else
    {
        while(zakoncz!="z")
        {
          cin>>kod;
          cin>>zakoncz;
        }
    }
    zapis<<kod;
    zapis.close();
    return 0;
}

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

int main()
{
    int i = 0; // licznik petli
    string zakoncz[256]; // tablica przechowujaca 256 lancuchow znakowych

    cout<<"Podaj numery kodow kreskowych\nAby zakonczyc program przycisnij przycisk 'z'\n\n";
    ofstream zapis("kody.txt");
    cin>>zakoncz[i];
    if(zakoncz[i]=="z")
    {
           cin.ignore();
    }
    else
    {
        while(zakoncz[i]!="z") 
        {
          i++; // zwiekszanie licznika o 1
          cin>>zakoncz[i]; // pobieranie lancucha znakowego do kolejnego elementu tablicy
        }
    }
    
    i = 0; // zerowanie licznika
    while(zakoncz[i]!="z") // zapis danych z tablicy do pliku wewnatrz petli
    {
      zapis << zakoncz[i] << endl;
      i++;
    }
    zapis.close();
    return 0;
}

Napisalem Ci dzialajacy kod z komentarzami.
Jesli masz jakies pytania co do niego to napisz.

Co w tym programie robi zapis i++; oraz czemu i =0

i = 0, poniewaz tablica zaczyna sie od elementu zerowego i konczy sie na elemencie rownym w tym przypadku 256 (czyli tak naprawde tablica jest 257-elementowa)

i++ jest rownoznaczne zapisowi i = i + 1; czyli zwiekszenie zmiennej i o 1.

Do kazdego z elementow tablicy odwolujemy sie przez indeks, czyli zaczynajac od 0 i zwiekszajac licznik o 1 przechodzimy po kolei przez kazdy element tablicy.

Wiem co robi i++ ale po co używamy tego w tym programie?

Zmieniłem trochę program i mam problem:

1. Aby zakończyć program muszę wpisać z kliknąć enter i ponownie z i ponownie kliknąć enter
2. Wpisując kod 5 i cenę 4 do pliku zapisuje się takie coś:
   50x22f6c0 40x22f6c0

PROGRAM:

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

int main()
{
    int i = 0;
    string kod[256];
    string cena[256];
    cout<<"Podaj numery kodow kreskowych\nAby zakonczyc program przycisnij przycisk 'z'\n\n";
    ofstream zapis("kody.txt");
    cout<<"Kod: ";
    cin>>kod[i];
    cout<<"Cena: ";
    cin>>cena[i];
    if(kod[i]=="z")
    {
           cin.ignore();
    }
    else
    {
        while(kod[i]!="z")
        {
          i++;
          cout<<"Kod: ";
          cin>>kod[i];
          cout<<"Cena: ";
          cin>>cena[i];
        }
    }
    i = 0;
    while(kod[i]!="z")    
    {
      zapis<<kod[i]<<
      zapis<<"\t";
      zapis<<cena[i]<<
      zapis<<"\n\n";
      i++;
    }
    zapis.close();
    return 0;
}

W petli while, 2 razy wczytujesz dane z klawiatury, raz do zmiennej kod, a drugi raz do zmiennej cena, wiec za kazdym razem musisz wpisac cos do obu zmiennych, dopeiro po wpisaniu sprawdzany jest warunek 'kod[i]!="z"', ktory mowi, ze jezeli wpisany ciag znakow do zmiennej kod znajdujacej sie pod indeksem tablicy 'i' jest rozny od ciagu znakowego "z" to wykonujemy dalej petle, wiec aby przerwac petle nalezy wpisac 'z' do zmiennej kod i cokolwiek do zmiennej cena.

Zapisuja Ci sie takei dziwne znaki, poniewaz napisales
zapis<<kod[i]<<
zapis<<cena[i]<<
zamiast
zapis<<kod[i];
zapis<<cena[i];

Vladq napisał(a)

Zapisuja Ci sie takei dziwne znaki, poniewaz napisales
zapis<<kod[i]<<
zapis<<cena[i]<<
zamiast
zapis<<kod[i];
zapis<<cena[i];

Za to dziękuje :)

A co do tego:

Vladq napisał(a)

W petli while, 2 razy wczytujesz dane z klawiatury, raz do zmiennej kod, a drugi raz do zmiennej cena, wiec za kazdym razem musisz wpisac cos do obu zmiennych, dopeiro po wpisaniu sprawdzany jest warunek 'kod[i]!="z"', ktory mowi, ze jezeli wpisany ciag znakow do zmiennej kod znajdujacej sie pod indeksem tablicy 'i' jest rozny od ciagu znakowego "z" to wykonujemy dalej petle, wiec aby przerwac petle nalezy wpisac 'z' do zmiennej kod i cokolwiek do zmiennej cena.

To nie wiem co mam poprawić

To zalezy jaki efekt chcesz uzyskac.
Jesli chcesz, zeby po wpisaniu do zmiennej kod lub cena petla od razu sie konczyla to mozna zrobic np. co takiego:

while(true)
        {
          i++;
          cout<<"Kod: ";
          cin>>kod[i];
          if(kod[i] == "z") break;
          cout<<"Cena: ";
          cin>>cena[i];
          if(cena[i] == "z") break;
        }

Oczywiscie warto tez wtedy lekko zmodyfikowac petle, ktora zapisuje do pliku, zeby sprawdzala rowniez czy przypadkiem w zmiennej cena nie pojawia sie "z"

while((kod[i]!="z") && (cena[i] != "z"))
{
      zapis<<kod[i];
      zapis<<"\t";
      zapis<<cena[i];
      zapis<<"\n\n";
      i++;
}

Dziękuje :)
Teraz się zastanawiam co do tego czy zakładać nowy wątek na odczytanie danych czy tutaj

A tak w ogóle to jak chcę np zaraz po włączeniu zamknąć program to muszę wpisać z kliknąć enter i znów z i znów enter... jak wpisuje liczby to jest ok ale jak chcę od razu to dzieje się tak jak napisałem