Multi file uploader extension checking not working all the times

Question

1.00/5 (2 votes)

See more:

Below is my code which checks for -b.doc in my selected files of that particular multiple fileuploader my problem is suppos if i choose
200-c.docx,201-C.docx,202.wma and 203.wma with same order it is saying validformat it is working some times and not working some times.

What I have tried:

i tried above code but it is not working when i do it multiple times

Posted 5-Sep-17 23:36pm

kav@94

Updated 6-Sep-17 22:31pm

v4

Add a Solution

Comments

Afzaal Ahmad Zeeshan 6-Sep-17 7:14am

What is the purpose of this? Be more specific so that we can provide a better solution.

3 solutions

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

koklimabc · Answer 1 · 2017-09-06T15:50:00

Solution 1

Should tried:

if (myfile.indexOf("-c.doc") != -1 || myfile.indexOf("-C.DOC") > != -1)

Posted 6-Sep-17 15:50pm

koklimabc

Comments

kav@94 7-Sep-17 1:02am

i had even tried with if (myfile.indexOf("-c.doc") != -1 || myfile.indexOf("-C.DOC") > != -1) but if i choose the files in
below order it is still saying valid format but it should say invalid format
because in the selected files there is an .mp3 file 400-C.doc,401-c.doc,402-C.doc,403.mp3

Patrice T · Answer 2 · 2017-09-06T19:19:00

Solution 2

You test the filename against lowercase and against uppercase, but any mix will fail.
You need to convert the filename to lowercase or uppercase before testing.

Posted 6-Sep-17 19:19pm

Patrice T

Comments

Patrice T 7-Sep-17 1:38am

You should learn to Google for such information.

Graeme_Grant · Answer 3 · 2017-09-06T19:28:00

Solution 3

C#

var myfile = chkFile.value.ToLower();
var validExtension = "-c.doc";
label.innerText = myfile.EndsWith(validExtension) ? "Valid Format" : "Invalid Format";

UPDATE: Here is a test to check that the above code works:

C#6+ version

C#

var files = new List<string> { "aaa.mp3", "aaa-c.mp3", "aAa-c.DoC", "aAa-c.doc" };
var validExtension = "-c.doc";
foreach (var file in files)
{
    Console.WriteLine($"{file} is  {(file.ToLower().EndsWith(validExtension) ? "Valid Format" : "Invalid Format")}");
}

Console.WriteLine("-- done --");
Console.ReadKey();

C# 5.0 or older:

C#

var files = new List<string> { "aaa.mp3", "aaa-c.mp3", "aAa-c.DoC", "aAa-c.doc" };
var validExtension = "-c.doc";
foreach (string file in files)
{
    Console.WriteLine(String.Format("{0} is  {1}", file, 
                      file.ToLower().EndsWith(validExtension)
                          ? "Valid Format"
                          : "Invalid Format");
}

Console.WriteLine("-- done --");
Console.ReadKey();

Output:

aaa.mp3 is  Invalid Format
aaa-c.mp3 is  Invalid Format
aAa-c.DoC is  Valid Format
aAa-c.doc is  Valid Format
-- done --

Posted 6-Sep-17 19:28pm

Graeme_Grant

Updated 6-Sep-17 21:13pm

v5

Comments

kav@94 7-Sep-17 1:48am

i tried in this way but still its not working

if (myfile.indexOf("-c.doc").toLowerCase() > 0 || myfile.indexOf("-C.DOC").toLowerCase() > 0 || myfile.indexOf("-C.doc").toLowerCase() > 0 || myfile.indexOf("-c.DOC").toLowerCase() > 0 || myfile.indexOf("-c.docx").toLowerCase() > 0 || myfile.indexOf("-C.DOCX").toLowerCase() > 0 || myfile.indexOf("-C.docx").toLowerCase() > 0 || myfile.indexOf("-c.DOCX").toLowerCase() > 0) {
label.innerText = "Valid Format";
}
else {
label.innerText = "Invalid Format";
chkFile.value = "";
}
}

Graeme_Grant 7-Sep-17 2:25am

Don't tell me what you are doing, read my solution and replace your code.

kav@94 7-Sep-17 2:46am

i tried your code only but its not working it is still accepting other extensions

Graeme_Grant 7-Sep-17 2:54am

Have a look at the update to the solution. Works fine. You have not implemented the change correctly. Run the test code in a new console app and you will see the same output as above.

kav@94 7-Sep-17 3:01am

i am getting error near foreach loop at var as syntax error and also near console.writeline as expected ')' this is javascript funtion not c# code

Graeme_Grant 7-Sep-17 3:04am

Code works fine. Syntax error could mean that you have keyed it wrong.

If you are using pre-C#6, ie: C#5 or older, then the Console line needs to use the old String.Format(...) to output the results.

kav@94 7-Sep-17 3:07am

its .net framework 3.5

Graeme_Grant 7-Sep-17 3:12am

Why such an old unsupported version of the Framework?

Added new version of the TEST that will work with C# 5.0 or older (Framework 3.5)

kav@94 7-Sep-17 3:26am

but still i am getting error near var files = new List<string> { "aaa.mp3", "aaa-c.mp3", "aAa-c.DoC", "aAa-c.doc" }; commas in this line and near foreach (string file in files) line moreover i cant keep the filenames statically in list it should be taben from the files that are choosed using fileuploader

Graeme_Grant 7-Sep-17 3:36am

You're a programmer, change the creation of the list of test files.

kav@94 7-Sep-17 3:40am

yes but i am new to this can you please help me out

Graeme_Grant 7-Sep-17 3:43am

How do you create a string list then add a list of strings to it? That is all that line of code is doing.

I have set the Framework version to 3.5 and it compiles and runs fine. The issue is something that you are doing. I can't help you any more than that!