Android Day of Week Calculator

[David Crow]

Pavlex4 wrote:

I have initialized Date date = new Date();

Which is not the one being passed to isDateValid().

"One man's wage rise is another man's price increase." - Harold Wilson

"Fireproof doesn't mean the fire will never come. It means when the fire comes that you will be able to withstand it." - Michael Simmons

"You can easily judge the character of a man by how he treats those who can do nothing for him." - James D. Miles

[Pavlex4]

So,what should I do?

[David Crow]

Maybe look at (or around) line 12.

"One man's wage rise is another man's price increase." - Harold Wilson

"Fireproof doesn't mean the fire will never come. It means when the fire comes that you will be able to withstand it." - Michael Simmons

"You can easily judge the character of a man by how he treats those who can do nothing for him." - James D. Miles

[Pavlex4]

You mean this "private Date date;" ?

[David Crow]

Correct.

"One man's wage rise is another man's price increase." - Harold Wilson

"Fireproof doesn't mean the fire will never come. It means when the fire comes that you will be able to withstand it." - Michael Simmons

"You can easily judge the character of a man by how he treats those who can do nothing for him." - James D. Miles

[Pavlex4]

What is problem with that line?

[Lost User]

Nothing.

[Pavlex4]

I added date = new Date(); to button onclicklistener but when I click button inside app it gets pressed and stays like that and nothing happens!!!

button.setOnClickListener(new View.OnClickListener()
{
@Override
public void onClick(View view)
{
String dayOfWeek;
boolean isValid;

            **date = new Date();**

            Calendar c = Calendar.getInstance();
            SimpleDateFormat sdf = new SimpleDateFormat("yyyyMMdd\_HHmmss");
            String strDate = sdf.format(c.getTime());

[Lost User]

You really need to make more effort to do some debugging of your code first. You have a problem which could be caused by any part of your code, so you need to do the troubleshooting to collect more information. Also you have a try/catch block in your code that reads:

        try
            {
                String getdate = spinner.getItemAtPosition(position).toString() + value
                        + textView2.getText().toString();
                Date date = sdf.parse(getdate);
            }
            catch (ParseException ex)
            {

            }

Do not do this, you are just hiding any problems that may occur. Put some proper code in the catch block to help you find errors.

[David Crow]

Can you not see that you have two Date objects (in separate scopes)?

"One man's wage rise is another man's price increase." - Harold Wilson

"Fireproof doesn't mean the fire will never come. It means when the fire comes that you will be able to withstand it." - Michael Simmons

"You can easily judge the character of a man by how he treats those who can do nothing for him." - James D. Miles

[Lost User]

Why check your own code when you can get it done for free at CodeProject? ;P