Оцените и скажите как оптимизировать?

@Chayn1k · Регистрация: 28.06.2016

Author24 — интернет-сервис помощи студентам

Это моя первая более мение обьемная и продуктивная программа. Написал для знакомых школьников, чтобы подтянули математику (люблю её

(математику)

)). Какие будут советы по оптимизации, очень волнует этот вопрос.

C++

#include<iostream>
#include<conio.h>
#include<math.h>
#include <stdlib.h>
#include<ctime>
#include<fstream>
using namespace std;
 
int main()
{
setlocale(0, "");
ofstream output("D://Otvet.txt", ios::app);
int a, b, s=0, p=0, np=0;
double k, o;
char x[4]={'*','-','+','/'};
int t2[10]={2, 4, 6, 8, 10, 12, 14, 16, 18, 20};
int t3[10]={3, 6, 9, 12, 15, 18, 21, 24, 27, 30};
int t4[10]={4, 8, 12, 16, 20, 24, 28, 32, 36, 40};
int t5[10]={5, 10, 15, 20, 25, 30, 35, 40, 45, 50};
int t6[10]={6, 12, 18, 24, 30, 36, 42, 48, 54, 60};
int t7[10]={7, 14, 21, 28, 35, 42, 49, 56, 63, 70};
int t8[10]={8, 16, 24, 32, 40, 48, 56, 64, 72, 80};
int t9[10]={9, 18, 27, 36, 45, 54, 63, 72, 81, 90};
char c;
srand(time(NULL));
while(s!=50)
{
    c=x[rand()%4];
 
switch(c)
{
    case '*': a=rand()%10;
          b=rand()%10;
          break;
    case '-':   b=rand()%51;
            a=rand()%51+b;
            break;
    case '+': a=rand()%101;
          b=rand()%101;
          break;
    case '/': b=rand()%10 + 2;
              if(b==2){a=t2[rand()%10];}
              if(b==3){a=t3[rand()%10];}
              if(b==4){a=t4[rand()%10];}
              if(b==5){a=t5[rand()%10];}
              if(b==6){a=t6[rand()%10];}
              if(b==7){a=t7[rand()%10];}
              if(b==8){a=t8[rand()%10];}
              if(b==9){a=t9[rand()%10];}
          break;
}
 
 
cout<<a<<' '<<c<<' '<<b<<"\nВаш ответ:";
if(c=='*'){k=a*b;}
if(c=='/'){k=a/b;}
if(c=='-'){k=a-b;}
if(c=='+'){k=a+b;}
cin>>o;
if(o==k)
{   const string stars(16,'*');
    cout<<stars<<"\n";
    cout<<"* Правильно!!! *"<<endl; p++;
    cout<<stars<<endl;
}
if(o!=k){ cout<<"Ты ошиблась"<<"\n"<<"Правельно:"<<k<<endl; np++;
}
s++;
}
cout<<"Правильных ответов:"<<p<<endl;
cout<<"Не правильных ответов:"<<np;
output<<"правильных ответов:"<<p<<"   "<<"Неправельных ответов:"<<np<<endl;
output.close();
getch(); 
return 0;
}

@kylroma · 05.08.2016, 18:58

Что программа делает?
Ну нельзя называть переменные такими именами. Ничего же не понятно.

@Doctor123 · 05.08.2016, 19:32

А в чём компилировали?
И какой вес экзешника получился?
Я предлагаю в моём проекте такое делать( del ).
Можем вместе перенести - я помогу.
Заодно оптимизируем - у меня тоже есть кого учить арифметике.
Получится компактно, просто и сразу оконное приложение (без использования консоли).

@Doctor123 · 05.08.2016, 22:14

Пока вот так оптимизировал, без использования своего проекта.
В рамках консольного приложения для MSVC 6.0

C++

#include "stdafx.h"
#include<conio.h>
#include<ctime>
#include<fstream>
#include<iostream>
using namespace std;
 
int main()
{
 
setlocale(0, "");
ofstream output("D://Otvet.txt", ios::app);
int a, b, s=0, p=0, np=0;
double k, o;
char x[4]={'*','-','+','/'};
char c;
srand(time(NULL));
while(s!=50)
{
    c=x[rand()%4];
 
    switch(c)
    {
        case '*': a=rand()%10;
                  b=rand()%10;
                  break;
 
        case '-': b=rand()%51;
                  a=rand()%51+b;
                  break;
 
        case '+': a=rand()%101;
                  b=rand()%101;
                  break;
 
        case '/': b=rand()%10 + 2;
                  k = rand()%10;
                  a = k * b;
                  break;
    }
    cout<<a<<' '<<c<<' '<<b<<"\nYour answer: ";
    if(c=='*'){k=a*b;}
    if(c=='-'){k=a-b;}
    if(c=='+'){k=a+b;}
    cin>>o;
    if(o==k)
    {  
        char stars[] = "*******************";
        cout<<stars<< "\n"; 
        cout<<"* It is right !!! *"<<endl; p++;
        cout<<stars<<endl;
    }
    if(o!=k){ cout<<"This is not true"<<"\n"<<"Right:"<<k<<endl; np++;
    }
    s++;
}
cout<<"Правильных ответов:"<<p<<endl;
cout<<"Неправильных ответов:"<<np;
output<<"правильных ответов:"<<p<<"   "<<"Неправильных ответов:"<<np<<endl;
output.close();
getch(); 
return 0;
}

Прикрепляю проект для MSVC 6.0 и экзешник там же в папке релиз.
Весит 220Kb - это очень много!

P.S. Моё предложение по переносу Вашей программы в мой проект по прежнему в силе...

@DevAlone · 05.08.2016, 22:46

Тут переписывать надо, а не оптимизировать...

@Doctor123 · 07.08.2016, 12:57

Перенёс в свой проект и оптимизировал. Проще и нагляднее уже, наверное, не сделать.
Вся логика находится на вкладке main.cpp

C++

int           op_1,                                                                                 //  Первый операнд
              op_2,                                                                                 //  Второй операнд
              res_;                                                                         //  Результат (правильный)
              
char     Operati_n,
      x[] = "*-+/";
 
#include "Init.h"
 
void INIT() 
{ 
    //  Knopka_2 = 1;                 // В самом начале кнопку "Restart"  считаем нажатой для печати первого задания                                
}
 
void START()
{       
     if(Knopka_2)                                                                //  Если кнопку "Restart" нажали, то..
     {  
        Knopka_2 = 0;   
        srand(Timer_CLK);   
        SetWindowText(hwndEdit,"");                                                    // Стираем содержимое поля ввода           
        Operati_n = x[rand()%4];
        switch(Operati_n)
        {
            case '*': op_1 = rand()%10;
                      op_2 = rand()%10;
                      res_ = op_1 * op_2;
                      break;
    
            case '-': op_2 = rand()%51;
                      op_1 = rand()%51 + op_2;
                      res_ = op_1 - op_2;
                      break;
    
            case '+': op_1 = rand()%101;
                      op_2 = rand()%101;
                      res_ = op_1 + op_2;
                      break;
    
            case '/': op_2 = rand()%10 + 2;
                      res_ = rand()%10;
                      op_1 = res_ * op_2;
                      break;
        }       
        wsprintf(str, "%d %c %d = ", op_1, Operati_n, op_2);
        Print(250, 102,     str);        
        Print(76,  300, "     ");                                                           //  Стираем старые строки
        Print(76,  320, "     ");                                                           //  Стираем старые строки
        Print(76,  340, "     ");                                                           //  Стираем старые строки
     }
     
     if(Knopka_3)                                                                 //  Если кнопку  "OK"  нажали, то..
     {
        Knopka_3 = 0;                                                              //  Обнулим запись о нажатии кнопки
        
        if(Transit[2][17] == res_)                                          //  Если ответ равен результату....
        { 
            Print(76,  300, "Совершенно верно !!!"); 
            
            Print(76,  320, "Результатом будет: ", res_); 
            
            Print(76,  340, "     "); 
        } 
        else                                                                                //  Если ответ НЕправильный
        {
            Print(76,  300, "Этого не может быть, Вы опять ошиблись..."); 
            
            Print(76,  320, "Ваш ответ: ", Transit[2][17]); 
            
            Print(76,  340, "А правильный ответ = ", res_);     
        }
     }      
}

Вся обвязка убрана из поля зрения и доступ к ней очень простой ( на случай внесения правок ).
Фильтр цифр и их прокрутка скроллом в поле ввода.
Вес экзешника без иконки 25Kb (+dllBridge 77Kb).

Хотел ещё изменить фон в поле ввода (белый сильно контрастирует), но не знаю как - подскажите...

@Chayn1k · 08.08.2016, 20:31 **[ТС]**

Doctor123, я скопировал код в dev-cpp и он материться на строку <cpp>include<Init.h></cpp>

@Doctor123 · 09.08.2016, 21:04

Вам нужно будет скачать весь проект.. только, что обновил его
А текст уроков копируется на вкладку main.cpp

Добавлено через 22 часа 27 минут
Дал поиграть ребёнку 6ти лет с программой. Нашлось несколько "детских" решений для накрутки положительного результата. Ребёнок готовится к школе ( с него требуют). В общем, у него был стимул хитрить.
На данном этапе считаю, что оптимизировано всё, что можно. Дальше только увеличение функционала.
Посмотреть, что получилось здесь.[cut]
Конструктивным вопросам буду рад...
PS Счёт ограничил примерно до 30-40.

@kylroma Одессит 243 / 88 / 44 Регистрация: 30.12.2013 Сообщений: 316 Записей в блоге: 2
	05.08.2016, 18:58	2
	Что программа делает? Ну нельзя называть переменные такими именами. Ничего же не понятно. 0

@Doctor123 22 / 22 / 8 Регистрация: 03.12.2013 Сообщений: 193
	05.08.2016, 19:32	3
	А в чём компилировали? И какой вес экзешника получился? Я предлагаю в моём проекте такое делать( del ). Можем вместе перенести - я помогу. Заодно оптимизируем - у меня тоже есть кого учить арифметике. Получится компактно, просто и сразу оконное приложение (без использования консоли). 0

@DevAlone 331 / 283 / 78 Регистрация: 02.08.2016 Сообщений: 1,008
	05.08.2016, 22:46	5
	Тут переписывать надо, а не оптимизировать... 0

@Chayn1k 0 / 0 / 0 Регистрация: 28.06.2016 Сообщений: 10
	08.08.2016, 20:31 [ТС]	7
	Doctor123, я скопировал код в dev-cpp и он материться на строку <cpp>include<Init.h></cpp> 0

@Doctor123 22 / 22 / 8 Регистрация: 03.12.2013 Сообщений: 193
	09.08.2016, 21:04	8
	Вам нужно будет скачать весь проект.. только, что обновил его А текст уроков копируется на вкладку main.cpp Добавлено через 22 часа 27 минут Дал поиграть ребёнку 6ти лет с программой. Нашлось несколько "детских" решений для накрутки положительного результата. Ребёнок готовится к школе ( с него требуют). В общем, у него был стимул хитрить. На данном этапе считаю, что оптимизировано всё, что можно. Дальше только увеличение функционала. Посмотреть, что получилось здесь.[cut] Конструктивным вопросам буду рад... PS Счёт ограничил примерно до 30-40. 0