"Naruszenie ochrony pamięci". Jak rozwiązać

0

Używam linuxa a mianowicie sabayon (pochodny od gentoo). Kompilator g++ 4.5.3. Chciałem zrobić programik zawierający wskaźniki, bo uczę się dopiero programowania. Nie wiem zabardzo na czym polega problem. Z tego co wiem na komputerze kolegi, który ma Windows program odpalił się. Wiem, że program nie jest dopracowany (wartość procent moze być dowolna itp.) jednak te błędy poprawię dopiero jak odpali mi się to, co mam teraz. Z góry dzięki za pomoc

#include <iostream>
#include <cstdio>

using namespace std;

int main()
{
   float maxpkt, *proc[6];
   short int oceny;


   cout << "Witaj w programie obliczajacym pkt dla spr." << endl << endl;
   cout << "Wprowadz teraz ilosc max punktow: ";
   cin >> maxpkt;
   getchar();
   cout << endl;

   do
   {
   cout << "Wpisz zakres ocen (5 lub 6): ";
   cin >> oceny;
   cout << endl;
   }while ((oceny !=5) && (oceny !=6));
   
   for (int i=0 ; i++ <= oceny; i++)
   {
     proc[i] = new float;
     cout << "ilosc max procent dla oceny "<< i++ << ":" ;
     cin >> *proc[i];
     getchar ();
     cout << endl;
   
   };
   
   cout << "Zostana teraz obliczone pkt dla poszczegolnych ocen" <<endl;
   cout << "Wcisnij klawisz ENTER";
   getchar ();
   cout << endl << endl;

   for (int j=0 ; j <= oceny ; j++)
   {
      cout << j+1 << " - " << *proc[j]*maxpkt/float(100) <<endl;
   };

   getchar ();
   return 0;
}
1

Tu i++ <= oceny prawdopodobnie wyłazisz poza zakres tablicy.

0

Raczej nie wychodzę poza tablicę... jest 6 elementów czyli 0 1 2 3 4 5 i=0 ilość ocen np 5. wtedy i++ czyli 1 2 3 4 5 takie są oceny, bo i++ mniejsze równe 5 i bez inkrementacji będzie 0 1 2 3 4 czyli 5 kolejnych funkcji przy liczbie ocen 6 wszystko będzie o jeden więcej, więc powinno być dobrze. Jeżeli źle rozumuję proszę mnie poprawić. Problem pojawia się przy wpisaniu pierwszej liczby procent dla oceny 1 czyli proc[0]. Po tym wyskakuje mi ten błąd.

1

Źle rozumujesz. Tu for (int i=0 ; i++ <= oceny; i++) i tu for (int j=0 ; j <= oceny ; j++) wylatujesz poza tablicę.
Przy pierwszej pętli dla zmiennej oceny równej 6 i będzie przyjmować wartości: 1 3 5 7.
Przy drugiej pętli dla oceny = 6 j będzie przyjmować wartości: 0 1 2 3 4 5 6

No i oczywiście poza wychodzeniem poza tablicę, odwołujesz się do niezaalokowanej pamięci(ze względu na to jak działa pierwsza pętla). No i nigdzie nie zwalniasz przydzielonej pamięci.

0

Cześć Tu znowu ja z już poprawionym kodem i błąd ten już nie występuje. Pojawił się natomiast inny. Przepraszam, że tak was męczę, ale nie ma kto mnie tego nauczyć dosłownie, bo wszyscy informatycy nie mają na to czasu.

#include <iostream>
#include <cstdio>

using namespace std;

int main()
{
   float maxpkt, *proc[6];
   short int oceny;


   cout << "Witaj w programie obliczajacym pkt dla spr." << endl << endl;
   cout << "Wprowadz teraz ilosc max punktow: ";
   cin >> maxpkt;
   getchar();
   cout << endl;

   do
   {
   cout << "Wpisz zakres ocen (5 lub 6): ";
   cin >> oceny;
   cout << endl;


   if ((oceny != 5) && (oceny!=6))

    {
     cout << "wprowadziles zla ilosc ocen. Wprowadz zares ocen ponownie" << endl;
    };

   }while ((oceny !=5) && (oceny !=6));
   
  do
  {
   for (int i=0 ; i < oceny; i++)
   {
     proc[i] = new float;
     cout << "ilosc max procent dla oceny "<< i+1 << ":" ;
     cin >> *proc[i];
     getchar ();
     cout << endl;
   
   if ((*proc[0] !< *proc[1]) || (*proc[1] !< *proc[2]) || (*proc[2] !< *proc[3]) || (*proc[3] !< *proc[4]) || (*proc [4] !< *proc[5]))
    {
     cout << "Podales niewlasciwy zakres liczb. Sprobuj ponownie.";
    };
   };
   } while ((*proc[0]) !< (*proc[1]) || (*proc[1]) !< (*proc[2]) || (*proc[2]) !< (*proc[3]) || (*proc[3]) !< (*proc[4]) || (*proc [4]) !< (*proc[5]));
   
   cout << "Zostana teraz obliczone pkt dla poszczegolnych ocen" <<endl;
   cout << "Wcisnij klawisz ENTER";
   getchar ();
   cout << endl << endl;

   for (int j=0 ; j+1 <= oceny ; j++)
   {
      cout << j+1 << " - " << *proc[j]*maxpkt/float(100) <<endl;
   };

   getchar ();
   return 0;
}

Błąd pojawia się przy if i while. Pewnie jest to jakiś błąd zapisu, ale niestety nie wiem jaki.
Z góry dziękuję za pomoc

0

((oceny != 5) && (oceny!=6))
Musisz mieć spełnione 2 warunki. Czyli zły warunek.
((oceny != 5) || (oceny!=6))
po co Ci średnik po klamrze za ifem ?
if ((oceny != 5) && (oceny!=6))

{
 cout << "wprowadziles zla ilosc ocen. Wprowadz zares ocen ponownie" << endl;
}; <- wywal
1

Dodam jeszcze, że nie zwalniasz pamięci którą alokujesz na stercie

0

Fakt zapomniałem o zwolnieniu pamięci.

((oceny != 5) && (oceny!=6))
Musisz mieć spełnione 2 warunki. Czyli zły warunek.
((oceny != 5) || (oceny!=6))

Jeżeli miałbym lub, to za każdym razem warunek byłby spełniony, przez co w kółko bym dostawał komunikat o błędnym wpisaniu ilości ocen i uruchomieniu pętli od nowa, czyli znowu bym musiał wpisać jakąś ocenę i znowu bym dostał komunikat o błędnym wpisaniu i tak w nieskończoność. Zawsze któryś z warunków byłby spełniony. Tak przynajmniej mi się wydaje.

kod razem ze zwolnieniem pamięci:

#include <iostream>
#include <cstdio>

using namespace std;

int main()
{
   float maxpkt, *proc[6];
   short int oceny;


   cout << "Witaj w programie obliczajacym pkt dla spr." << endl << endl;
   cout << "Wprowadz teraz ilosc max punktow: ";
   cin >> maxpkt;
   getchar();
   cout << endl;

   do
   {
   cout << "Wpisz zakres ocen (5 lub 6): ";
   cin >> oceny;
   cout << endl;


   if ((oceny != 5) && (oceny!=6))
    {
     cout << "wprowadziles zla ilosc ocen. Wprowadz zares ocen ponownie" << endl;
    }
    
   }while ((oceny !=5) && (oceny !=6));
   
   
   
  do
  {
   for (int i=0 ; i < oceny; i++)
   {
     proc[i] = new float;
     cout << "ilosc max procent dla oceny "<< i+1 << ":" ;
     cin >> *proc[i];
     getchar ();
     cout << endl;
	}
   
   if ((*proc[0] !< *proc[1]) || (*proc[1] !< *proc[2]) || (*proc[2] !< *proc[3]) || (*proc[3] !< *proc[4]) || (*proc [4] !< *proc[5]))  //tutaj błąd, który podałem wcześniej.
    {
     cout << "Podales niewlasciwy zakres liczb. Sprobuj ponownie." << endl << endl;
    }
   } while ((*proc[0]) !< (*proc[1]) || (*proc[1]) !< (*proc[2]) || (*proc[2]) !< (*proc[3]) || (*proc[3]) !< (*proc[4]) || (*proc [4]) !< (*proc[5]));  // tutaj też.
   
   cout << "Zostana teraz obliczone pkt dla poszczegolnych ocen" <<endl;
   cout << "Wcisnij klawisz ENTER";
   getchar ();
   cout << endl << endl;

   for (int j=0 ; j+1 <= oceny ; j++)
   {
      cout << j+1 << " - " << *proc[j]*maxpkt/float(100) <<endl;
      delete proc[j]; //dodane zwolnienie pamięci
   }


   getchar ();
   return 0;
}

Błędy pojawiają się przy while i if, do których dodałem komentarz. Nie pokazałem tego poprzednim razem.

0

Co oznacza zapis '!<' ?

0

Chodziło mi o to, żeby nie był mniejszy. Tak jak != oznacza nie jest równy to w ten sam sposób chciałem zrobić nie mniejszy, ale nie wiem czy to jest dobrze.

1

Nie mniejszy, to to samo co większy bądź równy ( >= )

1 użytkowników online, w tym zalogowanych: 0, gości: 1