Skip to content

Кутузова Анна Александровна Фт-320010#43

Open
kutuzova-ann wants to merge 2 commits intocppdevcourse:masterfrom
kutuzova-ann:master
Open

Кутузова Анна Александровна Фт-320010#43
kutuzova-ann wants to merge 2 commits intocppdevcourse:masterfrom
kutuzova-ann:master

Conversation

@kutuzova-ann
Copy link
Copy Markdown

Кутузова А.А. Фт-320010
added src/two-sum.cpp

added src/two-sum.cpp
added src/two-sum.cpp
Copy link
Copy Markdown
Collaborator

@czertyaka czertyaka left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Склонировал проект, переименовал файл srctwo-sum.cpp -> src/two-sum.cpp, тест прошел. Задание можете считать принятым, исправление на ваше усмотрение)

Comment thread srctwo-sum.cpp
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Файл назван srctwo-sum.cpp, а не two-sum.cpp. И находится в корневой директории проекта, а не в src/. Из-за этого не собираются тесты

Как поправить:

$ git mv srctwo-sum.cpp src/two-sum.cpp
$ git commit -m "moved file"
$ git push

Comment thread srctwo-sum.cpp
const int target,
std::size_t& index0,
std::size_t& index1
)
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

С самим кодом все хорошо, но надо соблюдать последовательность в стиле)

Между сигнатурой функции и телом обычно не ставят новую строку, пробелы в for между выражениями где-то поставлены, а где-то нет

Для домашнего задания в университете совсем не критично, но в реальных проектах обычно за этим следят

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