Быки и коровы, не правильно считает их

@TBIKC · Регистрация: 23.09.2010

Студворк — интернет-сервис помощи студентам

Не правильно считает быков и коров, помогите пожалуйста

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
#include <iostream>
#include <locale.h>
#include <cstdlib> // rand()
#include <ctime>
using namespace std;
int main()
{
    setlocale(0,"Russian");
    cout << endl;
 
    int ox = 0;
    int cow = 0;
    int intForRand;
    int randArray[3];
    int ourArray[3];
 
    srand(time(0)); 
 
    for(int i = 1;i<5;i++){
        do
        {intForRand = rand() % 9;
        randArray[i] = intForRand;}
        while(randArray[1] == 0);
 
        for(int j=1;j<4;j++){
           if(randArray[i] == randArray[j] && i != j )
           {                                    
                   intForRand = rand() % 9;
                   randArray[i] = intForRand;
           }
        }
    }
    for (int i=1;i<5;i++)
        cout << randArray[i];
 
    cout << endl << "BBedite 4isla" << endl;
    for(int j = 1;j<5;j++){
        cin >> ourArray[j];
        cout << endl;
 
        if((ourArray[j]<=9) && (ourArray[j]>=0) && (ourArray[1] != 0)){
            for(int i = 1;i<5;i++){
            if (randArray[i] == ourArray[j])
            {
            if ( i == j )
                ++ox;
            else
                ++cow;
            }
        }
        }
    }
    cout << "Byk: " << ox << endl;
    cout << "KoroB: " << cow << endl;
 
    if(ox == 4)
        cout << "You win!" << endl;
    else
        cout << "You lose!" << endl;
 
return 0;
}

Условия задачи, кто первый раз слышит
Быки и Коровы: простая, но увлекательная логическая игра.
Цель:отгадать число, состоящее из 4-х неповторяющихся цифр, которое автоматически создается при загрузке игры.
Правила: на цифровой клавиатуре вводится число из 4-х неповторяющихся цифр (впрочем, скрипт не позволит ввести одинаковые цифры). Результат отгадывания выражается в условных единицах - Быках и Коровах.
Бык - цифра входит в число и стоит на своем месте. Корова - цифра входит в число, но находится не на своем месте.

@Toshkarik · 25.02.2012, 09:55

Сообщение от TBIKC

C++
1
2
3
4
do
* * * * {intForRand = rand() % 9;
* * * * randArray[i] = intForRand;}
* * * * while(randArray[1] == 0);

Что это? Если должно быть все таки randArray[i] == 0, то советую почитать о бесконечной отсрочки. И еще - число четырехзначное, почему у массивов размер 3? Дальше даже не смотрел...

@TBIKC · 25.02.2012, 10:01 **[ТС]**

Потому что индексация с нуля начинается

Это код который генерирует число до тех пор пока 1 не будет равно 0, в нём проблем нету,

C++
1
2
3
4
       do
        {intForRand = rand() % 9;
        randArray[i] = intForRand;}
        while(randArray[1] == 0);

Только вот в 26 строки там j<5 должно быть

@Toshkarik · 25.02.2012, 10:07

Причем здесь индексация? Вы объявляете массив из 3 элементов.

Сообщение от TBIKC

Это код который генерирует число до тех пор пока 1 не будет равно 0, в нём проблем нету

Это как? Оо
Этот цикл выполняется до тех пор пока элемент массива с индексом 1 ( второй элемент ) равен 0, в случае если он равен и при этом i != 1 то цикл бесконечен...

ValeryS · 25.02.2012, 10:12

Сообщение от Toshkarik

И еще - число четырехзначное, почему у массивов размер 3?

Сообщение от TBIKC

Потому что индексация с нуля начинается

при чем здесь индексация??? он тебе про размер

Сообщение от TBIKC

int randArray[3];

массив размером 3
индексы 0 1 2

Сообщение от TBIKC

intForRand = rand() % 9;

а как бы нам тут 9 получить??
и если отрицательно будет что делать???

@TBIKC · 25.02.2012, 10:15 **[ТС]**

вот это странности ни разу не было зацикливаний
Внимательнее 1 :
C++
1 2 3 4
do {intForRand = rand() % 9; randArray[i] = intForRand;} while(randArray[1] == 0);

Выполняется до тех пор пока 1 число в массиве = 0

Что вы к нему привязались, он нормально отрабатывает)
тут проблема в строчках 45-52

Добавлено через 1 минуту
Оно отрицательным не получиться

C++
1
    srand(time(0));

@Toshkarik · 25.02.2012, 10:15

Сообщение от ValeryS

и если отрицательно будет что делать???

Если вы про rand() то она не генерирует отрицательных чисел, а только от 0 до RAND_MAX

Вот накидал немного...

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
#include <iostream>
#include <cstdlib>
 
/*
 *
 */
int main( int argc, char** argv ) {
    const int numberSize = 4, digitsSize = 10;
    int digits[ digitsSize ] = { 0, 1, 2, 3, 4, 5, 6, 7, 8, 9 },
    number[ numberSize ] = { 0 },
    userNumber[ numberSize ] = { 0 },
    bull = 0, cow = 0;
 
    for ( int i = 0; i < digitsSize; i++ )
    std::swap( digits[ i ], digits[ std::rand() % 10 ] );
 
    for ( int i = 0; i < numberSize; i++ ) {
    number[ i ] = digits[ i + 3 ];
 
    std::cout << "Please enter " << i + 1 << " digit of number: ";
    std::cin >> userNumber[ i ];
    }
 
    for ( int i = 0; i < numberSize; i++ ) {
    for ( int j = i; j < numberSize; j++ )
        if ( number[ i ] == userNumber[ i ] )
        i == j ? bull++ : cow++;
    }
 
    if ( bull == 4 )
    std::cout << "Congratulations! You win!" << std::endl;
    else
    std::cout << "Sorry, try again." << std::endl;
 
    return 0;
}

@diagon · 25.02.2012, 10:16

Сообщение от TBIKC

Выполняется до тех пор пока 1 число в массиве = 0

Выполняется до тех пор, пока второй элемент массива равен нулю.

Сообщение от ValeryS

и если отрицательно будет что делать???

rand не возвращает отрицательные значения...

@Toshkarik · 25.02.2012, 10:18

Сообщение от TBIKC

Выполняется до тех пор пока 1 число в массиве = 0

Сами же говорили, что индексация с 0 и тут же называете элемент с индексом 1 как первый...

Не по теме:

PS: Только дошло - зачем коров считать? :D

@TBIKC · 25.02.2012, 10:19 **[ТС]**

Спасибо за прогу, но хотелось бы всё же морю исправить

Добавлено через 28 секунд

Сообщение от Toshkarik

Сами же говорили, что индексация с 0 и тут же называете элемент с индексом 1 как первый...

Я не говорил такого, я сказал первое число

@Toshkarik · 25.02.2012, 10:20

Это второе число, первое числа, а точнее цифра, находится в randArray[ 0 ];

ValeryS · 25.02.2012, 10:30

Сообщение от TBIKC

Оно отрицательным не получиться

Сообщение от Toshkarik

Если вы про rand() то она не генерирует отрицательных чисел, а только от 0 до RAND_MAX

Пардон увлекся

но второй вопрос это не снимает

Сообщение от TBIKC

intForRand = rand() % 9;

будет 0 1 2 3 4 5 6 7 8

Сообщение от TBIKC

вот это странности ни разу не было зацикливаний

и не будет

только работает он не так как задумал(найдешь ошибку или подсказать?)
кстати почему 1( а не 0)

Сообщение от TBIKC

int randArray[3];

Сообщение от TBIKC

for(int i = 1;i<5;i++){

Сообщение от TBIKC

randArray[i] = intForRand;}

Как эти строки уживаются между собой?

@TBIKC · 26.02.2012, 14:43 **[ТС]**

сам не представляю)

@TBIKC · 27.02.2012, 16:06 **[ТС]**

Помогите пожалуйста, переделать

Jupiter · 27.02.2012, 16:20

Быки и коровы

@TBIKC · 27.02.2012, 16:26 **[ТС]**

я умею пользоваться поиском )
просто хочу чтобы изменить в моей программе код, а не новый ...

@dimcoder · 27.02.2012, 16:35

Решил немного покоментить код:

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
#include <iostream>
#include <locale.h>
#include <cstdlib> // rand()
#include <ctime>
using namespace std;  
int main()
{
    setlocale(0,"Russian");
    cout << endl;
 
    int ox = 0;
    int cow = 0;
    int intForRand;
    int randArray[3];//У вас что 3значное число? int randArray[4] надо писать
    int ourArray[3];//то же самое
 
    srand(time(0)); 
 
    for(int i = 1;i<5;i++){//индексацию надо с нуля начинать. И до 4, а не до 5.
        do
        {intForRand = rand() % 9;//обяхяательно заводить для этого переменную. А если первое число == 0?
        randArray[i] = intForRand;}
        while(randArray[1] == 0);//Не въехал зачем
 
        for(int j=1;j<4;j++){//опять индексация с 1
           if(randArray[i] == randArray[j] && i != j )//
           {                                    
                   intForRand = rand() % 9;//уже комментил
                   randArray[i] = intForRand;//
           }
        }
        }
        for (int i=1;i<5;i++)//уже комментил
            cout << randArray[i];
 
        cout << endl << "BBedite 4isla" << endl;
    for(int j = 1;j<5;j++){//комментил
        cin >> ourArray[j];//а где проверка? Здесь что-то типа гетчара нужен
        cout << endl;
 
        if((ourArray[j]<=9) && (ourArray[j]>=0) && (ourArray[1] != 0)){//не въехал
            for(int i = 1;i<5;i++){//комментил
                    if (randArray[i] == ourArray[j])//
                    {
                        if ( i == j )
                                ++ox;
                        else
                                ++cow;
                    }
            }
        }
    }
    cout << "Byk: " << ox << endl;//Ну разве что здесь без косяков :))))
    cout << "KoroB: " << cow << endl;
 
    if(ox == 4)
        cout << "You win!" << endl;
    else
        cout << "You lose!" << endl;
 
return 0;
}

А заполнить "наше" число так нельзя:

C++
1
2
3
4
5
a[0] = rand() % 9 + 1;
a[1] = rand() % 10;
a[2] = rand() % 10;
a[3] = rand() % 10;
//Take it simple

@TBIKC · 27.02.2012, 21:29 **[ТС]**

C++
1
2
3
        {intForRand = rand() % 9;//обяхяательно заводить для этого переменную. А если первое число == 0?
        randArray[i] = intForRand;}
        while(randArray[1] == 0);//Не въехал зачем

для проверки, чтобы первое число было не 0

Добавлено через 4 часа 51 минуту
Написал так, повторяются числа ... в сгенерированном коде

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
#include <iostream>
#include <locale.h>
#include <cstdlib> // rand()
#include <ctime>
using namespace std;
int main()
{
    setlocale(0,"Russian");
    cout << endl;
 
    int ox = 0;
    int cow = 0;
    int intForRand;
    int randArray[4];
    int ourArray[4];
 
    srand(time(0)); 
    randArray[0] = rand() % 9 + 1;
    randArray[1] = rand() % 10;
    randArray[2] = rand() % 10;
    randArray[3] = rand() % 10;
 
    for (int i=0;i<4;i++)
        cout << randArray[i];
 
    cout << endl << "BBedite 4isla" << endl;
    for(int j = 0;j<4;j++){
        cin >> ourArray[j];
        cout << endl;
 
        if((ourArray[j]<=9) && (ourArray[j]>=0) && (ourArray[0] != 0)){
            for(int i = 0;i<4;i++){
            if (randArray[i] == ourArray[j])
            {
            if ( i == j )
                ++ox;
            else
                ++cow;
            }
        }
        }
    }
    cout << "Byk: " << ox << endl;
    cout << "KoroB: " << cow << endl;
 
    if(ox == 4)
        cout << "You win!" << endl;
    else
        cout << "You lose!" << endl;
 
return 0;
}

@TBIKC · 28.03.2012, 08:22 **[ТС]**

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

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
#include <iostream>
#include <cstdlib> // rand()
#include <ctime>
using namespace std;  
int main()
{
    cout << endl;
 
    int ox = 0;
    int cow = 0;
    int intForRand;
    int randArray[4];
    int ourArray[4];
 
    srand(time(0)); 
 
    for(int i = 0;i<4;i++){
        do
        {intForRand = rand() % 9;
        randArray[i] = intForRand;}
        while(randArray[1] == 0);
 
        for(int j=0;j<4;j++){
           if(randArray[i] == randArray[j] && i != j )//
           {                                    
                   intForRand = rand() % 9;
                   randArray[i] = intForRand;
           }
        }
        }
        for (int i=0;i<4;i++)
            cout << randArray[i];
 
        cout << endl << "BBedite 4isla" << endl;
    for(int j = 0;j<4;j++){
        cin >> ourArray[j];
        cout << endl;
 
        if((ourArray[j]<=9) && (ourArray[j]>=0) && (ourArray[1] != 0)){
            for(int i = 0;i<4;i++){
                    if (randArray[i] == ourArray[j])//
                    {
                        if ( i == j )
                                ++ox;
                        else
                                ++cow;
                    }
            }
        }
    }
    cout << "Byk: " << ox << endl;
    cout << "KoroB: " << cow << endl;
 
    if(ox == 4)
        cout << "You win!" << endl;
    else
        cout << "You lose!" << endl;
 
return 0;
}

Новые блоги и статьи Все статьи Все блоги /
Реалии Hrethgir 01.03.2026 Нет, я не закончил до сих пор симулятор. Эта задача сложнее. Не получилось уйти в плавсостав, но оно и к лучшему, возможно. Точнее получалось - но сварщиком в палубную команду, а это значит, в моём. . .	Ритм жизни kumehtar 27.02.2026 Иногда приходится жить в ритме, где дел становится всё больше, а вовлечения в происходящее — всё меньше. Плотный график не даёт вниманию закрепиться ни на одном событии. Утро начинается с быстрых,. . .	SDL3 для Web (WebAssembly): Сборка библиотек: SDL3, Box2D, FreeType, SDL3_ttf, SDL3_mixer и SDL3_image из исходников с помощью CMake и Emscripten 8Observer8 27.02.2026 Недавно вышла версия 3. 4. 2 библиотеки SDL3. На странице официальной релиза доступны исходники, готовые DLL (для x86, x64, arm64), а также библиотеки для разработки под Android, MinGW и Visual Studio. . . .	SDL3 для Web (WebAssembly): Реализация движения на Box2D v3 - трение и коллизии с повёрнутыми стенами 8Observer8 20.02.2026 Содержание блога Box2D позволяет легко создать главного героя, который не проходит сквозь стены и перемещается с заданным трением о препятствия, которые можно располагать под углом, как верхнее. . .
Конвертировать закладки radiotray-ng в m3u-плейлист damix 19.02.2026 Это можно сделать скриптом для PowerShell. Использование . \СonvertRadiotrayToM3U. ps1 <path_to_bookmarks. json> Рядом с файлом bookmarks. json появится файл bookmarks. m3u с результатом. # Check if. . .	Семь CDC на одном интерфейсе: 5 U[S]ARTов, 1 CAN и 1 SSI Eddy_Em 18.02.2026 Постепенно допиливаю свою "многоинтерфейсную плату". Выглядит вот так: https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11617&stc=1&d=1771445347 Основана на STM32F303RBT6. На борту пять. . .	Камера Toupcam IUA500KMA Eddy_Em 12.02.2026 Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .	И ясному Солнцу zbw 12.02.2026 И ясному Солнцу, и светлой Луне. В мире покоя нет и люди не могут жить в тишине. А жить им немного лет.

@Toshkarik 1181 / 894 / 94 Регистрация: 03.08.2011 Сообщений: 2,461
	25.02.2012, 10:20
	Это второе число, первое числа, а точнее цифра, находится в randArray[ 0 ]; 0

@TBIKC web dev 107 / 93 / 9 Регистрация: 23.09.2010 Сообщений: 737
	26.02.2012, 14:43 [ТС]
	сам не представляю) 0

@TBIKC web dev 107 / 93 / 9 Регистрация: 23.09.2010 Сообщений: 737
	27.02.2012, 16:06 [ТС]
	Помогите пожалуйста, переделать 0

Jupiter Каратель 6610 / 4029 / 401 Регистрация: 26.03.2010 Сообщений: 9,273 Записей в блоге: 1
	27.02.2012, 16:20
	Быки и коровы 0

@TBIKC web dev 107 / 93 / 9 Регистрация: 23.09.2010 Сообщений: 737
	27.02.2012, 16:26 [ТС]
	я умею пользоваться поиском ) просто хочу чтобы изменить в моей программе код, а не новый ... 0