2010-04-06 9 views
21

Próbuję utworzyć mapę, gdzie klucz jest typu int, a wartość jest tablicąKorzystanie tablicę jako wartości mapy: nie widzi błędu

int red[3] = {1,0,0}; 
int green[3] = {0,1,0}; 
int blue[3]  = {0,0,1}; 

    std::map<int, int[3]> colours; 

colours.insert(std::pair<int,int[3]>(GLUT_LEFT_BUTTON,red)); //THIS IS LINE 24 ! 
colours.insert(std::pair<int,int[3]>(GLUT_MIDDLE_BUTTON,blue)); 
colours.insert(std::pair<int,int[3]>(GLUT_RIGHT_BUTTON,green)); 

Jednak gdy próbuję skompiluj ten kod, otrzymuję następujący błąd.

g++ (Ubuntu 4.4.1-4ubuntu8) 4.4.1 

In file included from /usr/include/c++/4.4/bits/stl_algobase.h:66, 
       from /usr/include/c++/4.4/bits/stl_tree.h:62, 
       from /usr/include/c++/4.4/map:60, 
       from ../src/utils.cpp:9: 
/usr/include/c++/4.4/bits/stl_pair.h: In constructor ‘std::pair<_T1, _T2>::pair(const _T1&, const _T2&) [with _T1 = int, _T2 = int [3]]’: 
../src/utils.cpp:24: instantiated from here 
/usr/include/c++/4.4/bits/stl_pair.h:84: error: array used as initializer 
/usr/include/c++/4.4/bits/stl_pair.h: In constructor ‘std::pair<_T1, _T2>::pair(const std::pair<_U1, _U2>&) [with _U1 = int, _U2 = int [3], _T1 = const int, _T2 = int [3]]’: 
../src/utils.cpp:24: instantiated from here 
/usr/include/c++/4.4/bits/stl_pair.h:101: error: array used as initializer 
In file included from /usr/include/c++/4.4/map:61, 
       from ../src/utils.cpp:9: 
/usr/include/c++/4.4/bits/stl_map.h: In member function ‘_Tp& std::map<_Key, _Tp, _Compare, _Alloc>::operator[](const _Key&) [with _Key = int, _Tp = int [3], _Compare = std::less<int>, _Alloc = std::allocator<std::pair<const int, int [3]> >]’: 
../src/utils.cpp:30: instantiated from here 
/usr/include/c++/4.4/bits/stl_map.h:450: error: conversion from ‘int’ to non-scalar type ‘int [3]’ requested 
make: *** [src/utils.o] Error 1 

Naprawdę nie widzę, gdzie jest błąd. Lub nawet jeśli wystąpił błąd.

+0

Dowiedz się czegoś każdy dzień :) – Tom

Odpowiedz

33

Nie można kopiować tablic według takich wartości.

Oto kilka rozwiązań, ale polecam nr 4 do swoich potrzeb:

1) Użyj std::vector zamiast tablicy

2) Za pomocą mapy wskaźników do tablic z 3 elementów.

int red[3] = {1,0,0}; 
int green[3] = {0,1,0}; 
int blue[3]  = {0,0,1}; 
std::map<int,int(*)[3]> colours; 
colours.insert(std::pair<int,int(*)[3]>(GLUT_LEFT_BUTTON,&red)); 
colours.insert(std::pair<int,int(*)[3]>(GLUT_MIDDLE_BUTTON,&blue)); 
colours.insert(std::pair<int,int(*)[3]>(GLUT_RIGHT_BUTTON,&green)); 
//Watch out for scope here, you may need to create the arrays on the heap. 

3) Użyj boost tuples zamiast tablic 3 elementów.

4) Zamiast korzystać z tablicy, należy utworzyć nową strukturę, która zajmuje 3 elementy. Stwórz mapę. Lub zawiń tablicę w strukturę i to też zadziała.

struct Triple 
{ 
    int color[3]; 
}; 

//Later in code 
Tripple red = {1, 0, 0}, green = {0, 1, 0}, blue = {0, 0, 1}; 
std::map<int,Triple> colours; 
colours.insert(std::pair<int,Triple>(GLUT_LEFT_BUTTON,red)); 
colours.insert(std::pair<int,Triple>(GLUT_MIDDLE_BUTTON,blue)); 
colours.insert(std::pair<int,Triple>(GLUT_RIGHT_BUTTON,green)); 
+0

Dzięki, fajne alternatywne rozwiązanie również. – Tom

+0

około 3) To jest mały projekt, nie mogę dodać doładowania, ale zajmie spojrzenie, dzięki. – Tom

+0

@Dom: dodano 4), co może być najprostszym rozwiązaniem. –

6

Tablice nie są pierwszymi konstruktorami w C++. Nie są to wartości wymagające wartości Copy Constructiblestd::map. Możesz użyć boost::array lub std::vector.

1

tablice nie mogą być przechowywane dane w standardowym kontenerze (std::pair)

3

Inną alternatywą jest umieszczenie macierzy do struktury dozowania:

struct Wrapper { int value[3]; }; 

    // ... 
    Wrapper red = {{1,0,0}};  
    std::map<int,Wrapper> colours;  
    colours.insert(std::pair<int,Wrapper>(1, red)); 
0

zastosowanie std :: make_pair:

int red[3] = {1,0,0}; 
int green[3] = {0,1,0}; 
int blue[3]  = {0,0,1}; 
std::map<int,int(*)[3]> colours; 
colours.insert(std::make_pair(GLUT_LEFT_BUTTON,&red)); 
colours.insert(std::make_pair(GLUT_MIDDLE_BUTTON,&blue)); 
colours.insert(std::make_pair(GLUT_RIGHT_BUTTON,&green)); 
5

Użyj std::tr1::array.

typedef std::tr1::array<int, 3> Triple; 
Triple red = {1, 0, 0}; 
Triple green = {0, 1, 0}; 
Triple blue = {0, 0, 1}; 
std::map<int, Triple> colours; 
colours.insert(std::make_pair(GLUT_LEFT_BUTTON, red)); 
colours.insert(std::make_pair(GLUT_MIDDLE_BUTTON, blue)); 
colours.insert(std::make_pair(GLUT_RIGHT_BUTTON, green)); 
+1

znany również teraz jako po prostu 'std :: array' ;-) czyniąc to naprawdę jedyną wciąż istotną odpowiedź. dziwne, jak ma tak mało głosów ... –

5

nie mapować do int [], zamiast mapować do int * tak:

#include <iostream> 
#include <map> 
using namespace std; 
int main(){ 
    std::map<int,int*> colors; 
    int red[] = {3,7,9}; 
    colors[52] = red; 
    cout << colors[52][1]; //prints 7 
    colors[52][1] = 11; 
    cout << colors[52][1]; //prints 11 
    return 0; 
} 
0

podejście z użyciem struktury w C++

int MAX_DATA_PER_INSTR = 8; 
//struct to hold the values. remember to write the constructor 
struct InstrChar 
{ 
    InstrChar(int in[MAX_DATA_PER_INSTR]) { 
    //c alternative is memcopy 
    std::copy(in, in+MAX_DATA_PER_INSTR, data); 
    } 
    int data[MAX_DATA_PER_INSTR]; 
}; 

// create a key value pair 
std::map <int, InstrChar> address_instructions; 
std::map <int, InstrChar>::iterator it; 

// sample array, 8 elements 
int xp[MAX_DATA_PER_INSTR ] = {31,4,3,4,4,3,1,2}; 
address_instructions.insert(std::pair<int, InstrChar>(PC, xp)); 
it = address_instructions.find(PC); 
InstrChar buf1 = it->second; 
//integer pointer to the array, can be dereferenced as *p, *(p+1), .... //*(p+7) 
int *p = buf1.data; 

//in case you need to print these values out. They can also be referred to as buf1.data[0], buf1.data[1], buf1.data[2] 
printf("%d\n", (*p)); 
printf("%d\n", *(p+1)); 
printf("%d\n", *(p+2)); 
printf("%d\n", *(p+3)); 
printf("%d\n", *(p+4)); 
printf("%d\n", *(p+5)); 
printf("%d\n", *(p+6)); 
printf("%d\n", *(p+7)); 
Powiązane problemy