Is there a better way to do this?
#Struct
storm basin
mild pasture
wary stratus
#define RGB2GL 1.0f / 255.0f
...
float r = 224.0f * RGB2GL;
leaden grotto
Is there a better way to do this?
that struct is kind of useless
iron hazelBOT
@leaden grotto has reached level 3. GG!
storm basin
that struct is kind of useless
why
iron hazelBOT
@storm basin has reached level 14. GG!
leaden grotto
because you can't pass to any functions unless their type is Color_sky
it should be an instance of an rgb struct
leaden grotto
you aren't passing the whole struct in, just its individual fields
storm basin
yeah
leaden grotto
either way, that's a bad design imo
storm basin
can you send better code
im not a pro
so idk wym by "an instance of an rgb struct"
leaden grotto
struct rgb_t {
float r, g, b, a;
};
// assuming this is at global scope
static rgb_t constexpr COLOR_SKY { .r = ..., .b = ..., .g = ..., .a = ...};
replace the ellipsis with the values you used above
like this, you can freely define other colors as well:
void whatever(){
rgb_t const red { .r = 1.f, .b = 0.f, .g = 0.f, .a = 1.f };
// blah blah blah
}
btw im not sure this syntax will work
it is only supported by c++ since c++17 i think
storm basin
it works but without the rgba names
just nums
and no dots
so like cpp .......... COLOR_SKY {1, 2, 3, 4};
leaden grotto
it is only supported by c++ since c++17 i think
it's actually since c++20
which is funny because this is available in c99
wary stratus
that struct is kind of useless
lol I didnt even notice
thorny dust
Is there a better way to do this?
struct Vec4f{
float x,y,z,w;
Vec4f(){}
Vec4f(float x,float y,float z,float w) : x(x),y(y),z(z),w(w){}
};
struct Vec4f skyColor = Vec4f(...);
to retrieve values simply do skyColor.x,y,z, or w
You already have a Vec4f object in the glm namespace
use it or make your own vec4f
leaden grotto
```cpp
struct Vec4f{
float x,y,z,w;
Vec4f(){}
Vec4f(float x,float y,...
you need not to define those constructors
thorny dust
it's a preference
thorny dust
like this, you can freely define other colors as well:
```cpp
void whatever(){
...
can you do this with classes too?
curious
wary stratus
can you do this with classes too?
using std::initializer_list as single constructor argument