Арифметические выражения

[oleg-w570]

No description provided.

[annapirova]

Олег, выглядит все аккуратно, но слишком мало тестов. Мне нужны не "тесты ради тестов", а гарантия того, что программа работает в разных случаях. Предлагаю добавить: - больше тестов на проверки (те же скобки) - тесты на то, что корректно разбилось на лексемы для простых случаев тоже - тесты на конвертацию в ПЗ и вычисления для простых случаев. Тонкие места: - там, где вы идете по строке и проверяете i+3й, i+1й символ. Надо проверять, что хватит на это символов. Может, строка кончится раньше (4+si , например). - Get() стека должен возвращать ссылку на элемент (так в stl определено) сб, 13 нояб. 2021 г. в 17:20, oleg-w570 ***@***.***>:

…

------------------------------ You can view, comment on, or merge this pull request online at: #11 Commit Summary - 26dff32 <26dff32> stack + gtest_stack - 11ddd3d <11ddd3d> арифметические выражения - b021d89 <b021d89> +натуральный логарифм - ede7bca <ede7bca> +гугл тесты File Changes (10 files <https://github.com/annapirova/mp2-lab3-arithmetic/pull/11/files>) - *M* include/arithmetic.h <https://github.com/annapirova/mp2-lab3-arithmetic/pull/11/files#diff-e7c8637dfd5658e9b657cdea238fc5e9989ca73980c75d8865c853a5f3077c18> (63) - *M* include/stack.h <https://github.com/annapirova/mp2-lab3-arithmetic/pull/11/files#diff-70dfe9a90569d6c8b418862d14aaef54d7dd654f0becd5f205566aa966342fb5> (78) - *M* samples/main_arithmetic.cpp <https://github.com/annapirova/mp2-lab3-arithmetic/pull/11/files#diff-3687db899db21cb258d39f63f9d1bcb7809eb7fe02b09cb5fcec43e84bc8e764> (8) - *M* sln/vc12/arithmetic/arithmetic.vcxproj <https://github.com/annapirova/mp2-lab3-arithmetic/pull/11/files#diff-87418effb3cb1abae75a58704cb41d8c334763ecff656498a6ba70966917abc0> (4) - *M* sln/vc12/gtest/gtest.vcxproj <https://github.com/annapirova/mp2-lab3-arithmetic/pull/11/files#diff-49386708c720e3a11e61753d95ec718cfb72d7c7f1d85a913f26793baa59a93e> (4) - *M* sln/vc12/sample/sample.vcxproj <https://github.com/annapirova/mp2-lab3-arithmetic/pull/11/files#diff-30aac2d01463c0e428f4af4a7c69644adb3ccd9c0721d5dede1c92353daea71f> (4) - *M* sln/vc12/tests/tests.vcxproj <https://github.com/annapirova/mp2-lab3-arithmetic/pull/11/files#diff-ec6319649a60c67bd96b015b6ca14c6436d961bf10e4657d7f5e6ed54cc79456> (4) - *M* src/arithmetic.cpp <https://github.com/annapirova/mp2-lab3-arithmetic/pull/11/files#diff-e68be49c27f3a5504e353e4b42da341bf4486a813f42bf594800fbeb5c58c4dd> (366) - *M* test/test_arithmetic.cpp <https://github.com/annapirova/mp2-lab3-arithmetic/pull/11/files#diff-a930d4165045a08c64724d204beb129005969521e73d6ec0024925427b4aa5f7> (123) - *M* test/test_stack.cpp <https://github.com/annapirova/mp2-lab3-arithmetic/pull/11/files#diff-28e83ea5ff40dd77d94f0369c0e94436ee4834bd32d18972491930af8a09b3f5> (43) Patch Links: - https://github.com/annapirova/mp2-lab3-arithmetic/pull/11.patch - https://github.com/annapirova/mp2-lab3-arithmetic/pull/11.diff — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#11>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ADL2AEIDL2QBVBLX4OVS2VTULZXZFANCNFSM5H6V6ZQA> .

-- С уважением, Анна Пирова

[oleg-w570]

По поводу проверки по строке на i+1й, i+2й и i+3й символы: я сначала тоже думал, что требуется доп проверка на конец строки, но сколько ни пробовал без неё ни разу программа не падала и ошибок не выдавала, только по какой причине я так до конца и не понял. Возможно ли, что последний элемент строки \0 (?) так вот с ним идёт сравнение -> получается false, и дальше проверка не продолжается?

[annapirova]

да, может быть, вы правы насчет проверки. Но канонически компилятор не гарантирует, что выражения в if будут проверены в том порядке, как написаны (то есть сначала i+1, потом i+2 и т.д.). Теоретически могут быть проверены в произвольном порядке. Тесты приняты. пт, 26 нояб. 2021 г. в 20:31, oleg-w570 ***@***.***>:

…

По поводу проверки по строке на i+1й, i+2й и i+3й символы: я сначала тоже думал, что требуется доп проверка на конец строки, но сколько ни пробовал без неё ни разу программа не падала и ошибок не выдавала, только по какой причине я так до конца и не понял. Возможно ли, что последний элемент строки \0 (?) так вот с ним идёт сравнение -> получается false, и дальше проверка не продолжается? — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#11 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ADL2AEKRQSQYCAS3I2DBQY3UN7AA7ANCNFSM5H6V6ZQA> .

-- С уважением, Анна Пирова