Left operand of comma operator has no effect [-Wunused-value]

Question

#include<iostream>
#include<string>
#include<cstdlib>
#include<ctime>
#include <algorithm>

using namespace std;
string los;
string z;
int iL, iP, i, j=2, n=30;

int main()
{

    srand(time(NULL));

    for(int i = 0; i < 30; i++ )
    {
        los =(( rand() %('d'-'a'))+'a');
        z=z+los;
    }

    cout << "Ciag znakow: " << z << endl;
    for(i=0; i<=n-2; i++)
      for(j=i+j; j<=n-1; j++)
           {
            iL=i;
            iP=j-1;
            while(iL<iP)
            {
                if(z[iL]!=z[iP])
                  break;
                else
                  iL=iL+1;
                  iP=iP-1;
                  
            }
            cout << z[i,j]; 
           }
           
        
            
    return 0;
}

Mam tutaj kod, który ma wypisać mi palindromy z losowego łańcucha znaków, składającego się z liter ABCD. Kiedy chce wypisać te wartości, wypisuje się ten sam łańcuch, na dodatek z pominięciem 2 pierwszych liter oraz pojawia się jeszcze ten błąd obok tego cout. Jakieś rozwiązanie?

Comments

https://en.wikipedia.org/wiki/Comma_operator

---

no niespecjalnie mi pomoże artykuł z wikipedii

Answer 1

nie rozumiem co ty chciałeś osiągnąć tymi iL oraz iP ale i tak nigdzie ich nie wykorzystujesz, podam ci kilka błędów które możesz poprawić:
1. To może nie jest aż taki błąd, ale nie potrzebujesz #include<string>, #include<cstdlib>, #include<algorithm> po prostu i tak nie wykorzystujesz tego w programie

2. Pętla druga mówi to że do j które jest zadeklarowane jako 2 dodajesz i oraz j no ok, w pierwszej turze pętli idzie od 2 do 29 ale 30 ale właśnie zostało w j więc przy kolejnym odtworzeniu odrazu pętla się zamknie no bo 30 jest nie jest mniejsze lub równe 29.

3. zmienne iL oraz iP do niczego ci w programie nie służą

4. tablica z w której masz wszystkie losowe znaki od A do D się nic nie zmienia, więc wypisujesz ją po prostu po kolei, a jak j masz ustawione na 2 to zaczyna ci od 2. dodam że nie wiem jak to ma działać, ale z tego co widać to z tablicy bierze tylko argument położony po prawo czyli w przypadku z[i,j] weźmie wartość j.

5. else nie ma nawiasów klamrowych