Форум программистов, компьютерный форум, киберфорум
C# для начинающих
Войти
Регистрация
Восстановить пароль
Блоги Сообщество Поиск Заказать работу  
 
Рейтинг 5.00/15: Рейтинг темы: голосов - 15, средняя оценка - 5.00
1 / 1 / 0
Регистрация: 13.02.2015
Сообщений: 11

Проверка кода на гавнокод и его оптимизация

17.02.2015, 22:19. Показов 3048. Ответов 13
Метки нет (Все метки)

Студворк — интернет-сервис помощи студентам
Здраствуйте! я начию програмировать и пишу совсем несложный код, но хочу спросить у знающих как мне правильно его оптимизировать и убрать/добавить что-то. Хочу сразу учиться грамотно писать и поэтому я здесь)
Так вот, предпоссылкой к этому коду стало пополнение счета телефона через терминал. Вот так вот сам себе придумал, я думаю с кода станет всё ясно.
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
using System;
 
namespace ReplenishmentPhone
{
    class Program
    {
        static void Main(string[] args)
        {
 
            double commision;
            bool exit = true;
           
 
            Console.WriteLine("Welcome to cash opperation");
            Console.WriteLine();
            Console.WriteLine("\n0.01 to 4.99 grn commision = 3%" +
                              "\n5.00 to 14.99 grn commision = 3.5%"+
                              "\n15.00 to more commision = 4%");
            Console.WriteLine();
           
            
            while (exit)
            {
                
                Console.Write("Insert your cash: ");
                double cash = Convert.ToDouble(Console.ReadLine());
 
                if (cash <= 4.99)
                {
                   commision = cash / 100 * 3;
                   cash = cash - commision;
                   Console.WriteLine("Credited to your account: {0} grn \nCommision = {1} grn", cash, commision);
                }
                else if (cash <= 5.00 & cash >= 14.99 )
                {
                    commision = cash / 100 * 3.5;
                    cash = cash - commision;
                    Console.WriteLine("Credited to your account: {0} grn \nCommision = {1} grn", cash, commision);
                }
                else if (cash >= 15.00)
                {
                    commision = cash / 100 * 4;
                    cash = cash - commision;
                    Console.WriteLine("Credited to your account: {0} grn \nCommision = {1} grn", cash, commision);
                }
  
                Console.WriteLine();
                Console.WriteLine("Do you want to Pay more? ");
                
                string askToExit = Console.ReadLine();
                if (askToExit == "n")
                {
                  Console.WriteLine("Spa-si-bo za to sho pi4kalu v nas bablo");
                  exit = false;  
                }
                    
            }
            Console.ReadLine();
        }
    }
}
0
Лучшие ответы (1)
Programming
Эксперт
39485 / 9562 / 3019
Регистрация: 12.04.2006
Сообщений: 41,671
Блог
17.02.2015, 22:19
Ответы с готовыми решениями:

Проверка кода сайта на ошибки и оптимизация
Всем привет! Я начинающий веб-разработчик. Написал свой первый сайт и хотел бы что-бы вы проверили его на всякого рода ошибки, неправильную...

Проверка кода и советы по его улучшению. Генерация случайных чисел и проверка на уникальность
Привет народ! Занимаюсь изучением Java, захотелось реализовать выборку случайных чисел для лотереи. Вроде бы все прописал и программа даже...

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

13
Жуткая тВарЬ
394 / 329 / 135
Регистрация: 06.02.2015
Сообщений: 962
Записей в блоге: 1
17.02.2015, 22:25
1. Ну во первых оформите код правильно, в тегах ибо в таком виде он плохо читаем.

2. Во вторых если Вы начинаете кодить, то "правильно писать" для Вас рано, для начала надо изучить сам язык, потом познакомиться с основными фреймворками и технологиями (узнать, что они могут и куда копать - не обязательно их изучать досконально).

3. В третьих, после всего вышеперечисленного, Вы должны обратить свой взор на основные патерны и шаблоны проектирования программ и спустя какое то время (наверно 1-2 года) у Вас появиться понимание правильного кода, хотя при желании можно и быстрее.

П.С. это моя личная точка зрения, прошу без злых коментариев.
1
 Аватар для Metall_Version
2152 / 1289 / 516
Регистрация: 04.03.2014
Сообщений: 4,092
17.02.2015, 22:29
KopeGuss, нормуль. только все числовые и строковые константы выноси в начало класса или метода, и делай их константами.

Добавлено через 51 секунду
Цитата Сообщение от KopeGuss Посмотреть сообщение
C#
1
if (cash <= 5.00 & cash >= 14.99 )
и юзай укороченные операторы логики
C#
1
if (cash <= 5.00 && cash >= 14.99 )
1
Жуткая тВарЬ
394 / 329 / 135
Регистрация: 06.02.2015
Сообщений: 962
Записей в блоге: 1
17.02.2015, 22:33
По коду:
У Вас все в одном методе, попробуйте разбить на классы и методы по принципу 1 метод 1 операция, 1 клас одна сущность (деньги, комиссия, управление операциями) - будет интересно и за счет маленьких участков кода будет проще не допускать ошибок .

Добавлено через 1 минуту
Цитата Сообщение от Metall_Version Посмотреть сообщение
и юзай укороченные операторы логики
тут бы надо объяснить разницу между & и && - во втором случае если первое условие не выполняется, то второе уже не будет проверяться
1
1 / 1 / 0
Регистрация: 13.02.2015
Сообщений: 11
17.02.2015, 22:45  [ТС]
amarf, Ну я тут новичек, и как код тут правильно вставлять незнаю) - просто нажал C# и вставил код)
к конструктивной критике отношусь положительно и агресии непроявляю) Так что спасибо Вашу точку мнения

Добавлено через 2 минуты
Metall_Version, имеешь ввиду сделать констатнтами % ставку и вывести ёё в начало кода?
За && прошу извинения, наверное незаметно вытер при корректеровке, так как с самого начала писал &&.

Добавлено через 2 минуты
amarf, С класами чуть более сложнее мне) Я начал писать был в класах, хотел вынести логику
C#
1
2
commision = cash / 100 * 3;
cash = cash - commision;
в отдельный клас, но чет намудрил, и решил писать всё в одном)
0
 Аватар для Metall_Version
2152 / 1289 / 516
Регистрация: 04.03.2014
Сообщений: 4,092
17.02.2015, 22:48
Лучший ответ Сообщение было отмечено KopeGuss как решение

Решение

Цитата Сообщение от KopeGuss Посмотреть сообщение
имеешь ввиду сделать констатнтами % ставку и вывести ёё в начало кода?
ага, но это так, приучение к хорошему стилю, на такой программке это вообще не критично
что-то критиковать и советовать на простейшей консольной проге не удасться, вот когда напишешь прогу здоровою, с парочкой классами хотя бы, тогда можно покритиковать
1
1 / 1 / 0
Регистрация: 13.02.2015
Сообщений: 11
17.02.2015, 22:56  [ТС]
Metall_Version, будем идти к усложнению заданий, и классовой работе) и может тогда я смогу писать что-то более интересное)))))
0
Жуткая тВарЬ
394 / 329 / 135
Регистрация: 06.02.2015
Сообщений: 962
Записей в блоге: 1
17.02.2015, 23:00
Цитата Сообщение от KopeGuss Посмотреть сообщение
к конструктивной критике отношусь положительно и агресии непроявляю
Поскольку Вы решили изучать программирование - это уже заслуживает похвалы и уважения, главное не бросить, оно много где пригодиться, особенно в работе (если таковая связана с компьютером как таковым)

То, что я написал это не критика это мой личный путь изучения языка в сокращенном варианте - и при этом я еще дофига чего не понимаю +).
Осознание, как правильно писать программы, появляется не сразу, а с опытом.

Присоединяюсь к Metall_Version, будем ждать от Вас более объектно ориентированных программ, с классами и объектами +) Удачи!
2
1 / 1 / 0
Регистрация: 13.02.2015
Сообщений: 11
17.02.2015, 23:04  [ТС]
amarf, Спасибо большое) Работа моя мало связана с компютерами, я каг бы психолог а вот кодинг увидел у знакомого, и меня чет зацепило) Но тут чет появилась неувязочка, я то математику в школе плохо знал) и стил мышления у меня тоже не особо програмирочный) я креативщик, а тут надо - цитирую стар трек "Логика Спок, Логика..." Но! я думаю, что всё получитсья
Спасибо Вам за хорошие слова
1
17.02.2015, 23:15

Не по теме:

Цитата Сообщение от KopeGuss Посмотреть сообщение
я то математику в школе плохо знал
я ее до сих пор не знаю..
точнее забыл все, всегда все вспоминать приходится, как-будто заново учить

0
1 / 1 / 0
Регистрация: 13.02.2015
Сообщений: 11
17.02.2015, 23:17  [ТС]
Metall_Version, Значит надо будет прислушиваться к словам Ленина) Еще раз спасибо!
0
17.02.2015, 23:55

Не по теме:

Цитата Сообщение от KopeGuss Посмотреть сообщение
я каг бы психолог
увлекался я как то этим грязным делом (именно лекции из ВУзов слушал и читал книги) но Бурлан со своей СВП и его упертые бараны-последователи вынесли мне мозг и я решил отдохнуть. В психологии тоже по полочкам надо уметь раскладывать и диферинцировать как и в программировании +)

0
1 / 1 / 0
Регистрация: 13.02.2015
Сообщений: 11
18.02.2015, 00:51  [ТС]
amarf, Я психолог управления, будущий тренер =) Так что там есть своя калабурда)
// как делать прозрачное сообщение "не по теме"?
0
18.02.2015, 01:03

Не по теме:

KopeGuss, это особая магия, доступная программистам высших категорий :wizard:

0
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
inter-admin
Эксперт
29715 / 6470 / 2152
Регистрация: 06.03.2009
Сообщений: 28,500
Блог
18.02.2015, 01:03
Помогаю со студенческими работами здесь

Оптимизация методом Ньютона (нахождение точки минимума). Оптимизация кода
MATLAB только начал осваивать. Попытался реализовать нахождение точки минимума методом Ньютона для функции 2*X12 - X1*X2 + 3*X22 -...

Оптимизация кода. Замер времени выполнения части кода.
Доброе утро. Есть желание посмотреть сколько времени занимает выполнение какого-то блока кода/отдельной функции или процедуры/программы...

Типы оптимизация: черная оптимизация, серая оптимизация и белая оптимизация
Много много лет назад, на заре становления профессии &quot;оптимизатора&quot; в какой то умной книжке был создан миф. Это миф о цветовой индефикации...

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

Оптимизация кода
Доброго времени суток. Получен рабочий код, но работающий очень медленно (пока что рекорд 56 часов =D), мне бы его оптимизировать чутка. ...


Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
14
Ответ Создать тему
Новые блоги и статьи
Ритм жизни
kumehtar 27.02.2026
Иногда приходится жить в ритме, где дел становится всё больше, а вовлечения в происходящее — всё меньше. Плотный график не даёт вниманию закрепиться ни на одном событии. Утро начинается с быстрых,. . .
SDL3 для Web (WebAssembly): Сборка SDL3 и Box2D из исходников с помощью CMake и Emscripten
8Observer8 27.02.2026
Недавно вышла версия 3. 4. 2 библиотеки SDL3. На странице официальной релиза доступны исходники, готовые DLL (для x86, x64, arm64), а также библиотеки для разработки под Android, MinGW и Visual Studio. . . .
SDL3 для Web (WebAssembly): Реализация движения на Box2D v3 - трение и коллизии с повёрнутыми стенами
8Observer8 20.02.2026
Содержание блога Box2D позволяет легко создать главного героя, который не проходит сквозь стены и перемещается с заданным трением о препятствия, которые можно располагать под углом, как верхнее. . .
Конвертировать закладки radiotray-ng в m3u-плейлист
damix 19.02.2026
Это можно сделать скриптом для PowerShell. Использование . \СonvertRadiotrayToM3U. ps1 <path_to_bookmarks. json> Рядом с файлом bookmarks. json появится файл bookmarks. m3u с результатом. # Check if. . .
Семь CDC на одном интерфейсе: 5 U[S]ARTов, 1 CAN и 1 SSI
Eddy_Em 18.02.2026
Постепенно допиливаю свою "многоинтерфейсную плату". Выглядит вот так: https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11617&stc=1&d=1771445347 Основана на STM32F303RBT6. На борту пять. . .
Камера Toupcam IUA500KMA
Eddy_Em 12.02.2026
Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .
И ясному Солнцу
zbw 12.02.2026
И ясному Солнцу, и светлой Луне. В мире покоя нет и люди не могут жить в тишине. А жить им немного лет.
«Знание-Сила»
zbw 12.02.2026
«Знание-Сила» «Время-Деньги» «Деньги -Пуля»
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2026, CyberForum.ru