Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[ADD CONTENT] C++ Basic Concepts #72

Merged
merged 20 commits into from
Mar 27, 2023
Merged

Conversation

harshbhosale01
Copy link
Contributor

@harshbhosale01 harshbhosale01 commented Feb 21, 2023

So far in Basics of C++, I have added usage of Pointer to a 2D array and Passing a 2D array to a Function as an argument in 1_cpp_basics. Moreover, Basics of OpenCV Mat function and its usage is explained in 2_cpp_basics.
Documentation is yet to be done.
Solves #57
CC: @VanshPanchal0308, @Aryan-karawale

Copy link
Collaborator

@amanchhaparia amanchhaparia left a comment

Choose a reason for hiding this comment

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

  • Add a table of content in the readme
  • Add content for type casting, enum, namespace etc
  • Change the formatting to code block where necessary.

1_cpp_basics/readme.md Outdated Show resolved Hide resolved
1_cpp_basics/readme.md Outdated Show resolved Hide resolved
1_cpp_basics/readme.md Outdated Show resolved Hide resolved
1_cpp_basics/readme.md Outdated Show resolved Hide resolved
1_cpp_basics/readme.md Outdated Show resolved Hide resolved
1_cpp_basics/readme.md Outdated Show resolved Hide resolved
1_cpp_basics/readme.md Outdated Show resolved Hide resolved
Copy link
Collaborator

@amanchhaparia amanchhaparia left a comment

Choose a reason for hiding this comment

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

  • Add comments in file 1_cpp_basics.cpp
  • Add instructions to run the example codes.

1_cpp_basics/1_cpp_basics.cpp Outdated Show resolved Hide resolved
1_cpp_basics/1_cpp_basics.hpp Outdated Show resolved Hide resolved
1_cpp_basics/2_cpp_basics.hpp Outdated Show resolved Hide resolved
@amanchhaparia amanchhaparia changed the title Generating Draft PR for Basics of C++ [ADD CONTENT] C++ Basic Concepts Mar 21, 2023
@amanchhaparia
Copy link
Collaborator

@harshbhosale01 @VanshPanchal0308 is this PR ready for final review?

@harshbhosale01
Copy link
Contributor Author

@amanchhaparia Yes, comments have been added to all codes. Please Review.

Copy link
Collaborator

@amanchhaparia amanchhaparia left a comment

Choose a reason for hiding this comment

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

  • Create a asset folder and place it all within it, instead of using imgur.
  • Output Images should be of linux terminal.

1_cpp_basics/README.md Show resolved Hide resolved
1_cpp_basics/README.md Outdated Show resolved Hide resolved
1_cpp_basics/README.md Show resolved Hide resolved
@harshbhosale01
Copy link
Contributor Author

Suggested changes, have been made @amanchhaparia
Please, have a look.

Copy link
Collaborator

@amanchhaparia amanchhaparia left a comment

Choose a reason for hiding this comment

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

remove a.out from the pointers folder

@harshbhosale01
Copy link
Contributor Author

@amanchhaparia Done

Copy link
Collaborator

@amanchhaparia amanchhaparia left a comment

Choose a reason for hiding this comment

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

Rename the subtopics folder with a numbering similar to the topics in 4_cv_basics

cout<<endl;
}
return 0;
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

Add a new line

display (a, 3, 4);
print (a, 3, 4);
return 0;
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

Add a new line

// call printid()
obj1.printid();
return 0;
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

Add a new line

//If condn is fullfilled it will get executed
}
return 0;
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

Add a new line

cout<<"The number is odd\n";
}
return 0;
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

Add a new line

A<int> d;
d.add(); //Calling the class method
return 0;
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

Add a new line

cout<<'\n';
cout<<"Addition of m and n is :"<<add(m,n); //Calling the template for float data type
return 0;
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

Add a new line

cout << "Sum = " << sum<<endl;

return 0;
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

Add a new line

<< "z = " << z << endl;

return 0;
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

Add a new line

}

return 0;
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

Add a new line

@amanchhaparia amanchhaparia merged commit f68a968 into SRA-VJTI:dev Mar 27, 2023
Khushi-Balia pushed a commit to Khushi-Balia/Pixels_Seminar that referenced this pull request Mar 28, 2023
* Adding Basics of C++

* Documentation of Basics of C++

* Updated Readme of C++

* code files updated

* Updated Readme of C++ with fun,class,templates

* Updated Readme of C++ with control_loops

* Final changes

* Updated Readme of C++ with comments

* Final changes

* Changed the readme name to README

* Updated Readme of C++ with comments

* Updated Readme of C++ with changes in output,images

* Adding assets folder

* Adding Image to readme

* Removing redundant files

* Final changes

* Corrected the formatting

* Changed naming format

---------

Co-authored-by: VanshPanchal0308 <vansh5chal0308@gmail.com>
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.

3 participants