Говнокод #27178 — C++ — Говнокод.ру

Говнокод: по колено в коде.

Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!

C++ / Говнокод #27178

#include <iostream>
#include <string>
using namespace std;
class Govnokod {
    bool _flag_dot;
    bool _flag_mant;
    int _index_mant;
    bool vetka1(int i, const string stroka) {
        for (int j = i++; j < stroka.length(); j++) {
            switch (stroka[j]) {
            case '.':  
                if (_flag_dot) return false;
                _flag_dot = true;
                break;     
            case '0' ... '9': break; 
            default:
                return false;
                break; }}
        return true;}
    bool vetka2_dalshe(const string stroka) {
        for (int j = 1; j < stroka.length(); j++) {
            switch (stroka[j]) {
            case '0' ... '9': break;
            default:
                return false;
                break; }} 
        return true; }
    bool vetka2(const string stroka) {
        switch (stroka[0]) {
        case '+':
        case '-':
            if (stroka.length() < 2) return false;  
            return vetka2_dalshe(stroka);
            break;   
        case '0' ... '9':
            return vetka2_dalshe(stroka);
            break;
        default:
            return false;
            break; }}
    bool mantissa(const string stroka) {
       for (int j = 0; j < stroka.length(); j++) {
           switch (stroka[j]) {
           case 'e':
           case 'E':            
               if ((_flag_mant) or (j == (stroka.length() - 1))) return false;
               _flag_mant = true;
               _index_mant = j;
               break; }}
       return true; }
    bool Dalshe(int i, const string stroka) {
        _flag_dot = false;
        _flag_mant = false;
        if (not mantissa(stroka)) return false;
        else if (_flag_mant) {
            string sub1 = stroka.substr(0, _index_mant);
            string sub2 = stroka.substr(_index_mant+1);
            return (vetka1(i, sub1) and vetka2(sub2)); }
        else return vetka1(i, stroka); }
    bool proverka(const string stroka) {
        switch (stroka[1]) {
        case '0' ... '9':
            return Dalshe(1, stroka);
            break;   
        default:
            return false;
            break; }}
    bool general_proverka(const string stroka) {
        switch (stroka[0]) {
        case '-':
        case '+':
            if (stroka.length() > 1) return proverka(stroka);
            else return false;
            break;  
        case '0' ... '9':
            return Dalshe(0, stroka);
            break; 
        default:
            return false;
            break; }}
    public:
    long double opros(char s) {
        string argument;
        while (true) {
            cout << "Введите значение " << s << ": ";
            getline(cin, argument);
            if (argument.length() == 0)
                cout << "Вы не ввели значение!" << endl;
            else if (not general_proverka(argument))
                cout << "Некорректное значение!" << endl;
            else 
                return strtold(argument.c_str(), nullptr); }
    }
    } obj;
int main() {
    for (char c = 'a'; c < 'd'; c++) {
        long double result = obj.opros(c);
        cout << "Значение: " << result << " -- корректное!" << endl;
    }
}

Решил попробовать в стиле ООП переписать.

Запостил:

Westnik_Govnokoda, 25 Декабря 2020

Комментарии (10) RSS

Westnik_Govnokoda 25.12.2020 17:42 # 0

Также решил написать функцию, которая бы вырезала все левые символы из строки, а затем результат этой функции передать аргументом методу general_proverka(). Однако столкнулся со странными багами: в cout без преобразования в Си-строку не отображается результат, но даже если и преобразование осуществить, все равно будто бы в переменной buf сохраняются её старые значения от предыдущих вызовов функции, но эта переменная не объявлена, как статичная...

string cut_incorrect_symbol(string stroka) {
int j, i;
string buf;
for (j = 0, i = 0; j < stroka.length(); j++) {
switch (stroka[j]) {
case '0' ... '9':
case '-':
case '+':
case '.':
case 'e':
case 'E':
buf[j-i] = stroka[j];
break;
default:
i++;
break;
}
}
cout << "Значение buf == " << buf << endl;
return buf;
}
Ответить
- bormand 25.12.2020 17:52 # +2
  
  > string buf;
  > buf[j-i] = stroka[j];
  
  Типичный вылет за границы буфера. В данном случае buf[...] вообще нельзя трогать т.к. конструктор по-умолчанию создаёт строку пустой (нулевой длины). Иначе ты перетираешь чужую память и получаешь неопределённое поведение.
  
  Можешь добавлять в неё символы push_back()'ом. Либо resize()'нуть до нужного размера перед тем как лезть к ней скобочками.
  Ответить
  - MAKAKA 25.12.2020 18:32 # 0
    
    такая хуйня со строками даже в руби не сработает, лол. А вот с массивом/списком чаров в скриптушне должна сработать
    Ответить
  - Westnik_Govnokoda 25.12.2020 19:10 # 0
    
    Большое спасибо, помогло.
    Ответить
- 1024-- 25.12.2020 23:00 # +1
  
  > general_proverka
  ryadovoy_ispytanie
  Ответить
  - Steve_Brown 28.12.2020 10:45 # 0
    
    general.where(general.uniform)
    general.where(general.orders)
    // Угадай мелодию
    Ответить
  - Мистер Хэнки 15.01.2021 11:47 # 0
    
    marshall_mathers
    Ответить
    - Desktop 15.01.2021 13:53 # 0
      
      Это что-то из .net?
      Ответить
      - inkanus_gray 15.01.2021 22:20 # 0
        
        Нет, это про сталинские самовары.
        Ответить
slesar_kip 25.08.2021 19:34 # 0

- Саламатина, закрыла бы ты рот свой лесбийский, - чуть не кинулся на неё Серый.
Ответить

Говнокод: по колено в коде.

C++ / Говнокод #27178

Комментарии (10) RSS

Добавить комментарий