0

Уважаемые программисты, у меня возникла проблема с написанием программ. Суть проблемы: не понимаю, в чем ошибка в f3, выдает: [Error] invalid array assignment, ругается на buf, в который я хочу записать переменную типа char из структуры. И вторая проблема: мне нужно отсортировать структуру по номеру группы, но он просто заменяет, а не сортирует. Как отсортировать структуру, НЕ применяя sort или что-то в таком духе?

#include <iostream>
#include <conio.h>
#include <stdlib.h>
#include <stdio.h>
#include <string.h>
#include <locale.h>
using namespace std;

const int N=3;
struct student {char fam[15];
    int kurs;
    char grup[3];
    float stip; };
void f1(student);

void f2(student parm[]);

void f3(student parm[]);

int main() { 
setlocale(LC_ALL, "rus");
int i; float maxs;
student stud[N];

//function 2
printf("\n rabota f2\n");
f2(stud);
getch();

maxs=0; 
for(i=0;i<N;i++)
if(stud[i].stip>maxs) maxs=stud[i].stip;
printf("\n stud - max stipendija %f rub ",maxs);
for(i=0; i<N; i++)
 if(stud[i].stip==maxs) printf ("\n%s",stud[i].fam);
getch();

//function f1
printf("\n rabota f1\n");
 for (i=0; i<N;i++) f1(stud[i]);
 getch();

//function f3
 printf("\n rabota f3\n");
 f3(stud);
 getch();

  }

 void f2(student parm[]){

 int i;
   for(i=0;i<N;i++) {
   printf("%d # Ñòóäåíò",i+1);
   printf("\nÔàìèëèÿ:"); cin>>parm[i].fam;
   printf("Êóðñ:"); cin>>parm[i].kurs;
   printf("Ãðóïïà:"); cin>>parm[i].grup;
   printf("Ñòèïåíäèÿ:"); cin>>parm[i].stip;
     }

    }
   void f1(student parm) {
    cout<<parm.fam<<" "<<parm.kurs<<" "<<parm.grup<<" "<<parm.stip<<"\n"; }

      void f3(student parm[]){
        int i,j=0, k;     // ñ÷åò÷èê öèêëîâ è òåêóùèé èíäåêñ ýëåìåíòà ìàññèâà
    char buf[3];
       cout<<"\nÑîðòèðîçêà ìàññèâà ìåòîäîì \"ïóçûðüêà\"\n";
       for(i=0;i<N;i++) cout<<parm[i].grup<<" ";
      for (i = 0; i < N-1; i++) { j++;
      for (k =0; k < N-1; k++) {
      if (parm[k].grup < parm[k+1].grup) {// îáìåíÿåì ê-é è (ê+1)-é ýëåìåíòû
       buf= parm[k].grup; parm[k].grup = parm[k+1].grup ; parm[k+1].grup = buf; } }
        // îòëàäî÷íàÿ ïå÷àòü - ñîñòîÿíèå
           // ìàññèâà ïîñëå î÷åðåäíîãî öèêëà ñîðòèðîâêè
      cout<<"\n-----"<<j<<"-----";
      for (k = 0; k < N; k++)
      cout<<parm[k].grup<<" ";
     cout<<"\n";
    }

     printf("\n massiv otsortirovan\n");
     for (i=0; i<N;i++) f1(parm[i]);
     getch();
    }
2
  • 1
    Я думаю, что Вы забыли приложить код, телепатией, конечно, обладаем, но не до такой степени.
    – KoVadim
    27 сен 2014 в 19:00
  • Вы правы! Исправила.. 28 сен 2014 в 7:16

2 ответа 2

1

Смотрите.

В языках типа C/C++ вы не можете копировать сложные штуки (типа массивов) простым присвоением. Ваш код

buf= parm[k].grup

пытается сделать именно это.

Поскольку вы пишете (судя по всему) на чистом C, вам стоит воспользоваться встроенным копированием памяти:

memcpy(/*куда*/ buf, /*откуда*/ parm[k].grup, /*сколько*/ 3);

Почему не strcpy? Потому что у вас строки фиксированной длины, а значит, имеют право не оканчиваться на \0.

По поводу сортировки: это ж у вас пузырьковая сортировка, да? Сейчас ваш код обмена обменивает лишь группу, а вам-то надо обменять целого студента! Вам лучше всего написать отдельную вспомогательную функцию swap(student* s1, student* s2), которая будет обменивать значения структур. И в нём обменивать не только группу, но и остальные поля. (Заодно текст программы станет более обозримым.)

7
  • 1
    @Хоббит, (маленькая подсказка для реализации swap()) а вот структуры (в отличие от массивов) можно копировать оператором присваивания (т.е. код swap() студентов будет почти таким же, как swap() двух чисел в массиве).
    – avp
    28 сен 2014 в 10:31
  • @avp: И правда! Хороший трюк, кстати. Я о нём совсем и забыл.
    – VladD
    28 сен 2014 в 10:40
  • @avp, только если структуры - Plain Old Data :)
    – Arkady
    29 сен 2014 в 8:18
  • 1
    @VladD, не думаю, что из-за длины. IMHO это потому, что массивы в Си очень близки к указателям. Фактически везде, кроме той области, где массив определяется (под него отводится память), массив и указатель не различимы. Поэтому операция копирования массива получается какой-то убогой, в одном месте ей можно воспользоваться, а во всех других -- нет. Размер же, явно не помеха. Если массив (размер фиксирован, но не ограничен) является частью структуры, то он будет замечательно копироваться (естественно, вместе с остальными полями структуры). -- POD -- ну, разумеется (другие просто не нужны).
    – avp
    29 сен 2014 в 9:45
  • 1
    Я сделала это! Просто после if заменять нужно всю структуру, а я заменяю отдельный элемент :) 30 сен 2014 в 16:57
0

Bидимо, вы используете массив или ссылку "buf". Создайте цикл или функцию, которая обменяет 2 элемента в структуре. В си нельзя обменять два массива или две структуры местами напрямую. Сортировать данные внутри структуры можно так же, как и данные вне. Используйте сортировку, которая вам больше понравится.

1
  • Я не очень понимаю, как правильно записать buf. А насчет сортировки - этими способами он просто заменяет мне числа, а мне нужно, чтобы он по позиции структуру отсортировал. :(( 28 сен 2014 в 7:38

Ваш ответ

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

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