Conversation
shannami
force-pushed
the
homework5Task2
branch
from
6b2db21 to
9103dd9
Compare
There was a problem hiding this comment.
Интересно, а здесь CMake файлов нет (да и не надо уже, наверное, сделаете отдельный PR для нужной домашки)
There was a problem hiding this comment.
Претензии к стеку такие же, как и в соседнем PR
| return c == '+' || c == '-' || c == '*' || c == '/'; | ||
| } | ||
|
|
||
| int isDigit(char c) |
There was a problem hiding this comment.
src/stack/task2.c
Outdated
| void infixToPostfix(char* infix, char* postfix) | ||
| { | ||
| struct Stack st = new(); | ||
| int rpn = 0; |
There was a problem hiding this comment.
Я абсолютно без идей, что это значит
src/stack/task2.c
Outdated
| return c >= '0' && c <= '9'; | ||
| } | ||
|
|
||
| void infixToPostfix(char* infix, char* postfix) |
There was a problem hiding this comment.
Идея с тем, чтобы заставить пользователя самому выделять выходную строку имеет место быть, но в таком случае нужно описать, сколько должен выделить пользователь, поскольку на самом деле это не очевидно. Кажется, что верхняя оценка 2 * strlen(infix).
src/stack/task2.c
Outdated
| if (top == '(') | ||
| break; |
There was a problem hiding this comment.
Кажется, здесь происходит какая-то ошибка, но пользователь об этом ни сном, ни духом :(
| postfix[rpn] = '\0'; | ||
| } |
There was a problem hiding this comment.
И возможно стоит ругаться на неподдерживаемые символы
WoWaster
left a comment
WoWaster
left a comment
There was a problem hiding this comment.
- Добавлять пробелы в вывод
- Сделать условие
ifболее явным. - Писать комментарии над
2 participants
Шалахина Анна дз 5.2