Подходящие фигурные скобки

Question

У меня сложное время, соответствующее фигурным скобкам в этом коде. Он продолжает указывать «else» без ошибки «if», но не уверен, как должен читаться синтаксис. Если кто-то может помочь, это будет оценено по достоинству. Вот код:

private void compileFactor() { 
     boolean its_a_variable = theInfo.isVar(); 
     if (isIdent(theToken)) { 
      String ident = theToken; 
      theToken = t.token();  // handles var and const cases! 

      IdentInfo theInfo = symTable.lookup(ident); 
     } 


      boolean its_a_variable = theInfo.isVar(); 
      int theAddr = theInfo.getAddr(); 
      boolean isGlobal = theInfo.getIsGlobal(); 
      int constValue = theInfo.getValue(); 

      if (its_a_variable) { // pld12: CHANGE THIS!! 
       int theAddr = theInfo.getAddr(); 
       boolean isGlobal = theInfo.getIsGlobal(); 
       if (theAddr == -1) t.error("undeclared identifier used in expr: "+ident); 
       if (isGlobal) cs.emit(Machine.LOAD, theAddr); 
       else cs.emit(Machine.LOADF, theAddr); 
      } else { 
       int constValue = theInfo.getValue(); 
       if (constValue = null) t.error("undeclared identifier used in expr: "+ident); 


       else { 
        cs.emitLOADI(theNumber); 
       } 


      else if (isNumber(theToken)) { 
       int theNumber = new Integer(theToken).intValue(); 
       cs.emitLOADINT(theNumber); 
       theToken = t.token(); 
      } 
      else if (equals(theToken, "(")) { // nothing to do to generate code! 
       accept("("); 
       compileExpr(); 
       accept(")"); 
      } 


     } 

Answer 1

Это else if происходит после else:

 else if (isNumber(theToken)) { 
      int theNumber = new Integer(theToken).intValue(); 
      cs.emitLOADINT(theNumber); 
      theToken = t.token(); 
     } 

Edit: фиксированные и отступы

private void compileFactor() { 
    if (isIdent(theToken)) { 
     String ident = theToken; 
     theToken = t.token();  // handles var and const cases! 

     IdentInfo theInfo = symTable.lookup(ident); 

     boolean its_a_variable = theInfo.isVar(); 
     int theAddr = theInfo.getAddr(); 
     boolean isGlobal = theInfo.getIsGlobal(); 
     int constValue = theInfo.getValue(); 

     if (its_a_variable) { // pld12: CHANGE THIS!! 
      int theAddr = theInfo.getAddr(); 
      boolean isGlobal = theInfo.getIsGlobal(); 
      if (theAddr == -1) t.error("undeclared identifier used in expr: "+ident); 
      if (isGlobal) cs.emit(Machine.LOAD, theAddr); 
      else cs.emit(Machine.LOADF, theAddr); 
     } else { 
      int constValue = theInfo.getValue(); 
      if (constValue = null) { 
       t.error("undeclared identifier used in expr: "+ident); 
      } else { 
       cs.emitLOADI(theNumber); 
      } 
     } 
    } else if (isNumber(theToken)) { 
     int theNumber = new Integer(theToken).intValue(); 
     cs.emitLOADINT(theNumber); 
     theToken = t.token(); 
    } else if (equals(theToken, "(")) { // nothing to do to generate code! 
     accept("("); 
     compileExpr(); 
     accept(")"); 
    } 
} 

Answer 2

Я подозреваю, что это ваша проблема.

  if (constValue = null) t.error("undeclared identifier used in expr: "+ident); 


      else { 
       cs.emitLOADI(theNumber); 
      } 

EDIT Если это будет?

  if (constValue = null) { 
       t.error("undeclared identifier used in expr: "+ident); 
      } else { 
       cs.emitLOADI(theNumber); 
      } 

Answer 3

бы предложить, если у вас возникли проблемы, соответствующие фигурные скобки, начать новое определение функции, только начните с добавления потока управления и ветвления (ifs, elses, ...) и используйте фигурные скобки на каждом блоке if/else if/else - даже на одном лайнере.

Когда вы получите это право, добавьте остальные функции.
Когда брекеты становятся без проблем, начните пропускать их для одного лайнера.

Answer 4

Несколько блокпостов

Я думаю, вам нужно будет закрыть скобки перед тем IsNumber (theToken)

}} else if (isNumber(theToken)) { 

и могут быть вам не нужна скобка после этой строки?

IdentInfo theInfo = symTable.lookup(ident); 
} 

Может быть, весь код такой?

private void compileFactor() { 
    boolean its_a_variable = theInfo.isVar(); 
    if (isIdent(theToken)) { 
     String ident = theToken; 
     theToken = t.token();  // handles var and const cases! 
     IdentInfo theInfo = symTable.lookup(ident); 

     boolean its_a_variable = theInfo.isVar(); 
     int theAddr = theInfo.getAddr(); 
     boolean isGlobal = theInfo.getIsGlobal(); 
     int constValue = theInfo.getValue(); 

     if (its_a_variable) { // pld12: CHANGE THIS!! 
      int theAddr = theInfo.getAddr(); 
      boolean isGlobal = theInfo.getIsGlobal(); 
      if (theAddr == -1) t.error("undeclared identifier used in expr: "+ident); 
      if (isGlobal) cs.emit(Machine.LOAD, theAddr); 
      else cs.emit(Machine.LOADF, theAddr); 
     } else { 
      int constValue = theInfo.getValue(); 
      if (constValue = null) 
       t.error("undeclared identifier used in expr: "+ident); 
      else { 
        cs.emitLOADI(theNumber); 
      } 
     } 
    } else if (isNumber(theToken)) { 
     int theNumber = new Integer(theToken).intValue(); 
     cs.emitLOADINT(theNumber); 
     theToken = t.token(); 
    } else if (equals(theToken, "(")) { // nothing to do to generate code! 
     accept("("); 
     compileExpr(); 
     accept(")"); 
    } 
} 

Answer 5

Я не хочу, чтобы начать размещение скобки священной войны, но именно поэтому я предпочитаю «открывающую скобку в следующей строке» идиоме. Я могу визуально отобразить открывающую и закрывающую фигурные скобки, даже если у меня нет IDE, чтобы помочь мне.

С учетом этого, IDE - ваш лучший друг здесь.

Я хотел рекомендацию немедленно при добавлении открытия и закрытия скобки к блоку перед заполнением его.

Сторонясь блоками одной строки без скобок помогут. На это не так много усилий добавить.

Answer 6

Это то, что структура выглядит следующим образом:

private void compileFactor() { 
    if (...) { 
    ... 
    } 
    if (...) { 
    ... 
    if (...) ... 
    if (...) ... 
    else ... 
    } else { 
    if (...) ... 
    else { 
     ... 
* } else if (...) { 
     ... 
    } else if (...) { 
     ... 
    } 
    } 

Где я поставил *, вы имеете еще следуя за другой. Просто добавив еще одну скобку до этого момента, она не исправит ее, так как это добавит else после другого.

Итак, в ваших условиях есть что-то принципиально неправильное, и вы в значительной степени должны переосмыслить его с самого начала.

Добавить скобки во всех операторах if, чтобы вы были уверены, что они действительно начинаются и заканчиваются там, где вы их ожидаете.

Answer 7

Если вы используете Visual Studio, вы можете использовать + «]» клавиши CTRL, чтобы соответствовать брекеты

Answer 8

Попробуйте комментируя следующий код:

function blah(){ //start function 
    if(...){  //open if 1 
    for(...){ //open for 

     //do something  

    }//close for 
    } //close if 1 
}//close function 

Я делаю это в Flash, хотя в некоторых langueages это пугает вещи.

Удачи вам!

Answer 9

Отмеченный код имеет 6 '}' и 7 '{'.

Последние «{» утверждение следующей выдержки является тот, который, как представляется, не имеет соответствия «}»

  else cs.emit(Machine.LOADF, theAddr); 
    } else {