How to Route a Horizontal Slider Value to My Code?

[anda_skoa]

hard to tell without knowing how you instantiate Gain and how you tried to connect it.

Btw, if you want to convert the value to be in the floating point range of 0 to 1 I would do that right there in Gain where you have access to the slider's range.
I.e. Gain::value(), setValue() and valueChanged() would all deal with the 0..1 range, which they then map to the actual range of the slider.

This allows you to change the "precision" at any time just by changing the slider's range.

Cheers,
_

[keenanw]

Thanks for the reply.

Well, for instance, in sine.cpp I tried this:

int tick( void *outputBuffer, void *inputBuffer, unsigned int nBufferFrames, double streamTime, RtAudioStreamStatus status, void *dataPointer )
  {
  Gain *volume = new volume;
 
  SineWave *sine = (SineWave *) dataPointer;
  register StkFloat *samples = (StkFloat *) outputBuffer;
  for ( unsigned int i=0; i<nBufferFrames; i++ )
      *samples++ = (sine->tick() * vel) * volume->valueChanged() / 100;
 
  return 0;
}

int tick( void *outputBuffer, void *inputBuffer, unsigned int nBufferFrames, double streamTime, RtAudioStreamStatus status, void *dataPointer )
  {
  Gain *volume = new volume;

  SineWave *sine = (SineWave *) dataPointer;
  register StkFloat *samples = (StkFloat *) outputBuffer;
  for ( unsigned int i=0; i<nBufferFrames; i++ )
      *samples++ = (sine->tick() * vel) * volume->valueChanged() / 100;

  return 0;
}

To copy to clipboard, switch view to plain text mode 

This results in a number of errors including "no matching function for call to 'Gain::valueChanged()'" and "cannot convert 'int*' to 'gain*' in initialization". So I'm asking a very general question. What is a good way to instantiate Gain in the first place in any c++ code? Any ideas or links to examples? I can't seem to find anything useful about this on the internet, probably because I don't know enough to google the right questions.

[ChrisW67]

The "no matching function" error arises because valueChanged() is a signal and its generated code is declared in the protected section of the class. If you want the value from the slider object directly then you want the value() function, not the signal indicating it has changed.

I assume that the "cannot convert 'int*' to 'gain*' in initialization" relates to this line:

Gain *volume = new volume;

Gain *volume = new volume;

To copy to clipboard, switch view to plain text mode 

The LHS of this is type "Gain*". Since there's no class called "volume" the compiler has assumed an int so the RHS is type "int*". It's not entirely clear what you are trying to achieve here with this circular reference. If, as I suspect, you intended to create a new Gain object:

Gain *volume = new Gain;

Gain *volume = new Gain;

To copy to clipboard, switch view to plain text mode 

then, had you succeeded, you would have always had value() == 50 from a slider that the user neither saw nor interacted with.

In a normal Qt program you would have a UI built containing widgets like your slider, ideally quite separate from the code that actually does substantial stuff in response. You can use Designer or hand code that UI. You would connect() the valueChanged() signal of your slider to a slot that reacts immediately to the changed value, for example pushing a new volume figure into your generator code. Assuming the tick() routine is part of the code you have pushed to another thread you could put the receiving slot in that QObject and store the current volume figure as a member variable in the generator thread.

Unless you want to do something like anda_skoa suggests to get a floating point or logarithmic slider then it strikes me that your Gain class could simply be a QSlider subclass, something like:

class Gain: public QSlider
{
  Q_OBJECT
public:
  explicit Gain(QWidget *p = 0): QSlider(Qt::Horizontal, p) {
    setRange(1, 100);
    setValue(50);
  }
};

class Gain: public QSlider
{
  Q_OBJECT
public:
  explicit Gain(QWidget *p = 0): QSlider(Qt::Horizontal, p) {
    setRange(1, 100);
    setValue(50);
  }
};

To copy to clipboard, switch view to plain text mode 

Do you want to allow a zero volume?

[keenanw]

Yeah, I really meant

Gain *volume = new Gain;

Gain *volume = new Gain;

To copy to clipboard, switch view to plain text mode 

I'll look into implementing the gui with Qt Designer, since I'm still having no luck. Thanks.