初学者 構造体の配列を関数の引数に渡す方法が知りたい

Question

前提・実現したいこと

c++初心者です

提示コードですが、draw関数に構造体の配列ichiを引数で渡したいのですがどうすればいいでしょうか？
配列でない構造体ではできるのですが...

発生している問題・エラーメッセージ

ques.cpp: In function ‘void draw(board*)’:
ques.cpp:18:8: error: no match for ‘operator>>’ (operand types are ‘std::ostream’ {aka ‘std::basic_ostream<char>’} and ‘std::string’ {aka ‘std::__cxx11::basic_string<char>’})
18 | cout>>ichi[0].maru>>endl;
| ~~~~^~~~~~~~~~~~~~
| | |
| | std::string {aka std::__cxx11::basic_string<char>}
| std::ostream {aka std::basic_ostream<char>}
In file included from /usr/include/c++/9/string:56,
from /usr/include/c++/9/bits/locale_classes.h:40,
from /usr/include/c++/9/bits/ios_base.h:41,
from /usr/include/c++/9/ios:42,
from /usr/include/c++/9/istream:38,
from /usr/include/c++/9/sstream:38,
from /usr/include/c++/9/complex:45,
from /usr/include/c++/9/ccomplex:39,
from /usr/include/x86_64-linux-gnu/c++/9/bits/stdc++.h:54,
from ques.cpp:1:
/usr/include/c++/9/bits/basic_string.tcc:1466:5: n

該当のソースコード

c++

#include <bits/stdc++.h>
#include<iostream>
#include<string>
#include<vector>
#include <array>
using namespace std;
using std::cout; using std::endl;
using std::vector; using std::string;

struct board{
  int x; 
  int y;  
  string maru;
};
  void draw(board ichi[])
	{
  cout>>ichi[0].maru>>endl;}

int main(){
  int i,k,j;
  board ichi[9];
  i=0;
 
  while(i<9){
	for(k=1;k<4;k++){
	  for(j=1;j<4;j++){
		ichi[i]={k,j,0,"○"};   
				i++;
	  }
	}
  }

  draw(ichi);

  return 0;
}

#補足
コンパイルエラーはまだ続いているのですが、膨大すぎるのでpick　upしてのせました

Comments

[kazuma-s]

ソースコードの貼り付けに失敗しているのを何とも思いませんか？
teratail のヘルプを見て、修正してください。

[tendon_udon]

ご指摘ありがとうございますm(__)m　修正しました

Answer 1

cout には << という演算子はあっても、>> という演算子はありません。
cin には >> があります。
エラーメッセージをちゃんと読みましょう。

Comments

[tendon_udon]

そのとおりでした
no match for ' operator'>> ってかいてありますね
ご指摘ありがとうございますm(__)m

Answer 2

配列を関数に渡す典型パターンをいくつか紹介しておきます。関数名はとりあえずfooにしておきます。

配列の先頭へのポインタを引数に取る

void foo(int* arr);

これは関数側からは要素数がわからないのでbuffer overrunの危険があります。やめましょう。

配列の先頭へのポインタと要素数を引数に取る

void foo(int* arr, std::size_t n);

これならbuffer overrunの危険がありません。C APIによく見られるパターンです

配列へのlvalue refreceを引数に取る

template<std::size_t N>
void foo(int (&arr)[N]);

この場合配列の要素数情報が型から消えないためbuffer overrunの危険がありませんが、要素数を可変にするためにtemplateが必要となるため、ヘッダーファイルに事実上書く必要があります。めったに使わない。

Cの配列を投げ捨ててstd::arrayを使う場合

template<std::size_t N>
void foo(std::array<int, N>& arr);

安全に扱えますが、呼び出し側に制約を与えるため利用ケースは限られます。どっちかというとconst std::array<int, N>&のほうがまだ使う。後述のイテレータのほうがよく使うので出番なし

イテレータのペアを受け取る

template<typename Iterator>
void foo(Iterator begin, Iterator end);

最頻出パターン。要素の先頭と終端の一つ先へのイテレータを渡す。それぞれstd::begin(), std::end()で取れる(C++20からはstd::ranges::begin()/std::ranges::end())。

引数が2個なのと、呼び出し側にボイラープレートが増えるのが問題。

また、template引数が本当にイテレータなのか検査が必要だが、C++11では安全に検査する方法はなく、C++14を待たなければならない・・・あれっ、そうだっけ？？まあなんにせよiterator_traits見たりするので面倒くさい。
https://stackoverflow.com/questions/25290462/how-to-define-is-iterator-type-trait

rangeを受け取る(C++20)

template<std::ranges::range Range>
void foo(Range&& r)

こういうかんじですね。C++20で追加されたconceptを使っています。呼び出し側も1引数で済むのでこれからの主流でしょう。

#include <ranges>
#include <iostream>
template<std::ranges::range Range>
void foo(Range&& r)
{
    std::cout << std::ranges::size(r) << std::endl;
}
int main()
{
    int arr[3]{};
    foo(arr);
}

https://wandbox.org/permlink/9TMsIsMpsYkjtl0y

Answer 3

#include <iostream>
#include <string>

struct board {
    int x;
    int y;
    std::string maru;
};

void draw(board ichi[]) {
  using namespace std;
  cout << ichi[0].maru << " " << ichi[3].maru << " " << ichi[6].maru << endl;
  cout << ichi[1].maru << " " << ichi[4].maru << " " << ichi[7].maru << endl;
  cout << ichi[2].maru << " " << ichi[5].maru << " " << ichi[8].maru << endl;
}

int main() {
    int i, k, j;
    board ichi[9];
    i = 0;

    while (i < 9) {
        for (k = 1; k < 4; k++) {
            for (j = 1; j < 4; j++) {
                ichi[i] = { k,j,"○" };
                i++;
            }
        }
    }

    draw(ichi);
    return 0;
}

Comments

[tendon_udon]

膨大なエラーの原因は<<を>>にしてたことだったんですね
ありがとうございます！！
また、includeやstdも重要だったのですね（全然わからなかったのでとりあえず入れとけの精神でincludeしまくってました
初期学習中に、ふと思いついたプログラムを実装したくなったのでつくったのですが、新しいプログラムを作るのは基本を終えてからにしようと思います

[episteme]

× 全然わからなかったのでとりあえず入れとけ
○ 目的の実現には不可欠なので確実に入れとく

[episteme]

あと
#include <bits/stdc++.h>
これはやめとけ。

どれを #include すべきかわかんなくなる。

...とはいえ「初学者にどれ#includeするかなんてわかるわけねーだろ。そんなことよりコード書いて動くことを体感させたい」ってことなら、わからんでもない。

[tendon_udon]

＃include<bits/stdc++.h>は包括的すぎて言語の理解を妨げるのですね…
気をつけますm(__)m

Answer 4

board ichi[9];で確保されるのはichi[0]からichi[8]までです。ichi[9]にアクセスした場合、何が起きるかわかりません。

Comments

[tendon_udon]

そうですね！
ichi[6].maruを抜かしてました…
ですが、修正してもコンパイルエラーが出てしまいます…

[maisumakun]

> 修正してもコンパイルエラーが出てしまいます…

コンパイルエラーが出るのでしたら、質問時点で書いてほしかったです。

どこにどのようなエラーが出ましたか？

[tendon_udon]

確かにそうすべきでした
ご指摘ありがとうございます
再編集しました