ironhead03 / IT_lab

0 stars 0 forks source link

Notes #1

Open AntonC9018 opened 2 days ago

AntonC9018 commented 2 days ago

Добавь .gitignore для exe файлов. Удали имеющиеся exe файлы через git rm --cached.


https://github.com/ironhead03/IT_lab/blob/e58de0a088e7ccfa0afc5c59fa1ed5d8798484a7/IT_lab2_CPP/IT_lab2_CPP.cpp#L22

Комментов на одном языке достаточно, на каком мне не принципиально.


https://github.com/ironhead03/IT_lab/blob/e58de0a088e7ccfa0afc5c59fa1ed5d8798484a7/IT_lab2_CPP/IT_lab2_CPP.cpp#L53

"because pointer was set to point at ..." -- обычно так говорят.

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


https://github.com/ironhead03/IT_lab/blob/e58de0a088e7ccfa0afc5c59fa1ed5d8798484a7/IT_lab4_CPP/IT_lab4.3_CPP.cpp

Смени encoding на UTF-8.


https://github.com/ironhead03/IT_lab/blob/e58de0a088e7ccfa0afc5c59fa1ed5d8798484a7/IT_lab4_CPP/IT_lab4.2_CPP.cpp

Недоделано


https://github.com/ironhead03/IT_lab/blob/e58de0a088e7ccfa0afc5c59fa1ed5d8798484a7/IT_lab4_CPP/IT_lab4.3_CPP.cpp#L17-L18

Если хочешь максимально грамотно детектить оверфлоу, почитай инфу. Если нет, убери это.

Во время дебаггинга использовал -- молодец. Сейчас оно уже не надо.

https://stackoverflow.com/questions/1815367/catch-and-compute-overflow-during-multiplication-of-two-large-integers


https://github.com/ironhead03/IT_lab/blob/e58de0a088e7ccfa0afc5c59fa1ed5d8798484a7/IT_lab5_CPP/IT_lab5.1_CPP.cpp#L21-L23

??


https://github.com/ironhead03/IT_lab/blob/e58de0a088e7ccfa0afc5c59fa1ed5d8798484a7/IT_lab5_CPP/IT_lab5.2_CPP.cpp#L6

Возвращай std::string_view, все проверки наружу вынеси. Алгоритм должен для любой строки работать.


https://github.com/ironhead03/IT_lab/blob/e58de0a088e7ccfa0afc5c59fa1ed5d8798484a7/IT_lab5_CPP/IT_lab5.2_CPP.cpp#L10-L11

Убери дебаггинг если закончил функцию.


https://github.com/ironhead03/IT_lab/blob/e58de0a088e7ccfa0afc5c59fa1ed5d8798484a7/IT_lab5_CPP/IT_lab5.2_CPP.cpp#L19

Может просто isSpace = true? Зачем паззл нарочно делать?)


https://github.com/ironhead03/IT_lab/blob/e58de0a088e7ccfa0afc5c59fa1ed5d8798484a7/IT_lab5_CPP/IT_lab5.2_CPP.cpp#L24

Это должно возвращатся, а не печататься.

Сделай еще одну функцию для теста, а findSpace очисти от всего, в ней оставь только логику. Никакой печати в ней.

AntonC9018 commented 2 days ago

https://github.com/ironhead03/IT_lab/blob/e58de0a088e7ccfa0afc5c59fa1ed5d8798484a7/IT_lab6_CPP/IT_lab6.2_CPP/main.cpp#L5

Не поэтому. Потому что определения у функции нет. Пересмотри видео. Диаграммы есть в примерах на гитхабе у меня.


https://github.com/ironhead03/IT_lab/blob/e58de0a088e7ccfa0afc5c59fa1ed5d8798484a7/IT_lab6_CPP/IT_lab6.2_CPP/main.cpp#L6

А тут ты понимаешь? Как тогда в прошлом пункте вообще не из той оперы?


https://github.com/ironhead03/IT_lab/blob/e58de0a088e7ccfa0afc5c59fa1ed5d8798484a7/IT_lab6_CPP/IT_lab6.2_CPP/main.cpp#L7

Класс??


https://github.com/ironhead03/IT_lab/blob/e58de0a088e7ccfa0afc5c59fa1ed5d8798484a7/IT_lab6_CPP/IT_lab6.2_CPP/main.cpp#L8

Где ты тут класс увидел?


https://github.com/ironhead03/IT_lab/blob/e58de0a088e7ccfa0afc5c59fa1ed5d8798484a7/IT_lab6_CPP/IT_lab6.2_CPP/main.cpp#L9

А сколько появлений? Ответь конкретно на вопрос.