Brainfuck interpreter in C 3Brainfuck InterpreterBrainfuck interpreter in CBrainfuck interpreter in...

How can I get through very long and very dry, but also very useful technical documents when learning a new tool?

What does "I’d sit this one out, Cap," imply or mean in the context?

How to write papers efficiently when English isn't my first language?

Are student evaluations of teaching assistants read by others in the faculty?

How does buying out courses with grant money work?

What is paid subscription needed for in Mortal Kombat 11?

Pole-zeros of a real-valued causal FIR system

Is expanding the research of a group into machine learning as a PhD student risky?

How to draw lines on a tikz-cd diagram

Why Were Madagascar and New Zealand Discovered So Late?

What happens if you roll doubles 3 times then land on "Go to jail?"

Is HostGator storing my password in plaintext?

How can a function with a hole (removable discontinuity) equal a function with no hole?

Implement the Thanos sorting algorithm

Short story about space worker geeks who zone out by 'listening' to radiation from stars

Tiptoe or tiphoof? Adjusting words to better fit fantasy races

Why escape if the_content isnt?

Increase performance creating Mandelbrot set in python

What Brexit proposals are on the table in the indicative votes on the 27th of March 2019?

Failed to fetch jessie backports repository

Did Dumbledore lie to Harry about how long he had James Potter's invisibility cloak when he was examining it? If so, why?

How does Loki do this?

Gears on left are inverse to gears on right?

Why not increase contact surface when reentering the atmosphere?



Brainfuck interpreter in C 3


Brainfuck InterpreterBrainfuck interpreter in CBrainfuck interpreter in JavaScriptBrainfuck Interpreter in JavaBrainFuck Interpreter in C++Brainfuck interpreter in ClojureBasic Brainfuck Interpreter in CJava Brainfuck Interpreter MethodRuby Brainfuck interpreterObject-oriented Brainfuck interpreter













0












$begingroup$


I created my brainfuck interpreter and I would like to know what can be done better and is the code clear and readable. I will be very thankful for opinions and suggestions.
Here is the code:



#include <stdio.h>
#include <stdlib.h>

#define ALLOCATION_ERROR 1
#define FILE_ERROR 2
#define OTHER_ERROR 3
#define TAPE_SIZE 30000

int main(int argc,char** argv){
if(argc!=2){
puts("Usage: main <source>");
exit(OTHER_ERROR);
}
unsigned char* tape=calloc(TAPE_SIZE,1);
if(tape==NULL){
fprintf(stderr,"Fatal: failed to allocate %zu bytes.n",(size_t)TAPE_SIZE*1);
exit(ALLOCATION_ERROR);
}
unsigned char* tape_ptr=tape;
FILE* input_file=fopen(argv[1],"rb");
if(input_file==NULL){
fprintf(stderr,"Error: failed to open file %sn",argv[1]);
exit(FILE_ERROR);
}
fseek(input_file,0,SEEK_END);
size_t code_size=(size_t)ftell(input_file);
fseek(input_file,0,SEEK_SET);
unsigned char* code=malloc(code_size+1);
if(code==NULL){
fprintf(stderr,"Fatal: failed to allocate %zu bytes.n",code_size+1);
exit(ALLOCATION_ERROR);
}
if(fread(code,1,code_size,input_file)!=code_size){
fprintf(stderr,"Error: failed to read from file %sn",argv[1]);
exit(FILE_ERROR);
}
code[code_size]=0;
for(unsigned char* code_ptr=code;*code_ptr;++code_ptr){
switch(*code_ptr){
case '>':
++tape_ptr;
break;
case '<':
--tape_ptr;
break;
case '+':
++*tape_ptr;
break;
case '-':
--*tape_ptr;
break;
case ',':
*tape_ptr=(unsigned char)getchar();
break;
case '.':
putchar(*tape_ptr);
fflush(stdout);
break;
case '[':
if(!*tape_ptr){
int loop=1;
while(loop){
++code_ptr;
if(*code_ptr=='[')
++loop;
if(*code_ptr==']')
--loop;
}
}
break;
case ']':
if(*tape_ptr){
int loop=1;
while(loop){
--code_ptr;
if(*code_ptr=='[')
--loop;
if(*code_ptr==']')
++loop;
}
}
break;
}
}
free(tape);
free(code);
}









share|improve this question









$endgroup$

















    0












    $begingroup$


    I created my brainfuck interpreter and I would like to know what can be done better and is the code clear and readable. I will be very thankful for opinions and suggestions.
    Here is the code:



    #include <stdio.h>
    #include <stdlib.h>

    #define ALLOCATION_ERROR 1
    #define FILE_ERROR 2
    #define OTHER_ERROR 3
    #define TAPE_SIZE 30000

    int main(int argc,char** argv){
    if(argc!=2){
    puts("Usage: main <source>");
    exit(OTHER_ERROR);
    }
    unsigned char* tape=calloc(TAPE_SIZE,1);
    if(tape==NULL){
    fprintf(stderr,"Fatal: failed to allocate %zu bytes.n",(size_t)TAPE_SIZE*1);
    exit(ALLOCATION_ERROR);
    }
    unsigned char* tape_ptr=tape;
    FILE* input_file=fopen(argv[1],"rb");
    if(input_file==NULL){
    fprintf(stderr,"Error: failed to open file %sn",argv[1]);
    exit(FILE_ERROR);
    }
    fseek(input_file,0,SEEK_END);
    size_t code_size=(size_t)ftell(input_file);
    fseek(input_file,0,SEEK_SET);
    unsigned char* code=malloc(code_size+1);
    if(code==NULL){
    fprintf(stderr,"Fatal: failed to allocate %zu bytes.n",code_size+1);
    exit(ALLOCATION_ERROR);
    }
    if(fread(code,1,code_size,input_file)!=code_size){
    fprintf(stderr,"Error: failed to read from file %sn",argv[1]);
    exit(FILE_ERROR);
    }
    code[code_size]=0;
    for(unsigned char* code_ptr=code;*code_ptr;++code_ptr){
    switch(*code_ptr){
    case '>':
    ++tape_ptr;
    break;
    case '<':
    --tape_ptr;
    break;
    case '+':
    ++*tape_ptr;
    break;
    case '-':
    --*tape_ptr;
    break;
    case ',':
    *tape_ptr=(unsigned char)getchar();
    break;
    case '.':
    putchar(*tape_ptr);
    fflush(stdout);
    break;
    case '[':
    if(!*tape_ptr){
    int loop=1;
    while(loop){
    ++code_ptr;
    if(*code_ptr=='[')
    ++loop;
    if(*code_ptr==']')
    --loop;
    }
    }
    break;
    case ']':
    if(*tape_ptr){
    int loop=1;
    while(loop){
    --code_ptr;
    if(*code_ptr=='[')
    --loop;
    if(*code_ptr==']')
    ++loop;
    }
    }
    break;
    }
    }
    free(tape);
    free(code);
    }









    share|improve this question









    $endgroup$















      0












      0








      0





      $begingroup$


      I created my brainfuck interpreter and I would like to know what can be done better and is the code clear and readable. I will be very thankful for opinions and suggestions.
      Here is the code:



      #include <stdio.h>
      #include <stdlib.h>

      #define ALLOCATION_ERROR 1
      #define FILE_ERROR 2
      #define OTHER_ERROR 3
      #define TAPE_SIZE 30000

      int main(int argc,char** argv){
      if(argc!=2){
      puts("Usage: main <source>");
      exit(OTHER_ERROR);
      }
      unsigned char* tape=calloc(TAPE_SIZE,1);
      if(tape==NULL){
      fprintf(stderr,"Fatal: failed to allocate %zu bytes.n",(size_t)TAPE_SIZE*1);
      exit(ALLOCATION_ERROR);
      }
      unsigned char* tape_ptr=tape;
      FILE* input_file=fopen(argv[1],"rb");
      if(input_file==NULL){
      fprintf(stderr,"Error: failed to open file %sn",argv[1]);
      exit(FILE_ERROR);
      }
      fseek(input_file,0,SEEK_END);
      size_t code_size=(size_t)ftell(input_file);
      fseek(input_file,0,SEEK_SET);
      unsigned char* code=malloc(code_size+1);
      if(code==NULL){
      fprintf(stderr,"Fatal: failed to allocate %zu bytes.n",code_size+1);
      exit(ALLOCATION_ERROR);
      }
      if(fread(code,1,code_size,input_file)!=code_size){
      fprintf(stderr,"Error: failed to read from file %sn",argv[1]);
      exit(FILE_ERROR);
      }
      code[code_size]=0;
      for(unsigned char* code_ptr=code;*code_ptr;++code_ptr){
      switch(*code_ptr){
      case '>':
      ++tape_ptr;
      break;
      case '<':
      --tape_ptr;
      break;
      case '+':
      ++*tape_ptr;
      break;
      case '-':
      --*tape_ptr;
      break;
      case ',':
      *tape_ptr=(unsigned char)getchar();
      break;
      case '.':
      putchar(*tape_ptr);
      fflush(stdout);
      break;
      case '[':
      if(!*tape_ptr){
      int loop=1;
      while(loop){
      ++code_ptr;
      if(*code_ptr=='[')
      ++loop;
      if(*code_ptr==']')
      --loop;
      }
      }
      break;
      case ']':
      if(*tape_ptr){
      int loop=1;
      while(loop){
      --code_ptr;
      if(*code_ptr=='[')
      --loop;
      if(*code_ptr==']')
      ++loop;
      }
      }
      break;
      }
      }
      free(tape);
      free(code);
      }









      share|improve this question









      $endgroup$




      I created my brainfuck interpreter and I would like to know what can be done better and is the code clear and readable. I will be very thankful for opinions and suggestions.
      Here is the code:



      #include <stdio.h>
      #include <stdlib.h>

      #define ALLOCATION_ERROR 1
      #define FILE_ERROR 2
      #define OTHER_ERROR 3
      #define TAPE_SIZE 30000

      int main(int argc,char** argv){
      if(argc!=2){
      puts("Usage: main <source>");
      exit(OTHER_ERROR);
      }
      unsigned char* tape=calloc(TAPE_SIZE,1);
      if(tape==NULL){
      fprintf(stderr,"Fatal: failed to allocate %zu bytes.n",(size_t)TAPE_SIZE*1);
      exit(ALLOCATION_ERROR);
      }
      unsigned char* tape_ptr=tape;
      FILE* input_file=fopen(argv[1],"rb");
      if(input_file==NULL){
      fprintf(stderr,"Error: failed to open file %sn",argv[1]);
      exit(FILE_ERROR);
      }
      fseek(input_file,0,SEEK_END);
      size_t code_size=(size_t)ftell(input_file);
      fseek(input_file,0,SEEK_SET);
      unsigned char* code=malloc(code_size+1);
      if(code==NULL){
      fprintf(stderr,"Fatal: failed to allocate %zu bytes.n",code_size+1);
      exit(ALLOCATION_ERROR);
      }
      if(fread(code,1,code_size,input_file)!=code_size){
      fprintf(stderr,"Error: failed to read from file %sn",argv[1]);
      exit(FILE_ERROR);
      }
      code[code_size]=0;
      for(unsigned char* code_ptr=code;*code_ptr;++code_ptr){
      switch(*code_ptr){
      case '>':
      ++tape_ptr;
      break;
      case '<':
      --tape_ptr;
      break;
      case '+':
      ++*tape_ptr;
      break;
      case '-':
      --*tape_ptr;
      break;
      case ',':
      *tape_ptr=(unsigned char)getchar();
      break;
      case '.':
      putchar(*tape_ptr);
      fflush(stdout);
      break;
      case '[':
      if(!*tape_ptr){
      int loop=1;
      while(loop){
      ++code_ptr;
      if(*code_ptr=='[')
      ++loop;
      if(*code_ptr==']')
      --loop;
      }
      }
      break;
      case ']':
      if(*tape_ptr){
      int loop=1;
      while(loop){
      --code_ptr;
      if(*code_ptr=='[')
      --loop;
      if(*code_ptr==']')
      ++loop;
      }
      }
      break;
      }
      }
      free(tape);
      free(code);
      }






      c interpreter brainfuck






      share|improve this question













      share|improve this question











      share|improve this question




      share|improve this question










      asked 10 mins ago









      DeBos99DeBos99

      315




      315






















          0






          active

          oldest

          votes











          Your Answer





          StackExchange.ifUsing("editor", function () {
          return StackExchange.using("mathjaxEditing", function () {
          StackExchange.MarkdownEditor.creationCallbacks.add(function (editor, postfix) {
          StackExchange.mathjaxEditing.prepareWmdForMathJax(editor, postfix, [["\$", "\$"]]);
          });
          });
          }, "mathjax-editing");

          StackExchange.ifUsing("editor", function () {
          StackExchange.using("externalEditor", function () {
          StackExchange.using("snippets", function () {
          StackExchange.snippets.init();
          });
          });
          }, "code-snippets");

          StackExchange.ready(function() {
          var channelOptions = {
          tags: "".split(" "),
          id: "196"
          };
          initTagRenderer("".split(" "), "".split(" "), channelOptions);

          StackExchange.using("externalEditor", function() {
          // Have to fire editor after snippets, if snippets enabled
          if (StackExchange.settings.snippets.snippetsEnabled) {
          StackExchange.using("snippets", function() {
          createEditor();
          });
          }
          else {
          createEditor();
          }
          });

          function createEditor() {
          StackExchange.prepareEditor({
          heartbeatType: 'answer',
          autoActivateHeartbeat: false,
          convertImagesToLinks: false,
          noModals: true,
          showLowRepImageUploadWarning: true,
          reputationToPostImages: null,
          bindNavPrevention: true,
          postfix: "",
          imageUploader: {
          brandingHtml: "Powered by u003ca class="icon-imgur-white" href="https://imgur.com/"u003eu003c/au003e",
          contentPolicyHtml: "User contributions licensed under u003ca href="https://creativecommons.org/licenses/by-sa/3.0/"u003ecc by-sa 3.0 with attribution requiredu003c/au003e u003ca href="https://stackoverflow.com/legal/content-policy"u003e(content policy)u003c/au003e",
          allowUrls: true
          },
          onDemand: true,
          discardSelector: ".discard-answer"
          ,immediatelyShowMarkdownHelp:true
          });


          }
          });














          draft saved

          draft discarded


















          StackExchange.ready(
          function () {
          StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fcodereview.stackexchange.com%2fquestions%2f216381%2fbrainfuck-interpreter-in-c-3%23new-answer', 'question_page');
          }
          );

          Post as a guest















          Required, but never shown

























          0






          active

          oldest

          votes








          0






          active

          oldest

          votes









          active

          oldest

          votes






          active

          oldest

          votes
















          draft saved

          draft discarded




















































          Thanks for contributing an answer to Code Review Stack Exchange!


          • Please be sure to answer the question. Provide details and share your research!

          But avoid



          • Asking for help, clarification, or responding to other answers.

          • Making statements based on opinion; back them up with references or personal experience.


          Use MathJax to format equations. MathJax reference.


          To learn more, see our tips on writing great answers.




          draft saved


          draft discarded














          StackExchange.ready(
          function () {
          StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fcodereview.stackexchange.com%2fquestions%2f216381%2fbrainfuck-interpreter-in-c-3%23new-answer', 'question_page');
          }
          );

          Post as a guest















          Required, but never shown





















































          Required, but never shown














          Required, but never shown












          Required, but never shown







          Required, but never shown

































          Required, but never shown














          Required, but never shown












          Required, but never shown







          Required, but never shown







          Popular posts from this blog

          Webac Holding Inhaltsverzeichnis Geschichte | Organisationsstruktur | Tochterfirmen |...

          What's the meaning of a knight fighting a snail in medieval book illustrations?What is the meaning of a glove...

          Salamanca Inhaltsverzeichnis Lage und Klima | Bevölkerungsentwicklung | Geschichte | Kultur und...