Książka telefocznina w cpp ;]

[bagsiu]

Witam pisze sobie w cpp książke telefoniczną na plikach, ale wywala mi pare błędów.
Co złego w tym kodzie ?

#include <iostream>
#include <conio.h>
#include <fstream>
int main() {
std::cout<<"                    *** KSIAZKA TELEFONICZNA ! *** \n ";
std::cout<<" Menu:  \n";
std::cout<<"1.Zapisz Numer ! \n";
std::cout<<"2.Odczytaj Numery ! \n";
std::cout<<"3.Tworca programu ! \n";
std::cout<<"4.Wylacz program ! \n";

int nWybor;
std::cin >> nWybor;

switch(nWybor) {
case 1:
     std::cout<<"           ***** Zapisywanie Numeru !!! ***** \n";
     std::cout<<"Wpisz numer: \n";
    int nNumer;
    std::cin >> nNumer;
   std::fstream plik;
  plik.open("numery.txt", std::ios::out);
  plik << nNumer;   
  plik.close();
 break;
case 2:
int nOdczyt;
std::cin >> nOdczyt;
std::fstream plik;
plik.open("numery.txt", std::ios::in);
plik >> nOdczyt;
break;
}
getch();
std::cout<<"By zakonczyc wcisnij dowolny klawisz...";

}

Błędy które wywala to :
27 C:\Documents and Settings\bagsiu\Pulpit\numery tel..cpp jump to case label
22 C:\Documents and Settings\bagsiu\Pulpit\numery tel..cpp crosses initialization of `std::fstream plik'

I inne, ale najbardziej chodzi mi o te dwa blędy, których nie potrafie rozwiązać :]
Z góry dzięki

 

[nowy_me]

x:\x\main.cpp(26) : error C2360: initialization of 'plik' is skipped by 'case' label

przenieś to nad switch:

std::fstream plik;

jeśli programujesz dla Windows to polecam darmowy kompilator Visual C++ Express Edition, gdyż jego komunikaty są bardziej zrozumiałe, a każdy błąd jest opisany po wpisaniu jego kodu do Googla
http://msdn.microsoft.com/en-us/library/61af7cx3(VS.71).aspx

 

[bagsiu]

Dzięki wielkie działa ;p
Zaraz przetestuje ten kompilator polecony przez ciebie : )
Dzięki jeszcze raz

 

[tobix10]

przenieś to nad switch:

Wystarczy całego case wziąć w nawiasy klamrowe {}, ale w tym przypadku lepiej przenieś nad, bo definiujesz to samo w obu case'ach.

 

[discovery44]

Jeszcze poradzę ja: użyj przesteni nazw std:

using namespace std;

Dzięki temu zaoszczędzisz sobie pisania std:: przed każdym składnikiem przestrzeni (cout, cin ect.) a kod programu będzie czytelniejszy.

// ex. (na wszelki wypadek):

#include <iostream>
using namespace std;
int main(){
    // std::cout << "A";
       cout << "A";
}