1 / 1 / 0
Регистрация: 02.03.2020
Сообщений: 106
|
||||||
1 | ||||||
Визуализация кода27.04.2020, 12:31. Показов 6305. Ответов 18
Метки нет (Все метки)
Здравствуйте, помогите сделать код визуально лучше, я из себя уже выжал что мог.
Кликните здесь для просмотра всего текста
0
|
27.04.2020, 12:31 | |
Ответы с готовыми решениями:
18
Визуализация Визуализация Примитивная 3D-визуализация Визуализация Алгоритма А* |
Just Do It!
|
||||||||||||||||
27.04.2020, 15:03 | 2 | |||||||||||||||
Сообщение было отмечено Evreii как решение
Решение
и ещё что всегда режет мой глаз и терзает мою душу) это когда часть кода спрятана за правой границей окна редактора. Код желательно(а для меня и обязательно) ВСЕГДА ДЕРЖАТЬ ПОЛНОСТЬЮ ВИДИМЫМ В ПРЕДЕЛАХ ВЕРТИКАЛЬНЫХ ГРАНИЦ РЕАДАКТОРА! Добавлено через 4 минуты Evreii, ещё пример:
и на экране такой вывод будет смотреться посимпотичнее т.к. похож на таблицу. или вот чисто по коду: у вас снарядом разорвало цикл for тут:
и тд и тп..
1
|
1 / 1 / 0
Регистрация: 02.03.2020
Сообщений: 106
|
||||||
27.04.2020, 17:42 [ТС] | 3 | |||||
Очень вам благодарен!
Добавлено через 40 минут Не могли бы вы еще как-то помочь избавиться от вложенности тут:
1
|
8739 / 4317 / 960
Регистрация: 15.11.2014
Сообщений: 9,760
|
||||||
27.04.2020, 17:52 | 4 | |||||
зачем на каждой итерации запрашиваешь размер?
в чем сакральный смысл использовать at ?и зачем ты на каждой новой строке его заново дергаешь? нафига ты везде юзаешь endl ?
0
|
1 / 1 / 0
Регистрация: 02.03.2020
Сообщений: 106
|
|
27.04.2020, 17:59 [ТС] | 5 |
0
|
Just Do It!
|
||||||
27.04.2020, 18:10 | 6 | |||||
если на то пошло, то правильно так:
0
|
8739 / 4317 / 960
Регистрация: 15.11.2014
Сообщений: 9,760
|
||||||||||||||||
27.04.2020, 18:26 | 7 | |||||||||||||||
считаю, что это категорически не правильно.
потому что функция должна выполнять строго одну задачу, не должна додумывать за клиентов, что им там вдруг ещё может захотеццо. а если не захочется? корректный дизайн:
если уж докалебываццо, тогда до последнего '\n'
0
|
Just Do It!
|
|
27.04.2020, 18:33 | 8 |
нет это не функция.
функция должна выдавать разный выход, который зависит от того, что у ней не входе. если я на вход подаю объект дисплея то вывод на дисплей, если же объект файла, то вывод в файл. Что тут может быть неясно? это общее, а не про код автора.
0
|
1 / 1 / 0
Регистрация: 02.03.2020
Сообщений: 106
|
||||||
27.04.2020, 18:36 [ТС] | 9 | |||||
XLAT, извините за навязчивость но эта вложенность не даёт мне покоя, я понимаю что от неё можно избавиться но вот проблема в исполнении. Я думал чтобы упростить просто взять и открывать всё время один и тот же файл например вот так:
0
|
8739 / 4317 / 960
Регистрация: 15.11.2014
Сообщений: 9,760
|
|||||||||||
27.04.2020, 18:45 | 10 | ||||||||||
"объект дисплея" - это твоя личная отсибятина.
std::cout - это не "объект дисплея". он вообще может быть не связан с черно-белым окошком консоли, а выводить в файлик, например. и делает он это в соответствии с стандартными правилами. ты же не ожидаешь от него, что в случае:
так с какой стати, ты считаешь что в этом случае обязан?
а в другом этак, значит хреновый у тебя дизайн.
0
|
Just Do It!
|
||||||
27.04.2020, 19:00 | 11 | |||||
вводите юзверя в заблуждение,
вот так уже нет:
Evreii, я бы посоветовал вам переписать этот код в ОПП стиле. глобальные переменные это жирный источник багов, часто трудноуловимых, если такого кода у вас много. к file могут иметь доступ кто угодно из любого места программы. А что если file уже был открыт где-то ещё? и тд и тп... Добавлено через 6 минут если ты ничего не понял, так и напиши что, мол, "я ничего не понял".
0
|
8739 / 4317 / 960
Регистрация: 15.11.2014
Сообщений: 9,760
|
|
27.04.2020, 19:04 | 12 |
0
|
Just Do It!
|
|||||||||||
27.04.2020, 23:22 | 13 | ||||||||||
расскажи мне, что твоя отсебятина лучше
но, извини я не доктор Добавлено через 2 часа 26 минут хм, нужно было сразу написать, что код у вас не рабочий?! вот демка, в которой упростить без удаления функционала не получится: демка по теме:
эта демка в деле тут: Задание с раздела "Чтение строк и файлов в С++"
0
|
0 / 0 / 0
Регистрация: 10.10.2022
Сообщений: 3
|
|
20.10.2022, 21:33 | 14 |
XLAT, а мне можно помочь? только я ничего пока не понимаю
0
|
0 / 0 / 0
Регистрация: 10.10.2022
Сообщений: 3
|
|
20.10.2022, 22:05 | 16 |
Добавлено через 23 секунды
платно) как вариант
0
|
0 / 0 / 0
Регистрация: 10.10.2022
Сообщений: 3
|
|
20.10.2022, 22:11 | 18 |
вообще, я серьёзно.
0
|
4773 / 2582 / 894
Регистрация: 29.11.2010
Сообщений: 5,591
|
||||||
21.10.2022, 01:39 | 19 | |||||
Девиз для код-ревью.
В рамочку на стеночку. ))) Добавлено через 2 часа 41 минуту Нам всем уже помочь нельзя. Да, хорошая идея. Ещё лучше разделить чтение имени файла и открытие файла по разным функциям. И переходите на fstream. Для красоты, разумеется. Пример:
0
|
21.10.2022, 01:39 | |
21.10.2022, 01:39 | |
Помогаю со студенческими работами здесь
19
Визуализация графов Визуализация расчетов Визуализация графика С++ OpenGL Визуализация алгоритмов сортировки Визуализация результатов моделирования Визуализация быстрой сортировки Искать еще темы с ответами Или воспользуйтесь поиском по форуму: |