aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorzlg <zlg@zlg.space>2015-01-30 00:55:59 -0800
committerzlg <zlg@zlg.space>2015-01-30 00:55:59 -0800
commit1f92ea2a90d98ef150a8546cf5695a73ba68312d (patch)
treedef01f396ff869a6c8ab99a9c838ba57b6de4a64
parentEnsure 1-02's solution compiles (diff)
downloadknr-1f92ea2a90d98ef150a8546cf5695a73ba68312d.tar.gz
knr-1f92ea2a90d98ef150a8546cf5695a73ba68312d.tar.bz2
knr-1f92ea2a90d98ef150a8546cf5695a73ba68312d.tar.xz
knr-1f92ea2a90d98ef150a8546cf5695a73ba68312d.zip
Solve Exercise 5-18: error-recovering `dcl`
This exercise was a real pain in the ass. The original use of getch() and ungetch() was inferior to a line-based approach. It really doesn't seem like the way a parser should be built, but it taught me a little about the order of recursion. I'm convinced that a debugger is necessary if you want to build a good parser. This implementation doesn't recurse in a natural way, or even enough to stack data type prefixes. Hopefully by the time I get to 5-20 I'll have enough ideas to implement something that *does* recurse well.
-rw-r--r--ch5/5-18_dcl-error-recovery.c172
1 files changed, 172 insertions, 0 deletions
diff --git a/ch5/5-18_dcl-error-recovery.c b/ch5/5-18_dcl-error-recovery.c
new file mode 100644
index 0000000..9a22764
--- /dev/null
+++ b/ch5/5-18_dcl-error-recovery.c
@@ -0,0 +1,172 @@
+#include <stdio.h>
+#include <string.h>
+#include <ctype.h>
+
+/* The C Programming Language: 2nd Edition
+ *
+ * Exercise 5-18: Make `dcl` recover from input errors.
+ *
+ * Notes: The book does a poor job of describing what the reader should
+ * do here. Given that 5-20 calls for a refactor of `dcl`, my solution
+ * will simply complain at missing parens and brackets, and silently
+ * accept semicolons and comments at the end of the input line. Those are
+ * considered "errors" to the book's version of the program, and so this
+ * solution will fix them.
+ *
+ * This solution does not handle more complex data types such as const,
+ * static, long ("long long ..."), {un}signed, etc. It also won't handle
+ * arbitrary parentheses. Really simple stuff.
+ */
+
+#define MAXTOKEN 100
+#define BUFSIZE 100
+
+enum { NAME, PARENS, BRACKETS, TYPE };
+
+void dcl(void);
+void dirdcl(void);
+int gettoken(void);
+char buf[BUFSIZE];
+int bufp = 0;
+int tokentype;
+
+char token[MAXTOKEN];
+char name[MAXTOKEN];
+char datatype[MAXTOKEN];
+char out[1000];
+int get_line(char *s, int lim);
+
+void dcl(void) {
+ int ns;
+ for (ns = 0; gettoken() == '*'; ns++) {
+ }
+ dirdcl();
+ while (ns-- > 0) {
+ strcat(out, " pointer to");
+ }
+}
+
+void dirdcl(void) {
+ int type;
+ if (tokentype == '(') {
+ dcl();
+ if (tokentype != ')') {
+ printf("error: missing )\n");
+ }
+ } else if (tokentype == TYPE) {
+ strcat(datatype, " ");
+ strcat(datatype, token);
+ // check again for another type
+ gettoken();
+ } else if (tokentype == NAME) {
+ strcpy(name, token);
+ } else {
+ printf("error: expected name or (dcl)\n");
+ }
+ while ((type = gettoken()) == PARENS || type == BRACKETS) {
+ if (type == PARENS) {
+ strcat(out, " function returning");
+ } else {
+ strcat(out, " array");
+ strcat(out, token);
+ strcat(out, " of");
+ }
+ }
+}
+
+int gettoken(void) {
+ char *p = token;
+ while (buf[bufp] == ' ' || buf[bufp] == '\t') {
+ bufp++;
+ }
+ if (buf[bufp] == '/') {
+ if (buf[bufp+1] == '/' || buf[bufp+1] == '*') {
+ while (buf[bufp] != '\n') {
+ bufp++;
+ }
+ }
+ return tokentype = buf[bufp];
+ } else if (buf[bufp] == '(') {
+ if (buf[++bufp] == ')') {
+ strcpy(token, "()");
+ bufp++;
+ return tokentype = PARENS;
+ } else {
+ return tokentype = '(';
+ }
+ } else if (buf[bufp] == '[') {
+ while (buf[bufp] != ']') {
+ if (buf[bufp + 1] == '[') {
+ return tokentype = buf[bufp + 1];
+ }
+ if (isspace(buf[bufp])) {
+ bufp++;
+ continue;
+ }
+ *p++ = buf[bufp++];
+ }
+ *p++ = buf[bufp++];
+ *p = '\0';
+ return tokentype = BRACKETS;
+ } else if (isalpha(buf[bufp])) {
+ *p++ = buf[bufp++];
+ while (isalnum(buf[bufp])) {
+ *p++ = buf[bufp++];
+ }
+ *p = '\0';
+ /* check types */
+ if (strcmp(token, "int") == 0 ||
+ strcmp(token, "double") == 0 ||
+ strcmp(token, "char") == 0) {
+ return tokentype = TYPE;
+ }
+ return tokentype = NAME;
+ } else {
+ return tokentype = buf[bufp++];
+ }
+}
+
+/* Returns length of line including \n
+ */
+int get_line(char *s, int lim) {
+ int c, i;
+ for (i = 0; i < lim - 1 && (c=getchar()) != EOF && c != '\n'; ++i) {
+ *s++ = c;
+ }
+ if (c == '\n') {
+ *s++ = c;
+ i++;
+ }
+ *s = '\0';
+ return i;
+}
+
+int main(void) {
+ int len = 0;
+ while ((len = get_line(buf, BUFSIZE)) > 0 && buf[len - 1] != EOF) {
+ if (len == 1 && buf[bufp] == '\n') {
+ continue;
+ }
+ bufp = 0;
+ datatype[0] = 0;
+ gettoken();
+ if (tokentype == TYPE) {
+ strcpy(datatype, token);
+ }
+ out[0] = '\0';
+ name[0] = '\0';
+ dcl();
+ if (tokentype == ')') {
+ printf("error: too many closing parens\n\n");
+ } else if (tokentype == ']') {
+ printf("error: too many closing brackets\n\n");
+ } else if (tokentype == '(') {
+ printf("error: too many opening parens\n\n");
+ } else if (tokentype == '[') {
+ printf("error: too many opening brackets\n\n");
+ } else {
+ printf("%s: %s %s\n\n", name, out, datatype);
+ }
+ }
+ return 0;
+}
d>1-80/+67 Whoops. 2018-09-03Branch off from master with pytest, tox, clickzlg16-778/+779 This commit is huge, but contains everything needed for a "proper" build system built on pytest + tox and a CLI built with click. For now, this branch will contain all new vgstash development activity until it reaches feature parity with master. The CLI is installed to pip's PATH. Only the 'init', 'add', and 'list' commands work, with only two filters. This is pre-alpha software, and is therefore not stable yet. 2018-03-18Flesh out filter types and ownership statuszlg3-82/+144 It's time for a refactor to a module; the functionality and interface are clashing. 2018-03-18README.mdown: break line correctlyzlg1-1/+1 2018-03-18add 'playlog' list filterzlg2-2/+9 This filter is used to get an idea of which games you're currently playing through, so you can prioritize games to play when you're bored and detect it when you've beaten a game but haven't marked it as such. 2018-03-13Update helpers a bitzlg1-2/+9 At present, user modification is needed to make these seamless. vgup() may need to be axed in favor of telling the user to make an alias. 2018-03-13Make VGSTASH_DB_LOCATION point to a filezlg2-21/+20 It used to point to a directory, which would then look for .vgstash.db. This behavior was kind of backwards and I don't remember why I did it that way. This change gives users more control over where they put their DB. Be sure to update your environment variable if you have it set! 2016-11-18Remove settings from helpers.shZe Libertine Gamer1-5/+0 Sourcing them in .bash_profile screws up login if they're set. 2016-11-15Correct phrasing in README.Ze Libertine Gamer1-4/+4 2016-11-13DerpZe Libertine Gamer1-0/+1 2016-11-03Improve error handling in shell scriptsZe Libertine Gamer4-3/+23 2016-10-24Correct run_again, add recursionZe Libertine Gamer1-0/+4 Loops and functions -- oh my, what a useful combination. :) 2016-10-21Add quotes to correct behavior for arglistZe Libertine Gamer1-1/+1 2016-10-14updater.sh: add recursion, error handlingZe Libertine Gamer1-43/+101 2016-10-14Correct pipe-handling behaviorZe Libertine Gamer1-1/+9 2016-10-12Clarify a method to move between platformsZe Libertine Gamer1-2/+5 Also correct a typo.