Skip to content

fix : get just the project#21

Open
ToavinaGitHub wants to merge 6 commits into
mohsinenur:masterfrom
ToavinaGitHub:dev
Open

fix : get just the project#21
ToavinaGitHub wants to merge 6 commits into
mohsinenur:masterfrom
ToavinaGitHub:dev

Conversation

@ToavinaGitHub
Copy link
Copy Markdown

No description provided.

Copy link
Copy Markdown

@Imran-imtiaz48 Imran-imtiaz48 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review
The original code had duplicate connection blocks, which were unnecessary. Here are some improvements made in the reviewed code:

  1. Removed Duplicate Code: The duplicate connection code was removed for clarity.
  2. Use of Variables: Introduced variables for the database connection parameters to make the code more readable and maintainable.
  3. Improved Error Handling: Used mysqli_connect_error() to provide more detailed error messages if the connection fails.
    These changes enhance the readability, maintainability, and robustness of the code.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants