JacksonSteel Ответов: 2

[Leetcode 5 ]на самом деле не могу найти, что не так в моем решении для самой длинной проблемы подстроки палиндрома


input:
babad
abbd

output:
ad
bb

expected:
bab
bb

#include<iostream>
using namespace std;
class Solution {
public:
        string longestPalindrome(string s) {
        int maxlength=1;
        bool ispalindromic[1000][1000]={false};
        for(int i=0;i<s.length();i++)
                ispalindromic[i][i]=1;
                
        for(int l=2;l<s.length();l++){
        for(int i=0;i<s.length()-1; i++){
            int j=i+l-1;
            if(l==2&&s[i]==s[j]){
                ispalindromic[i][j]=1;
            maxlength=max(maxlength,j-i+1);
                continue;}
            if(ispalindromic[i+1][j-1]&&s[i]==s[j]){
                ispalindromic[i][j]=1;
            maxlength=max(maxlength,j-i+1);
            }
        }}
        for(int i=0;i<s.length();i++){
        int j=i+maxlength-1;
            if(ispalindromic[i][j]){
                   return s.substr(i,j);
                }
            }
        return s.substr(0,1);
        }
};


Что я уже пробовал:

Сначала я создал ispalindromic[1000][1000] и убедился, что каждый алфавит сам по себе является палиндромным. Затем я проверяю палиндром от длины 2 и так далее. Всякий раз, когда ispalindromic становится истинным, код обновляет maxlength, так что в конце концов код может просто использовать maxlength для печати самого длинного палиндрома.

I also added cout<<j<<endl; to see whether it's j's fault. if i input abbd, j is 2 which is right, and program prints right answer bb, but if I input acddc, j is still right(3), but the program prints ddc which is wrong.
 
if(ispalindromic[i][j]){
            	   cout<<j<<endl;
                   return s.substr(i,j);
                }

2 Ответов

Рейтинг:
2

OriginalGriff

Компиляция не означает, что ваш код верен! :смеяться:
Подумайте о процессе разработки как о написании электронного письма: успешная компиляция означает, что вы написали электронное письмо на правильном языке - например, на английском, а не на немецком, - а не то, что письмо содержало сообщение, которое вы хотели отправить.

Итак, теперь вы входите во вторую стадию разработки (на самом деле это четвертая или пятая, но вы перейдете к более ранним стадиям позже): тестирование и отладка.

Начните с рассмотрения того, что он делает, и как это отличается от того, что вы хотели. Это важно, потому что это дает вам информацию о том, почему он это делает. Например, если программа предназначена для того, чтобы позволить пользователю ввести число, а он удваивает его и печатает ответ, то если ввод / вывод был таким:

Input   Expected output    Actual output
  1            2                 1
  2            4                 4
  3            6                 9
  4            8                16
Тогда совершенно очевидно, что проблема заключается в бите, который удваивает его - он не прибавляет себя к себе или умножает его на 2, он умножает его на себя и возвращает квадрат входного сигнала.
Таким образом, вы можете посмотреть на код, и очевидно, что он находится где-то здесь:
int Double(int value)
   {
   return value * value;
   }

К счастью, у вас есть инструмент, который поможет вам выяснить, что происходит: отладчик. Как вы его используете, зависит от вашей системы компилятора, но быстрый поиск в Google имени вашей IDE и "отладчика" должен дать вам необходимую информацию.

Как только у вас появится идея, что может пойти не так, начните использовать отладчик, чтобы выяснить, почему. Поместите точку останова в первую строку метода и запустите приложение. Когда он достигнет точки останова, отладчик остановится и передаст управление вам. Теперь вы можете запускать свой код построчно (так называемый "одноступенчатый") и просматривать (или даже изменять) содержимое переменных по мере необходимости (черт возьми, вы даже можете изменить код и повторить попытку, если вам это нужно).
Подумайте о том, что должна делать каждая строка кода перед ее выполнением, и сравните это с тем, что она действительно делала, когда вы использовали кнопку "Step over" для выполнения каждой строки по очереди. Он сделал то, что вы ожидали? Если да, то переходите к следующей строке.
Если нет, то почему? Чем это отличается?

Надеюсь, это поможет вам определить, в какой части этого кода есть проблема и в чем она заключается.
Это навык, и его стоит развивать, поскольку он помогает вам как в реальном мире, так и в развитии. И, как и все навыки, он только улучшается при использовании!


JacksonSteel

Я добавил соиь<&ЛТ;Дж&ЛТ;<епси; до="" см.="" ли="" это="" J и="" вине="".если="" я="" ввода="" abbd,="" ж="" является="" 2="" который="" права="" и="" в="" программе="" печатает="" ответа="" вв,="" но="" acddc,="" еще="" право(3),="" он="" цифрового="" неверно.

<pre="">if(ispalindromic[i][j]){
соиь<&ЛТ;Дж&ЛТ;<епси;
возвратить С. функцию substr(я,J);
}

Patrice T

ваше сообщение повреждено

JacksonSteel

https://www.paste.org/108420

Рейтинг:
0

Patrice T

Совет: Научитесь правильно делать отступы в вашем коде, это покажет его структуру и поможет чтению и пониманию. Это также помогает выявлять структурные ошибки.

#include<iostream>
using namespace std;
class Solution {
    public:
    string longestPalindrome(string s) {
        int maxlength=1;
        bool ispalindromic[1000][1000]={false};
        for(int i=0;i<s.length();i++)
            ispalindromic[i][i]=1;

        for(int l=2;l<s.length();l++){
            for(int i=0;i<s.length()-1; i++){
                int j=i+l-1;
                if(l==2&&s[i]==s[j]){
                    ispalindromic[i][j]=1;
                    maxlength=max(maxlength,j-i+1);
                continue;}
                if(ispalindromic[i+1][j-1]&&s[i]==s[j]){
                    ispalindromic[i][j]=1;
                    maxlength=max(maxlength,j-i+1);
                }
            }}
            for(int i=0;i<s.length();i++){
                int j=i+maxlength-1;
                if(ispalindromic[i][j]){
                    return s.substr(i,j);
                }
            }
            return s.substr(0,1);
        }
    }

Стиль отступа - Википедия[^]

Профессиональные редакторы программистов имеют эту функцию и другие, такие как сопоставление скобок и подсветка синтаксиса.
Блокнот++ Главная Страница[^]
личные[^]
-----
В коде с отступом:
#include<iostream>
using namespace std;
class Solution {
    public:
    string longestPalindrome(string s) {
        int maxlength=1;
        bool ispalindromic[1000][1000]={false};
        for(int i=0;i<s.length();i++)
            ispalindromic[i][i]=1;

        for(int l=2;l<s.length();l++){
            for(int i=0;i<s.length()-1; i++){
                int j=i+l-1;
                if(l==2&&s[i]==s[j]){
                    ispalindromic[i][j]=1;
                    maxlength=max(maxlength,j-i+1);
                continue;}
                if(ispalindromic[i+1][j-1]&&s[i]==s[j]){
                    ispalindromic[i][j]=1;
                    maxlength=max(maxlength,j-i+1);
                }
            }} // the double } here is saving 1 keystroke at cost of obfuscationg the code strubture
            for(int i=0;i<s.length();i++){
                int j=i+maxlength-1;
                if(ispalindromic[i][j]){
                    return s.substr(i,j);
                }
            }
            return s.substr(0,1); // This return is inside a for loop, surprising !
            // it prevent the loop from looping !
        }
    }

В вашем коде я вижу maxlength на протяжении всего палиндрома, но я не вижу ни одного maxposition для позиции палиндрома.
-----
Ваш код ведет себя не так, как вы ожидаете, или вы не понимаете, почему !

Существует почти универсальное решение: запускайте свой код на отладчике шаг за шагом, проверяйте переменные.
Отладчик здесь, чтобы показать вам, что делает ваш код, и ваша задача-сравнить с тем, что он должен делать.
В отладчике нет никакой магии, он не знает, что должен делать ваш код, он не находит ошибок, он просто помогает вам, показывая, что происходит. Когда код не делает того, что ожидается, вы близки к ошибке.
Чтобы увидеть, что делает ваш код: просто установите точку останова и посмотрите, как работает ваш код, отладчик позволит вам выполнять строки 1 на 1 и проверять переменные по мере их выполнения.

Отладчик - Википедия, свободная энциклопедия[^]

Освоение отладки в Visual Studio 2010 - руководство для начинающих[^]
Базовая отладка с помощью Visual Studio 2010 - YouTube[^]

1.11 — отладка программы (пошаговое выполнение и останова) | выучить C++[^]

Отладчик здесь только для того, чтобы показать вам, что делает ваш код, и ваша задача-сравнить его с тем, что он должен делать.

[Обновление]
Ваш код очень сложный.
Попробуйте вручную решить проблему и посмотреть, нужен ли вам огромный логический массив, чтобы получить ответ.


JacksonSteel

j-это maxposition, и зачем возвращать s.substr(0,1); предотвращать циклирование цикла ?

Patrice T

Посмотрите на ispalindromic, где вы его инициализируете ?

JacksonSteel

bool ispalindromic[1000][1000]={false};

Patrice T

Инициализация массива не является его объявлением.
Инициализация массива - это придание начального значения каждому элементу.
C не стирайте переменные автоматически, вы должны сделать это явно.

Ладно, моя беда, я пропустил инициализацию.

Patrice T

Если вы хотите понять, что не так в вашем коде, вам нужно запустить его на отладчике и посмотреть, что именно он делает.