Brainfuck interpreter in C 3Brainfuck 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

What is the opposite of 'gravitas'?

How did Doctor Strange see the winning outcome in Avengers: Infinity War?

How to check is there any negative term in a large list?

Trouble understanding the speech of overseas colleagues

Unreliable Magic - Is it worth it?

Where does the Z80 processor start executing from?

Do all network devices need to make routing decisions, regardless of communication across networks or within a network?

Large drywall patch supports

How to pronounce the slash sign

How can I kill an app using Terminal?

How did Arya survive the stabbing?

Is HostGator storing my password in plaintext?

Does "every" first-order theory have a finitely axiomatizable conservative extension?

Implement the Thanos sorting algorithm

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

Proof of work - lottery approach

How does it work when somebody invests in my business?

System.debug(JSON.Serialize(o)) Not longer shows full string

Sequence of Tenses: Translating the subjunctive

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

Go Pregnant or Go Home

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

I'm in charge of equipment buying but no one's ever happy with what I choose. How to fix this?

when is out of tune ok?



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 11 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

          名間水力發電廠 目录 沿革 設施 鄰近設施 註釋 外部連結 导航菜单23°50′10″N 120°42′41″E / 23.83611°N 120.71139°E / 23.83611; 120.7113923°50′10″N 120°42′41″E / 23.83611°N 120.71139°E / 23.83611; 120.71139計畫概要原始内容臺灣第一座BOT 模式開發的水力發電廠-名間水力電廠名間水力發電廠 水利署首件BOT案原始内容《小檔案》名間電廠 首座BOT水力發電廠原始内容名間電廠BOT - 經濟部水利署中區水資源局

          Prove that NP is closed under karp reduction?Space(n) not closed under Karp reductions - what about NTime(n)?Class P is closed under rotation?Prove or disprove that $NL$ is closed under polynomial many-one reductions$mathbfNC_2$ is closed under log-space reductionOn Karp reductionwhen can I know if a class (complexity) is closed under reduction (cook/karp)Check if class $PSPACE$ is closed under polyonomially space reductionIs NPSPACE also closed under polynomial-time reduction and under log-space reduction?Prove PSPACE is closed under complement?Prove PSPACE is closed under union?

          Is my guitar’s action too high? Announcing the arrival of Valued Associate #679: Cesar Manara Planned maintenance scheduled April 23, 2019 at 23:30 UTC (7:30pm US/Eastern)Strings too stiff on a recently purchased acoustic guitar | Cort AD880CEIs the action of my guitar really high?Μy little finger is too weak to play guitarWith guitar, how long should I give my fingers to strengthen / callous?When playing a fret the guitar sounds mutedPlaying (Barre) chords up the guitar neckI think my guitar strings are wound too tight and I can't play barre chordsF barre chord on an SG guitarHow to find to the right strings of a barre chord by feel?High action on higher fret on my steel acoustic guitar