How can i clean my messy code

[suslucoder]

Hi everyone. Im a newbie, because of that i apolige. I have a messy code. I want to clean it and want it shorter. How can i do it?

 if(item->parent() == gps)
  {
    if(scln == "A")
    {
        seri[0]->setColor(Qt::blue);
        seri[0]->setName(gpsList.at(0));
        chart->addSeries(seri2[0]);
        seri[0]->attachAxis(axisX);
        seri[0]->attachAxis(axisY);
        chart->legend()->setVisible(true);
        queuee[0].enqueue(0.25);

    }

    if(scln == "B")
    {
        seri[1]->setName(gpsList.at(1));
        seri[1]->setColor(Qt::green);
        chart->addSeries(seri2[1]);
        seri[1]->attachAxis(axisX);
        seri[1]->attachAxis(axisY);
        chart->legend()->setVisible(true);
          queuee[1].enqueue(0.96);
    }

    if(scln == "C")
    {
        seri[2]->setName(gpsList.at(2));
        seri[2]->setColor(Qt::yellow);
        chart->addSeries(seri[2]);
        seri[2]->attachAxis(axisX);
        seri[2]->attachAxis(axisY);
        chart->legend()->setVisible(true);
        queuee[2].enqueue(0.39);
    }
    if(scln == "D")
    {
        seri[3]->setName(gpsList.at(3));
        seri[3]->setColor(Qt::darkBlue);
        chart->addSeries(seri[3]);
        seri[3]->attachAxis(axisX);
        seri[3]->attachAxis(axisY);
        chart->legend()->setVisible(true);
        queuee[3].enqueue(0.66);
    }

[keskiverto]

Do not double-post. Other thread: http://www.cplusplus.com/forum/general/277206/

[JRManx]

Here:
Don't use a bunch of if statements like that. Use else if instead. And add an else at the end for error checking.

if (item->parent () == gps)
{
	if (scln == "A")
	{
		seri[0]->setColor (Qt::blue);
		seri[0]->setName (gpsList.at(0));
		
		chart->addSeries (seri2[0]);
		
		seri[0]->attachAxis (axisX);
		seri[0]->attachAxis (axisY);
		
		chart->legend()->setVisible (true);
		
		queuee[0].enqueue (0.25);
	}
	else if (scln == "B")
	{
		seri[1]->setName (gpsList.at(1));
		seri[1]->setColor (Qt::green);
		
		chart->addSeries (seri2[1]);
		
		seri[1]->attachAxis (axisX);
		seri[1]->attachAxis (axisY);
		
		chart->legend()->setVisible (true);
		
		queuee[1].enqueue (0.96);
	}
	else if (scln == "C")
	{
		seri[2]->setName (gpsList.at(2));
		seri[2]->setColor (Qt::yellow);
		
		chart->addSeries (seri[2]);
		
		seri[2]->attachAxis (axisX);
		seri[2]->attachAxis (axisY);
		
		chart->legend()->setVisible (true);
		
		queuee[2].enqueue (0.39);
	}
	else if (scln == "D")
	{
		seri[3]->setName (gpsList.at(3));
		seri[3]->setColor (Qt::darkBlue);
		
		chart->addSeries (seri[3]);
		
		seri[3]->attachAxis (axisX);
		seri[3]->attachAxis (axisY);
		
		chart->legend()->setVisible (true);
		
		queuee[3].enqueue (0.66);
	}
	// Add an else in here for error checking
	else
	{
		std::cout << "Error. SCLN must be A, B, C, or D. \n";
	}
}

And yeah, please don't double-post. It fills up the website with unnecessary junk that we have to wade through, and it causes people to think you may be a spammer.

Best,
max

[Ganado]

Please do not bump double-posts. Thank you.

[JRManx]

"Bump"? What do you mean exactly? Are you referring to me and @keskiverto? If so, why not? I don't think anyone on this forum likes double-posters, thus the warning to the OP.

[mbozzi]

When one replies to a thread it appears at (is "bumped") to the top of the post queue.

Forum etiquette is to keep all the conversation in one place and also to get useless threads out of the feed. If someone says "this is a duplicate of <link>", ideally one should follow the link and comment there.