Форум программистов, компьютерный форум, киберфорум
Наши страницы

С++ для начинающих

Войти
Регистрация
Восстановить пароль
 
rennnorb
9 / 9 / 2
Регистрация: 28.05.2014
Сообщений: 135
Завершенные тесты: 1
#1

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

23.08.2014, 16:04. Просмотров 1011. Ответов 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
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
23.08.2014, 16:04
Здравствуйте! Я подобрал для вас темы с ответами на вопрос Нужна проверка, имеет ли судоку решение (C++):

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

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

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

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

Определить, имеет ли решение система уравнений - C++
По введенным значениями коэффициентов линейных уравнений определить имеет ли система этих уравнений решение, если имеет, то единственно ли...

Определить, имеет ли уравнение хотя бы одно вещественное решение - C++
9. Для заданных вещественных чисел а, b, c определить, имеет ли уравнения ax2+bx+c=0, хотя бы одно вещественное решение.

2
MrGluck
Модератор
Эксперт CЭксперт С++
7491 / 4606 / 692
Регистрация: 29.11.2010
Сообщений: 12,597
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
18258 / 6383 / 440
Регистрация: 30.03.2009
Сообщений: 17,665
Записей в блоге: 28
24.08.2014, 14:56 #3
Мало ли, вдруг пригодится
Решение Судоку
0
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
24.08.2014, 14:56
Привет! Вот еще темы с ответами:

Посчитать сумму элементов массива, при которых квадратное уравнение имеет решение - C++
Дана матрица a (а1,а2,а3...аn) Найти сумму тех элементов,при которых уравнение X^2+3ax+5=0 Если таких элементов нет,вывести 0. ...

Для заданных вещественных чисел а, b, c определить, имеет ли уравнения ax2+bx+c=0, хотя бы одно вещественное решение. - C++
Для заданных вещественных чисел а, b, c определить, имеет ли уравнения ax2+bx+c=0, хотя бы одно вещественное решение. помогите решить на с++

Генератор паролей (нужна проверка) - C++
Здравствуйте. Написал простенький генератор паролей задаваемой пользователем длины. Все вроде правильно работает, просто, пожалуйста,...

нужна решение задач в с++ - C++
1) Решить задачу. Функция у=2х, если х&gt;0, и у=х+10 в других случиях. 2) Решить задачу. Дан одномерный массив А(К). Определить сумму...


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

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

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