19 / 10 / 6
Регистрация: 28.05.2014
Сообщений: 140
1

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

23.08.2014, 16:04. Показов 2440. Ответов 2
Метки нет (Все метки)

Здравтсвуйте!

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

Критика кода приветствуется.
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;
}




Нужна помощь: программа ставит цифру в первую подходящую клетку. По идее после каждого действия нужна проверка имеет ли наше судоку ришение, но как это сделать я почти не представляю.
__________________
Помощь в написании контрольных, курсовых и дипломных работ, диссертаций здесь
0
Programming
Эксперт
94731 / 64177 / 26122
Регистрация: 12.04.2006
Сообщений: 116,782
23.08.2014, 16:04
Ответы с готовыми решениями:

проверка судоку
Помогите пожалуйста написать программу , которая проверяет правильность решения судоку. Мне нужно...

решение судоку
Судоку размера n называется квадрат со стороной n2, разделенный на n2 средних квадратов со стороной...

Решение Судоку
Здравствуйте! Заранее извеняюсь за повтор, писал такую тему в алгоритмах, но там глухо :( ...

Генерация и решение судоку
Никто не знает ,если ли статья про решение и ,главное, генерацию судоку с различными эвристиками с...

2
Форумчанин
Эксперт CЭксперт С++
8193 / 5043 / 1437
Регистрация: 29.11.2010
Сообщений: 13,453
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 и не могу понять что она делает (названия параметров, члена класса, да и самой функции выбраны неудачно). При попытке создать поток считывания с файла хорошо бы проверить, а создался ли он вообще. Также лучше проверять на успех считывание.
И т.д. и т.п.
0
Evg
Эксперт CАвтор FAQ
21264 / 8280 / 637
Регистрация: 30.03.2009
Сообщений: 22,635
Записей в блоге: 30
24.08.2014, 14:56 3
Мало ли, вдруг пригодится
Решение Судоку
0
IT_Exp
Эксперт
87844 / 49110 / 22898
Регистрация: 17.06.2006
Сообщений: 92,604
24.08.2014, 14:56
Помогаю со студенческими работами здесь

Докажите, что если линейная система с целыми коэффициентами имеет какой-то решение, то она имеет решение в Q
Здравствуйте. Помогите с доказательством, пожалуйста: &quot;Докажите, что если линейная система с...

Имеет ли решение система уравнений? Если имеет - найдите корни (Pascal -> C#)
Определите, имеет ли решение система ax+by=c, dx+ey=f Если имеет, найдите корни. uses crt; var...

Алгоритм генерации судоку - нужна помощь
Сразу извиняюся за возможное повторение темы! Необходима помощь в составлении алгоритма...

1C8.2 нужна обработка решения игры судоку
1C8.2 нужна обработка решения игры судоку


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

Или воспользуйтесь поиском по форуму:
3
Ответ Создать тему
Опции темы

КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2023, CyberForum.ru