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.

Sign up for GitHub

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

Jump to bottom

Refine input handling #2

Open
kbuffardi opened this issue Sep 21, 2024 · 0 comments · May be fixed by #8, #50 or #52
Open

Refine input handling #2

kbuffardi opened this issue Sep 21, 2024 · 0 comments · May be fixed by #8, #50 or #52

Comments

@kbuffardi
Copy link
Contributor

Currently, the program only creates a list of 1-word favorites. However, we want to allow the user to enter an entire line of input, even with spaces, such as Software Engineering as a single item in the list. We probably want to use getline to capture a line of input instead of a space-delimited string.

Also, the user enters done to finish their input. However, the program currently displays done as the last item in the list. We should exclude the done command from the list.
Ian-J-S added a commit to Ian-J-S/MyFave that referenced this issue Sep 25, 2024
@Ian-J-S
Enables multiple word favorites, closes ChicoState#2
ea46d4d
codestafa pushed a commit to codestafa/MyFave that referenced this issue Sep 25, 2024
@codestafa2
added line to ignore done and handle spaces in input, closes ChicoSta…
0b2125c 
…te#2
TaigaODonnell added a commit to TaigaODonnell/MyFave that referenced this issue Sep 25, 2024
@TaigaODonnell
Refined input handling, closes ChicoState#2
fc8dafd
scblotcky added a commit to scblotcky/MyFave that referenced this issue Sep 25, 2024
@scblotcky
Added getline and exclude 'done', closes ChicoState#2
e749183
RyanTheRyan-code pushed a commit to RyanTheRyan-code/MyFave that referenced this issue Sep 25, 2024
Fixed main to get whole line and not include done, Closes ChicoState#2
245e5cb
jarrettgaither added a commit to jarrettgaither/MyFave that referenced this issue Sep 25, 2024
@jarrettgaither
Closes ChicoState#2
9dd6517
CarlosF03R added a commit to CarlosF03R/MyFave that referenced this issue Sep 25, 2024
@CarlosF03R
Resolved entire line of input, Closes ChicoState#2
a0fdba0
rosalep added a commit to rosalep/MyFave that referenced this issue Sep 25, 2024
@rosalep
Implemented getline and fixed final list displayed in main.cpp, closes …
a7845e6 
…ChicoState#2
VicaWorth pushed a commit to VicaWorth/MyFave that referenced this issue Sep 25, 2024
resolving single itemed list, closes ChicoState#2
3d6e51a
CharcoalNuggets added a commit to CharcoalNuggets/MyFave that referenced this issue Sep 25, 2024
@CharcoalNuggets
Added getline functionality, Resolves ChicoState#2
ab9b7fc
lukemaloney03 pushed a commit to lukemaloney03/MyFave that referenced this issue Sep 25, 2024
feat: updated input handler to extract entire line as input, closes C…
41e385d 
…hicoState#2
big-al188 added a commit to big-al188/MyFave that referenced this issue Sep 25, 2024
@big-al188
Refined input handling closes ChicoState#2
ece1b76
Miguel9088 added a commit to Miguel9088/MyFave that referenced this issue Sep 25, 2024
@Miguel9088
updating the main.cpp file closing ChicoState#2
406443b
nrakocevic added a commit to nrakocevic/MyFave that referenced this issue Sep 25, 2024
@nrakocevic
created a continous integration yml file, added condition to exclude …
37a3383 
…done, and replace cin with getline, closes ChicoState#2
evanalba added a commit to evanalba/MyFave that referenced this issue Sep 25, 2024
@evanalba
feat, fix: refine input handling and set up continuous integration. c…
499602f 
…loses ChicoState#1 closes ChicoState#2.
natesmitty1738 added a commit to natesmitty1738/MyFave that referenced this issue Sep 25, 2024
@natesmitty1738
refine input handling, fixes ChicoState#2
c7f41e5
DAC098 added a commit to DAC098/MyFave that referenced this issue Sep 25, 2024
@DAC098
updates to fix ChicoState#2
3ed65bb 
added in the ability to retrieve single line input from the user. also
checking if the input give is the word done and not adding it to the
list.
Miguel9088 added a commit to Miguel9088/MyFave that referenced this issue Sep 25, 2024
@Miguel9088
updating the main.cpp file, closing ChicoState#2
89f0aa7
LWinn10126 added a commit to LWinn10126/MyFave that referenced this issue Sep 25, 2024
@LWinn10126
Multiple Word Favorites and removes done, closes ChicoState#1, closes C…
aa97fa0 
…hicoState#2
CharcoalNuggets added a commit to CharcoalNuggets/MyFave that referenced this issue Sep 25, 2024
@CharcoalNuggets
Added getline functionality and should cut done from list, Resolves C…
9be2e54 
…hicoState#2
MikeD76 added a commit to MikeD76/MyFave that referenced this issue Sep 25, 2024
@MikeD76
Added refined input handling feature, closes ChicoState#2
52ec97b
thomasdevine01 added a commit to thomasdevine01/MyFave that referenced this issue Sep 25, 2024
@thomasdevine01
added multi word support and removes unexpected behavior, resolves is…
dd74b57 
…sue ChicoState#2
blgrigg pushed a commit to blgrigg/MyFave that referenced this issue Sep 25, 2024
Refines input handling, closes ChicoState#2
9380ec7
IvanGav added a commit to IvanGav/MyFave that referenced this issue Sep 25, 2024
@IvanGav
improved line reading; set up CICD for building success - closes Chic…
0dbf4a2 
…oState#1, closes ChicoState#2
@Kamranaway Kamranaway linked a pull request Sep 25, 2024 that will close this issue
Open
cmgomes5 added a commit to cmgomes5/MyFave that referenced this issue Sep 25, 2024
@cmgomes5
Added workflow for github actions, first half of Issue ChicoState#2
6dafc3f
CS-Brandon added a commit to CS-Brandon/MyFave that referenced this issue Sep 25, 2024
@CS-Brandon
fixing the code issue ChicoState#2
5274a56
jbaeee referenced this issue in jbaeee/MyFave Sep 25, 2024
@jbaeee
added getline and removed done, close #2
eedf705
clmcguire added a commit to clmcguire/MyFave that referenced this issue Sep 25, 2024
@clmcguire
Create c-cpp.yml - addresses issue ChicoState#2
cf80c63
asa789 added a commit to asa789/MyFave that referenced this issue Sep 25, 2024
@asa789
Resloved Input Handling, closes ChicoState#2
509c3fb
scblotcky added a commit to scblotcky/MyFave that referenced this issue Sep 25, 2024
@scblotcky
Changed exit word to 'exit' instead of 'done', closes ChicoState#2
dfc910b
clmcguire added a commit to clmcguire/MyFave that referenced this issue Sep 25, 2024
@clmcguire
Adding status badge to READ.ME addresses issue ChicoState#2
0bab796
cmgomes5 added a commit to cmgomes5/MyFave that referenced this issue Sep 25, 2024
@cmgomes5
Added workflow badge to readme. Closes Issue ChicoState#2
3e4a67c
@rosalep rosalep mentioned this issue Sep 25, 2024
Open
leszakal added a commit to leszakal/MyFave that referenced this issue Sep 25, 2024
@leszakal
Add input handling for including spaces; exclude done from list, closes
d2f1dea 
ChicoState#2
akleinecke added a commit to akleinecke/MyFave that referenced this issue Sep 25, 2024
@akleinecke
Modified main to use getline and pop last element, closes ChicoState#2
c4d6ef4
akleinecke added a commit to akleinecke/MyFave that referenced this issue Sep 25, 2024
@akleinecke
Modified main to use getline and pop last element, closes ChicoState#2
ac83fa6
nrakocevic added a commit to nrakocevic/MyFave that referenced this issue Sep 25, 2024
@nrakocevic
created a continous integration yml file, added condition to exclude …
bf94f63 
…done, and replace cin with getline, closes ChicoState#2
evangoertzen added a commit to evangoertzen/MyFave that referenced this issue Sep 25, 2024
@evangoertzen
refine input handling and exclude 'done' from list. Resolves ChicoSta…
17946d3 
…te#2
Shalom-cmd added a commit to Shalom-cmd/MyFave that referenced this issue Sep 25, 2024
@Shalom-cmd
Update ci.yml closes ChicoState#2
91d0c21
jbaeee referenced this issue in jbaeee/MyFave Sep 25, 2024
set up continuous integration, closes #2
8b8e3c0
natesmitty1738 added a commit to natesmitty1738/MyFave that referenced this issue Sep 25, 2024
@natesmitty1738
refine input handling, fixes ChicoState#2
198476d
the-real-brogrammer pushed a commit to the-real-brogrammer/MyFave that referenced this issue Sep 25, 2024
Change input to getline, exclude 'done' from results. Closing ChicoSt…
fc46b12 
…ate#2
nrakocevic added a commit to nrakocevic/MyFave that referenced this issue Sep 25, 2024
@nrakocevic
created a continous integration yml file, added condition to exclude …
1285a39 
…done, and replace cin with getline, closes ChicoState#2
klmenigoz added a commit to klmenigoz/MyFave that referenced this issue Sep 25, 2024
@klmenigoz
forgot to add message, resolved ChicoState#2
9e04a53
jbaeee referenced this issue in jbaeee/MyFave Sep 25, 2024
@jbaeee
added status badge, closes #2
88add02
CharcoalNuggets added a commit to CharcoalNuggets/MyFave that referenced this issue Sep 25, 2024
@CharcoalNuggets
Added getline and handles trailing done, Resolves ChicoState#2
15e9126
CarlosF03R added a commit to CarlosF03R/MyFave that referenced this issue Sep 25, 2024
@CarlosF03R
Resolved lines of input, closes ChicoState#2
5b24296
chrisheesh added a commit to chrisheesh/MyFave that referenced this issue Sep 25, 2024
@chrisheesh
closes ChicoState#2
3cde482
chrisheesh added a commit to chrisheesh/MyFave that referenced this issue Sep 25, 2024
@chrisheesh
Merge branch 'main' of github.com:chrisheesh/MyFave
afd3b32 
merge changes to resolve ChicoState#1 and resolve ChicoState#2
@chrisheesh chrisheesh linked a pull request Sep 25, 2024 that will close this issue
Open
nrakocevic added a commit to nrakocevic/MyFave that referenced this issue Sep 25, 2024
@nrakocevic
created a continous integration yml file, added condition to exclude …
876f5ff 
…done, and replace cin with getline, closes ChicoState#2
njmartinez2 added a commit to njmartinez2/MyFave that referenced this issue Sep 25, 2024
@njmartinez2
Added getline and excluded done from the list, closes ChicoState#2
f104e6c
BHM03 added a commit to BHM03/MyFaveBH that referenced this issue Sep 25, 2024
@BHM03
Enabled sentence Favorites, fixed ChicoState#2
7007e38
@klmenigoz klmenigoz mentioned this issue Sep 25, 2024
Open
@vbearwood vbearwood mentioned this issue Sep 25, 2024
Open
BHM03 added a commit to BHM03/MyFaveBH that referenced this issue Sep 25, 2024
@BHM03
Excluded done, fixed ChicoState#2
5a3ec2c
@LeoMalinen2 LeoMalinen2 linked a pull request Sep 25, 2024 that will close this issue
Open
Shalom-cmd added a commit to Shalom-cmd/MyFave that referenced this issue Sep 25, 2024
@Shalom-cmd
set up github actions ci closes ChicoState#2
de9d535
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
1 participant
@kbuffardi