Посчитать m последних разрядов числа n

05.01.2013, 13:33. **Показов** 726. **Ответов** 2

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

Всем привет!

В свободное от работы время занимаюсь программированием - подтягиваю свой уровень до того, который был когда-то раньше, во время обучения в ВУЗе. Для начала решил озаботиться азами, для этого использую С++ Builder 6 и сборник лабораторных работ для каких-то первокурсников (супруга преподаватель).

Задачки решаются, но возникает вопрос, насколько они кустарно сделаны. Мне было бы интересно послушать критики на мой код.

Модераторам просьба: посоветуйте, как оформлять темы - отдельно под каждую лабу? (их там с пару десятков и для каждой много вариантов. Я делаю на выбор, что нравится)

Срочности никакой, никому ничего не сдаю, стипендии не жажду (зарплаты хватает)), сроки не горят.

Заранее спасибо за ваше время.

Начнем
Из последнего, что у меня вызвало сначала затруднения, а потом, когда получилось - сомнения (а не наставил ли я тут костылей?)
ЛР№ 4, вариант 6
Пользователь вводит натуральные сисла m, n. Получить сумму последних m чисел числа n.

Мой вариант:

C++

#include <vcl.h>
#pragma hdrstop
#include <iostream.h>
#include <math.h>
//---------------------------------------------------------------------------
 
#pragma argsused
int main(int argc, char* argv[])
{
   int n = 0;
   int m = 0;
   double Y = 0;
   int temp = 0;
   cout<<"Enter n (some integer value > 0)\n";
   cin>>n;
   cout<<"Enter m (some integer value > 0)\n";
   cin>>m;
   if ((n>0) && (m>0))
   {
      for (int i = 1; i <=m; i++)
      {
         temp = fmod(n,10);
         Y = Y + temp;
         n = (n - temp)/10;
         cout<<"Y = "<<Y<<"\n";
      }
      cout<<"Resulting Y = "<<Y<<"\n";
   }
   else
   {
      cout<<"Incorrect values are entered!\n";
   }
   system("PAUSE");
   return 0;
}

Не проверял вводимые числа на цельность, потому что компилятор их округляет в меньшую сторону, т.е. если пользователь введет 1 или больше, то будет 1, 2, 3 и т.д. что удовлетворяет условиям. Если введет 0.99999 или меньше - будет сообщение о неправильном вводе.

P. S.: правильно ли я оформил? Инклюды пихать, или можно на них экономить?

Croessmah · 05.01.2013, 14:38

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

Конкретно Вашу задачу можно решить разными способами и не факт, что все они уже найдены.

C++

#include <iostream>
#include <cmath>
int main(){
    int n,m,summ=0;
    std::cout<<"Enter n->";
    std::cin>>n;
    std::cout<<"Enter m->";
    std::cin>>m;
    for(int n1=abs(n), m1=m;n1 && m1>0;summ+=n1%10,n1/=10,--m1);
    std::cout<<summ<<std::endl;
    return 0;
}

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

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

посоветуйте, как оформлять темы - отдельно под каждую лабу? (их там с пару десятков и для каждой много вариантов. Я делаю на выбор, что нравится)

Запрещено создавать темы с множеством вопросов во всех разделах, кроме разделов платных услуг. Один вопрос - одна тема.
Правила форума
По каким возникнут вопросы, по тем и спрашивайте.
Заодно и путаницы не возникнет.

@Issues · 05.01.2013, 14:40

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

for (int i = 1; i <=m; i++)
* * * {
* * * * *temp = fmod(n,10);
* * * * *Y = Y + temp;
* * * * *n = (n - temp)/10;
* * * * *cout<<"Y = "<<Y<<"\n";
* * * }

я бы немного укоротил.

C++

1 2	for (int i = 0; i < m && n != 0; i++, n /= 10) Y += n % 10;

	05.01.2013, 14:40