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

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

+1

01
02
03
04
05
06
07
08
09
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

//Попробуйте программно смоделировать разговор людей.Всего есть 10 людей.Каждый человек имеет имя(строку) и возраст(число).
//Возраст каждого человека генерируется рандомно из диапазона от 20 до 40, а имена состоят из следующего списка: 
//Александр, Андрей, Анастасия, Ирина, Наталья, Павел, Роман, Светлана, Сергей, Татьяна.
//Любой человек способен выполнять два действия :
//-здороваться с другим человеком;
//-рассказывать о себе.
//Люди делятся на 3 типа(разные классы) :
//    Тип №1 : Формалисты.Здороваются со всеми следующим образом : Здравствуйте, <имя>!, где <имя> — это имя человека, с которым он здоровается.
//    Тип №2 : Неформалы.Со всеми здороваются : Привет, <имя>!.
//    Тип №3 : Реалисты.Если возраст собеседника меньше / равен или не больше, чем на 5 лет, то здороваются следующим образом : Привет, <имя>!, в противном случае — Здравствуйте, <имя>!.
//    В программной реализации приветствие должно быть реализовано как полиморфный метод, принимающий человека в качестве параметра и возвращающий строку.
//    Рассказ о человеке является строкой формата "Меня зовут Вася, мой возраст 21 лет и я неформал" (вместо Вася используется имя любого другого человека, 
//    вместо 21 возраст этого человека, вместо неформал может быть формалист, либо реалист).Как видите у людей с грамматикой не всё в порядке, 
//    и они говорят лет после любого числа — непорядок, это нужно обязательно исправить.
//    Программа должна показать информацию обо всех людях.Затем все люди должны поздороваться друг с другом в следующем порядке : 
//    первый здоровается со вторым, затем второй с первым, а затем первый с третьим, третий с первым и т.д.Например :
//    Петя : Привет, Вася!
//    Вася : Здравствуйте, Петя!
//    Петя : Привет, Женя!
//    Женя : Привет, Петя!


unique_ptr<Person>& createPerson()
{
    switch (getRandomNumber(0, 2))
    {
    case 0:
        {
        unique_ptr<Person> person = make_unique<Formalist>(RANDNAME_0_10, RANDAGE_20_40);
        return person; 
        }
    case 1:
        {
        unique_ptr<Person> person = make_unique<Neformal>(RANDNAME_0_10, RANDAGE_20_40);
        return person;
        }
    case 2:
        {
        unique_ptr<Person> person = make_unique<Realist>(RANDNAME_0_10, RANDAGE_20_40);
        return person;
        }
    default:
        {
        unique_ptr<Person> person = nullptr;
        return person;
        }
    };
}

void createGroup(vector<unique_ptr<Person>>& group, int8_t size)
{
    unique_ptr<Person> tempPerson = nullptr;
    for (int i = 0; i < size; ++i)
    {
        tempPerson = std::move(createPerson());
        group.push_back(tempPerson);
    }
}

Не могу понять как инициализировать массив в createGroup() через unique_pointers. Нужно ли было изначально возвращать shared pointers из createPerson()? Или использовать перемещения с unique_pointers? И куда по правильному впихивать using namespace std; если не в global scope? Спасибо заранее

Полный код
https://repl.it/@radionnazmiev/Ravesli#Ravesli_12_H/main.cpp

radionnazmiev, 06 Декабря 2020

Комментарии (54)

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

+1

01
02
03
04
05
06
07
08
09
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

#include <iostream>
#include <ctime>
#include <random>
#include <windows.h>
#include <cstdint>                             // Для int8_t;
#include <string>   
#include <string_view> 
#include <array>                               
#include <iomanip>                           // setw();

using namespace std;

const array <string, 94> arr = { "олицетворение", "патология", "субтропики", "лекарь", "предосторожность", "монархист",
        "социум", "клякса", "борщевик", "лемур", "старьевщик", "марсельеза", "послушание", "прикол", "шифровка", "фантом", "субмарина",
        "гладиолус", "сообщество", "гениальность", "трансформация", "пальпация", "усушка", "благополучие", "капелла", "недоумение",
        "траверс", "проныра", "полоскание", "побережье", "закись", "кровопиец", "сыродел", "продвижение", "совместимость", "двойственность",
        "сабботаж", "рассылка", "куратор", "прорицание", "синтез", "негодяй", "паралич", "тяжелоатлет", "освободитель", "кунжут",
        "мордобой", "ностальгия", "хлорофилл", "первооткрыватель", "гипотенуза", "шкатулка", "психиатр", "маэстро", "прыть",
        "пустомеля", "здравоохранение", "алиментщик", "параметр", "аннотация", "пудинг", "прыжок", "пневматика", "штырь", "фасовщик",
        "совместитель", "свиновод", "яйцеварка", "зверобой", "тундра", "вогнутость", "рикошет", "компромисс", "фоторепортер",
        "кордебалет", "нумизмат", "гипербола", "желтизна", "брезент", "ботфорты", "гостинец", "символист", "акустика", "психика",
        "удивление", "уховертка", "снежинка", "меценат", "атомоход", "контрабас", "брюква", "миловидность", "спринтер", "пунктир" };

int randNumber(int min, int max)
{
    random_device rd;  //Will be used to obtain a seed for the random number engine
    mt19937 gen(rd()); //Standard mersenne_twister_engine seeded with rd()
    uniform_int_distribution<int> distrib(min, max);
    return distrib(gen);
}

string_view randString()
{
    return (string_view)arr[randNumber(0, arr.size())];
}

void printGuessedChar(string_view input, string_view randWord)
{
    for (int i = 0; i < randWord.size(); ++i)
    {
        for (int j = 0; j < input.size(); ++j)
        {
            if (randWord[i] == input[j])
            {
                cout << randWord[i];
                break;
            }
        }
        cout << '*';
    }
    cout << endl;
}

int randNumber(int min, int max)
{
    random_device rd;  //Will be used to obtain a seed for the random number engine
    mt19937 gen(rd()); //Standard mersenne_twister_engine seeded with rd()
    uniform_int_distribution<int> distrib(min, max);
    return distrib(gen);
}

void theGame()
{
    cout << "Добро пожаловать в игру 'Виселица'!" << endl;
    string_view randWord = randString();
    cout << "Компьютер сгенерировал слово" << endl;
    cout << "Слово - ";
    for (int i = 0; i < randWord.size(); ++i)
    {
        cout << '*';
    };
    cout << endl;
    int8_t count = 0;
    string input;
    do
    {
        cout << "У вас " << 6 - count << " попыток угадать слово." << endl << "Введите букву: ";
        if (cin.fail()) // если предыдущее извлечение оказалось неудачным,
        {
            cin.clear(); // то возвращаем cin в 'обычный' режим работы
            cin.ignore(32767, '\n'); // и удаляем значения предыдущего ввода из входного буфера
        }
        cin >> setw(1) >> input[count];
        printGuessedChar(input, randWord);
        ++count;
    } while (count != 6);
}

int main()
{
    SetConsoleCP(1251);
    SetConsoleOutputCP(1251);
    srand((unsigned)time(nullptr));
    theGame();
}

Заколебался уже с этим кодом. Не получается впихнуть пользовательский ввоз в массив...Первая попытка освоить std::string и std::string_view. Ну и в целом советы по коду приветствуются. Благодарю
//Давайте напишем популярную игру, которая называется «Виселица».В игре вам нужно угадать слово, которое загадала программа, буква за буквой.
//Игрок угадывает одну букву за раз и может ошибиться только 6 раз(после этого он проигрывает).
//Необходимый функционал вашей программы :
//создайте массив слов(например, поместите туда 40 слов) и рандомным образом выберите 1 слово для угадывания;
//программа должна выводить длину всего слова и отображать буквы, которые угадал игрок;
//после каждого неудачного угадывания, программа должна сообщить игроку, сколько у него осталось попыток неверно указать букву, прежде чем он проиграет;
//если человек указал букву, которую ранее уже угадывал, и она не дублируется в слове, то не наказывайте его, а просто предоставьте возможность угадать букву еще раз.
//Например, компьютер загадал слово INTERESTING :
//
//Добро пожаловать в игру "Виселица"!
//Слово - _ _ _ _ _ _ _ _ _ _ _
//Угадайте букву : S
//Верно - _ _ _ _ _ _ S _ _ _ _
//Угадайте следующую букву : F
//Неверно!Такой буквы нет, у вас осталось 5 попыток неверно указать букву!
//...

radionnazmiev, 30 Ноября 2020

Комментарии (33)

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

0

01
02
03
04
05
06
07
08
09
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

#include <iostream>
#include <ctime>
#include <string>
#include <random>
#include <algorithm>
#include <iomanip> // для ограничения количества вводимых симолов для std::cin

void compUsrWthCmptr(std::string userInput, std::string computerInput)
{
    std::reverse(userInput.begin(), userInput.end());//  Считаем правильно угаданные позиции
    std::reverse(computerInput.begin(), computerInput.end());
    int guessedPositions{ 0 };
    for (int i = 0; (i < userInput.length()) && (i < computerInput.length()); ++i)
    {
        if (userInput[i] == computerInput[i])
        {
            guessedPositions++;
        }
    }
    std::string::iterator it_userInput;
    std::string::iterator it_computerInput;
    it_userInput = std::unique(userInput.begin(), userInput.end()); // Удаляем повторяющиеся цифры
    userInput.resize(std::distance(userInput.begin(), it_userInput));
    it_computerInput = std::unique(computerInput.begin(), computerInput.end());
    computerInput.resize(std::distance(computerInput.begin(), it_computerInput));
    int guessedDigits{ 0 }; //  Считаем количество правильно угаданных цифр без учета повторяющихся
    for (int i = 0; i < userInput.length(); ++i)
    {
        for (int x = 0; x < computerInput.length(); ++x)
        {
            if (userInput[i] == computerInput[x])
            {
                guessedDigits++;
            }
        }
    }
    std::cout << "  Угадано: " << guessedDigits << ". Соответствует своим разрядам: " << guessedPositions << std::endl << std::endl;
};
void startTheGame()
{
    int pcsRandomNumber = getRandomNumber(0, 999);      //Загаданое число.
    std::cout << "  Компьютер загадал трехзначное число от 0 до 999!\n" << "  Это: " << pcsRandomNumber << std::endl << std::endl;
    std::string pcNumber{ std::to_string(pcsRandomNumber) };
    bool win = false;
    do
    {
        int usersGuess = getUsersGuess();
        std::string guess{ std::to_string(usersGuess) };
        std::cout << "  Ваш вариант : " << guess << std::endl;
        compUsrWthCmptr(guess, pcNumber);
        if (usersGuess == pcsRandomNumber)
        {
            win = true;
            std::cout << "  *** Вы угадали число " << pcsRandomNumber << "!***\n";
        }
    } while (!win);
};
int getUsersGuess()
{
    while (true) // цикл продолжается до тех пор, пока пользователь не введет корректное значение
    {
        std::cout << "  Введите коректное значение: ";
        int a;
        std::cin >> std::setw(3) >> a;
        if (std::cin.fail()) // если предыдущее извлечение оказалось неудачным,
        {
            std::cin.clear(); // то возвращаем cin в 'обычный' режим работы
            std::cin.ignore(32767, '\n'); // и удаляем значения предыдущего ввода из входного буфера
            std::cout << "  Предыдущее извлечение оказалось неудачным. Попытайтесь еще раз.\n\n";
        }
        else
        {
            if (a >= 1000 || a < 0)
            {
                std::cin.ignore(32767, '\n'); // удаляем лишние значения
                std::cout << "  Введенное число вне требуемого диапазонате. Попытайтесь еще раз.\n\n";
            }
            else
            {
                std::cin.ignore(32767, '\n'); // удаляем лишние значения
                return a;
            }
        }
    }
}
int getRandomNumber(int min, int max)
{
    return static_cast<int>(rand() % (max - min + 1) + min);
}

int main()
{
    setlocale(LC_ALL, "Russian");
    srand(static_cast<unsigned int>(time(0)));
    startTheGame();
    return 0;
}

Начинающий говнокодер просит оценить его код. Где/что можно улучшить если возможно. Благодарю
//Напишите программу реализующую игру «Угадай число».Компьютер загадывает число от 0 до 999 (используйте генерацию случайных чисел),
//а пользователь угадывает его.На каждом шаге угадывающий делает предположение, а задумавший число — сообщает, сколько цифр из числа угаданы
//и сколько из угаданных цифр занимают правильные позиции в числе.Например, если задумано число 725 и выдвинуто предположение,
//что задумано число 523, то угаданы две цифры(5 и 2) и одна из них занимает верную позицию.

radionnazmiev, 25 Ноября 2020

Комментарии (29)

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

+2

01
02
03
04
05
06
07
08
09
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

ShipType Ship::getShipTypeByLength(int length)
{
    switch (length) {
        case 1: return BOAT;
        case 2: return CRUISER;
        case 3: return DESTROYER;
        case 4: return BATTLESHIP;
        case 5: return AIRCRAFT_CARRIER;

        default:
            std::cout << "No ship meets the given length: " << length << std::endl;
            return ERROR_SHIP;
    }
}

int Ship::getShipLengthByType(ShipType type)
{
    switch (type) {
        case BOAT: return 1;
        case CRUISER: return 2;
        case DESTROYER: return 3;
        case BATTLESHIP: return 4;
        case AIRCRAFT_CARRIER: return 5;

        default:
            std::cout << "No ship meets the given type: " << type << std::endl;
            return 0;
    }
}

int Ship::getShipAmountByType(ShipType type)
{
    switch (type) {
        case BOAT: return 4;
        case CRUISER: return 3;
        case DESTROYER: return 2;
        case BATTLESHIP: return 1;
        case AIRCRAFT_CARRIER: return 1;

        default:
            std::cout << "No ship meets the given type: " << type << std::endl;
            return 0;
    }
}

Coordinates Ship::getFirstBlockCoordinatesByShipData(int x, int y, int length, Orientation orientation)
{
    Coordinates result;
    if (orientation == HORIZONTAL) {
        result.x = x - (length / 2);
        result.y = y;
    } else {
        result.x = x;
        result.y = y - (length / 2);
    }
    return result;
}

Coordinates Ship::getLastBlockCoordinatesByShipData(int x, int y, int length, Orientation orientation)
{
    Coordinates result;
    if (orientation == HORIZONTAL) {
        result.x = x + (length / 2) + (length % 2) - 1;
        result.y = y;
    } else {
        result.x = x;
        result.y = y + (length / 2) + (length % 2) - 1;
    }
    return result;
}

Вот некоторые полезные функции из игры «Морской Бой», которую я зачем-то написал.

oaoaoammm, 21 Ноября 2020

Комментарии (44)

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

+1

01
02
03
04
05
06
07
08
09
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

#include <iostream>
using namespace std;

int del(int s)
{
	int k = 0;
	for (int i = 1;i <= 20;i++)
	{
		if (s % i == 0)
		{
			k++;
		};

	};
	return k == 20;

}

int main()
{
	unsigned long int q = 1;
	for (int i = 1;i <= 20;i++)
	{
		q = q*i;
	}
	for (int i=1;i<=q;i++)
	{
		if (del(i))
		{
			cout << i;
			break;
		}
	}
	return 0;
}

2520 - самое маленькое число, которое делится без остатка на все числа от 1 до 10.

Какое самое маленькое число делится нацело на все числа от 1 до 20?

Что за неоптимизированное говно я сейчас сделал.........

NikEral7, 19 Ноября 2020

Комментарии (467)

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

0

1
2
3
4

pituhs.reserve(kurochkas.size());
for (auto& kurochka : kurochkas) {
    pituhs.push_back(kurochka.snesti_jajichko());
}

reserve заебал.
Коллега (да, тот же самый) пихает его везде.
Я понимаю зачем это нужно, но блядь, я читаю на одну строку больше, чем мог бы, и лишний раз напрягаю мозг.
И это сливается со словом «reverse».

Кстати, оптимизаторы могли бы такую хуйню сами детектить, и резервировать сами.

3_dar, 12 Ноября 2020

Комментарии (122)

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

+2

1

Class::Class(Pethu pethu) : pethu(std::move(pethu)) {

std::move заебал. Просто взял, блядь, — и заебал!
Чем это лучше передачи по ссылке?

guestinxo, 12 Ноября 2020

Комментарии (26)

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

+1

1
2

using T = int;
0 .T::~T();

Чем-то меня эта (полностью валидная) коньструкция очаровала.

gost, 12 Ноября 2020

Комментарии (25)

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

0

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20

void two(string str, int *mstr,int l)    //Замена цифр в строке
{
    string base[10] = { "ноль","один","два","три","четыре","пять","шесть","семь","восемь","девять" };    //Строки для замены
    int i,j,t;

    for (i = 0;i < l;i++)
    {
        str.insert(mstr[i]+1, base[atoi(&str[mstr[i]])]);    //Вставка после числа в строку
        t = base[atoi(&str[mstr[i]])].length();        //Смещение последующих чисел в строке
        str.erase(mstr[i], 1);        //Удаление цифры в строке

        for (j = i; j < l;j++)        //Новые позиции чисел в строке
        {
            mstr[j] += t-1;
        }

    }

    cout << "Изменённая строка: " << str << endl;
}

По заданию требовалось обработать символьную строку так, чтобы цифры записывались числительными. В этой функции идет замена цифр в строке. Массив str - строка символов, mstr - int массив, куда записывается позиция числа в строке (т.е в строке aaaa1aaa mstr[0]=4), а l - количество чисел в строке.
Код не мой. Это ад. Неработающий.
Сидел, ржал.

ShadowCat, 07 Ноября 2020

Комментарии (33)

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

0

01
02
03
04
05
06
07
08
09
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

//библиотеки cuda_runtime.h и device_launch_parameters.h
//для работы с cyda
#include "cuda_runtime.h"
#include "device_launch_parameters.h"
#include<vector>
#include<string>//для getline
#include <stdio.h>
#include<fstream>
using namespace std;
__global__ void Upload_to_GPU(unsigned long long  *Number,unsigned long long  *Stepn, bool *Stop,unsigned long long  *INPUT,unsigned long long  *max) {
	int thread = threadIdx.x;
	unsigned long long  MAX_DEGREE_OF = max[0];
    int X = thread;
	unsigned long long  Calculated_number = 1;
	unsigned long long  Current_degree_of_number = 2;
    unsigned long long   Original_numberP = INPUT[0];
	Stop[thread] = false;
	bool BREAK = false;
	if (X!=0&&X!=1) {
		while (!BREAK) {
			if (Current_degree_of_number <= MAX_DEGREE_OF) {
				Calculated_number = 1;
				for (int counter = 0; counter < Current_degree_of_number; counter++) {
				 Calculated_number	*=X;
				}
				if (Calculated_number == Original_numberP) {
					Stepn[thread] = Current_degree_of_number;
					Number[thread] = X;
					Stop[thread] = true;
					BREAK = true;
				}
				Current_degree_of_number++;
			}
			else { BREAK = true; }
		}
	}
}

https://habr.com/post/525892/
> Сравнение времени выполнения алгоритма на CPU и GPU

gost, 31 Октября 2020

Комментарии (70)

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