Форум программистов, компьютерный форум CyberForum.ru

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

Восстановить пароль Регистрация
 
Рейтинг: Рейтинг темы: голосов - 9, средняя оценка - 4.89
TBIKC
web dev
 Аватар для TBIKC
104 / 91 / 1
Регистрация: 23.09.2010
Сообщений: 738
25.02.2012, 09:27     Быки и коровы, не правильно считает их #1
Не правильно считает быков и коров, помогите пожалуйста
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-х неповторяющихся цифр (впрочем, скрипт не позволит ввести одинаковые цифры). Результат отгадывания выражается в условных единицах - Быках и Коровах.
Бык - цифра входит в число и стоит на своем месте. Корова - цифра входит в число, но находится не на своем месте.
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
25.02.2012, 09:27     Быки и коровы, не правильно считает их
Посмотрите здесь:

C++ [C++] подскажите Алгоритм игры "быки и коровы"
C++ Быки и коровы
Быки и коровы C++
C++ Быки и коровы. Комментарии к коду.
C++ Игра быки и коровы
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
Toshkarik
 Аватар для Toshkarik
1139 / 856 / 50
Регистрация: 03.08.2011
Сообщений: 2,381
Завершенные тесты: 1
25.02.2012, 09:55     Быки и коровы, не правильно считает их #2
Цитата Сообщение от TBIKC Посмотреть сообщение
C++
1
2
3
4
do
* * * * {intForRand = rand() % 9;
* * * * randArray[i] = intForRand;}
* * * * while(randArray[1] == 0);
Что это? Если должно быть все таки randArray[i] == 0, то советую почитать о бесконечной отсрочки. И еще - число четырехзначное, почему у массивов размер 3? Дальше даже не смотрел...
TBIKC
web dev
 Аватар для TBIKC
104 / 91 / 1
Регистрация: 23.09.2010
Сообщений: 738
25.02.2012, 10:01  [ТС]     Быки и коровы, не правильно считает их #3
Потому что индексация с нуля начинается

Это код который генерирует число до тех пор пока 1 не будет равно 0, в нём проблем нету,
C++
1
2
3
4
       do
        {intForRand = rand() % 9;
        randArray[i] = intForRand;}
        while(randArray[1] == 0);
Только вот в 26 строки там j<5 должно быть
Toshkarik
 Аватар для Toshkarik
1139 / 856 / 50
Регистрация: 03.08.2011
Сообщений: 2,381
Завершенные тесты: 1
25.02.2012, 10:07     Быки и коровы, не правильно считает их #4
Причем здесь индексация? Вы объявляете массив из 3 элементов.

Цитата Сообщение от TBIKC Посмотреть сообщение
Это код который генерирует число до тех пор пока 1 не будет равно 0, в нём проблем нету
Это как? Оо
Этот цикл выполняется до тех пор пока элемент массива с индексом 1 ( второй элемент ) равен 0, в случае если он равен и при этом i != 1 то цикл бесконечен...
ValeryS
Модератор
6377 / 4843 / 442
Регистрация: 14.02.2011
Сообщений: 16,048
25.02.2012, 10:12     Быки и коровы, не правильно считает их #5
Цитата Сообщение от Toshkarik Посмотреть сообщение
И еще - число четырехзначное, почему у массивов размер 3?
Цитата Сообщение от TBIKC Посмотреть сообщение
Потому что индексация с нуля начинается
при чем здесь индексация??? он тебе про размер
Цитата Сообщение от TBIKC Посмотреть сообщение
int randArray[3];
массив размером 3
индексы 0 1 2
Цитата Сообщение от TBIKC Посмотреть сообщение
intForRand = rand() % 9;
а как бы нам тут 9 получить??
и если отрицательно будет что делать???
TBIKC
web dev
 Аватар для TBIKC
104 / 91 / 1
Регистрация: 23.09.2010
Сообщений: 738
25.02.2012, 10:15  [ТС]     Быки и коровы, не правильно считает их #6
вот это странности ни разу не было зацикливаний
Внимательнее 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
 Аватар для Toshkarik
1139 / 856 / 50
Регистрация: 03.08.2011
Сообщений: 2,381
Завершенные тесты: 1
25.02.2012, 10:15     Быки и коровы, не правильно считает их #7
Цитата Сообщение от 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
Higher
 Аватар для diagon
1920 / 1186 / 49
Регистрация: 02.05.2010
Сообщений: 2,925
Записей в блоге: 2
25.02.2012, 10:16     Быки и коровы, не правильно считает их #8
Цитата Сообщение от TBIKC Посмотреть сообщение
Выполняется до тех пор пока 1 число в массиве = 0
Выполняется до тех пор, пока второй элемент массива равен нулю.

Цитата Сообщение от ValeryS Посмотреть сообщение
и если отрицательно будет что делать???
rand не возвращает отрицательные значения...
Toshkarik
 Аватар для Toshkarik
1139 / 856 / 50
Регистрация: 03.08.2011
Сообщений: 2,381
Завершенные тесты: 1
25.02.2012, 10:18     Быки и коровы, не правильно считает их #9
Цитата Сообщение от TBIKC Посмотреть сообщение
Выполняется до тех пор пока 1 число в массиве = 0
Сами же говорили, что индексация с 0 и тут же называете элемент с индексом 1 как первый...

Не по теме:

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

TBIKC
web dev
 Аватар для TBIKC
104 / 91 / 1
Регистрация: 23.09.2010
Сообщений: 738
25.02.2012, 10:19  [ТС]     Быки и коровы, не правильно считает их #10
Спасибо за прогу, но хотелось бы всё же морю исправить

Добавлено через 28 секунд
Цитата Сообщение от Toshkarik Посмотреть сообщение
Сами же говорили, что индексация с 0 и тут же называете элемент с индексом 1 как первый...
Я не говорил такого, я сказал первое число
Toshkarik
 Аватар для Toshkarik
1139 / 856 / 50
Регистрация: 03.08.2011
Сообщений: 2,381
Завершенные тесты: 1
25.02.2012, 10:20     Быки и коровы, не правильно считает их #11
Это второе число, первое числа, а точнее цифра, находится в randArray[ 0 ];
ValeryS
Модератор
6377 / 4843 / 442
Регистрация: 14.02.2011
Сообщений: 16,048
25.02.2012, 10:30     Быки и коровы, не правильно считает их #12
Цитата Сообщение от 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
web dev
 Аватар для TBIKC
104 / 91 / 1
Регистрация: 23.09.2010
Сообщений: 738
26.02.2012, 14:43  [ТС]     Быки и коровы, не правильно считает их #13
сам не представляю)
TBIKC
web dev
 Аватар для TBIKC
104 / 91 / 1
Регистрация: 23.09.2010
Сообщений: 738
27.02.2012, 16:06  [ТС]     Быки и коровы, не правильно считает их #14
Помогите пожалуйста, переделать
Jupiter
Каратель
Эксперт C++
6543 / 3963 / 226
Регистрация: 26.03.2010
Сообщений: 9,273
Записей в блоге: 1
Завершенные тесты: 2
27.02.2012, 16:20     Быки и коровы, не правильно считает их #15
Быки и коровы
TBIKC
web dev
 Аватар для TBIKC
104 / 91 / 1
Регистрация: 23.09.2010
Сообщений: 738
27.02.2012, 16:26  [ТС]     Быки и коровы, не правильно считает их #16
я умею пользоваться поиском )
просто хочу чтобы изменить в моей программе код, а не новый ...
dimcoder
Полярный
 Аватар для dimcoder
449 / 422 / 66
Регистрация: 11.09.2011
Сообщений: 1,108
27.02.2012, 16:35     Быки и коровы, не правильно считает их #17
Решил немного покоментить код:
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
web dev
 Аватар для TBIKC
104 / 91 / 1
Регистрация: 23.09.2010
Сообщений: 738
27.02.2012, 21:29  [ТС]     Быки и коровы, не правильно считает их #18
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;
}
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
28.03.2012, 08:22     Быки и коровы, не правильно считает их
Еще ссылки по теме:

C++ Не правильно считает
Быки и коровы C++
Не правильно считает C++

Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
TBIKC
web dev
 Аватар для TBIKC
104 / 91 / 1
Регистрация: 23.09.2010
Сообщений: 738
28.03.2012, 08:22  [ТС]     Быки и коровы, не правильно считает их #19
Помогите пожалуйста
Вот последняя редакция кода, только быков и коров не правильно считает
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;
}
Yandex
Объявления
28.03.2012, 08:22     Быки и коровы, не правильно считает их
Ответ Создать тему
Опции темы

Текущее время: 08:49. Часовой пояс GMT +3.
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2016, vBulletin Solutions, Inc.
Рейтинг@Mail.ru