annotate scripts/specfun/isprime.m @ 19041:0976f9fccbbd

isprime.m: Verify that input is a real number (bug #43041) * isprime.m: Use isreal() to validate input. Put input checking first in function. Add input validation tests.
author Rik <rik@octave.org>
date Thu, 21 Aug 2014 16:05:09 -0700
parents d63878346099
children 920a400929ca
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
17744
d63878346099 maint: Update copyright notices for release.
John W. Eaton <jwe@octave.org>
parents: 14363
diff changeset
1 ## Copyright (C) 2000-2013 Paul Kienzle
10664
faff5367cc05 second isprime rewrite
Jaroslav Hajek <highegg@gmail.com>
parents: 10657
diff changeset
2 ## Copyright (C) 2010 VZLU Prague
5827
1fe78adb91bc [project @ 2006-05-22 06:25:14 by jwe]
jwe
parents:
diff changeset
3 ##
1fe78adb91bc [project @ 2006-05-22 06:25:14 by jwe]
jwe
parents:
diff changeset
4 ## This file is part of Octave.
1fe78adb91bc [project @ 2006-05-22 06:25:14 by jwe]
jwe
parents:
diff changeset
5 ##
1fe78adb91bc [project @ 2006-05-22 06:25:14 by jwe]
jwe
parents:
diff changeset
6 ## Octave is free software; you can redistribute it and/or modify it
1fe78adb91bc [project @ 2006-05-22 06:25:14 by jwe]
jwe
parents:
diff changeset
7 ## under the terms of the GNU General Public License as published by
7016
93c65f2a5668 [project @ 2007-10-12 06:40:56 by jwe]
jwe
parents: 5827
diff changeset
8 ## the Free Software Foundation; either version 3 of the License, or (at
93c65f2a5668 [project @ 2007-10-12 06:40:56 by jwe]
jwe
parents: 5827
diff changeset
9 ## your option) any later version.
5827
1fe78adb91bc [project @ 2006-05-22 06:25:14 by jwe]
jwe
parents:
diff changeset
10 ##
1fe78adb91bc [project @ 2006-05-22 06:25:14 by jwe]
jwe
parents:
diff changeset
11 ## Octave is distributed in the hope that it will be useful, but
1fe78adb91bc [project @ 2006-05-22 06:25:14 by jwe]
jwe
parents:
diff changeset
12 ## WITHOUT ANY WARRANTY; without even the implied warranty of
1fe78adb91bc [project @ 2006-05-22 06:25:14 by jwe]
jwe
parents:
diff changeset
13 ## MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
1fe78adb91bc [project @ 2006-05-22 06:25:14 by jwe]
jwe
parents:
diff changeset
14 ## General Public License for more details.
1fe78adb91bc [project @ 2006-05-22 06:25:14 by jwe]
jwe
parents:
diff changeset
15 ##
1fe78adb91bc [project @ 2006-05-22 06:25:14 by jwe]
jwe
parents:
diff changeset
16 ## You should have received a copy of the GNU General Public License
7016
93c65f2a5668 [project @ 2007-10-12 06:40:56 by jwe]
jwe
parents: 5827
diff changeset
17 ## along with Octave; see the file COPYING. If not, see
93c65f2a5668 [project @ 2007-10-12 06:40:56 by jwe]
jwe
parents: 5827
diff changeset
18 ## <http://www.gnu.org/licenses/>.
5827
1fe78adb91bc [project @ 2006-05-22 06:25:14 by jwe]
jwe
parents:
diff changeset
19
1fe78adb91bc [project @ 2006-05-22 06:25:14 by jwe]
jwe
parents:
diff changeset
20 ## -*- texinfo -*-
11431
0d9640d755b1 Improve docstrings for all isXXX functions.
Rik <octave@nomad.inbox5.com>
parents: 10664
diff changeset
21 ## @deftypefn {Function File} {} isprime (@var{x})
0d9640d755b1 Improve docstrings for all isXXX functions.
Rik <octave@nomad.inbox5.com>
parents: 10664
diff changeset
22 ## Return a logical array which is true where the elements of @var{x} are
0d9640d755b1 Improve docstrings for all isXXX functions.
Rik <octave@nomad.inbox5.com>
parents: 10664
diff changeset
23 ## prime numbers and false where they are not.
0d9640d755b1 Improve docstrings for all isXXX functions.
Rik <octave@nomad.inbox5.com>
parents: 10664
diff changeset
24 ##
19041
0976f9fccbbd isprime.m: Verify that input is a real number (bug #43041)
Rik <rik@octave.org>
parents: 17744
diff changeset
25 ## @code{isprime} is appropriate if the maximum value in @var{x} is not too
0976f9fccbbd isprime.m: Verify that input is a real number (bug #43041)
Rik <rik@octave.org>
parents: 17744
diff changeset
26 ## large (< 1e15). For larger values special purpose factorization code
0976f9fccbbd isprime.m: Verify that input is a real number (bug #43041)
Rik <rik@octave.org>
parents: 17744
diff changeset
27 ## should be used.
5827
1fe78adb91bc [project @ 2006-05-22 06:25:14 by jwe]
jwe
parents:
diff changeset
28 ##
11431
0d9640d755b1 Improve docstrings for all isXXX functions.
Rik <octave@nomad.inbox5.com>
parents: 10664
diff changeset
29 ## @example
0d9640d755b1 Improve docstrings for all isXXX functions.
Rik <octave@nomad.inbox5.com>
parents: 10664
diff changeset
30 ## @group
0d9640d755b1 Improve docstrings for all isXXX functions.
Rik <octave@nomad.inbox5.com>
parents: 10664
diff changeset
31 ## isprime (1:6)
0d9640d755b1 Improve docstrings for all isXXX functions.
Rik <octave@nomad.inbox5.com>
parents: 10664
diff changeset
32 ## @result{} [0, 1, 1, 0, 1, 0]
0d9640d755b1 Improve docstrings for all isXXX functions.
Rik <octave@nomad.inbox5.com>
parents: 10664
diff changeset
33 ## @end group
0d9640d755b1 Improve docstrings for all isXXX functions.
Rik <octave@nomad.inbox5.com>
parents: 10664
diff changeset
34 ## @end example
5827
1fe78adb91bc [project @ 2006-05-22 06:25:14 by jwe]
jwe
parents:
diff changeset
35 ## @seealso{primes, factor, gcd, lcm}
1fe78adb91bc [project @ 2006-05-22 06:25:14 by jwe]
jwe
parents:
diff changeset
36 ## @end deftypefn
1fe78adb91bc [project @ 2006-05-22 06:25:14 by jwe]
jwe
parents:
diff changeset
37
11469
c776f063fefe Overhaul m-script files to use common variable name between code and documentation.
Rik <octave@nomad.inbox5.com>
parents: 11431
diff changeset
38 function t = isprime (x)
7125
f084ba47812b [project @ 2007-11-08 02:29:23 by jwe]
jwe
parents: 7017
diff changeset
39
19041
0976f9fccbbd isprime.m: Verify that input is a real number (bug #43041)
Rik <rik@octave.org>
parents: 17744
diff changeset
40 if (nargin != 1)
0976f9fccbbd isprime.m: Verify that input is a real number (bug #43041)
Rik <rik@octave.org>
parents: 17744
diff changeset
41 print_usage ();
0976f9fccbbd isprime.m: Verify that input is a real number (bug #43041)
Rik <rik@octave.org>
parents: 17744
diff changeset
42 elseif (! isreal (x) || any ((x < 0 | x != fix (x))(:)))
0976f9fccbbd isprime.m: Verify that input is a real number (bug #43041)
Rik <rik@octave.org>
parents: 17744
diff changeset
43 error ("isprime: X must be a non-negative integer");
0976f9fccbbd isprime.m: Verify that input is a real number (bug #43041)
Rik <rik@octave.org>
parents: 17744
diff changeset
44 endif
0976f9fccbbd isprime.m: Verify that input is a real number (bug #43041)
Rik <rik@octave.org>
parents: 17744
diff changeset
45
0976f9fccbbd isprime.m: Verify that input is a real number (bug #43041)
Rik <rik@octave.org>
parents: 17744
diff changeset
46 maxn = max (x(:));
0976f9fccbbd isprime.m: Verify that input is a real number (bug #43041)
Rik <rik@octave.org>
parents: 17744
diff changeset
47 ## generate prime table of suitable length.
0976f9fccbbd isprime.m: Verify that input is a real number (bug #43041)
Rik <rik@octave.org>
parents: 17744
diff changeset
48 maxp = min (maxn, max (sqrt (maxn), 1e7)); # FIXME: threshold not optimized.
0976f9fccbbd isprime.m: Verify that input is a real number (bug #43041)
Rik <rik@octave.org>
parents: 17744
diff changeset
49 pr = primes (maxp);
0976f9fccbbd isprime.m: Verify that input is a real number (bug #43041)
Rik <rik@octave.org>
parents: 17744
diff changeset
50 ## quick search for table matches.
0976f9fccbbd isprime.m: Verify that input is a real number (bug #43041)
Rik <rik@octave.org>
parents: 17744
diff changeset
51 t = lookup (pr, x, "b");
0976f9fccbbd isprime.m: Verify that input is a real number (bug #43041)
Rik <rik@octave.org>
parents: 17744
diff changeset
52 ## take the rest.
0976f9fccbbd isprime.m: Verify that input is a real number (bug #43041)
Rik <rik@octave.org>
parents: 17744
diff changeset
53 m = x(x > maxp);
0976f9fccbbd isprime.m: Verify that input is a real number (bug #43041)
Rik <rik@octave.org>
parents: 17744
diff changeset
54 if (! isempty (m))
0976f9fccbbd isprime.m: Verify that input is a real number (bug #43041)
Rik <rik@octave.org>
parents: 17744
diff changeset
55 ## there are still possible primes. filter them out by division.
0976f9fccbbd isprime.m: Verify that input is a real number (bug #43041)
Rik <rik@octave.org>
parents: 17744
diff changeset
56 if (maxn <= intmax ("uint32"))
0976f9fccbbd isprime.m: Verify that input is a real number (bug #43041)
Rik <rik@octave.org>
parents: 17744
diff changeset
57 m = uint32 (m);
0976f9fccbbd isprime.m: Verify that input is a real number (bug #43041)
Rik <rik@octave.org>
parents: 17744
diff changeset
58 elseif (maxn <= intmax ("uint64"))
0976f9fccbbd isprime.m: Verify that input is a real number (bug #43041)
Rik <rik@octave.org>
parents: 17744
diff changeset
59 m = uint64 (m);
0976f9fccbbd isprime.m: Verify that input is a real number (bug #43041)
Rik <rik@octave.org>
parents: 17744
diff changeset
60 else
0976f9fccbbd isprime.m: Verify that input is a real number (bug #43041)
Rik <rik@octave.org>
parents: 17744
diff changeset
61 warning ("isprime: X contains integers too large to be tested");
10664
faff5367cc05 second isprime rewrite
Jaroslav Hajek <highegg@gmail.com>
parents: 10657
diff changeset
62 endif
19041
0976f9fccbbd isprime.m: Verify that input is a real number (bug #43041)
Rik <rik@octave.org>
parents: 17744
diff changeset
63 pr = cast (pr(pr <= sqrt (maxn)), class (m));
0976f9fccbbd isprime.m: Verify that input is a real number (bug #43041)
Rik <rik@octave.org>
parents: 17744
diff changeset
64 for p = pr
0976f9fccbbd isprime.m: Verify that input is a real number (bug #43041)
Rik <rik@octave.org>
parents: 17744
diff changeset
65 m = m(rem (m, p) != 0);
0976f9fccbbd isprime.m: Verify that input is a real number (bug #43041)
Rik <rik@octave.org>
parents: 17744
diff changeset
66 if (length (m) < length (pr) / 10)
0976f9fccbbd isprime.m: Verify that input is a real number (bug #43041)
Rik <rik@octave.org>
parents: 17744
diff changeset
67 break;
0976f9fccbbd isprime.m: Verify that input is a real number (bug #43041)
Rik <rik@octave.org>
parents: 17744
diff changeset
68 endif
0976f9fccbbd isprime.m: Verify that input is a real number (bug #43041)
Rik <rik@octave.org>
parents: 17744
diff changeset
69 endfor
0976f9fccbbd isprime.m: Verify that input is a real number (bug #43041)
Rik <rik@octave.org>
parents: 17744
diff changeset
70 pr = pr(pr > p);
0976f9fccbbd isprime.m: Verify that input is a real number (bug #43041)
Rik <rik@octave.org>
parents: 17744
diff changeset
71 mm = arrayfun (@(x) all (rem (x, pr)), m);
0976f9fccbbd isprime.m: Verify that input is a real number (bug #43041)
Rik <rik@octave.org>
parents: 17744
diff changeset
72 m = m(mm);
10664
faff5367cc05 second isprime rewrite
Jaroslav Hajek <highegg@gmail.com>
parents: 10657
diff changeset
73 if (! isempty (m))
19041
0976f9fccbbd isprime.m: Verify that input is a real number (bug #43041)
Rik <rik@octave.org>
parents: 17744
diff changeset
74 m = cast (sort (m), class (x));
0976f9fccbbd isprime.m: Verify that input is a real number (bug #43041)
Rik <rik@octave.org>
parents: 17744
diff changeset
75 t |= lookup (m, x, "b");
10664
faff5367cc05 second isprime rewrite
Jaroslav Hajek <highegg@gmail.com>
parents: 10657
diff changeset
76 endif
7125
f084ba47812b [project @ 2007-11-08 02:29:23 by jwe]
jwe
parents: 7017
diff changeset
77 endif
f084ba47812b [project @ 2007-11-08 02:29:23 by jwe]
jwe
parents: 7017
diff changeset
78
5827
1fe78adb91bc [project @ 2006-05-22 06:25:14 by jwe]
jwe
parents:
diff changeset
79 endfunction
9984
d1cc2e0ddf55 isprime: produce logical result
John W. Eaton <jwe@octave.org>
parents: 7125
diff changeset
80
11469
c776f063fefe Overhaul m-script files to use common variable name between code and documentation.
Rik <octave@nomad.inbox5.com>
parents: 11431
diff changeset
81
14363
f3d52523cde1 Use Octave coding conventions in all m-file %!test blocks
Rik <octave@nomad.inbox5.com>
parents: 14138
diff changeset
82 %!assert (isprime (3), true)
f3d52523cde1 Use Octave coding conventions in all m-file %!test blocks
Rik <octave@nomad.inbox5.com>
parents: 14138
diff changeset
83 %!assert (isprime (4), false)
f3d52523cde1 Use Octave coding conventions in all m-file %!test blocks
Rik <octave@nomad.inbox5.com>
parents: 14138
diff changeset
84 %!assert (isprime (magic (3)), logical ([0, 0, 0; 1, 1, 1; 0, 0, 1]))
f3d52523cde1 Use Octave coding conventions in all m-file %!test blocks
Rik <octave@nomad.inbox5.com>
parents: 14138
diff changeset
85
19041
0976f9fccbbd isprime.m: Verify that input is a real number (bug #43041)
Rik <rik@octave.org>
parents: 17744
diff changeset
86 %% Test input validation
9984
d1cc2e0ddf55 isprime: produce logical result
John W. Eaton <jwe@octave.org>
parents: 7125
diff changeset
87 %!error isprime ()
d1cc2e0ddf55 isprime: produce logical result
John W. Eaton <jwe@octave.org>
parents: 7125
diff changeset
88 %!error isprime (1, 2)
19041
0976f9fccbbd isprime.m: Verify that input is a real number (bug #43041)
Rik <rik@octave.org>
parents: 17744
diff changeset
89 %!error <X must be a non-negative integer> isprime (i)
0976f9fccbbd isprime.m: Verify that input is a real number (bug #43041)
Rik <rik@octave.org>
parents: 17744
diff changeset
90 %!error <X must be a non-negative integer> isprime ([1 2; -3 4])
0976f9fccbbd isprime.m: Verify that input is a real number (bug #43041)
Rik <rik@octave.org>
parents: 17744
diff changeset
91 %!error <X must be a non-negative integer> isprime ([1 2; 3.1 4])
14363
f3d52523cde1 Use Octave coding conventions in all m-file %!test blocks
Rik <octave@nomad.inbox5.com>
parents: 14138
diff changeset
92