Никогда не передавайте долго как параметр? - PullRequest
0 голосов
/ 22 февраля 2019

Программный вопрос:

с учетом сетки чисел, сколько различных простых чисел вы можете найти в матрице, учитывая, что вы можете прочитать строки или их часть в вертикальной форме, горизонтальная или диагональная ориентация, в обоих направлениях?

БОЛЬШОЙ РЕДАКТИРОВАТЬ: Для любого плохого чтения души, я решил это, добавив if (n > INT_MAX) return false; в первую строку под моей функцией!

Моя программа не падает, если параметр isPrime не равен const ll &n, и мне интересно, почему.Это потому, что вы никогда не должны передавать long long в качестве параметра, потому что он такой большой?Или это потому, что если вы добавите int к целому long long, он больше не станет long long?

Вводом, который я использую для проверки своей работы, является:

40
0251677085866837460317708637021446063144
8812262220360202463050064531874436437682
5251855367278508848642345043775871434078
0042675865438283025822603307175060748288
5672321632434878440388701468545837465571
3448326728143606881852187616524878044060
8876415778774852362710315274652021065556
1406474838287088561242126854006826771778
7827443331184330371521043472218803550383
6318874838447276075161123302780187880165
0884752758538865306583258450386821283658
1260362124615176735303563717773657467333
2580363145201308707655341168610513145546
4142635386876777348215436708254351251288
5301330463217260626047132625527161775404
8620446353006857360714856156584322276288
0813375760405334773480860674772272733638
6715558007108501053612008324501255710425
8840634327383685827335506853781648727036
8827728873376824454871536655067801443735
0664640563836487481174816586572628815173
7186752536147276768154002317573417465332
4438770023402783205544064640821537621225
4162442401558771474140203865162080237721
5008757506737224070577338578644664641338
2155803687408638660278862273674652462840
2118148017744113203720114756276821067158
4838003412436782114402742024145245437315
5161343527676283186170466281455700086618
7723886261287175705152273086317588317188
6653360024271146551000054710768617586846
0050014847531086708661266564560614115164
3351156208161708784441387827072734346251
0457546342466313073230326436563643506534
3837451141488371231210888733717540046582
3334248265835234158638343058444640886465
0173240266426385002380821305357684721128
0437020214873361352055818843664073456138
3858604586068245520287541000014334760058
5840781588142205318614583635575571714673

Мой код здесь:

#include <iostream>
#include <set>
#include <vector>

typedef long long ll;

const int DIRECTIONS[][2] = {{1, 0}, {0, 1}, {1, 1}, {-1, 0}, {0, -1}, {-1, -1}, {1, -1}, {-1, 1}};

bool isPrime(const int &n) {
        if (n != 2) {
                if (n < 2 || !(n & 1)) return false;
                for (ll i = 3; i * i <= n + 1; i += 2) {
                        if (n % i == 0) return false;
                }
        }
        return true;
}

int main() {
        int length = 0;
        std::set<ll> primes{};
        char c;
        std::cin >> length;
        std::vector<std::vector<ll>> grid;
        for (int i = 0; i < length; i++) {
                std::vector<ll> row;
                for (int j = 0; j < length; j++) {
                        std::cin >> c;
                        row.push_back(c - '0');
                }
                grid.push_back(row);
        }
        for (int i = 0; i < length; i++) {
                for (int j = 0; j < length; j++) {
                        for (auto d : DIRECTIONS) {
                                int tempi = i, tempj = j;
                                ll base10 = 0;
                                while (i < length && j < length && i >= 0 && j >= 0) {
                                        base10 *= 10;
                                        base10 += grid[i][j];
                                        if (isPrime(base10)) primes.insert(base10);
                                        i += d[0];
                                        j += d[1];
                                }
                                i = tempi;
                                j = tempj;
                        }
                }
        }
        std::cout << "Number of primes: " << primes.size() << std::endl;
        /*std::cout << "List of primes: " << std::flush;
        for (auto prime : primes) {
                std::cout << prime << " " << std::flush;
        }*/
        return 0;
}

1 Ответ

0 голосов
/ 22 февраля 2019

Программа вызывает неопределенное поведение в детали с:

base10 *= 10;
base10 += grid[i][j]

, потому что эти вычисления иногда превышают максимальное значение long long.

long long может содержать не более19 десятичных цифр.Если целью является проверка простых чисел до 40 цифр, вам придется перейти на другой подход.

...