Необъяснимо, но факт

@ferr3t · Регистрация: 18.04.2018

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

Всем привет. Учусь сейчас С++, пишу в CodeBlocks. Позавчера любая написанная программа (более сложная, чем cout << "Hello World!") выводила очень странные результаты. И вот сегодня, столкнувшись с десятой неверно работающей программой, я что-то заподозрил. Пошел в онлайн-комплилятор, и он выдал верный ответ.

Переустанавливал CodeBlocks, все перепроверил, вроде всё должно быть верно. Но работает неправильно. Что такое, помогите, мне к ЕГЭ надо готовиться

nd2 · 17.05.2018, 17:50

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

помогите

Не видя кода?

Добавлено через 53 секунды

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

пишу в CodeBlocks

Компилятор там какой подключен (название, версия)?

@ferr3t · 17.05.2018, 17:51 **[ТС]**

C++

#include <iostream>
 
using namespace std;
 
int main()
{
    int m, N, k;
    string fullName;
 
    cin >> N;
 
    string one[N], two[N], three[N];
    for (int i = 0; i < N; i++)
    {
        one[i] = ' ';
        two[i] = ' ';
        three[i] = ' ';
    }
    for (int i = 0; i < N; i++)
    {
        cin >> fullName;
        for (int j = 0; j < 3; j++)
        {
            cin >> m;
            if (m == 2) k++;
        }
        if (k == 3) three[i] = fullName;
        if (k == 2) two[i] = fullName;
        if (k == 1) one[i] = fullName;
    }
    for (int i = 0; i < N; i++)
    {
        cout << three[i] << endl;
    }
    for (int i = 0; i < N; i++)
    {
        cout << two[i] << endl;
    }
    for (int i = 0; i < N; i++)
    {
        cout << one[i] << endl;
    }
    return 0;
}

Вот код. Условия задачи: На вход программе подаются сведения о сдаче экзаменов учениками 9-х классов некоторой средней школы. В первой строке сообщается количество учеников N, которое не меньше 10, но не превосходит 100, каждая из следующих N строк имеет следующий формат: <Фамилия> <Имя> <оценки>, где <Фамилия> - строка, состоящая не более чем из 20 символов, <Имя> — строка, состоящая не более чем из 15 символов, <оценки> - через пробел три целых числа, соответствующие оценкам по пятибалльной системе. <Фамилия> и <Имя>, а также <Имя> и <оценки> разделены одним пробелом. Пример входной строки:

Иванов Петр 4 2 4

Требуется написать как можно более эффективную программу (укажите используемую версию языка программирования, например, Borland Pascal 7.0), которая будет выводить на экран фамилии и имена неуспевающих учеников (имеющих по результатам экзаменов хотя бы одну двойку), располагая их в порядке уменьшения числа двоек.

Добавлено через 50 секунд
nd2, GNU GCC Compiler

nd2 · 17.05.2018, 17:56

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

C++

1
2
3

one[i] = ' '; 
two[i] = ' '; 
three[i] = ' ';

Это для чего сделано?

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

GNU GCC Compiler

У тебя Linux?

@ferr3t · 17.05.2018, 17:57 **[ТС]**

nd2, винда. Это для того, чтобы он корректно выводил заполненные строки. Я, если честно, не уверен, что оно так должно быть сделано.

Он ведь там не каждый элемент массива заполнит фамилией, а только те, в которых двоечники. А остальные элементы останутся неопределёнными. Вот я их и "опустошаю"

nd2 · 17.05.2018, 18:01

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

cin >> fullName;

И что ты здесь вводишь? Вот это?

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

Иванов Петр

@ferr3t · 17.05.2018, 18:02 **[ТС]**

nd2, да

Добавлено через 51 секунду
nd2, ну хотя нет, подожди. Я понимаю, он через пробел будет следующую переменную уже считывать. Я пробовал вводить просто букву А, он все равно не работает

nd2 · 17.05.2018, 18:02

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

Вот я их и "опустошаю"

std::string по умолчанию пустыми создаются, поэтому не нужно их "опутсошать", тем более, через присваивание пробела.

@ferr3t · 17.05.2018, 18:06 **[ТС]**

nd2, окей, убрал это. В любом случае, он мне выводит пустые строки в CodeBlocks, а в онлайн-компиляторе (https://www.onlinegdb.com/online_c++_compiler) выводит верный ответ

Добавлено через 2 минуты
nd2, к слову, убрал это "опустошение" и запустил проект в онлайне - ничего мне не вывел

nd2 · 17.05.2018, 18:13

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

В любом случае, он мне выводит пустые строки в CodeBlocks

Скрин работы покажи, чтоб твой ввод был виден.

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

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

C++

k++

У тебя где k инициализируется?

@ferr3t · 17.05.2018, 18:13 **[ТС]**

nd2,

@ferr3t · 17.05.2018, 18:14 **[ТС]**

nd2, шок. k не инициализировался. Сделал ему k = 0 и прога сработала. Даже как-то стыдно

@Operok · 17.05.2018, 18:18

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

C++

1 2	cin >> N; string one[N], two[N], three[N];

Undefined behavior...
Да нормальный компилятор такое даже пропустить не должен!

@ferr3t · 17.05.2018, 18:18 **[ТС]**

Operok, почему?

nd2 · 17.05.2018, 18:22

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

Да нормальный компилятор такое даже пропустить не должен!

Это нестандартное расширение GCC.

@ferr3t · 17.05.2018, 18:23 **[ТС]**

nd2, и слава богу

nd2 · 17.05.2018, 18:23

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

Сделал ему k = 0 и прога сработала.

Смотря где сделал. В конце каждой итерации внешнего цикла k нужно обнулять.

@ferr3t · 17.05.2018, 18:27 **[ТС]**

nd2, да-да, так и сделал, спасибо)

Добавлено через 2 минуты
nd2, только вот у меня вопрос. Он теперь после каждой выведенной как бы пропускает строку из-за << endl. Это необходимо, но как сделать так, чтобы он после элементов с пустым значением не пропускал?

nd2 · 17.05.2018, 18:29

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

и слава богу

Но расширение нестандартное, поэтому такой код считается непереносимым.

@Operok · 17.05.2018, 18:29

ferr3t, потому что конструкция вида:

C++

1	SomeType var[size];

это объявление "статического" массива, размер которого известен при компиляции, а вы определяете размер в рантайме (вводя его в консоль) и что будет находиться в переменных one, two, three я представления не имею. Запомните, размер любой переменной на этапе компиляции должен быть известен.

C++

size_t N = 0;
cin >> N;
string* one = new string[N];
 
// поработали
 
delete[] one;

а лчше так

C++

1
2
3

size_t N = 0;
cin >> N;
std::vector<string> one(N);

@ferr3t 0 / 0 / 0 Регистрация: 18.04.2018 Сообщений: 20
	17.05.2018, 18:13 [ТС]	11
	nd2, Миниатюры 0

@ferr3t 0 / 0 / 0 Регистрация: 18.04.2018 Сообщений: 20
	17.05.2018, 18:14 [ТС]	12
	nd2, шок. k не инициализировался. Сделал ему k = 0 и прога сработала. Даже как-то стыдно 0

@ferr3t 0 / 0 / 0 Регистрация: 18.04.2018 Сообщений: 20
	17.05.2018, 18:23 [ТС]	16
	nd2, и слава богу 0

nd2 3438 / 2817 / 1249 Регистрация: 29.01.2016 Сообщений: 9,426
	17.05.2018, 18:29	19
	Сообщение от ferr3t и слава богу Но расширение нестандартное, поэтому такой код считается непереносимым. 0

@ferr3t 0 / 0 / 0 Регистрация: 18.04.2018 Сообщений: 20
		1
	Необъяснимо, но факт 17.05.2018, 17:45. Показов 1371. Ответов 24 Метки нет (Все метки) Всем привет. Учусь сейчас С++, пишу в CodeBlocks. Позавчера любая написанная программа (более сложная, чем cout << "Hello World!") выводила очень странные результаты. И вот сегодня, столкнувшись с десятой неверно работающей программой, я что-то заподозрил. Пошел в онлайн-комплилятор, и он выдал верный ответ. Переустанавливал CodeBlocks, все перепроверил, вроде всё должно быть верно. Но работает неправильно. Что такое, помогите, мне к ЕГЭ надо готовиться 0

nd2 3438 / 2817 / 1249 Регистрация: 29.01.2016 Сообщений: 9,426
	17.05.2018, 17:50	2
	Сообщение от ferr3t помогите Не видя кода? Добавлено через 53 секунды Сообщение от ferr3t пишу в CodeBlocks Компилятор там какой подключен (название, версия)? 0

@ferr3t 0 / 0 / 0 Регистрация: 18.04.2018 Сообщений: 20
	17.05.2018, 17:57 [ТС]	5
	nd2, винда. Это для того, чтобы он корректно выводил заполненные строки. Я, если честно, не уверен, что оно так должно быть сделано. Он ведь там не каждый элемент массива заполнит фамилией, а только те, в которых двоечники. А остальные элементы останутся неопределёнными. Вот я их и "опустошаю" 0

nd2 3438 / 2817 / 1249 Регистрация: 29.01.2016 Сообщений: 9,426
	17.05.2018, 18:01	6
	Сообщение от ferr3t cin >> fullName; И что ты здесь вводишь? Вот это? Сообщение от ferr3t Иванов Петр 0

@ferr3t 0 / 0 / 0 Регистрация: 18.04.2018 Сообщений: 20
	17.05.2018, 18:02 [ТС]	7
	nd2, да Добавлено через 51 секунду nd2, ну хотя нет, подожди. Я понимаю, он через пробел будет следующую переменную уже считывать. Я пробовал вводить просто букву А, он все равно не работает 0

nd2 3438 / 2817 / 1249 Регистрация: 29.01.2016 Сообщений: 9,426
	17.05.2018, 18:02	8
	Сообщение от ferr3t Вот я их и "опустошаю" std::string по умолчанию пустыми создаются, поэтому не нужно их "опутсошать", тем более, через присваивание пробела. 0

@ferr3t 0 / 0 / 0 Регистрация: 18.04.2018 Сообщений: 20
	17.05.2018, 18:06 [ТС]	9
	nd2, окей, убрал это. В любом случае, он мне выводит пустые строки в CodeBlocks, а в онлайн-компиляторе (https://www.onlinegdb.com/online_c++_compiler) выводит верный ответ Добавлено через 2 минуты nd2, к слову, убрал это "опустошение" и запустил проект в онлайне - ничего мне не вывел 0

@ferr3t 0 / 0 / 0 Регистрация: 18.04.2018 Сообщений: 20
	17.05.2018, 18:18 [ТС]	14
	Operok, почему? 0

nd2 3438 / 2817 / 1249 Регистрация: 29.01.2016 Сообщений: 9,426
	17.05.2018, 18:22	15
	Сообщение от Operok Да нормальный компилятор такое даже пропустить не должен! Это нестандартное расширение GCC. 0

nd2 3438 / 2817 / 1249 Регистрация: 29.01.2016 Сообщений: 9,426
	17.05.2018, 18:23	17
	Сообщение от ferr3t Сделал ему k = 0 и прога сработала. Смотря где сделал. В конце каждой итерации внешнего цикла k нужно обнулять. 0

@ferr3t 0 / 0 / 0 Регистрация: 18.04.2018 Сообщений: 20
	17.05.2018, 18:27 [ТС]	18
	nd2, да-да, так и сделал, спасибо) Добавлено через 2 минуты nd2, только вот у меня вопрос. Он теперь после каждой выведенной как бы пропускает строку из-за << endl. Это необходимо, но как сделать так, чтобы он после элементов с пустым значением не пропускал? 0

Необъяснимо, но факт

Решение