Форум программистов, компьютерный форум CyberForum.ru

Хороший или плохой тон программирования - C++

Войти
Регистрация
Восстановить пароль
Другие темы раздела
C++ Найти сумму максимального и минимального чисел строки, у которой произведение элементов нечетно http://www.cyberforum.ru/cpp/thread109643.html
Найти сумму максимального и минимального чисел строки, у которой произведение элементов нечетно. Посмотрите код. Тут есть ошибки? Очень нужно. //--------------------------------------------------------------------------- #include <vcl.h> #include <conio.h> #include <stdlib.h> #include <iostream.h>
C++ Вопрос о стеке На сколько я знаю, подпрограмма которая выполняется рекурсивно, кидает в стек копию своих параметров, и так пока стек не переполнится... У меня такая штука: void f(int i){ char num; do{ ... num=getch(); switch(num){ http://www.cyberforum.ru/cpp/thread109582.html
книга Windows 2000 API Р.Саймон где скачать диск с программами? C++
книга Windows 2000 API Р.Саймон где скачать диск с программами который идет к книге?
C++ Заменить все положительные элементы массива, кратные пяти, на максимальный элемент
Требования к выполнение контрольной работы: 1. Каждый пункт меню и вывод текста меню выполняется в подпрограмме. 2. Если в начале работы сразу выбирается пункт ниже первого (массив не задан), получить массив генерацией его элементов в интервале от –100 до 100 3. После выполнения каждого пункта меню (кроме последнего), программа снова возвращается в меню. 4. Вычисление суммы, количества,...
C++ Компиляция .cpp http://www.cyberforum.ru/cpp/thread109255.html
Доброго времени суток, уважаемым форумчанам. Пишет чайник. Я хочу установить скрипт баннерообменной сети, выбрал (долго искал и выбирал) TBN. Там необходимо компилировать файлы. Вот что пишут в мануале: Откомпилируйте файл engine.cpp, разместите его в cgi-bin вашего сервера под именем engine.cgi Откомпилируйте файл cronjob.cpp, разместите его в произвольном месте вашего сервера вне ...
C++ Календарь Дни недели пронумерованы следующим образом: 1 — понедельник, 2 — вторник, ... , 6 — суббота, 7 — воскресенье. Дано целое число K, лежащее в диапазоне 1–365, и целое число N, лежащее в диапазоне 1–7. Написать программу, которая определяет номер дня недели для K-го дня года, если известно, что в этом году 1 января было днем недели с номером N. Входные данные Со стандартного потока ввода вводится... подробнее

Показать сообщение отдельно
Evg
Эксперт С++Автор FAQ
 Аватар для Evg
16946 / 5351 / 329
Регистрация: 30.03.2009
Сообщений: 14,393
Записей в блоге: 26
04.04.2010, 14:37     Хороший или плохой тон программирования
Цитата Сообщение от Atrus Посмотреть сообщение
Какой возражение?
Возражение "Evg, это отнюдь не искусственный способ"

Цитата Сообщение от Atrus Посмотреть сообщение
ifstream - это класс. Он так же на вход принимает массив значений и на выходе формирует вектор значений.
1. При чём тут вообще ifstream? Речь шла о использовании goto внутри процедуры, реализующей некий алгоритм. Если вдруг ты не знаешь, ifstream - это вовсе не алгоритм
2. ifstream - это класс, а потому он не может принимать на вход какие-либо значения. Ровно как и не может принимать значения экземпляр класса.
3. Никакого вектора значений ни в одном методе класса ifstream нет. Ни на входе, ни на выходе

Цитата Сообщение от Atrus Посмотреть сообщение
И все-таки это класс. И никто не жалуется, что он не реализован одной функцией.
Кто тебе сказал такую ерунду? Если прочёл в книге, то либо выбрось эту книгу в помойку, либо прочти ещё раз, чтобы быть уверенным, что ты правильно понял, что там написано

Цитата Сообщение от Atrus Посмотреть сообщение
в С++ int, float, long и.т.п это так же классы. Тоже полная ерунда?
Я уверен, что вы пользуетесь компилятором С++ и абсолютно не жалуетесь, что какой-нибудь double является классом.
Всё-таки выбрось книгу в помойку

Цитата Сообщение от Atrus Посмотреть сообщение
Почему бы мне не оформить решение СЛАУ в виде класса, ведь это будет гораздо читабельнее за счет декомпозиции, чем 50-70 строк кода щедро усыпанных комментариями и перемежающимся разными сомнительными решениями типа goto или while - break.
Честно говоря, я не знаю, что такое СЛАУ. Но твои слова сильно смахивают на то, что ты предлагаешь вместо процедурного подхода использовать объектно ориентированный только ради того, чтобы избежать использования goto (который в данном случае скорее всего вообще не понадобится)

Вот прототип конкретной программы. Это фрагмент исходников компилятора. Функция распределения ресурсов для переменной. Нужно разобраться, переменную надо класть на регистр или в стек. На регистр нельзя класть переменные, на которые брался адрес, нельзя класть массивы, нельзя класть большие структуры, в некоторых случаях нежелательно класть маленькие структуры, нежелательно класть переменные, у которых мало использований (лучше оставить регистры для часто используемых переменных), ещё всякие тонкости (я описал только понятные случаи). Процедура распределения ресурсов выглядела примерно так:

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
if (<на переменную не брался адрес и переменная не большая структура и не массив>)
{
  /* Пытаемся распределить на регистр */
 
  /* Если структура, то анализируем её
   * Если в структуре есть другие структуры, для которых есть целиковые пересылки,
   * то выдирать их из регистра будет слишком накладно, а потому лучше распределить
   * в стек. Если структура содержит в себе массив, то её нельзя держать на регистре и т.п. */
  вычисление_промежуточных_признаков
  if (признаки_плохие)
    goto Stack;
 
  /* Проверяем использование переменной. Редко используемые кладём в стек */
  вычисление_промежуточных_признаков
  if (признаки_плохие)
    goto Stack;
   
  несколько аналогичных фрагментов
 
  распределение перменной на регистр
} else
{
  /* Переменную распределяем в стек */
 Stack:
    распределение перменной на регистр
}
Перепиши этот код без использования goto. Особенно, учитывая, что разные блоки внутри первого if'а могут использовать результаты вычисления (т.е. промежуточные переменные) более ранних блоков (в той же ветке if'а). Да ещё и так, чтобы было понятно. А вот в данном случае с использованием goto никаких непоняток нет, ибо код написан в том же порядке, в каком идёт процесс мышления (не привязанный к языку программирования).

Данный фрагмент процедуры представляет собой так называемый "код верхнего уровня", логику которого крайне нежелательно пилить на процедуры (чтобы весь процесс был перед глазами, а не размазан по нескольким процедурам). Более мелкие блоки, отвечающие за вычисление какого-то конкретного свойства, можно и нужно выделять в отдельные процедуры, ибо для понимания ОБЩЕГО ПРОЦЕССА не надо знать, как в деталях работает тот или иной фрагмент алгоритма, ибо в большинстве случаев правильное название функции, отвечающей за фрагмент, и комментарий в одну строку делают этот момент хорошо понятным. Но ОБЩИЙ ПРОЦЕСС (самый верхний уровень) пилить на подпроцессы только ради того, чтобы избежать goto - это совершенно точно снизит понимаемость. Проверено на многолетнем опыте
 
Текущее время: 16:22. Часовой пояс GMT +3.
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2017, vBulletin Solutions, Inc.
Рейтинг@Mail.ru