Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Lesson1part1 #114

Open
wants to merge 4 commits into
base: AOsypchuk
Choose a base branch
from
Open

Conversation

AOsypchuk
Copy link

created a game guess the number

@AOsypchuk AOsypchuk changed the base branch from master to AOsypchuk April 10, 2019 20:16
@galkinletter galkinletter added the ready The PR is ready for review label Apr 11, 2019
printf("enter number: ");
scanf("%i", &num);

if (num < 0 || num > 9)
Copy link

Choose a reason for hiding this comment

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

Please, check your code style. Braces have to be on the same line

if (num < 0 || num > 9)
{
printf("the number is bigger than 9 or smaller then 0.\n");
exit(1);
Copy link

Choose a reason for hiding this comment

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

exit(1) means that your program crashed.

@a-vodka a-vodka added changes requested The PR review revealed issues which should be fixed (set by reviewer). and removed ready The PR is ready for review labels Apr 25, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changes requested The PR review revealed issues which should be fixed (set by reviewer).
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants