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 command-line options #1

Closed
4 tasks
PedroAS7 opened this issue Sep 27, 2024 · 1 comment · Fixed by #13
Closed
4 tasks

Add command-line options #1

PedroAS7 opened this issue Sep 27, 2024 · 1 comment · Fixed by #13
Labels
feature Mandatory feature

Comments

@PedroAS7
Copy link
Collaborator

PedroAS7 commented Sep 27, 2024

We'll need command-line options to control how the program will run. We'll be using picocli (required).
The command-line options that will need to be handled:

  • -i, --input-file: image to hide information in, or that has as a message hidden in it. Takes a string as input
  • -o, --output-file: image with the message hidden in it (in case input file does not have it), or with the message removed (in case it does). Takes a string as input
  • -m, --message-file: file containing the message to be hidden (in case input file does not have it), or where the message will be saved to (in case it does). Takes a string as input
  • -b, --bits: number of bits to change in every pixel's colors. Takes an int $n, 1 \leq n \leq 8$ as input, where $n$ is a power of 2 (so, acceptable values are 1, 2, 4, 8)
@PedroAS7 PedroAS7 added the feature Mandatory feature label Sep 27, 2024
@lentidas
Copy link
Owner

I think the input file, output file and message file are better expressed as picocli parameters (@CommandLine.Parameter() annotation) instead of options. The options are better suited for things like the number of bits or for activating encryption, eventually.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Mandatory feature
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants