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

Насколько плохо написан код? - C++

Восстановить пароль Регистрация
Другие темы раздела
C++ Реализовать класс "Очередь" http://www.cyberforum.ru/cpp-beginners/thread1817825.html
Помогите разобраться при создании очереди чисел все просто и понятно, к примеру есть поле value туда я помещаю числа и оттуда могу их их взять: head = new element ; tail = head; head->value = x; head->adr=0; size = 1;
C++ Реализовать поиск слова в текстовом файле Необходимо создать консольное приложение, которое по заданному пользователем слову будет обращаться в файл txt с текстом и искать заданное слово, при нахождении такого слова вывести об этом сообщение, если нет то нет, слово должно искаться с учетом регистров, даже если оно в файле будет забориком написано, как можно проще, я даун в программировании,пока что, помогите плиз http://www.cyberforum.ru/cpp-beginners/thread1817812.html
В считывание с файла возникает ошибка C++
Так случилось, что я сменил версию visual studio и работающая программа перестала нормально работать (возникает точка остановки). #include "stdafx.h" #include <iostream> #include <conio.h> #include <time.h> #include <cmath> #include <fstream> #include <algorithm> using namespace std;
Вывод пирамиды чисел в консоли C++
Подскажите как реализовать вывод вот такой пирамиды чисел: ___1___ __121__ _12321_ 1234321 ....
C++ Выбрать вариант не имеющий ошибки и к каждому написать почему этот вариант правильный/неправильный http://www.cyberforum.ru/cpp-beginners/thread1817794.html
Прошу помогите, я плохо понимаю c++, да и простите если не в той теме, в общем, дали задание: Выбрать вариант не имеющий ошибки и к каждому написать почему этот вариант правильный/неправильный. (а) if (num == 10) else num = 0; (c)
C++ Вывести на консоль подстроку, ограниченную заданными символами есть строка, которая выглядит так: "admin::::Владислав,Павленко: pavlenko2k16" как вывести на экран имя, которое находится между "::::" и ","? Добавлено через 31 минуту сделал делив текст string str = database;//ïðèñâàèâàåì string separ(","); //ðàçäåëèòåëü string::size_type pos = str.find(",");//èùåì ïîëîæåíèå ðàçäåëèòåëÿ string first = str.substr(0, pos);//äî ðàçäåëèòåëÿ ... подробнее

Показать сообщение отдельно
_stanislav
26 / 26 / 8
Регистрация: 16.08.2014
Сообщений: 246
Завершенные тесты: 2
03.10.2016, 13:07     Насколько плохо написан код?
Цитата Сообщение от gru74ik Посмотреть сообщение
А возвращаясь к вопросу ТС? Насколько плохо написан его код? На что бы стоило обратить внимание чисто стилистически и концептуально?
Я думаю что нужно выработать свой стиль приближенный к стилю большинства программистов. Без заскоков в ту или иную сторону.

Добавлено через 7 минут
Вот мой стиль. Стандарт, читается легко (я так думаю)
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
// *.h
 
#include "IState.h"
 
#ifndef DEFINE_STATE_MACHINE
#define DEFINE_STATE_MACHINE
 
namespace lexer
{
    class StateMachine
    {
    public:
        StateMachine();
        ~StateMachine();
 
        void Set(char);
        void Reset();
 
        StateId GetState() { return stateId; }
 
    private:
        static const int size = 4;
        IState* states[size];
 
        StateId stateId;
 
        static StateId Skip(char);
        StateId CheckState();
    };
}
 
#endif
 
// *.cpp
 
#include "StateMachine.h"
 
#include "StateFunction.h"
#include "StateName.h"
#include "StateNumber.h"
#include "StateOperation.h"
 
#include <cctype>
 
lexer::StateMachine::StateMachine()
{
    states[0] = new StateFunction();
    states[1] = new StateName();
    states[2] = new StateNumber();
    states[3] = new StateOperation();
 
    stateId = StateId::NON;
}
lexer::StateMachine::~StateMachine()
{
    for (int i = 0; i < size; i++)
        delete states[i];
}
 
void lexer::StateMachine::Set(char ch)
{
    for (int i = 0; i < size; i++)
        states[i]->Set(ch);
 
    stateId = CheckState();
 
    if (stateId == StateId::BAD)
        stateId = Skip(ch);
}
void lexer::StateMachine::Reset()
{
    for (int i = 0; i < size; i++)
        states[i]->Reset();
 
    stateId = StateId::NON;
}
 
lexer::StateId lexer::StateMachine::Skip(char ch)
{
    switch (ch)
    {
    case '^':
    case '*':
    case '/':
    case '+':
    case '-':
    case ';':
    case '(':
    case ')':
        return StateId::BAD;
 
    default:
        if (isspace(ch) )
            return StateId::BAD;
    }
 
    return StateId::NON;
}
 
lexer::StateId lexer::StateMachine::CheckState()
{
    for (int i = 0; i < size; i++)
        if (!states[i]->Good() && !states[i]->Bad() )
            return StateId::NON;
 
    for (int i = 0; i < size; i++)
        if (states[i]->Good() )
            return states[i]->GetId();
 
    return StateId::BAD;
}
 
Текущее время: 14:59. Часовой пояс GMT +3.
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2016, vBulletin Solutions, Inc.
Рейтинг@Mail.ru