По заданным параметрам ввести граф используя заданные правила - C++

@LaMusic · Регистрация: 07.09.2014

По заданным параметрам a,b,c,d ввести граф(список рёбер) использующие следующие правила h={14,если d mod 3=0; 13,если d mod 3=1; 15,если d mod 3=2;}

Программа вводит вершины a,b,c,d и получает n(список вершин) и количество рёбер.
(i,j) Э x a*i+b*j/c mod d=<1

Вот что сам сделал точнее попытался сделать.

C++

#include "stdafx.h"
#include <iostream>
#include <stdlib.h>
 
using namespace std;
 
int _tmain(int argc, _TCHAR* argv[])
{
    
    int a,b,c,d,n,x,i,j;
 
    cout<<"Введите параметры";
    cin>>a>>b>>c>>d;
    if 
        d mod 3=0 then 14
    else
    if d mod 3=1 then 13
    else
    if d mod 3=2 then 15
 for (i=1;i<10;i++) do
    for (j=1;j<10;j++) do
    if x=a*i+b*i/c mod d=<1
        then
        cout<<"Граф:"<<x;
    return 0;
}

Добавлено через 21 час 48 минут
Я просто застрял на выводе списков вершин и количестве рёбер.
То есть я правила описал в условном операторе,открыл циклы и уже по заданной формуле считаю и тогда лишь вывожу.

Точнее я так по алгоритму рассчитывал,но вижу что формулу или не туда впихиваю или зря циклы открыл.

@LaMusic · 10.09.2014, 11:07 **[ТС]**

Ругается на a,b,c,d,n которые в скобках,но убрать их не получается так как в главной программе я хочу вывести их результаты,которые я считаю в процедура mnreber.
и такой вид ошибки в строках 12 и 25 - write: вероятно, инициализатор, использующий стиль функции, является определением функции.

Но если я например удалю из скобок буквы a,b,c,d,n то программа компилируется но не работает уже на вводе.

C++

#include "stdafx.h"
#include <iostream>
#include <math.h>
#include <stdlib.h>
 
using namespace std;
 
void write(a,b,c,d,n)
{
    if (d % 3 == 0)
    n = 14;
  else
  if (d % 3 == 1) 
    n = 13;
  else
  if (d % 3 == 2)
    n = 15;
  cout<<"Количество вершин = " <<n;
}
 
void mnreber(a,b,c,d,n)
{
 int i, j, f;
  double k;
 
  for (i = 0 ;i<n;i++){
    for (j = 0;j<n;j++){
      k = ((a * i) + (b * j)) / c;
      f = floor(k);
      if (f % d<= 1) 
      {
        cout<<i<<' '<<j;
      }
      else
      cout<<"Пусто!";
}
}
}
 
 
int _tmain(int argc, _TCHAR* argv[])
{   
  setlocale(LC_ALL,"Russian");
  int a,b,c,d,n;
  cout<<"Введите значения параметров";
  cout<<'a = ';
  cin>>a;
  cout<<'b = ';
  cin>>b;
  cout<<'c = ';
  cin>>c;
  cout<<'d = ';
  cin>>d;
  write(a,b,c,d,n);
  mnreber(a,b,c,d,n);
  system("pause");
    return 0;
}

@LaMusic · 12.09.2014, 15:17 **[ТС]**

Уже не нужно.

@LaMusic · 17.09.2014, 22:27 **[ТС]**

Снова актуально:

C++

// laba_1.cpp: По заданным параметрам a,b,c,d ввести граф(список рёбер) использующие следующие правила
//Программа вводит вершины a,b,c,d и получает n(список рёбер)
 
#include "stdafx.h"
#include <iostream>
#include <math.h>
#include <stdlib.h>
 
using namespace std;
 
int MyArr[15][15];
 
void write(int a, int b,int c,int  d,int n)
{
    if (d % 3 == 0)
    n = 14;
  else
  if (d % 3 == 1) 
    n = 13;
  else
  if (d % 3 == 2)
    n = 15;
  cout<<"Количество вершин = " <<n;
}
 
void reber(int a, int b,int c,int  d,int n)
{
 int i, j, f;
  bool k;
 
  for (i = 0 ;i<n;i++){
    for (j = 0;j<n;j++){
      f = (a * i) + (b * j) / c;
      if (f % d<= 1) 
      {
        cout<<i<<' '<<j;
      }
      else
      cout<<"Пусто!";
}
}
for (i = 0 ;i<n;i++){
    for (j = 0;j<n;j++){
        if (MyArr[i, j] == MyArr[j, i])  k = true;
      else
        k = false;
  if (k = true) 
    cout<<"Неор";
  else
    cout<<"Ор";
    }
}
}
 
 
int _tmain(int argc, _TCHAR* argv[])
{   
  setlocale(LC_ALL,"Russian");
  int a,b,c,d,n;
  cout<<"Введите значения параметров";
  cout<<'a = ';
  cin>>a;
  cout<<'b = ';
  cin>>b;
  cout<<'c = ';
  cin>>c;
  cout<<'d = ';
  cin>>d;
  write(a,b,c,d,n);
  reber(a,b,c,d,n);
  system("pause");
    return 0;
}

Программа компилируется но не считает,вот что на выходе:

@LaMusic · 18.09.2014, 21:24 **[ТС]**

С вводом проблемы решил,но до конца не работает всё равно,вот что выдаёт(Внизу на рисунке)

Он не видит процедуру reber хотя она в main определена

C++

#include "stdafx.h"
#include <iostream>
#include <math.h>
#include <stdlib.h>
 
using namespace std;
 
int a,b,c,d,n;
int MyArr[15][15];
 
void write(int a, int b,int c,int  d,int n)
{
    if (d % 3 == 0)
    n = 14;
  else
  if (d % 3 == 1)
    n = 13;
  else
  if (d % 3 == 2)
    n = 15;
  cout<<"Количество вершин = " <<n;
}
 
void reber(int a, int b,int c,int  d,int n)
{
 int i, j, f;
  bool k;
 for (i = 0 ;i<n;i++)
 {
    for (j = 0;j<n;j++)
    {
      f = (a * i) + (b * j) / c;
      if (f % d<= 1)
      {
        cout<<i<<' '<<j;
      }
      else
    cout<<"Пусто!";
}
}
for (i = 0 ;i<n;i++)
{
    for (j = 0;j<n;j++)
    {
        if (MyArr[i][j] == MyArr[j][i])  k = true;
      else
        k = false;
  if (k = true)
    cout<<"Неор";
  else
    cout<<"Ор";
    }
}
}
 
 
int _tmain(int argc, _TCHAR* argv[])
{  
  setlocale(LC_ALL,"Russian");
  cout<<"Введите значения параметров\n";
  cout<<"a = ";
  cin>>a;
  cout<<"b = ";
  cin>>b;
  cout<<"c = ";
  cin>>c;
  cout<<"d = ";
  cin>>d;
  write(a,b,c,d,n);
  reber(a,b,c,d,n);
  system("pause");
    return 0;
}

MrGluck · 18.09.2014, 21:37

При передачи в функцию переменной по значению, внутри функции ведётся работа с её локальной копией, оригинал остаётся неизменным.
При объявлении локальных переменных (а это происходит при объявлении параметров функции) с теми же именами, что и глобальные переменные, при работе компилятор отдаёт предпочтение локальным. (Что стандарт по этому поводу говорит я искать ленюсь, сообщаю лишь результаты, основанные на опыте работы с gcc и VS). У вас глобальные переменные a, b, c, d, n не меняются. Вот так вот вы и узнали, что глобальные переменные - зло.

Перенесите объявление a, ... n в main и сделайте передачу в функции переменных, которые должны меняться по ссылке. Хотя в функции write, например, вы могли бы и возвращать значение, записывая его в n. Название правда функций у вас неочевидное, впрочем как и переменных. Однако, если названия a, b, c, d - неинформативные, то вот write должен вести запись куда-либо во внешний источник.

Добавлено через 1 минуту
В 48 строке присваивание вместо сравнения, для последнего используют оператор ==

@zss · 18.09.2014, 21:43

Можно
void write(int a, int b,int c,int d,int n) заменить на void write()
void reber(int a, int b,int c,int d,int n) - на void reber()
А переменные оставить глобальными.

Хотя так делать не рекомендуется.
Глобальными переменные делаются в исключительных случаях,
но тогда им даются осмысленные имена (Velosity, Pi и т.п.).

@zss · 18.09.2014, 21:49

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

В 48 строке присваивание

k уже логическая переменная. Зачем писать масло-масляное.
Пишем

C++

if(k)

@LaMusic · 18.09.2014, 22:15 **[ТС]**

Спасибо вам,заработало но проблемы уже с выводом

На 1-ом рисунке показано как должно выводить(код написанный на Паскале),а на 2-ом где всё вместе намешано,это уже моё))(при том что вводимые значения одинаковые),в чём еще может быть ошибка.

А глобальные переменные это иногда зло,но без них порой никак.

MrGluck · 18.09.2014, 23:32

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

А глобальные переменные это иногда зло,но без них порой никак.

В вашем случае ещё "как".
LaMusic, исправленный код выложите, телепаты в отпуске.

@LaMusic · 20.09.2014, 21:11 **[ТС]**

Вот код.Считает правильно а вот вывод хромает,пытался вчера и сегодня сам разобрать,уже циклы закрывал и открывал в других местах но всё равно всё вперемешку.

C++

#include "stdafx.h"
#include <iostream>
#include <math.h>
#include <stdlib.h>
 
using namespace std;
 
int a,b,c,d,n;
int MyArr[15][15];
 
void write()
{
    if (d % 3 == 0)
    n = 14;
  else
  if (d % 3 == 1) 
    n = 13;
  else
  if (d % 3 == 2)
    n = 15;
  cout<<"Количество вершин = " <<n;
}
 
void reber()
{
 int i, j, f;
  bool k;
 for (i = 0 ;i<n;i++)
 {
    for (j = 0;j<n;j++)
    {
      f = (a * i) + (b * j) / c;
      if (f % d<= 1) 
      {
        cout<<' '<<i;
        cout<<' '<<j;
      }
      else
    cout<<"Пусто!";
}
}
for (i = 0 ;i<n;i++)
{
    for (j = 0;j<n;j++)
    {
        if (MyArr[i][j] == MyArr[j][i])  k = true;
      else
        k = false;
  if (k = true) 
    cout<<"Неор";
  else
    cout<<"Ор";
    }
}
}
 
 
int _tmain(int argc, _TCHAR* argv[])
{   
  setlocale(LC_ALL,"Russian");
  cout<<"Введите значения параметров\n";
  cout<<"a = ";
  cin>>a;
  cout<<"b = ";
  cin>>b;
  cout<<"c = ";
  cin>>c;
  cout<<"d = ";
  cin>>d;
  write();
  reber();
  system("pause");
    return 0;
}

@LaMusic · 24.09.2014, 20:16 **[ТС]**

Можно закрывать)

@LaMusic 34 / 0 / 0 Регистрация: 07.09.2014 Сообщений: 34
	12.09.2014, 15:17 [ТС]	#3
	Уже не нужно. 0

MrGluck Модератор 7727 / 4763 / 731 Регистрация: 29.11.2010 Сообщений: 13,014
	18.09.2014, 21:37	#6
	При передачи в функцию переменной по значению, внутри функции ведётся работа с её локальной копией, оригинал остаётся неизменным. При объявлении локальных переменных (а это происходит при объявлении параметров функции) с теми же именами, что и глобальные переменные, при работе компилятор отдаёт предпочтение локальным. (Что стандарт по этому поводу говорит я искать ленюсь, сообщаю лишь результаты, основанные на опыте работы с gcc и VS). У вас глобальные переменные a, b, c, d, n не меняются. Вот так вот вы и узнали, что глобальные переменные - зло. Перенесите объявление a, ... n в main и сделайте передачу в функции переменных, которые должны меняться по ссылке. Хотя в функции write, например, вы могли бы и возвращать значение, записывая его в n. Название правда функций у вас неочевидное, впрочем как и переменных. Однако, если названия a, b, c, d - неинформативные, то вот write должен вести запись куда-либо во внешний источник. Добавлено через 1 минуту В 48 строке присваивание вместо сравнения, для последнего используют оператор == 0

@zss Модератор 6831 / 6393 / 2136 Регистрация: 18.12.2011 Сообщений: 16,768 Завершенные тесты: 1
	18.09.2014, 21:43	#7
	Можно void write(int a, int b,int c,int d,int n) заменить на void write() void reber(int a, int b,int c,int d,int n) - на void reber() А переменные оставить глобальными. Хотя так делать не рекомендуется. Глобальными переменные делаются в исключительных случаях, но тогда им даются осмысленные имена (Velosity, Pi и т.п.). 0

@LaMusic 34 / 0 / 0 Регистрация: 07.09.2014 Сообщений: 34
	18.09.2014, 22:15 [ТС]	#9
	Спасибо вам,заработало но проблемы уже с выводом На 1-ом рисунке показано как должно выводить(код написанный на Паскале),а на 2-ом где всё вместе намешано,это уже моё))(при том что вводимые значения одинаковые),в чём еще может быть ошибка. А глобальные переменные это иногда зло,но без них порой никак. 0 Миниатюры

MrGluck Модератор 7727 / 4763 / 731 Регистрация: 29.11.2010 Сообщений: 13,014
	18.09.2014, 23:32	#10
	Сообщение от LaMusic А глобальные переменные это иногда зло,но без них порой никак. В вашем случае ещё "как". LaMusic, исправленный код выложите, телепаты в отпуске. 0

@LaMusic 34 / 0 / 0 Регистрация: 07.09.2014 Сообщений: 34
	24.09.2014, 20:16 [ТС]	#12
	Можно закрывать) 0

Опции темы

С++ для начинающих

По заданным параметрам ввести граф используя заданные правила - C++