0

Программа выполняется правильно, но число в конце просто убивает, как его убрать? http://pastebin.com/g6J2GnU8

5
  • printf("%d", a[20]); В массиве нет двадцатого элемента (он начинается с нуля, поэтому полседний элемент девятнадцатый), и эта конструкция не нужна, она использует память, лежащую вне массива.
    – etki
    28 мар 2014 в 7:56
  • спасибо. без этой конструкции программа не меняет 2 последних элемента т.е 20 и 19
    – himiko
    28 мар 2014 в 8:00
  • 1
    @himiko, эта конструкция вообще ничего не меняет, только выводит
    – etki
    28 мар 2014 в 8:31
  • Не вдаваясь сильно в логику кода: for (i = 0; i < 20; i++) { if (a[i] != a[i + 1] && i%2==0) { temp = a[i]; Вы сравниваете 20-й элемент массива с 21 несуществующим, нужно изменить условие цикла на: for (i = 0; i < 19; i++)
    – MDJHD
    28 мар 2014 в 9:14
  • да, я уже поняла и исправила) благодарю)
    – himiko
    28 мар 2014 в 9:15

2 ответа 2

2
#include "stdafx.h"
#include <iostream>
#include <stdio.h>
#include <conio.h>
#include <math.h>
#include <cstdio>
#include <locale.h>

int main(void)
{
        int i, temp;
        int a[20];
        for (i = 0; i <20; i++)
        {
                printf("massiv[%d] = ", i);
                scanf_s("%d", &a[i]);
        }

        for (i = 0; i < 20; i+=2)
        {
                temp = a[i];
                a[i] = a[i + 1];
                a[i + 1] = temp;
                printf("%d %d ", a[i], a[i+1]);
        }

        system("pause");
        return 0;
}
1
  • идеально!) большое спасибо за помощь:)
    – himiko
    28 мар 2014 в 9:26
0
#include "stdafx.h"
#include <iostream>
#include <stdio.h>
#include <conio.h>
#include <math.h>
#include <cstdio>
#include <locale.h>

Кто все эти люди? Реально нужен только stdio. Запомни, Visual Studio - это отшень плёхо.

Переменная temp нужна только во втором цикле, а объявлена перед первым. Это отшень плёхо.

system("pause");

И не надо смешивать две задачи в одном куске кода. Сначала меняем, потом выводим.

for( i = 0; i < 20; ) {
    temp = a[i];
    a[i] = a[++i];
    a[i++] = temp;
}

for( i = 0; i < 20; ++i ) {
    printf("%d ", a[i]);
}
7
  • 2
    @VadimTukaev, я еще раз попрошу не поднимать старые вопросы.
    – etki
    22 апр 2014 в 18:38
  • 1
    Честно говоря, просто не обращаю внимания на дату. Мне всегда казалось, что правильный ответ ценен сам по себе. Допустим, у кого-то возникнет такая же задача и он нагуглит эту страницу. P.S. Сейчас посмотрел - вопрос от 28 марта. Меньше месяца назад - это тоже старый вопрос? А сколько вопрос считается новым, полчаса? 22 апр 2014 в 19:04
  • 1
    @VadimTukaev > А са это йа пуду пить тибя толстый шелесный палка. Как это по-рюски? Йа-йа, льом, льом! Вот это вот гуглить будут?
    – etki
    22 апр 2014 в 19:23
  • Это надо не гуглить, это надо делать. 22 апр 2014 в 19:41
  • 1
    @VadimTukaev, слушайте, ну честно, у нас тут так не принято - поднимать отвеченные/прошлогодние вопросы и пытаться задеть собеседника. Цель сообщества - не ответить на максимальное количество вопросов, а поддерживать этот, эм, поток вопросов в сбалансированном состоянии.
    – etki
    22 апр 2014 в 19:47

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.