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

Что не правильно в коде, что посоветуете дописать - C++

Восстановить пароль Регистрация
 
Zerstoren93
Сообщений: n/a
15.05.2013, 22:14     Что не правильно в коде, что посоветуете дописать #1
Дано задание (Ввести натуральное число N и последовательность действительных чисел Y1, , …,Yn . Рассчитать сумму Z1...Zn. Где Zi (если 0<Yi<10 , 1 в противном случае). Пытался с пустого места, что-нибудь да написать и получилось это. Посоветуйте, что и как сделать ?


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
#include <iostream>
#include <string>
using namespace std;
 
int main()    
{int n;
int i;
int z[10];
int sum=0;
int mass[10];
cout<<"vvedite n menhe<10:"; 
cin>>n;
     // будем вводить последовательнность чисел 
    for ( i=0;i<n;i++)
    {cout << "[" << i + 1 << "]" << ": ";
            cin >> mass[i];
            
            cout << " massiv:\n ";}
    for ( i = 0; i < n; ++i) {
        cout << mass[i] << " ";}
 
        if (mass[i]>=0 && mass[i]<=10)
        {cout<<z[i]<<" ";}
        {z[i]=mass[i];}
        
 
        else
        {z[i]=1;}
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
15.05.2013, 22:14     Что не правильно в коде, что посоветуете дописать
Посмотрите здесь:

Что не правильно в этом коде? C++
Пишу крестики-нолики. Что не правильно в этом коде? C++
C++ Что посоветуете написать?
что не правильно в коде? C++
Что такое перегрузка оператора? Что у меня в коде за ошибка? C++
Объясните что именно делает "x%d, y%d: " или что это в даном коде C++
C++ Что в этом коде не правильно?
C++ Решил изучить новую низкоуровневую область в программировании, что посоветуете?

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

Или воспользуйтесь поиском по форуму:
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
vadtsyb
9 / 9 / 1
Регистрация: 12.05.2013
Сообщений: 23
15.05.2013, 22:49     Что не правильно в коде, что посоветуете дописать #2
Меня смущает только этот фрагмент:
C++
1
2
3
4
5
if (mass[i]>=0 && mass[i]<=10)
{cout<<z[i]<<" ";}
{z[i]=mass[i];}
else
{z[i]=1;}
Отдельные условия надо брать в свои скобки, и комбинация операторских скобок странная. На мой взгляд, нужно так:
C++
1
2
3
4
5
if ((mass[i] >= 0) && (mass[i] <= 10))
  z[i]=mass[i];
else
  z[i]=1;
cout<<z[i]<<" ";
Или даже так (с тернарной операцией):
C++
1
2
z[i]=(mass[i] >= 0) && (mass[i] <= 10) ? mass[i] : 1;
cout<<z[i]<<" ";
Yandex
Объявления
15.05.2013, 22:49     Что не правильно в коде, что посоветуете дописать
Ответ Создать тему
Опции темы

Текущее время: 07:06. Часовой пояс GMT +3.
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2016, vBulletin Solutions, Inc.
Рейтинг@Mail.ru