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

Нужна проверка, имеет ли судоку решение - C++

Восстановить пароль Регистрация
 
rennnorb
8 / 8 / 2
Регистрация: 28.05.2014
Сообщений: 123
23.08.2014, 16:04     Нужна проверка, имеет ли судоку решение #1
Здравтсвуйте!

В данный момент пишу программу которая решает судоку:

Критика кода приветствуется.
main.cpp
Кликните здесь для просмотра всего текста

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
#include <fstream>
#include "sudoku.h"
using namespace std;
int main()
{
     sudoke su;
     su.getFi();
     su.print();
     for(char c=49; c<53; c++)
     {
          for(int x=0; x<2; x++)
               for(int y=0; y<2; y++)
               {
                    if(su.rowNeed(x, c)&&su.colNeed(y, c))
                         su.set(x, y, c);   
               }
     }
 
     su.set(0, 0, 2, true);
 
     for(char c=49; c<53; c++)
     {
          for(int x=2; x<4; x++)
               for(int y=0; y<2; y++)
               {
                    if(su.rowNeed(x, c)&&su.colNeed(y, c))
                         su.set(x, y, c);   
               }
     }
 
     su.set(0, 0, 2, true);
 
     for(char c=49; c<53; c++)
     {
          for(int x=2; x<4; x++)
               for(int y=2; y<4; y++)
               {
                    if(su.rowNeed(x, c)&&su.colNeed(y, c))
                         su.set(x, y, c);   
               }
     }
 
     su.set(0, 0, 2, true);
 
     for(char c=49; c<53; c++)
     {
          for(int x=0; x<2; x++)
               for(int y=2; y<4; y++)
               {
                    if(su.rowNeed(x, c)&&su.colNeed(y, c))
                         su.set(x, y, c);   
               }
     } 
     //знаю что четыре цикла это губо и не красиво,
     //но ничего лучше придумать не смог
     su.print();
     cout<<endl;
     return 0;
}



sudoke.h
Кликните здесь для просмотра всего текста

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
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
#include <fstream>
#include <iostream>
#include <vector>
using namespace std;
class sudoke
{
private:
    char fi[4][4];
 
 
public:
     void getFi();//иициализируем поле из файла in
     void print();//выводим поле
     void printColl(int number);//выводим колонку
     void printRow(int number);//выводим строку
     //bool isGood();
 
     //void setOne(int a){one=a;}
     //void setTwo(int a){two=a;}
     //void setThree(int a){three=a;}
     //void setFour(int a){four=a;}
 
     //int getOne(){return one;}
     //int getTwo(){return two;}
     //int getTehree(){return three;}
     //int getFour(){return four;}
 
     void set(int x, int y, char a, bool _new);//устанавливаем элемент поля с координатами
                                                                    //х и  у на а, если _new истина
                                                                    //то очищаем массив чисел из квадрата
 
      bool colNeed(int number, char a);//истино, если в колонке нет числа а
      bool rowNeed(int number, char a);//истинно, если в строке нет числа а
 
      sudoke();
 
};
 
sudoke::sudoke()
{
      for(int a=0; a<4; a++)
            for(int b=0; b<4; b++)
                 fi[a][b]=0;
      //one = two = three = four = 0;
}
 
 
 
void sudoke::getFi()
{
    ifstream in("in");
    char buff=0;
    for(int a=0; a<4; a++)
        for(int b=0; b<4; b++)
            in>>fi[a][b];   
}
 
 
void sudoke::print()
{
    for(int a=0; a<4; a++)
    {
        if(a==2)
        {
            cout<<"--+--"<<endl;
        }
        for(int b=0; b<4; b++)
        {
            if(b==2)
                cout<<'|';
            cout<<fi[a][b];
        }
        cout<<endl;
    }
    cout<<endl<<endl;
}
 
void sudoke::printColl(int number)
{
    for(int i=0; i<4; i++)
        cout<<fi[i][number]<<endl;
}
 
void sudoke::printRow(int number)
{
    for(int i=0; i<4; i++)
        cout<<fi[number][i]<<endl;
}
 
 
bool sudoke::colNeed(int number, char a)
{
    for(int i=0; i<4; i++)
        if(fi[i][number] == a)
            return false;
    return true;
}
 
bool sudoke::rowNeed(int number, char a)
{
    for(int i=0; i<4; i++)
        if(fi[number][i] == a)
            return false;
    return true;
}
 
void sudoke::set(int x, int y, char a, bool _new=false)
{
     static vector<char> ve;
 
     if(_new)
     {
          //не знаю как обратиться
          //к переменной из метода,
          //пришлось делать вот так
 
          ve.clear();
          return;
     }
 
 
     if(fi[x][y]=='0')
     {
         for(int i=0; i<ve.size(); i++)//если число уже есть в квадрате...
               if(ve[i]==a)
                    return;
          fi[x][y] = a;
     }
     ve.push_back(fi[x][y]);
     cout<<endl;
     return;
}




Нужна помощь: программа ставит цифру в первую подходящую клетку. По идее после каждого действия нужна проверка имеет ли наше судоку ришение, но как это сделать я почти не представляю.
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
23.08.2014, 16:04     Нужна проверка, имеет ли судоку решение
Посмотрите здесь:

решение судоку C++
C++ Решение Судоку
Посчитать сумму элементов массива, при которых квадратное уравнение имеет решение C++
проверка судоку C++
C++ Генератор паролей (нужна проверка)
C++ Для заданных вещественных чисел а, b, c определить, имеет ли уравнения ax2+bx+c=0, хотя бы одно вещественное решение.
Определить, имеет ли решение система уравнений C++
нужна решение задач в с++ C++
Нужна проверка кода (система обмена сообщениями) C++
C++ Определить, имеет ли уравнение хотя бы одно вещественное решение
Тоже нужна проверка на ошибки Windows C++
C++ Генерация и решение судоку

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

Или воспользуйтесь поиском по форуму:
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
MrGluck
Ворчун
Эксперт С++
 Аватар для MrGluck
5583 / 3062 / 344
Регистрация: 29.11.2010
Сообщений: 8,254
23.08.2014, 18:07     Нужна проверка, имеет ли судоку решение #2
http://habrahabr.ru/post/173795/
http://habrahabr.ru/post/113837/
http://www.sudoku-club.ru/howto.html

Не беря во внимание сам алгоритм, слишком всё завязано на магические числа, названия методов неинформативны, в main сплошной дубляж + воздействие на класс идёт извне. Реализацию пишите в файл sudoku.cpp. Почему sudokE?
Строить логику внутри функции относительно его параметра-флага не по фен-шую (если конечно распределение - не единственное её предназначение). Что такое fi - непонятно. То есть допустим я смотрю на тело функции colNeed и не могу понять что она делает (названия параметров, члена класса, да и самой функции выбраны неудачно). При попытке создать поток считывания с файла хорошо бы проверить, а создался ли он вообще. Также лучше проверять на успех считывание.
И т.д. и т.п.
Evg
Эксперт С++Автор FAQ
 Аватар для Evg
16934 / 5339 / 328
Регистрация: 30.03.2009
Сообщений: 14,343
Записей в блоге: 26
24.08.2014, 14:56     Нужна проверка, имеет ли судоку решение #3
Мало ли, вдруг пригодится
Решение Судоку
Yandex
Объявления
24.08.2014, 14:56     Нужна проверка, имеет ли судоку решение
Ответ Создать тему
Опции темы

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