Remove duplicate code in TopicPublishInfo

[zyx333]

The code block below in the selectOneMessageQueue method is identical to another overloaded method (2nd code block).

rocketmq/client/src/main/java/org/apache/rocketmq/client/impl/producer/TopicPublishInfo.java

Lines 74 to 76 in 7d67484

	int index = this.sendWhichQueue.incrementAndGet();
	int pos = index % this.messageQueueList.size();
	MessageQueue mq = this.messageQueueList.get(pos);

rocketmq/client/src/main/java/org/apache/rocketmq/client/impl/producer/TopicPublishInfo.java

Lines 85 to 90 in 7d67484

	public MessageQueue selectOneMessageQueue() {
	int index = this.sendWhichQueue.incrementAndGet();
	int pos = index % this.messageQueueList.size();
	return this.messageQueueList.get(pos);
	}

IMO, it can be rewritten like this.

for (int i = 0; i < this.messageQueueList.size(); i++) {
         MessageQueue mq = selectOneMessageQueue()
         if (!mq.getBrokerName().equals(lastBrokerName)) {
               return mq;
         }
 }

If it's acceptable, I will submit a pull request.

[Oliverwqcwrw]

IMO, It's a good idea, welcome your PR :-)

[AdityaP7649]

Hi there! I'm interested in working on this issue. It aligns with my skills. Could you please assign this issue to me, or guide me on how to get started? Thanks!

[benedikt-roth]

@Oliverwqcwrw I would like to work on this in case it is not picked up yet.

[farukhfrk]

I would like to work on this, Could you please assign it to me?

[ThestralWarrior]

i see this hasn't been assigned yet, can I work on this?

[bitWizrd]

@Oliverwqcwrw Kindly to ask if there is somebody working for this issue? lf not, I'm glad to undertake it.

[Oliverwqcwrw]

Thank you very much for your attention to the community, I noticed that the latest code has changed, please take a look at the latest code to check whether the problem still exists :)

[ling0900]

Is it possible for this issue to be closed in the near future? If not, I'll try to follow up on it.