Компилируется, но не выполняется программа

@zuck127 · Регистрация: 24.02.2016

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

Есть код, содержащий функцию сортировки выбором
Все компилируется , однако при запуске выдает просто черную консоль, и ничего не происходит

C++

#include <iostream>
#include <fstream>
using namespace std;
int i,j;
void SelectionSort(int A[], int n) //сортировка выбором
{
int count, key;
for (i=0; i<n-1; i++)
{
count=A[i]; key=i;
for (j=i+1; j<n; j++)
if (A[j]<A[key]) key=j;
if (key!=i)
{
A[i]=A[key];
A[key]=count;
}
}
}
int main(){
fstream F; 
F.open("nosort.txt", ios::in);
int size;
int x;
while(!F.eof()){
    F>>x;
    size++;
}
int arr[size];
int i=0;
F.seekg( 0 , ios::beg );
while(!F.eof()){
    F>>arr[i];
    i++;
}
F.close();
SelectionSort(arr, size);
F.open("sort.txt", ios::out);
for(i=0;i<=size;i++){
F<<arr[i]<< endl;
    }
F.close();
}

@hoggy · 24.02.2016, 00:38

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

однако при запуске выдает просто черную консоль, и ничего не происходит

сделайте так:

C++

#include <iostream>
#include <fstream>
using namespace std;
int i,j;
void SelectionSort(int A[], int n) //сортировка выбором
{
int count, key;
for (i=0; i<n-1; i++)
{
count=A[i]; key=i;
for (j=i+1; j<n; j++)
if (A[j]<A[key]) key=j;
if (key!=i)
{
A[i]=A[key];
A[key]=count;
}
}
}
int main(){
 
cout <<"hello, zuck127!"<<endl;
 
fstream F; 
F.open("nosort.txt", ios::in);
int size;
int x;
while(!F.eof()){
    F>>x;
    size++;
}
int arr[size];
int i=0;
F.seekg( 0 , ios::beg );
while(!F.eof()){
    F>>arr[i];
    i++;
}
F.close();
SelectionSort(arr, size);
F.open("sort.txt", ios::out);
for(i=0;i<=size;i++){
F<<arr[i]<< endl;
    }
F.close();
}

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

@zuck127 · 24.02.2016, 00:43 **[ТС]**

Это конечно здорово) но мне нужно чтобы в файл записывался отсортированный массив, а этого не происходит после запуска

@hoggy · 24.02.2016, 00:56

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

нужно чтобы в файл записывался отсортированный массив, а этого не происходит после запуска

C++

#include <iostream>
#include <fstream>
using namespace std;
int i,j;
 
void SelectionSort(int A[], int n) 
{
    int count, key;
    for (i=0; i<n-1; i++)
    {
        count=A[i]; key=i;
        for (j=i+1; j<n; j++)
            if (A[j]<A[key]) 
                key=j;
 
        if (key!=i)
            A[i]=A[key],
            A[key]=count;
    }
}
 
int main(){
 
    cout <<"hello, zuck127!"<<endl;
 
    fstream F; 
    F.open("nosort.txt", ios::in);
    if (!F)
        cout << "error: nosort.txt not found" << endl;
 
    int size;
    int x;
    while(!F.eof())
        F>>x,
        size++;
    
    int arr[size];   //<--- detected ugly code
    int i=0;
    F.seekg( 0 , ios::beg );
    while(!F.eof())
        F>>arr[i],
        i++;
    
    F.close();
    SelectionSort(arr, size);
    F.open("sort.txt", ios::out);
 
    if (!F)
        cout << "error: can`t write sort.txt" << endl;
 
    for(i=0;i<=size;i++)
        F<<arr[i]<< endl;
 
    F.close();
}

@zuck127 · 24.02.2016, 01:00 **[ТС]**

Не работает. Выводит "Hello, zuck127". Консоль висит. В файл ничего не записывается
p.s. знаю, что код ужасный)

@hoggy · 24.02.2016, 01:09

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

int size;

C++

1	int size = 0;

@zuck127 · 24.02.2016, 01:13 **[ТС]**

Так же висит консоль и ничего не меняется

@igor_petrov · 24.02.2016, 02:21

zuck127,

1. Нельзя объявлять в коде вот так массив заведомо-неизвестного размера. Только если size - константа.

C++

1	int arr[size];

Вместо этого используйте

C++

1
2
3

int *arr = new int[size];
<...>
delete []arr;

2. Вторая ошибка - вы не инициализируете переменную size. Задайте ее по-умолчанию равной нулю.

3. Открытие потока можно отслеживать таким кодом. Может быть программа даже не открывает этот файл, потому что не может его найти.

C++

fstream F;
F.open("nosort.txt", ios::in);
 
if(F.is_open()) {
    //все хорошо, поток открыт для чтения
} else {
    //выводим ошибку и закрываем программу
}

Все, что бросается в глаза на первый взгляд. Тестируйте

@elephants · 24.02.2016, 02:38

C++

#include <iostream>
#include <fstream>
using namespace std;
int i, j;
void SelectionSort(int A[], int n) //сортировка выбором
{
    int count, key;
    for(i = 0; i<n - 1; i++)
    {
        count = A[i]; key = i;
        for(j = i + 1; j<n; j++)
        if(A[j]<A[key]) key = j;
        if(key != i)
        {
            A[i] = A[key];
            A[key] = count;
        }
    }
}
int main(){
    fstream F;
    F.open("nosort.txt", ios::in);
    int size = 0;
    int x;
    while(!F.eof()){
        F >> x;
        size++;
    }
    int *arr = new int[size];
    int i = 0;
    F.seekg(0, ios::beg);
    while(!F.eof()){
        F >> arr[i];
        i++;
    }
    F.close();
    SelectionSort(arr, size);
    F.open("sort.txt", ios::out);
    for(i = 0; i < size; i++){
        F << arr[i] << '\n';
    }
    F.close();
    delete[] arr;
    std::cout << "Done!\n";
    std::cin.get();
    return 0;
}

В первую очередь, как уже выше писали, массив надо создавать динамически. Так же была ошибка в последнем цикле for в проверочном условии, вместо

C++

1	for(i=0;i<=size;i++)

надо

C++

1	for(i=0;i<size;i++)

В первом случае в файл дополнительно записывались случайные данные в памяти за границей массива.

@zuck127 0 / 0 / 0 Регистрация: 24.02.2016 Сообщений: 17
	24.02.2016, 00:43 [ТС]	3
	Это конечно здорово) но мне нужно чтобы в файл записывался отсортированный массив, а этого не происходит после запуска 0

@zuck127 0 / 0 / 0 Регистрация: 24.02.2016 Сообщений: 17
	24.02.2016, 01:00 [ТС]	5
	Не работает. Выводит "Hello, zuck127". Консоль висит. В файл ничего не записывается p.s. знаю, что код ужасный) 0

@zuck127 0 / 0 / 0 Регистрация: 24.02.2016 Сообщений: 17
	24.02.2016, 01:13 [ТС]	7
	Так же висит консоль и ничего не меняется 0