From b7451f8db6281f1335383b7c8a1ba164ddc5ba89 Mon Sep 17 00:00:00 2001
From: Aleksey Lobanov <alekseylobanov1@gmail.com>
Date: Sun, 7 Jun 2015 13:15:16 +0300
Subject: [PATCH] Deleted skiping of events and some refactoring

---
 src/crossgen.hpp         | 10 +++++-----
 wxCrossGen/main.cpp      | 19 ++++++-------------
 wxCrossGen/wxCrossGen.mk |  2 +-
 3 files changed, 12 insertions(+), 19 deletions(-)

diff --git a/src/crossgen.hpp b/src/crossgen.hpp
index 13ef766..ca7c996 100644
--- a/src/crossgen.hpp
+++ b/src/crossgen.hpp
@@ -54,7 +54,7 @@ void readGrid(wxString path, GridType &grid){
         wxLogDebug(str);
         for (unsigned int j = 0; j < str.size(); ++j)
             grid.at(j).at(i) = str.at(j);
-        i++;
+        ++i;
     }
     wxLogDebug(wxT("Прочитана сетка размером %d x %d"), 
       static_cast<int>(grid.size()), static_cast<int>(grid.at(0).size()));
@@ -80,10 +80,10 @@ void generateWordInfo(GridType &grid, std::vector<WordInfo> &winfos){
                         size_t cur_len = 1;
                         bool cont = true;
                         while ((j + cur_len < grid.at(0).size()) && cont){
-                            cur_len++;
+                            ++cur_len;
                             if (grid.at(i).at(j+cur_len-1) != CELL_CLEAR){
                                 cont = false;
-                                cur_len--;
+                                --cur_len;
                             }
                         }
                         exist = true;
@@ -102,7 +102,7 @@ void generateWordInfo(GridType &grid, std::vector<WordInfo> &winfos){
                         size_t cur_len = 1;
                         bool cont = true;
                         while ((i + cur_len < grid.size()) && cont){
-                            cur_len++;
+                            ++cur_len;
                             if (grid.at(i+cur_len-1).at(j) != CELL_CLEAR){
                                 cont = false;
                                 cur_len--;
@@ -119,7 +119,7 @@ void generateWordInfo(GridType &grid, std::vector<WordInfo> &winfos){
                     }
                 if (exist){
                     exist = false;
-                    cur_ind++;
+                    ++cur_ind;
                 }
             }
         }
diff --git a/wxCrossGen/main.cpp b/wxCrossGen/main.cpp
index 3c053c1..75a38cc 100644
--- a/wxCrossGen/main.cpp
+++ b/wxCrossGen/main.cpp
@@ -31,17 +31,14 @@ MainFrame::MainFrame(wxWindow* parent, int id, const wxString& title, const wxPo
 }
 
 
-void MainFrame::set_properties()
-{
+void MainFrame::set_properties() {
     SetTitle(_("Генератор кроссвордов"));
     SetSize(wxSize(700, 500));
     SetMinSize(wxSize(600,340));
 }
 
 
-void MainFrame::do_layout()
-{
-    // begin wxGlade: MainFrame::do_layout
+void MainFrame::do_layout() {
     wxBoxSizer* sizer_1 = new wxBoxSizer(wxVERTICAL);
     wxBoxSizer* sizer_4 = new wxBoxSizer(wxHORIZONTAL);
     wxBoxSizer* sizer_3 = new wxBoxSizer(wxHORIZONTAL);
@@ -59,7 +56,6 @@ void MainFrame::do_layout()
     sizer_1->Add(sizer_4, 0, wxEXPAND, 0);
     SetSizer(sizer_1);
     Layout();
-    // end wxGlade
 }
 
 
@@ -69,8 +65,7 @@ BEGIN_EVENT_TABLE(MainFrame, wxFrame)
 END_EVENT_TABLE();
 
 void MainFrame::OnbtnPathClick(wxCommandEvent &event) {
-    event.Skip();
-    wxFileDialog dlgOpen(this, wxT("Open XYZ file"), wxEmptyString, wxEmptyString,
+    wxFileDialog dlgOpen(this, wxT("Открыть файл кроссворда"), wxEmptyString, wxEmptyString,
     wxT("Файлы кроссворда (*.cross)|*.cross"), wxFD_OPEN|wxFD_FILE_MUST_EXIST);
     
     if (dlgOpen.ShowModal() == wxID_CANCEL)
@@ -118,7 +113,7 @@ void MainFrame::SetGridImage(GridType &grid, size_t w) {
         dc.DrawText(wxString::Format(wxT("%d"),winfos.at(i).ind), sq_w*winfos.at(i).x, sq_h*winfos.at(i).y);
     }
     
-    if (_words.size() > 0) {
+    if ( _words.size() > 0 ) {
         using std::vector;
         vector< vector< bool > > usedCells(
             grid.size(), 
@@ -127,8 +122,8 @@ void MainFrame::SetGridImage(GridType &grid, size_t w) {
         cur_f = dc.GetFont();
         cur_f.SetPointSize(sq_h*0.6);
         dc.SetFont(cur_f);
-        for (size_t i = 0; i < winfos.size(); ++i){
-            if (winfos.at(i).direct == true){
+        for ( size_t i = 0; i < winfos.size(); ++i ) {
+            if ( winfos.at(i).direct == true ) {
                 for (size_t j = 0; j < winfos.at(i).len; ++j)
                     if ( !usedCells.at(winfos.at(i).x+j).at(winfos.at(i).y) ) {
                         dc.DrawText(
@@ -160,8 +155,6 @@ void MainFrame::SetGridImage(GridType &grid, size_t w) {
 }
 
 void MainFrame::OnbtnGenerateClick(wxCommandEvent &event) {
-    event.Skip();
-    
     if ( !_isDictLoaded ) {
         readDict(wxT("big_cross_ru.txt"), _dict);
         _isDictLoaded = true;
diff --git a/wxCrossGen/wxCrossGen.mk b/wxCrossGen/wxCrossGen.mk
index 3ad1fb1..ae86bfc 100644
--- a/wxCrossGen/wxCrossGen.mk
+++ b/wxCrossGen/wxCrossGen.mk
@@ -13,7 +13,7 @@ CurrentFileName        :=
 CurrentFilePath        :=
 CurrentFileFullPath    :=
 User                   :=Aleksey Lobanov
-Date                   :=06/06/15
+Date                   :=07/06/15
 CodeLitePath           :="/home/alex/.codelite"
 LinkerName             :=/usr/bin/g++-4.8
 SharedObjectLinkerName :=/usr/bin/g++-4.8 -shared -fPIC