Prateek Krishna Ответов: 5

Почему он показывает ошибку сегментации (SIGSEGV)...задача состоит в том, чтобы напечатать самый большой простой фактор.


Given a number N, the task is to find the largest prime factor of that number.

Input:
The first line of input contains an integer T, denoting the number of test cases. Then T test cases follow. Each test case contains an integer N.

Output:
For each test case, in a new line, print the largest prime factor of N.

Constraints:
1 <= T <= 100
2 <= N <= 10^10

Example:
Input:
2
6
15
Output:
3
5


Что я уже пробовал:

#include <bits/stdc++.h>
using namespace std;

void check(long long int n)
{
    long long int p=2,i;
    bool prime[n+1];
    memset(prime,true,sizeof(prime));
    for(p=2;p*p<=n;p++)
    {
        if(prime[p])
        {
          for(i=2*p;i<=n;i+=p)
          prime[i]=false;
        }
    }
    if(prime[n])
    {
        cout<<n<<endl;
    }
    else
    {
    for(i=n/2;i>=2;i--)
   {
       if(prime[i])
       {
         if(n%i==0)
         {
         cout<<i;
         break;
         }
       }
   }
   cout<<"\n";
    }
}


int main() {
	int t;
	cin>>t;
	while(t--)
	{
	    long long int n;
	    cin>>n;
	    if(n==1)
	    cout<<n<<endl;
	    else if(n==2)
	    cout<<n<<endl;
	    else if(n==3)
	    cout<<n<<endl;
	    else
	    check(n);
	}
	return 0;
}

CPallini

Уверены ли вы, что противопоказания верны (ваш код не падает в таких ограничениях)?

Prateek Krishna

Да,

Ограничения:
1 <= T <= 100
2 <= N <= 10^10

CPallini

Извини, что я неправильно понял.
Вы можете (то есть вы должен) реализовать его без выделения prime массив.

Prateek Krishna

как же так?

5 Ответов

Рейтинг:
2

Patrice T

Цитата:
Почему он показывает ошибку сегментации (SIGSEGV)

Это означает, что где-то в вашем коде вы пытаетесь получить доступ к памяти, которой не владеете.
Используя отладчик, вы увидите, что на самом деле делает ваш код, и вы увидите, где возникает проблема.

Ваш код ведет себя не так, как вы ожидаете, или вы не понимаете, почему !

Существует почти универсальное решение: запускайте свой код на отладчике шаг за шагом, проверяйте переменные.
Отладчик здесь, чтобы показать вам, что делает ваш код, и ваша задача-сравнить с тем, что он должен делать.
В отладчике нет никакой магии, он не знает, что должен делать ваш код, он не находит ошибок, он просто помогает вам, показывая, что происходит. Когда код не делает того, что ожидается, вы близки к ошибке.
Чтобы увидеть, что делает ваш код: просто установите точку останова и посмотрите, как работает ваш код, отладчик позволит вам выполнять строки 1 на 1 и проверять переменные по мере их выполнения.

Отладчик - Википедия, свободная энциклопедия[^]

Освоение отладки в Visual Studio 2010 - руководство для начинающих[^]
Базовая отладка с помощью Visual Studio 2010 - YouTube[^]

1.11 — отладка программы (пошаговое выполнение и останова) | выучить C++[^]

Отладчик здесь только для того, чтобы показать вам, что делает ваш код, и ваша задача-сравнить его с тем, что он должен делать.


Рейтинг:
2

CPallini

Не имеет смысла (пытаться) выделить массив, 10^10 элементы: среда выполнения не может предоставить вам его, и на самом деле он вам не нужен.
Вы можете хранить только фактические простые числа в a vector Однако существуют гораздо более простые решения такой проблемы (вам позволено найти их самостоятельно, это не так).
вопрос Гуглить немного).


Maciej Los

5ed!

CPallini

Спасибо!

Рейтинг:
1

Richard MacCutchan

Скорее всего из-за следующего:

bool prime[n+1];
memset(prime,true,sizeof(prime));

То sizeof оператор-это операция времени компиляции. Но поскольку компилятор не знает, насколько велик массив, он не может правильно вычислить значение. Вы не должны использовать динамические массивы таким образом, так как не все компиляторы поддерживают эту функцию. Используйте C++ new оператор таким образом гарантирует, что вы получите правильный результат.


0x01AA

В 5
"Используйте новый оператор C++": или в качестве альтернативы что-то вроде (n+1)*sizeof(bool)
В любом случае memset(prime,true,...); выглядит подозрительно для меня

Richard MacCutchan

Вам не нужно использовать sizeof при выделении в стеке или использовании new. Спецификация типа четко определяет, какое количество байтов следует выделить для каждого элемента.

Например, если вы ввели

int foo[2 * sizeof(int)];

в нем будет выделено 8 пунктов, а не 2.

Рейтинг:
1

KarstenK

используйте отладчик для поиска проблем в коде.

совет: сделайте некоторый вывод, чтобы быстрее найти его там, где ваши проблемы все еще являются. Я бы снова предположил, что какое-то деление с нулем находится внутри невыполненной работы.


Maciej Los

5ed!

Рейтинг:
0

OriginalGriff

Компиляция не означает, что ваш код верен! :смеяться:
Подумайте о процессе разработки как о написании электронного письма: успешная компиляция означает, что вы написали письмо на правильном языке - например, на английском, а не на немецком, - а не то, что письмо содержало сообщение, которое вы хотели отправить.

Итак, теперь вы входите во вторую стадию разработки (на самом деле это четвертая или пятая, но вы перейдете к более ранним стадиям позже): тестирование и отладка.

Начните с рассмотрения того, что он делает, и как это отличается от того, что вы хотели. Это важно, потому что это дает вам информацию о том, почему он это делает. Например, если программа предназначена для того, чтобы позволить пользователю ввести число, а затем удвоить его и напечатать ответ, то если бы ввод / вывод был таким:

Input   Expected output    Actual output
  1            2                 1
  2            4                 4
  3            6                 9
  4            8                16
Тогда совершенно очевидно, что проблема заключается в бите, который удваивает его - он не прибавляет себя к себе или умножает его на 2, он умножает его на себя и возвращает квадрат входного сигнала.
Таким образом, вы можете посмотреть на код, и очевидно, что он находится где-то здесь:
int Double(int value)
   {
   return value * value;
   }

Как только у вас появится идея, что может пойти не так, начните использовать отладчик, чтобы выяснить, почему. Поместите точку останова в первую строку метода и запустите приложение. Когда он достигнет точки останова, отладчик остановится и передаст управление вам. Теперь вы можете запускать свой код построчно (так называемый "одноступенчатый") и просматривать (или даже изменять) содержимое переменных по мере необходимости (черт возьми, вы даже можете изменить код и повторить попытку, если вам это нужно).
Подумайте о том, что должна делать каждая строка кода перед ее выполнением, и сравните это с тем, что она действительно делала, когда вы использовали кнопку "Step over" для выполнения каждой строки по очереди. Он сделал то, что вы ожидали? Если да, то переходите к следующей строке.
Если нет, то почему? Чем это отличается?
Надеюсь, это поможет вам определить, в какой части этого кода есть проблема и в чем она заключается.
Это навык, и его стоит развивать, поскольку он помогает вам как в реальном мире, так и в развитии. И, как и все навыки, он только улучшается при использовании!


Maciej Los

5ed!