двойное освобождение или повреждение (fasttop): 0x0000000000063d070 *** сито программа С++

Я пишу решетчатую программу на С++. Но для каждого допустимого ввода программа всегда выдает результат с 4 найденными простыми числами и «2 3 5», независимо от того, как изменяется ввод. При попытке запустить программу через консоль выдает ошибку о том, что двойное освобождение или повреждение (фасттоп): 0x0000000000063d070***. Кстати, я новичок в С++. Кроме того, я пытаюсь правильно отформатировать вывод, но они просто летают.

Это желаемый формат.

2   3   5   7  11  13  17  19  23  29  31  37  41  43  47  53  59  61  67  71

73 79 83 89 97 101 103 107 109 113 127 131 137 139 149 151 157 163 167 173 179 181 191 193 197 199 211 223 227 229 233 239 241 251 257 263 269 271 277 281 283 293 307 311 313 317 331 337 347 349 353 359 367 373 379 383 389 397 401 409 419 421 431 433 439 443 449 457 461 463 467 479 487 491 499 503 509 521 523 541 547 557 563 569 571 577 587 593 599 601 607 613 617 619 631 641 643 647 653 659 661 673 677 683 691 701 709 719 727 733 739 743 751 757 761 769 773 787 797 809 811 821 823 827 829 839 853 857 859 863 877 881 883 887 907 911 919 929 937 941 947 953 967 971 977 983 991 997


person Echo111    schedule 26.05.2016    source источник
comment
PrimesSieve(limit).~PrimesSieve(); -- Почему вы явно вызываете деструктор?   -  person PaulMcKenzie    schedule 26.05.2016
comment
Так это не правильно называть? Или нам не нужно его вызывать? Я новичок в С++. Мне жаль.   -  person Echo111    schedule 26.05.2016
comment
Нет необходимости явно вызывать деструкторы. Когда объект выходит за пределы области видимости, автоматически вызывается деструктор. Все, что вы в конечном итоге делаете, это вызываете деструктор, а затем деструктор вызывается снова - это приведет к двойным бесплатным ошибкам, как вы видите сейчас. Кроме того, в какой книге или учебнике по С++ вы видите деструкторы, называемые так? Бьюсь об заклад, вы не нашли его (если только он не является продвинутым и не обсуждает размещение-новое)   -  person PaulMcKenzie    schedule 26.05.2016
comment
Кроме того, я пытаюсь отформатировать его красиво. Но цифры просто летают...   -  person Echo111    schedule 26.05.2016
comment
Оформление - другой вопрос. Сначала побеспокойтесь о сбое — красивый вывод ничего не значит, если программа не выдает результатов.   -  person PaulMcKenzie    schedule 26.05.2016
comment
результаты все правильные. Я просто в замешательстве. ......Программа выдает правильный интервал для 1-значных и 3-значных чисел, но программа не работает для интервала для 2-значных чисел, и она выводит 8 пробелов перед 2(что должно быть нет) при нахождении простых чисел до 2.   -  person Echo111    schedule 26.05.2016
comment
PrimesSieve(limit).~PrimesSieve(); - эта строка создает временный объект, затем вызывает деструктор временного объекта, затем уничтожает временный объект (вызывая его деструктор снова)   -  person user253751    schedule 26.05.2016


Ответы (1)


Помимо того, что ваше двойное освобождение вызвано явным вызовом деструктора, как сказал @PaulMcKenzie в комментариях, ваша проблема с выводом только первых нескольких простых чисел связана с этой строкой:

int n = sizeof(is_prime_);

is_prime_ — это указатель, поэтому его размер фиксируется во время компиляции (вероятно, 4 или 8 байт в зависимости от вашей системы).

У вас уже есть limit_ в качестве значения, вы должны использовать его, чтобы определить n.

person The Dark    schedule 26.05.2016
comment
Программа выдает правильный интервал для однозначных и трехзначных чисел, но программа не работает для интервала для двузначных чисел и выводит 8 пробелов перед 2 (которых не должно быть) при нахождении простых чисел. до 2. Я запутался. Как такое могло произойти?... - person Echo111; 26.05.2016
comment
setw задает ширину поля вывода - в него не нужно добавлять количество цифр в текущем значении, так как оно уже учтет это. Просто используйте cout <<setw(max_prime_width) << i; - person The Dark; 26.05.2016