Skip to content

1lab - Calc#1

Open
sirgay1 wants to merge 22 commits into
mainfrom
1lab
Open

1lab - Calc#1
sirgay1 wants to merge 22 commits into
mainfrom
1lab

Conversation

@sirgay1

@sirgay1 sirgay1 commented Oct 31, 2025

Copy link
Copy Markdown
Owner

No description provided.

@azya0

azya0 commented Nov 22, 2025

Copy link
Copy Markdown

При написание README.md файла, как можно догадаться из расширения, используется синтаксис Markdown. То, что написано у вас просто нечитаемо.

При этом я не понимаю, как собрать и скомпилировать вашу программу.

Ну просто запустите .exe файл и проверяйте

Не сработает. Мы учимся работать на c++, а не x86_win11 исполняемых файлах (так же я не могу таким образом гарантировать, что исполняемый файл ваш).

Вообще, если у вас есть желание приложить скомпилированную программу к вашему github-репозиторию, то для этого есть релизы. .exe файлу не место в исходном коде проекта. (Но это так, лирическое отступление, а не решение вашей проблемы)

Пока что 0. Если/когда захотите исправить, пришлите на почту новое письмо

@azya0

azya0 commented Nov 23, 2025

Copy link
Copy Markdown

Сергей, вы на 3 курсе ПМИ. Когда вы вручную компилировали исполняемый файл, а потом и .dll к нему, вас не посетила мысль, что вы делаете что-то неправильно? Пора уже учиться работать с системами сборки, по типу cmake.

О том, как вы собрались запускать main.cpp даже спрашивать неудобно

Ну и как вишенка на торте: при запуске вашей команды из README.md, она просто не работает

image

Пожалуйста, убедитесь, что у вас всё работает, прежде чем отправить всё на повторную проверку

@azya0

azya0 commented Dec 18, 2025

Copy link
Copy Markdown

Из плюсов: проект +- работает, если не вдаваться в детали
Из минусов: архитектура ужасна, расширять/рефакторить проект невозможно. 2 унарных плагина было реализовано

Как вишенка на торте:

pos++;
    switch (c) {
    case '+': return { TokenKind::Plus };
    case '-': return { TokenKind::Minus };
    case '*': return { TokenKind::Mul };
    case '/': return { TokenKind::Div };
    case '(': return { TokenKind::LParen };
    case ')': return { TokenKind::RParen };
    case ',': return { TokenKind::Comma };

10 баллов без права пересдачи

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants