Оцените приложение. Попробуйте его сломать

@Laborant666 · Регистрация: 30.06.2020

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

Собственно, прошу потестировать приложение. В случае незапуска, ошибок, отказа работы прошу рассказать, что было сделано, на какой конфигурации пк. В принципе в работе я не сомневаюсь, больше жду идей и советов по улучшению. Суть приложения кликать за вас и оберегать вашу мышь в беспощадных кликерах

Комментарий модератора

П.5.18.Правил
Запрещено размещать задания и решения в виде картинок и других файлов с их текстом.

@NQUARE · 26.01.2021, 15:40

Laborant666, Не вот прям конечно, но у меня g++ (x86_64-win32-seh-rev0, Built by MinGW-W64 project) 8.1.0 агрится:

Code
1
2
main.cpp:21:15: warning: passing NULL to non-pointer argument 1 of 'void srand(unsigned int)' [-Wconversion-null]                                               
     srand(NULL);

Думаю надо srand((unsigned int)time(NULL));

Добавлено через 53 минуты
Да и вся программа в целом не очень правильно и кривовато написана. Не работает, виснет после ввода данных. Практически все функции WinAPI не правильно используются. Крч там править и править. Вечером сделаю.

Алексей1153 · 26.01.2021, 15:51

Сообщение от Laborant666

потому что если оставить в теле цикла Sleep(10), то итераций цикла будет 100 в секунду всегда, а с моим вариантом слипа будет от одной итерации до двадцати.

Sleep(10) - это означает, что цикл практически вообще не будет нагружать ЦП, так как время будет постоянно отдаваться другим потокам
А в твоём варианте нагрузка будет чуть почти такая же (то есть - никакая), но всё же чуть-чуть побольше, так как будет выполняться деление

Очевидно, что Sleep(10) - выбор очевидный (блин, тавталогия вышла

) . Он проще и "производительнее"

@Laborant666 · 26.01.2021, 22:02 **[ТС]**

Сообщение от NQUARE

Да и вся программа в целом не очень правильно и кривовато написана. Не работает, виснет после ввода данных. Практически все функции WinAPI не правильно используются. Крч там править и править. Вечером сделаю.

Да? Странно, у меня всё работало хорошо. Косяк по поводу несоответствия выбранных кликов с реальным их количеством (не критично потому и не фикшу), а так стабильно работает
Буду рад глянуть на рефакторинг кода, если можно - комментариев накидать, что на что изменили и почему

Добавлено через 2 минуты
NQUARE, Кстати, криво работает какая версия кода из темы? Код в самой теме один, код в 10м сообщении в обсуждении уже подправленный (там 100% нагрузка на ядро исправлена)

Добавлено через 16 минут

Сообщение от Алексей1153

А в твоём варианте нагрузка будет чуть почти такая же (то есть - никакая), но всё же чуть-чуть побольше, так как будет выполняться деление

Но в Sleep(1000\count_click) значение count_click известно до входа в цикл. Это не избавляет от проблемы рассчета выражения каждую итерацию? Думал раз известны все составляющие рассчета до входа в цикл - обсчет произойдет один раз, а не каждую итерацию. Получается желательно всегда передавать уже готовое значение, даже если в формуле всё известно до входа в цикл?

Алексей1153 · 27.01.2021, 07:19

Laborant666, это же не константа (как enum, enum class, constexpr или, не приснись на ночь, макрос ). Оптимизатор может перенести перед циклом, а может и не перенести.
Дело не в этом.
А в том, что по влиянию на быстродействие обе конструкции одинаковы (какие-то микросекунды, если они и будут, никто не заметит). А из одинакового нужно выбирать то, что проще. Проще для написания и для чтения

@NQUARE · 27.01.2021, 14:30

Сообщение от Laborant666

Буду рад глянуть на рефакторинг кода, если можно - комментариев накидать, что на что изменили и почему

Ок, пока руки ещё не доходили. Но у меня комментарии длинные получаются)

Сообщение от Laborant666

Косяк по поводу несоответствия выбранных кликов с реальным их количеством (не критично потому и не фикшу)

Да не, я не про это. У меня даже кликать не начинает, зависает после вывода инструкции. Версия кода последняя, все сообщения читал.

Добавлено через 1 минуту
А зачем кста надо было random делать? Типа чтобы просто эффект был или что?

@Laborant666 · 27.01.2021, 14:40 **[ТС]**

Сообщение от NQUARE

А зачем кста надо было random делать? Типа чтобы просто эффект был или что?

А это я выпилить забыл часть кода. Прога кликер же, некоторые игры блочат кнопку, если в одном и том же месте кликается слишком часто, вот рандом был сделан для того, что б кликало в радиусе 5 пикселей

Добавлено через 27 секунд

Сообщение от NQUARE

У меня даже кликать не начинает, зависает после вывода инструкции.

Странно, у друзей тестировал - работало

Добавлено через 40 секунд

Сообщение от NQUARE

Ок, пока руки ещё не доходили. Но у меня комментарии длинные получаются)

Ничего страшного, мне будет интересно узнать где я коряво пишу, всё же профессионально я не обучался

Добавлено через 4 минуты

Сообщение от NQUARE

У меня даже кликать не начинает, зависает после вывода инструкции.

Не реагирует программа вообще никак? +\- с нумпада надо жать. В версии кода из 10го сообщения для кликов надо держать постоянно +

@NQUARE · 27.01.2021, 20:25

Сообщение от Laborant666

Не реагирует программа вообще никак? +\- с нумпада надо жать. В версии кода из 10го сообщения для кликов надо держать постоянно +

Нет, вообще никак, полный 0

@Laborant666 · 27.01.2021, 20:29 **[ТС]**

Сообщение от NQUARE

Нет, вообще никак, полный 0

Хм. А вы из под-винды запускаете?

Добавлено через 1 минуту
Возможно коды клавиш не совпадают у меня и у вас, хотя по идее должны, с других пк стартовала прога

@NQUARE · 27.01.2021, 20:46

Сообщение от Laborant666

Хм. А вы из под-винды запускаете?

Ну канеш.

Сообщение от Laborant666

Возможно коды клавиш не совпадают у меня и у вас, хотя по идее должны, с других пк стартовала прога

Не. Точно нет. Во первых они везде одинаковые (по ASCII таблице) а во вторых я перепробовал ещё некоторые. К стати по ASCII таблице у вас коды не правильные, я недавно тож прогу делал и смотрел код в ASCII для +, он там 46 ли 42 ли, но ни как не 10x.

@Laborant666 · 27.01.2021, 22:47 **[ТС]**

NQUARE, GetKeyState работает не по ascii таблице, а именно по кодам клавиш. ASCII это коды символов. Если б я использовал сравнение через kbhit() где по ascii и сравниваешь - окно должно было быть всегда активно.GetKeyState отлавливает нажатия всегда

@NQUARE · 27.01.2021, 23:11

А почему интересно тогда у меня с GetKeyState и ASCII всё норм работало и работает. Хмм...

@Laborant666 · 28.01.2021, 17:22 **[ТС]**

Сообщение от NQUARE

А почему интересно тогда у меня с GetKeyState и ASCII всё норм работало и работает. Хмм...

Может это особенности среды? Я из VS 2019 запускал и писал. Совершенно не понимаю почему у вас нужны другие коды для работы, не те что работают у меня

Добавлено через 3 минуты
NQUARE, не понимаю, по документации GetKeyState действительно работает через ascii, но у меня работает почему-то через другую таблицу. Я если в параметр вобью код из ascii - у меня GetKeyState не сработает. В ascii есть коды цифро-буквенные, в той таблице, которую я использовал все коды лишь циферные

Добавлено через 5 минут
NQUARE, что самое забавное я делал цикл от 0 до 256 и прогонял его, в цикле было условие вернуть i когда сработает условие if( GetKeyState(i) < 0). И возвращало не значение из ascii

Добавлено через 1 минуту
Мне лень было в таблице значения каждый раз искать, я программку делал, что б зажать клавишу и получить код через секунду. На основе этих кодов и написал код, который у меня работает, а у вас почему-то нет. Что-то по-разному настроено?

Добавлено через 4 часа 34 минуты
NQUARE, проверьте работает у вас этот код?

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
#include <iostream>
#include <Windows.h>
int key = 404;
int main()
{
    std::cout << "Press a key\n";
    Sleep(3000);
 
    for (int d = 0; d < 256; d++)
    {
        if (GetKeyState(d) < 0)
        {
            key = d;
            break;
        }
    }
 
    if (key != 404)
        std::cout << "Code of Key is - " << key << "\n";
    else
        std::cout << "Can`t find a key\n";
 
    Sleep(5000);
 
    return 0;
}

Добавлено через 2 минуты
Я этим кодом находил требуемые коды клавиш для запуска. Минус на нумпаде должен выдать 109, плюс - 107, звездочка - 106. Плюс/равно без шифта на основной клавиатуре - 187, минус/нижний прочерк без шифта на основной - 189. Совпадают ли ваши значения с моими?

Добавлено через 4 минуты
NQUARE, вот кстати таблица с кодами клавиш, значения которой я использовал. Они с результатами моей программы, что даёт код нажатой клавиши - совпадают. Только почему-то таблица в справочнике borland c++
http://netcode.ru/cpp/?artID=234

Добавлено через 11 минут
NQUARE, вот. http://jquery.page2page.ru/ind... в_и_клавиш у меня в GetKeyState возвращаются коды клавиш. ascii при изменении раскладки выдаст разные значения на одну и ту же клавишу. GetKeyState при любой раскладке выдает одинаковые значения. Жду ваших ответов с тестами моего кода, интересно найти проблему разного поведения на моём и вашем пк

@NQUARE · 29.01.2021, 13:17

Сообщение от Laborant666

Может это особенности среды

У меня Sublime Text 4 и MinGW

Добавлено через 1 минуту

Сообщение от Laborant666

не понимаю, по документации GetKeyState действительно работает через ascii

Да, вот и я про то же.

Добавлено через 1 минуту

Сообщение от Laborant666

но у меня работает почему-то через другую таблицу. Я если в параметр вобью код из ascii - у меня GetKeyState не сработает

А у меня всё наоборот.

Добавлено через 2 минуты

Сообщение от Laborant666

что самое забавное я делал цикл от 0 до 256 и прогонял его, в цикле было условие вернуть i когда сработает условие if( GetKeyState(i) < 0). И возвращало не значение из ascii

Что у вас за странное условие? Что оно значит? Зачем вы сравниваете, так ведь запутаться можно. без ничего надо вообще, функция ведь true-false возвращает.

Добавлено через 1 минуту

Сообщение от Laborant666

В ascii есть коды цифро-буквенные

В смысле??? Где вы их там нашли?? Там только цифры от 0 до 256.

@oleg-m1973 · 29.01.2021, 13:17

Сообщение от NQUARE

Что у вас за странное условие? Что оно значит? Зачем вы сравниваете, так ведь запутаться можно. без ничего надо вообще, функция ведь true-false возвращает.

Нет, она возвращает состояние клавиши

Return value
Type: SHORT

The return value specifies the status of the specified virtual key, as follows:

If the high-order bit is 1, the key is down; otherwise, it is up.
If the low-order bit is 1, the key is toggled. A key, such as the CAPS LOCK key, is toggled if it is turned on. The key is off and untoggled if the low-order bit is 0. A toggle key's indicator light (if any) on the keyboard will be on when the key is toggled, and off when the key is untoggled.

@NQUARE · 29.01.2021, 13:23

Сообщение от Laborant666

вот кстати таблица с кодами клавиш, значения которой я использовал. Они с результатами моей программы, что даёт код нажатой клавиши - совпадают. Только почему-то таблица в справочнике borland c++

Зачем искать всякие таблице в недрах интернета? Она же не для функции GetKeyState().

Добавлено через 2 минуты

Сообщение от Laborant666

вот. http://jquery.page2page.ru/ind... в_и_клавиш

Стоп. Ну дык там же та самая ASCII таблица...

Добавлено через 1 минуту

Сообщение от oleg-m1973

Нет, она возвращает состояние клавиши

Ну это же практически true/false
У меня вот так всё работает на клавише '+':

C++
1
if (GetAsyncKeyState(48)) deg = !deg;

@Laborant666 · 29.01.2021, 13:52 **[ТС]**

Сообщение от NQUARE

Стоп. Ну дык там же та самая ASCII таблица..

Верхняя таблица это коды символов. Они частично совпадают с ascii. Ниже таблица это коды клавиш w и ц вернут один код, ибо это одна и та же клавиша. Вот потому то и не ascii в GetKeyState. В MSDN написано ascii, но видать давно не редачили.

Добавлено через 2 минуты

Сообщение от NQUARE

Ну это же практически true/false
У меня вот так всё работает на клавише '+':

Не тру\фолс. Сравнение с нулем - выдаст тру при зажатой клавише. Без сравнения вообще - нажал кнопку - тру. И будет тру пока ещё раз не нажмешь. Сравнение больше нуля - будет тру при отжатой клавише, вам правильно сказали оно возрвщает состояние клавиши. Плюс вы в своём примере использует GetAsyncKeyState. А это немного другое. Мой код запустите, что я вот вчера кидал и позажимайте клавиши и скажите какая клавиша какой код показала

Добавлено через 2 минуты
NQUARE, запустите этот код и покажите что у вас показывает. Запустили, зажали клавишу после надписи и через три секунды получили код зажатой клавиши. Протыкайте +(плюс) -(минус) *(звезда) на нумпаде, =(равно) -(минус) на основной клавиатуре

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
#include <iostream>
#include <Windows.h>
int key = 404;
int main()
{
    std::cout << "Press a key\n";
    Sleep(3000);
 
    for (int d = 0; d < 256; d++)
    {
        if (GetKeyState(d) < 0)
        {
            key = d;
            break;
        }
    }
 
    if (key != 404)
        std::cout << "Code of Key is - " << key << "\n";
    else
        std::cout << "Can`t find a key\n";
 
    Sleep(5000);
 
    return 0;
}

Новые блоги и статьи Все статьи Все блоги /
Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы, точка. Programma_Boinc 23.12.2025 Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы, точка. Рецензия / Мнение Это мой обзор планшета X220 с точки зрения школьника. Недавно я решила попытаться уменьшить свой. . .	PhpStorm 2025.3: WSL Terminal всегда стартует в ~ and_y87 14.12.2025 PhpStorm 2025. 3: WSL Terminal всегда стартует в ~ (home), игнорируя директорию проекта Симптом: После обновления до PhpStorm 2025. 3 встроенный терминал WSL открывается в домашней директории. . .	Как объединить две одинаковые БД Access с разными данными VikBal 11.12.2025 Помогите пожалуйста !! Как объединить 2 одинаковые БД Access с разными данными.	Новый ноутбук volvo 07.12.2025 Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro	Музыка, написанная Искусственным Интеллектом volvo 04.12.2025 Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .
От async/await к виртуальным потокам в Python IndentationError 23.11.2025 Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .	Поиск "дружественных имён" СОМ портов Argus19 22.11.2025 Поиск "дружественных имён" СОМ портов На странице: https:/ / norseev. ru/ 2018/ 01/ 04/ comportlist_windows/ нашёл схожую тему. Там приведён код на С++, который показывает только имена СОМ портов, типа,. . .	Сколько Государство потратило денег на меня, обеспечивая инсулином. Programma_Boinc 20.11.2025 Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .	Ломающие изменения в C#.NStar Alpha Etyuhibosecyu 20.11.2025 Уже можно не только тестировать, но и пользоваться C#. NStar - писать оконные приложения, содержащие надписи, кнопки, текстовые поля и даже изображения, например, моя игра "Три в ряд" написана на этом. . .	Мысли в слух kumehtar 18.11.2025 Кстати, совсем недавно имел разговор на тему медитаций с людьми. И обнаружил, что они вообще не понимают что такое медитация и зачем она нужна. Самые базовые вещи. Для них это - когда просто люди. . .

Алексей1153 фрилансер 6444 / 5637 / 1128 Регистрация: 11.10.2019 Сообщений: 14,994
	27.01.2021, 07:19
	Laborant666, это же не константа (как enum, enum class, constexpr или, не приснись на ночь, макрос ). Оптимизатор может перенести перед циклом, а может и не перенести. Дело не в этом. А в том, что по влиянию на быстродействие обе конструкции одинаковы (какие-то микросекунды, если они и будут, никто не заметит). А из одинакового нужно выбирать то, что проще. Проще для написания и для чтения 1

@Laborant666 208 / 101 / 20 Регистрация: 30.06.2020 Сообщений: 848
	27.01.2021, 22:47 [ТС]
	NQUARE, GetKeyState работает не по ascii таблице, а именно по кодам клавиш. ASCII это коды символов. Если б я использовал сравнение через kbhit() где по ascii и сравниваешь - окно должно было быть всегда активно.GetKeyState отлавливает нажатия всегда 0

@NQUARE Искусственный интеллект 112 / 82 / 34 Регистрация: 26.08.2020 Сообщений: 590 Записей в блоге: 1
	27.01.2021, 23:11
	А почему интересно тогда у меня с GetKeyState и ASCII всё норм работало и работает. Хмм... 0