Program z błędem użyciem wskaźnika C++

pytanie zadane 7 kwietnia 2018 w C i C++ przez Tomeko Nowicjusz (150 p.)
edycja 7 kwietnia 2018 przez Tomeko

Witam!

Proszę o pomoc, w programie poniżej wyświetla się błąd o treści "Element Project26.exe wyzwolił punkt przerwania."

#include <iostream>
using namespace std;
int ile;
int main()
{
    cout << "Ile liczb w tablicy: ";
    cin >> ile;
    int *tablica;
    tablica = new int[ile];
    for (int i = 0; i < ile; i++)
    {
        cout << (int)tablica << endl;
        if (i != ile - 1) tablica++;
    }
    delete[] tablica;   //   BLAD W TYM MIEJSCU
    tablica = NULL;
    return 0;
}

Dodam jeszcze, że programuję w Visual Studio 2017

komentarz 7 kwietnia 2018 przez Mariusz08 Maniak (62,300 p.)

Kod bez bloczka.

komentarz 7 kwietnia 2018 przez Tomeko Nowicjusz (150 p.)

Jakiego bloczka?

2	komentarz 7 kwietnia 2018 przez niezalogowany Jak wstawić kod źródłowy

2 odpowiedzi

odpowiedź 7 kwietnia 2018 przez Patrycjerz Mędrzec (192,320 p.)
wybrane 7 kwietnia 2018 przez Tomeko

Najlepsza

if (i != ile - 1) tablica++;

Modyfikujesz tutaj wskaźnik, który przechowuje adres zaalokowanej pamięci. Później próbujesz usunąć pamieć wskazywaną przez taki zmodyfikowany wskaźnik, a przecież jest to błędem. Operator delete musi otrzymywać adres wygenerowany przez operator new, żaden inny. Jeśli chcesz modyfikować adres, stwórz kopię wskaźnika i pracuj na niej.

cout << (int)tablica << endl;

Po co rzutujesz adres na int?

Pokaż 11 poprzednich komentarzy

komentarz 7 kwietnia 2018 przez Mateusz Tocha Bywalec (2,560 p.)
edycja 7 kwietnia 2018 przez Mateusz Tocha

ale pętla się już nie wykona, prawda? u mnie niestety jak wchodze w debugger to po przejsciu lini gdzie jest new[ile] z wcześniejsza deklaracja na -1. Kompilacja gcc + cmake debug gdb.

Pewnie jakąś flagę musiałbym włączyć żeby to tak działo jak mówisz

komentarz 7 kwietnia 2018 przez Patrycjerz Mędrzec (192,320 p.)

Racja, mój błąd. Pętla przestanie działać. Oczywiście takie dziwne rzutowania nie powinny mieć miejsca i lepiej zastosować, tak jak napisałeś, typ bez znaku.

komentarz 7 kwietnia 2018 przez mokrowski Mędrzec (155,640 p.)

Standard mówi że dla typów signed, nie wiadomo czy się licznik przekręci. Dla typów unsigned wiadomo. Operacje przekroczenia zakresu dla signed są UB.

komentarz 8 kwietnia 2018 przez Tomeko Nowicjusz (150 p.)

Czyli unsigned int może być?

komentarz 8 kwietnia 2018 przez mokrowski Mędrzec (155,640 p.)

@Tomeko, dla wszelkiego rodzaju liczników stosuj typ std::size_t. On jest przewidziany do obsługi tablic, kontenerów, określania wielkości obiektu itp... Jest unsigned i najczęściej long (na arch. 32-bit lub większych). Do stosowania typów mniejszych jako liczniki, powinieneś mieć jakiś bardzo ważki powód. Np. arch. wbudowana z mikrokontrolerem 8-bitowym w którym i tak nie ma więcej niż 255 bajtów pamięci.

odpowiedź 7 kwietnia 2018 przez mokrowski Mędrzec (155,640 p.)

Dla wersji języka przed C++11, może to wyglądać tak:

#include <iostream>
#include <cstddef>
#include <stdint.h>
#include <cstdlib>
#include <new>

int main()
{
    std::size_t ile = 0;
    std::cout << "Ile liczb w tablicy: ";
    std::cin >> ile;
    int * tablica = new(std::nothrow) int[ile];
    if(tablica == NULL) {
    	std::cerr << "Alokacja tablicy nie powiodła się.\n";
    	return EXIT_FAILURE;
    }
    for (size_t i =  0; i < ile; ++i)
    {
        std::cout << reinterpret_cast<intptr_t>(&(tablica[i])) << '\n';
    }
    delete[] tablica;
    tablica = NULL;
    return EXIT_SUCCESS;
}