annotate toys/other/factor.c @ 1411:dd336488a69b draft

factor: catch integer overflow. Now factor 9223372036854775783 (largest positive 64 bit signed prime) takes a couple minutes but gives the right answer.
author Rob Landley <rob@landley.net>
date Fri, 01 Aug 2014 18:50:46 -0500
parents ee0109c35b34
children 89384d54d49a
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
1410
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
1 /* factor.c - Factor integers
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
2 *
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
3 * Copyright 2014 Rob Landley <rob@landley.net>
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
4 *
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
5 * No standard, but it's in coreutils
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
6
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
7 USE_FACTOR(NEWTOY(factor, 0, TOYFLAG_USR|TOYFLAG_BIN))
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
8
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
9 config FACTOR
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
10 bool "factor"
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
11 default y
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
12 help
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
13 usage: factor NUMBER...
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
14
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
15 Factor integers.
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
16 */
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
17
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
18 #include "toys.h"
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
19
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
20 static void factor(char *s)
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
21 {
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
22 long l, ll;
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
23
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
24 l = strtol(s, &s, 0);
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
25 if (*s) {
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
26 error_msg("%s: not integer");
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
27 return;
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
28 }
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
29
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
30 printf("%ld:", l);
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
31
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
32 // Negative numbers have -1 as a factor
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
33 if (l < 0) {
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
34 printf(" -1");
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
35 l *= -1;
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
36 }
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
37
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
38 // Deal with 0 and 1 (and 2 since we're here)
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
39 if (l < 3) {
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
40 printf(" %ld\n", l);
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
41 return;
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
42 }
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
43
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
44 // Special case factors of 2
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
45 while (l && !(l&1)) {
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
46 printf(" 2");
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
47 l >>= 1;
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
48 }
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
49
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
50 // test odd numbers.
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
51 for (ll=3; ;ll += 2) {
1411
dd336488a69b factor: catch integer overflow.
Rob Landley <rob@landley.net>
parents: 1410
diff changeset
52 long lll = ll*ll;
dd336488a69b factor: catch integer overflow.
Rob Landley <rob@landley.net>
parents: 1410
diff changeset
53
dd336488a69b factor: catch integer overflow.
Rob Landley <rob@landley.net>
parents: 1410
diff changeset
54 if (lll>l || lll<ll) {
1410
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
55 if (l>1) printf(" %ld", l);
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
56 break;
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
57 }
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
58 while (!(l%ll)) {
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
59 printf(" %ld", ll);
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
60 l /= ll;
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
61 }
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
62 }
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
63 xputc('\n');
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
64 }
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
65
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
66 void factor_main(void)
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
67 {
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
68 if (toys.optc) {
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
69 char **ss;
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
70
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
71 for (ss = toys.optargs; *ss; ss++) factor(*ss);
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
72 } else for (;;) {
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
73 char *s = 0;
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
74 size_t len = 0;
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
75
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
76 if (-1 == getline(&s, &len, stdin)) break;
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
77 factor(s);
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
78 }
ee0109c35b34 Add factor.
Rob Landley <rob@landley.net>
parents:
diff changeset
79 }