Ввод\вывод массива через функции

@KrL^^ · Регистрация: 01.10.2008

Помогите плз исправить) Нужно осуществить ввод\вывод массива

C

#include "stdio.h"
#include "conio.h"
 
#define N 10
#define M 10
void output (int n, int m,float a [N][M]);
void input (int n, int m, float a [N][M]);
//void sort (int n, int m, float b [], float *S);
int main()
{
    float a[N][M], S, S1;
    int i,j, n, m;
    scanf_s("%d %d", &n, &m);
    printf("vvod");
    input(n, m, a);
    printf("vivod");
    output(n, m, a);
    _getch();
}
 
void input (int n, int m, float a[N][M])
{
    int i, j;
    for(i=0;i<n;i++)
        for(j=0;j<m;j++)
    
        scanf_s("%f", &a[i][j]);
        
    
}
void output (int n, int m,float a[N][M])
{
    int i, j;
    for(i=0;i<n;i++)
    {
        for(j=0;j<m;j++)
            printf("a[%d %d]= %f",i+1, j+1, a[i,j]);
        printf("\n");
    }
}

На выводе нули((
Ошибка как я понимаю во вводе. Как правильно?

@Alik-Soldier · 30.06.2009, 15:46

C++

#include "stdafx.h"
#include <iostream>
#include <conio.h>
#include <stdio.h>
using namespace std;
#define N 2
#define M 2
void output (float a [N][M]);
void input (float a [N][M]);
//void sort (int n, int m, float b [], float *S);
int main()
{
        float a[N][M];
        printf("vvedite massiv\n");
        input(a);
        printf("massiv:\n");
        output(a);
        _getch();
}
 
void input (float a[N][M])
{
        for(int i=0;i<N;i++)
                for(int j=0;j<M;j++)
                    cin>>a[i][j];
        printf("\n");       
}
void output (float a[N][M])
{
        for(int i=0;i<N;i++)
        {
                for(int j=0;j<M;j++)
                       cout<<a[i][j]<<" ";
                printf("\n");
        }
}

@KrL^^ · 30.06.2009, 16:07 **[ТС]**

Спасибо, но не много не так) Необходимо сделать выбор размера массива до 10х10. Нас за это дерут) для этого используются m,n

З.Ы. Нашел ошибку)
printf("a[%d %d]= %f",i+1, j+1, a[i,j])

@Alik-Soldier · 30.06.2009, 20:01

Ну я и сделал N*M. Ты же взял их в define. Зачем брать лишние переменные,если они уже есть)

@KrL^^ · 30.06.2009, 23:03 **[ТС]**

Ну а без N u M не будет массив инициализироваться) а n u m используются уже в самой проге. хз нам препод так показывал..

@Alik-Soldier · 30.06.2009, 23:04

Да я понял о чем ты говориш)))

@IT_Exp · 30.06.2009, 23:13

вот эту функцию -

C

1	void output (float a[N][M])

завалить можно так:

C

1 2	float arr[1][M]; output(arr);

(:

@odip · 01.07.2009, 00:24

А потому что матрицы нужно делать нормально.

C

1	double **arr;

И никаких констант - M-ы,N-ы.

@KrL^^ · 01.07.2009, 02:54 **[ТС]**

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

А потому что матрицы нужно делать нормально.
double **arr;

И никаких констант - M-ы,N-ы.

хм.. не понял) можно подробней?)

@odip · 01.07.2009, 10:36

C

#include <stdlib.h>
 
 
void func( void ) {
 
#define M 10
#define N 20
 
double **arr= NULL;
int i, j;
 
arr= malloc( M*sizeof(double*) );
for ( i= 0; i<M; i++ ) {
    arr[i]= malloc( N*sizeof(double) );
}
 
for ( i= 0; i<M; i++ ) {
    for ( j= 0; j<N; j++ ) {
        arr[i][j]= 0.0;
    }
}
 
}

@pigah · 05.07.2009, 07:35

C++

#include "stdafx.h"
#include <iostream>
#include <windows.h>
#include <cmath>
#include <iomanip>
#include <limits.h>
#include <conio.h>
 
using namespace std;
 
const int N=10,A=-10,B=20;
int i;
 
template <class T>void формирование(T x[],int n=N,int a=A,int b=B);
template <class B>void вывод(B x[],int n=N);
 
int _tmain(int argc, _TCHAR* argv[]){
 
    SetConsoleOutputCP(1251);
    srand(GetTickCount());  
    int a[N];
    double d[N];
    формирование(a);
    cout<<"\n\tМассив a:\n";
    i = 0;
    вывод(a);
    cout<<"\n\n";
    формирование(d);
    cout<<"\tМассив d:\n";
    i = 0;
    вывод(d);
    cout<<"\n\n";
    return 0;
}
 
 
template <class T>void формирование(T x[],int n,int a,int b){
    int i;
    for(i=0;i<n;i++){
        if(sizeof(T)==4)
            x[i]=a+rand()%(b-a+1);
        else
            x[i] = double (a+rand()%(b-a+1))/1.1;
    }//for
}
 
template <class B>void вывод(B x[],int n){//рекурсивная функция 
    if(i<n){
        cout<<setw(12)<<x[i];
        if((i+1)%5 == 0)cout<<"\n";
        i++;
        вывод(x);  
    }   
    //return;
}

@Юляшка · 05.07.2009, 13:04

Можно вопросик?Разве не лучше вместо #define использовать const? Это суть задачи не меняет,просто эта директива устарела....

@odip · 05.07.2009, 13:33

define устарело ?
Очень смешно

))))))

@Юляшка · 05.07.2009, 13:55

Чего смешно?Совсем не смешно!!!!

@odip · 05.07.2009, 14:07

Открой любой системный h-файл и посмотри есть ли там define-ы.
Потом смейся.

@Юляшка · 05.07.2009, 14:09

Так я ж не говорю,что его нельзя использовать....Просто он уже устарел...

@odip · 05.07.2009, 14:13

Хорошо, компнем глубже.
Что с точки зрения программиста значит "устарел" в твоем понимании ?

accept · 06.07.2009, 05:30

Сообщение от Юляшка

Разве не лучше вместо #define использовать const?

чем больше переменных в глобалке, тем хуже для программы
define даёт возможность убрать его в любой момент, так как он нужен только для мнгновенной замены всех значений
то есть, иными словами, он никак не регистрируется в программе, его как бы нет

глобальная переменная: на первых порах это не видно, но сначала от неё начинает зависеть одна маленькая функция, потом на ней строится большая, потом ещё больше
в результате ты имеешь огромную функцию, а в С++ там вообще классы, которая так плотно зависит от этой переменной в глобалке, что при переносе этого класса тебе приходится таскать за ним эту переменную, потому что переделывать всё просто лень, а без него уже никуда
потом у тебя появляются два три класса, каждый работает с массивами и структурами, в каждой структуре указатель, а почему указатель ? да потому что, если сделать массив, нужно будет ещё одну прилипалу делать, а её, как известно, ничем уже не вытащишь потом, потому что без неё никак, а место для неё одно - тама, где будет видно её в области видимости
ну, вот и прикинь, тебе константу для размера массива нужно будет объявлять в protected: оно тебе надо ? а убрать захочешь что делать будешь ?

ты объявляешь десять констант: A B C D E ...
и все они равны 100
и тебе пофигу, они всё равно неважные, просто на их места установится сотка и такой код пойдёт на компиляцию, у них минус такой, что их нельзя проверить в процессе компиляции, то есть ты не можешь встроить средство для контроля размера массива (типа, а правильный размер у массива, а подходит ли он для дальнейшего ?), то есть, что вставится - то вставится

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

дефайны можно передефайнить, я даже не знаю, в чём было бы преимущество const int, если бы в #define добавили опцию const, может только если бы было #define FVALUE 2.2 можно было бы однозначнее задать размер для 2.2, но его и так задать можно через суффикс #define FVALUE 2.2f

а сам define можно по-разному применять

C

#include <stdio.h>
 
#define LOOP  { int i; for (i = 0; i < 5; i++) printf("Number %d\n", i+1); }
 
main()
{
    LOOP
    return 0;
}

@odip 7175 / 3234 / 81 Регистрация: 17.06.2009 Сообщений: 14,164
	05.07.2009, 14:13	17
	Хорошо, компнем глубже. Что с точки зрения программиста значит "устарел" в твоем понимании ? 0

@KrL^^ 0 / 0 / 0 Регистрация: 01.10.2008 Сообщений: 30
	30.06.2009, 16:07 [ТС]	3
	Спасибо, но не много не так) Необходимо сделать выбор размера массива до 10х10. Нас за это дерут) для этого используются m,n З.Ы. Нашел ошибку) printf("a[%d %d]= %f",i+1, j+1, a[i,j]) 0

@Alik-Soldier 9 / 9 / 2 Регистрация: 01.05.2009 Сообщений: 127
	30.06.2009, 20:01	4
	Ну я и сделал N*M. Ты же взял их в define. Зачем брать лишние переменные,если они уже есть) 0

@KrL^^ 0 / 0 / 0 Регистрация: 01.10.2008 Сообщений: 30
	30.06.2009, 23:03 [ТС]	5
	Ну а без N u M не будет массив инициализироваться) а n u m используются уже в самой проге. хз нам препод так показывал.. 0

@Alik-Soldier 9 / 9 / 2 Регистрация: 01.05.2009 Сообщений: 127
	30.06.2009, 23:04	6
	Да я понял о чем ты говориш))) 0

@KrL^^ 0 / 0 / 0 Регистрация: 01.10.2008 Сообщений: 30
	01.07.2009, 02:54 [ТС]	9
	Сообщение от odip А потому что матрицы нужно делать нормально. double **arr; И никаких констант - M-ы,N-ы. хм.. не понял) можно подробней?) 0

@Юляшка 3 / 3 / 0 Регистрация: 14.12.2008 Сообщений: 30
	05.07.2009, 13:04	12
	Можно вопросик?Разве не лучше вместо #define использовать const? Это суть задачи не меняет,просто эта директива устарела.... 0

@odip 7175 / 3234 / 81 Регистрация: 17.06.2009 Сообщений: 14,164
	05.07.2009, 13:33	13
	define устарело ? Очень смешно )))))) 0

@Юляшка 3 / 3 / 0 Регистрация: 14.12.2008 Сообщений: 30
	05.07.2009, 13:55	14
	Чего смешно?Совсем не смешно!!!! 0

@odip 7175 / 3234 / 81 Регистрация: 17.06.2009 Сообщений: 14,164
	05.07.2009, 14:07	15
	Открой любой системный h-файл и посмотри есть ли там define-ы. Потом смейся. 0

@Юляшка 3 / 3 / 0 Регистрация: 14.12.2008 Сообщений: 30
	05.07.2009, 14:09	16
	Так я ж не говорю,что его нельзя использовать....Просто он уже устарел... 0